[ http://issues.apache.org/jira/browse/TOMAHAWK-738?page=comments#action_12443114 ] sean schofield commented on TOMAHAWK-738: -----------------------------------------
Again, nobody is saying there's not a problem here its just this solution merely covers up a problem that exists in the implementation. At a minimum we should check instanceof StateHolder on the restore call but it would be better (IMO) to address the root cause of the problem that prevents saveAttachedState to be called with List or whatever (as it clearly should be able to according to the HTML docs/spec.) > SaveState fails with a java.util.List implementation other than ArrayList > ------------------------------------------------------------------------- > > Key: TOMAHAWK-738 > URL: http://issues.apache.org/jira/browse/TOMAHAWK-738 > Project: MyFaces Tomahawk > Issue Type: Bug > Components: UISaveState > Affects Versions: 1.1.5-SNAPSHOT > Reporter: Cagatay Civici > Assigned To: Cagatay Civici > Fix For: 1.1.5-SNAPSHOT > > > restoreAttachedState of UIComponentBase wraps the lists as an ArrayList so > restoring values fails when a list implementation other than an arraylist is > used. > An example; > private LinkedList list; > private String name; > private String surname; > public LinkedList getList() { > list = new LinkedList(); > list.add(name); > list.add(surname); > return list; > } > public void setList(LinkedList list) { > name = (String)list.get(0); > surname = (String)list.get(1); > } -- 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
