[ 
https://issues.apache.org/jira/browse/MFCOMMONS-9?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12715160#action_12715160
 ] 

Jing Wu commented on MFCOMMONS-9:
---------------------------------

This will have dependency on trinidad-maven-plugin. Patches for both 
trinidad-maven-plugin and trinidad are attached.

For the plugin, the following changes were made:

1. 
$plugin_root\maven-faces-plugin\src\main\java\org\apache\myfaces\trinidadbuild\plugin\faces\generator\GeneratorHelper.java
 

- added a method named isKnowTypeSet(...), used by the tag generator to 
recognize Set<String> type.

2. 
$plugin_root\maven-faces-plugin\src\main\java\org\apache\myfaces\trinidadbuild\plugin\faces\generator\TrinidadComponentTagGenerator.java
 

 - the generator that generates setStringSetProperty(...) method on tag files.

 

For trinidad, the following changes were made:

 1. 

$trinidad_root\trinidad-api\src\main\java\org\apache\myfaces\trinidad\webapp\UIXComponentELTag.java
 

- this is the parent of all tag classes, which implements 
setStringSetProperty(...) that all generated tag classes depend 

 on, it converts the string to a set, and set it on the bean.

 2. 

 
$trinidad_root\trinidad-impl\src\main\java\org\apache\myfaces\trinidadinternal\share\expl\Coercions.java
 

 - add coercion case to convert String to Set.

 3. 

 
$trinidad_root\trinidad-impl\src\main\java\org\apache\myfaces\trinidadinternal\share\xml\XMLUtils.java
 

 - the actual implementation of converting String to Set, consumed by 
Coercions.java

 4. 

 
$trinidad_root\trinidad-impl\src\main\java\org\apache\myfaces\trinidadinternal\taglib\util\TagUtils.java
 

 - add a method getStringSet(...) that will be used by generated tag classes.

 


> Support java.util.Set as property type
> --------------------------------------
>
>                 Key: MFCOMMONS-9
>                 URL: https://issues.apache.org/jira/browse/MFCOMMONS-9
>             Project: MyFaces Commons
>          Issue Type: Improvement
>          Components: myfaces-commons-components
>    Affects Versions: 1.0.1-SNAPSHOT
>         Environment: This is generic, not specific to any OS or 
> software/hardware specifications.
>            Reporter: Jing Wu
>            Priority: Minor
>   Original Estimate: 48h
>  Remaining Estimate: 48h
>
> For some properties, java.util.Set is the best suitable type, trinidad should 
> support java.util.Set as the property type. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to