[ 
http://issues.apache.org/jira/browse/MYFACES-1284?page=comments#action_12374492 
] 

Gert Vanthienen commented on MYFACES-1284:
------------------------------------------

Currently, in order to make a Tomahawk component DisplayValueOnlyCapable, you 
have to make these changes to a component's XML file:
- specify  generateDisplayValueOnlyMethods="true" in order to get the 
isSetDisplayValueOnly()-method generated
- specify a boolean field named displayValueOnly
- specify a String field named displayValueOnlyStyle
- specify a String field named displayValueOnlyStyleClass

Wouldn't it make sense to specify all this in a single attribute (e.g. 
isDisplayValueOnlyCapable="true") instead of repeating the same definitions 
over and over again?  Not only would this simplify the XML file, it would also 
make future changes to the DisplayValueOnlyCapable interface easier to 
implement.

In my opinion, the same goes for a set of other Tomahawk interfaces (e.g. 
UserRoleAware, EscapeCapable).

> Generating component source code
> --------------------------------
>
>          Key: MYFACES-1284
>          URL: http://issues.apache.org/jira/browse/MYFACES-1284
>      Project: MyFaces Core
>         Type: Improvement

>   Components: build process
>     Reporter: Gert Vanthienen
>     Priority: Minor
>  Attachments: build-tools-generator.patch, tomahawk-generator.patch
>
> The component code generator has been re-added to the Myfaces build process 
> by Manfred Geiler, but there is still some work to be done.  For a 
> description, see also 
> http://mail-archives.apache.org/mod_mbox/myfaces-dev/200604.mbox/[EMAIL 
> PROTECTED]

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to