[ https://issues.apache.org/jira/browse/MYFACES-4036?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15298713#comment-15298713 ]
Leonardo Uribe commented on MYFACES-4036: ----------------------------------------- Thinking about this issue and reading the comments, I have realized there is something wrong in the current UIData implementation. The problem is call saveState(...) and restoreState(...) at the end on invoke application phase could lead to inconsistencies. The reason is there is some information in _rowStates that is not saved with the state, so when the state is restored in the render request, that information is lost, which means the input values are not persisted when validation fails. In other words, UIData state saving algorithm must be stable under this scenario (action request - render request). I need some time to think about it and fix it, so I'll answer the question about how to do it properly when I have solved this issue. > UIData state is not restorable when rowStatePreserved is set to true > -------------------------------------------------------------------- > > Key: MYFACES-4036 > URL: https://issues.apache.org/jira/browse/MYFACES-4036 > Project: MyFaces Core > Issue Type: Bug > Affects Versions: 2.2.10-SNAPSHOT > Environment: WebSphere Liberty 8.5.5.8, MyFaces 2.2.8, IBM JSF > Portlet Bridge 1.0 > Reporter: Hank Ibell > Priority: Minor > > A JSF portlet with input fields inside a h:dataTable component will lose > their local values if validation fails, even when rowStatePreserved is set to > true. > The IBM JSF portlet bridge calls processSaveState() on UIViewRoot so that the > bridge can restore the view state on their RENDER request. After looking at > UIData, it looks like _initialDescendantFullComponentState is lost between > Portal's ACTION and RENDER requests (a new UIData object is created on the > RENDER request). Unfortunately, UIData's _rowDeltaStates will not be restored > without its initial state. > It looks like UIData should also save/restore > _initialDescendantFullComponentState in its saveSate()/restoreState() methods > since it may be needed to restore _rowDeltaStates. -- This message was sent by Atlassian JIRA (v6.3.4#6332)