[richfaces-issues] [JBoss JIRA] Moved: (RF-7606) SelectUtils.getConvertedUISelectManyValue more restrictive than it needs to be

[Nick Belaevski (JIRA)]

     [ https://jira.jboss.org/jira/browse/RF-7606?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Nick Belaevski moved AJSF-143 to RF-7606:
-----------------------------------------

       Project: RichFaces  (was: Ajax4jsf)
           Key: RF-7606  (was: AJSF-143)
    Issue Type: Feature Request  (was: Quality Risk)


> SelectUtils.getConvertedUISelectManyValue more restrictive than it needs to be
> ------------------------------------------------------------------------------
>
>                 Key: RF-7606
>                 URL: https://jira.jboss.org/jira/browse/RF-7606
>             Project: RichFaces
>          Issue Type: Feature Request
>         Environment: JBoss 4.2.2.GA, RichFaces 3.2.1.GA, windows xp sp2 x86
>            Reporter: Andr? Fernandes
>            Priority: Minor
>
> on line 189 of the org.ajax4jsf.util.SelectUtils we see 
> if (List.class.isAssignableFrom(valueType))
> ....
> But I think it don't need to be List. 
> It should be Collection because it is less restrictive and works just fine. :)
> I found this when I was using an Entity with a Collection<AClass> and a 
> <rich:pickList ...>
>   <s:selectItems ...></s:selectItems>
> </rich:pickList> 
> on the view.
> When I was posting back to the server, it was throwing an exception (think it is IllegalArgumentException).
> Then I get the source for RichFaces and find out that the SelectUtils.getConvertedUISelectManyValue was not expecting a Collection, but instead it was expecting a List.
> Then I change my entity to have a List<AClass> and it works just fine.
> If the line 189 just change to if (Collection.class.isAssignableFrom(valueType)) it should work.

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