[ http://issues.apache.org/jira/browse/MYFACES-462?page=all ] Mathias Broekelmann reopened MYFACES-462: -----------------------------------------
Just found in the spec: Java-Doc for javax.faces.component.Stateholder.isTransient() -- snip -- If true, the Object implementing this interface must not participate in state saving or restoring. -- snap -- I think we should follow this and revert it. At least make it consistent with processRestoreState which also checks the transient flag. > UIComponent.processSaveState implementation not correct > ------------------------------------------------------- > > Key: MYFACES-462 > URL: http://issues.apache.org/jira/browse/MYFACES-462 > Project: MyFaces > Type: Bug > Components: JSR-127 > Versions: Nightly Build > Reporter: Oliver Rossmueller > Assignee: Oliver Rossmueller > Fix For: Nightly Build > > I think the test for child.isTransient in line 449 of UIComponentBase has to > be removed as the javadoc says > * Call the processSaveState() method of all facets and children of > this UIComponent in the order determined by a call to getFacetsAndChildren(). > There is no "unless the facet of child is transient" so processSaveState has > to be called for all facets/children whether they are transient or not -- 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
