[
https://issues.apache.org/jira/browse/PIVOT-520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12876823#action_12876823
]
Greg Brown edited comment on PIVOT-520 at 6/8/10 5:56 PM:
----------------------------------------------------------
Just to be a bit more clear about what specifically I am disagreeing with -
consider this Java code:
PushButton myButton = new PushButton();
You wouldn't expect that calling myButton.getName() now would return
"myButton", right? That's because there is no association between a Java
variable name and the "name" bean property, The Java compiler isn't going to
magically detect that PushButton defines a setName() method and call it for
you. You need to do that yourself:
myButton.setName("myButton");
This would be the equivalent code in WTKX:
<PushButton wtkx:id="myButton" name="myButton"/>
WTKX IDs and the proposed "name" bean property simply address different use
cases. A WTKX ID defines a variable that you can immediately reference in code,
either in Java (via Bindable) or in script that you embed in the WTKX file, and
the name becomes a key by which you can later obtain a reference to a value
that you can use in code. IMO, it would introduce needless complexity to some
highly critical code (WTKXSerializer) if we try to combine these two concepts.
was (Author: gbrown):
Just to be a bit more specific about what specifically I am disagreeing
with - consider this Java code:
PushButton myButton = new PushButton();
You wouldn't expect that calling myButton.getName() now would return
"myButton", right? That's because there is no association between a Java
variable name and the "name" bean property, The Java compiler isn't going to
magically detect that PushButton defines a setName() method and call it for
you. You need to do that yourself:
myButton.setName("myButton");
This would be the equivalent code in WTKX:
<PushButton wtkx:id="myButton" name="myButton"/>
WTKX IDs and the proposed "name" bean property simply address different use
cases. A WTKX ID defines a variable that you can immediately reference in code,
either in Java (via Bindable) or in script that you embed in the WTKX file, and
the name becomes a key by which you can later obtain a reference to a value
that you can use in code. IMO, it would introduce needless complexity to some
highly critical code (WTKXSerializer) if we try to combine these two concepts.
> add a name property to Component
> --------------------------------
>
> Key: PIVOT-520
> URL: https://issues.apache.org/jira/browse/PIVOT-520
> Project: Pivot
> Issue Type: Improvement
> Components: wtk
> Reporter: Appddevvv
> Priority: Minor
> Fix For: 1.6
>
>
> Add a name property to Component
> a) A getter/setter
> b) A method to Container for Component getNamedComponent(String componentName)
> The name should default to an the wtkx:id if one is specified for that
> component or if the name is not specified, null. If a name is specified and
> no wtkx:id is specified , it should become the wtkx:id of that component. The
> name must be a valid java identifier, otherwise an serialization exception
> should be thrown. The name used in the component should be left and right
> whitespace trimmed.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.