[jira] [Comment Edited] (OFBIZ-4035) change the id Attribute for input fields in the model macro form renderer from String to FlexibleStringExpander
[
https://issues.apache.org/jira/browse/OFBIZ-4035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17042669#comment-17042669
]
Michael Brohl edited comment on OFBIZ-4035 at 2/23/20 10:07 AM:
Two PRs created:
[https://github.com/apache/ofbiz-framework/pull/22|https://github.com/apache/ofbiz-framework/pull/22]
[https://github.com/apache/ofbiz-framework/pull/23|https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
was (Author: danwatford):
Two PRs created:
[[https://github.com/apache/ofbiz-framework/pull/22||https://github.com/apache/ofbiz-framework/pull/22]
[[https://github.com/apache/ofbiz-framework/pull/23||https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
> change the id Attribute for input fields in the model macro form renderer
> from String to FlexibleStringExpander
> ---
>
> Key: OFBIZ-4035
> URL: https://issues.apache.org/jira/browse/OFBIZ-4035
> Project: OFBiz
> Issue Type: Improvement
> Components: ALL COMPONENTS
>Affects Versions: Trunk
>Reporter: Sascha Schumann
>Assignee: Daniel Watford
>Priority: Major
> Labels: patch-available
> Attachments: OFBIZ-4035_ModelFormField.java.patch
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> hi there,
> here a little useful improvement. i changed the ID Field of an input field in
> in the macro form renderer from a normal String to a flexible String
> expander.
> The advantage is that i can use ${} variables in the id description, that
> helps me for example creating automatically unique id's.
> Cheers
> Sascha
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Comment Edited] (OFBIZ-4035) change the id Attribute for input fields in the model macro form renderer from String to FlexibleStringExpander
[
https://issues.apache.org/jira/browse/OFBIZ-4035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17042669#comment-17042669
]
Michael Brohl edited comment on OFBIZ-4035 at 2/23/20 10:06 AM:
Two PRs created:
[[https://github.com/apache/ofbiz-framework/pull/22||https://github.com/apache/ofbiz-framework/pull/22]
[[https://github.com/apache/ofbiz-framework/pull/23||https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
was (Author: danwatford):
Two PRs created:
[[https://github.com/apache/ofbiz-framework/pull/22||https://github.com/apache/ofbiz-framework/pull/23]
[[https://github.com/apache/ofbiz-framework/pull/23||https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
> change the id Attribute for input fields in the model macro form renderer
> from String to FlexibleStringExpander
> ---
>
> Key: OFBIZ-4035
> URL: https://issues.apache.org/jira/browse/OFBIZ-4035
> Project: OFBiz
> Issue Type: Improvement
> Components: ALL COMPONENTS
>Affects Versions: Trunk
>Reporter: Sascha Schumann
>Assignee: Daniel Watford
>Priority: Major
> Labels: patch-available
> Attachments: OFBIZ-4035_ModelFormField.java.patch
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> hi there,
> here a little useful improvement. i changed the ID Field of an input field in
> in the macro form renderer from a normal String to a flexible String
> expander.
> The advantage is that i can use ${} variables in the id description, that
> helps me for example creating automatically unique id's.
> Cheers
> Sascha
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Comment Edited] (OFBIZ-4035) change the id Attribute for input fields in the model macro form renderer from String to FlexibleStringExpander
[
https://issues.apache.org/jira/browse/OFBIZ-4035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17042669#comment-17042669
]
Michael Brohl edited comment on OFBIZ-4035 at 2/23/20 10:06 AM:
Two PRs created:
[[https://github.com/apache/ofbiz-framework/pull/22||https://github.com/apache/ofbiz-framework/pull/23]
[[https://github.com/apache/ofbiz-framework/pull/23||https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
was (Author: danwatford):
Two PRs created:
[[https://github.com/apache/ofbiz-framework/pull/22||https://github.com/apache/ofbiz-framework/pull/23]
[https://github.com/apache/ofbiz-framework/pull/22|https://github.com/apache/ofbiz-framework/pull/23]
[]|https://github.com/apache/ofbiz-framework/pull/23]
[[https://github.com/apache/ofbiz-framework/pull/23||https://github.com/apache/ofbiz-framework/pull/23]
[https://github.com/apache/ofbiz-framework/pull/23]
[]|https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
> change the id Attribute for input fields in the model macro form renderer
> from String to FlexibleStringExpander
> ---
>
> Key: OFBIZ-4035
> URL: https://issues.apache.org/jira/browse/OFBIZ-4035
> Project: OFBiz
> Issue Type: Improvement
> Components: ALL COMPONENTS
>Affects Versions: Trunk
>Reporter: Sascha Schumann
>Assignee: Daniel Watford
>Priority: Major
> Labels: patch-available
> Attachments: OFBIZ-4035_ModelFormField.java.patch
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> hi there,
> here a little useful improvement. i changed the ID Field of an input field in
> in the macro form renderer from a normal String to a flexible String
> expander.
> The advantage is that i can use ${} variables in the id description, that
> helps me for example creating automatically unique id's.
> Cheers
> Sascha
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Comment Edited] (OFBIZ-4035) change the id Attribute for input fields in the model macro form renderer from String to FlexibleStringExpander
[
https://issues.apache.org/jira/browse/OFBIZ-4035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17042669#comment-17042669
]
Michael Brohl edited comment on OFBIZ-4035 at 2/23/20 10:05 AM:
Two PRs created:
[[https://github.com/apache/ofbiz-framework/pull/22||https://github.com/apache/ofbiz-framework/pull/23]
[https://github.com/apache/ofbiz-framework/pull/22|https://github.com/apache/ofbiz-framework/pull/23]
[]|https://github.com/apache/ofbiz-framework/pull/23]
[[https://github.com/apache/ofbiz-framework/pull/23||https://github.com/apache/ofbiz-framework/pull/23]
[https://github.com/apache/ofbiz-framework/pull/23]
[]|https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
was (Author: danwatford):
Two PRs created:
[https://github.com/apache/ofbiz-framework/pull/22|https://github.com/apache/ofbiz-framework/pull/23]
[https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
> change the id Attribute for input fields in the model macro form renderer
> from String to FlexibleStringExpander
> ---
>
> Key: OFBIZ-4035
> URL: https://issues.apache.org/jira/browse/OFBIZ-4035
> Project: OFBiz
> Issue Type: Improvement
> Components: ALL COMPONENTS
>Affects Versions: Trunk
>Reporter: Sascha Schumann
>Assignee: Daniel Watford
>Priority: Major
> Labels: patch-available
> Attachments: OFBIZ-4035_ModelFormField.java.patch
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> hi there,
> here a little useful improvement. i changed the ID Field of an input field in
> in the macro form renderer from a normal String to a flexible String
> expander.
> The advantage is that i can use ${} variables in the id description, that
> helps me for example creating automatically unique id's.
> Cheers
> Sascha
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Comment Edited] (OFBIZ-4035) change the id Attribute for input fields in the model macro form renderer from String to FlexibleStringExpander
[
https://issues.apache.org/jira/browse/OFBIZ-4035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17042669#comment-17042669
]
Michael Brohl edited comment on OFBIZ-4035 at 2/23/20 10:04 AM:
Two PRs created:
[|https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
was (Author: danwatford):
Two PRs created:
[https://github.com/apache/ofbiz-framework/pull/22|https://github.com/apache/ofbiz-framework/pull/23]
[https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
> change the id Attribute for input fields in the model macro form renderer
> from String to FlexibleStringExpander
> ---
>
> Key: OFBIZ-4035
> URL: https://issues.apache.org/jira/browse/OFBIZ-4035
> Project: OFBiz
> Issue Type: Improvement
> Components: ALL COMPONENTS
>Affects Versions: Trunk
>Reporter: Sascha Schumann
>Assignee: Daniel Watford
>Priority: Major
> Labels: patch-available
> Attachments: OFBIZ-4035_ModelFormField.java.patch
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> hi there,
> here a little useful improvement. i changed the ID Field of an input field in
> in the macro form renderer from a normal String to a flexible String
> expander.
> The advantage is that i can use ${} variables in the id description, that
> helps me for example creating automatically unique id's.
> Cheers
> Sascha
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Comment Edited] (OFBIZ-4035) change the id Attribute for input fields in the model macro form renderer from String to FlexibleStringExpander
[
https://issues.apache.org/jira/browse/OFBIZ-4035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17042669#comment-17042669
]
Michael Brohl edited comment on OFBIZ-4035 at 2/23/20 10:04 AM:
Two PRs created:
[https://github.com/apache/ofbiz-framework/pull/22|https://github.com/apache/ofbiz-framework/pull/23]
[https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
was (Author: danwatford):
Two PRs created:
[https://github.com/apache/ofbiz-framework/pull/22|https://github.com/apache/ofbiz-framework/pull/23]
[https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
> change the id Attribute for input fields in the model macro form renderer
> from String to FlexibleStringExpander
> ---
>
> Key: OFBIZ-4035
> URL: https://issues.apache.org/jira/browse/OFBIZ-4035
> Project: OFBiz
> Issue Type: Improvement
> Components: ALL COMPONENTS
>Affects Versions: Trunk
>Reporter: Sascha Schumann
>Assignee: Daniel Watford
>Priority: Major
> Labels: patch-available
> Attachments: OFBIZ-4035_ModelFormField.java.patch
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> hi there,
> here a little useful improvement. i changed the ID Field of an input field in
> in the macro form renderer from a normal String to a flexible String
> expander.
> The advantage is that i can use ${} variables in the id description, that
> helps me for example creating automatically unique id's.
> Cheers
> Sascha
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
[jira] [Comment Edited] (OFBIZ-4035) change the id Attribute for input fields in the model macro form renderer from String to FlexibleStringExpander
[
https://issues.apache.org/jira/browse/OFBIZ-4035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17042669#comment-17042669
]
Michael Brohl edited comment on OFBIZ-4035 at 2/23/20 10:04 AM:
Two PRs created:
[https://github.com/apache/ofbiz-framework/pull/22|https://github.com/apache/ofbiz-framework/pull/23]
[https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
was (Author: danwatford):
Two PRs created:
[|https://github.com/apache/ofbiz-framework/pull/23]
Since the core functionality required by this ticket was already implemented,
the first PR introduces documentation in the widget-form.xsd describing how the
functionality can be used.
The second PR extends the first PR and introduces unit tests. The potentially
controversial issue is that to implement the tests I felt it was necessary to
mock out static methods, functionality that's not available from Mockito, and
therefore brought in a test dependency on JMockit.
To provide unit tests for MacroFormRenderer without using a mocking library
like JMockit will require the class to be refactored which I deemed a bit of a
risky change at this point, particularly without existing tests to prove
correct behaviour is maintained.
> change the id Attribute for input fields in the model macro form renderer
> from String to FlexibleStringExpander
> ---
>
> Key: OFBIZ-4035
> URL: https://issues.apache.org/jira/browse/OFBIZ-4035
> Project: OFBiz
> Issue Type: Improvement
> Components: ALL COMPONENTS
>Affects Versions: Trunk
>Reporter: Sascha Schumann
>Assignee: Daniel Watford
>Priority: Major
> Labels: patch-available
> Attachments: OFBIZ-4035_ModelFormField.java.patch
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> hi there,
> here a little useful improvement. i changed the ID Field of an input field in
> in the macro form renderer from a normal String to a flexible String
> expander.
> The advantage is that i can use ${} variables in the id description, that
> helps me for example creating automatically unique id's.
> Cheers
> Sascha
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
