[richfaces-issues] [JBoss JIRA] Work started: (RF-3899) Tree component doesn´t work with adviseNodeOpened, immediate="true" and switchType="ajax" or switchType="server"

Tree component doesn´t work with adviseNodeOpened, immediate="true" and switchType="ajax" or switchType="server" ---------------------------------------------------------------------------------------------------------------- Key: RF-3899 URL: https://jira.jboss.org/jira/browse/RF-3899 Project: RichFaces Issue Type: Bug Affects Versions: 3.2.1 Environment: BEA Weblogic 9.2 Sun Facelets 1.1.14 Microsoft Internet Explorer 6.0.2900.2180 and Mozilla Firefox 2.0.0.15. Reporter: Mauricio Noda Assignee: Nick Belaevski Fix For: 3.2.2 Attachments: TreeState.java Tree component renders the tree incorrectly when adviseNodeOpened returns false in an initial request, but returns true in a subsequent request. The tree node expand icon shows as expanded, but the node children are not rendered. Only happens when using immediate="true" and switchType set as "ajax" or "server". Works fine in client mode or when immediate is set to false. Works but triggers unwanted validation errors when expanding/collapsing nodes: <rich:tree adviseNodeOpened="#{bean.adviseNodeOpened}" changeExpandListener="#{bean.changeExpandListener}"> <!-- tree content --> </rich:tree> Works but is too slow with large trees: <rich:tree immediate="true" switchType="client" adviseNodeOpened="#{bean.adviseNodeOpened}" changeExpandListener="#{bean.changeExpandListener}"> <!-- tree content --> </rich:tree> Initial request works, but subsequent requests doesn´t if adviseNodeOpened return changes: <rich:tree immediate="true" adviseNodeOpened="#{bean.adviseNodeOpened}" changeExpandListener="#{bean.changeExpandListener}"> <!-- tree content --> </rich:tree> After some bugtracking, I found this: Richfaces 3.2.1, inside org.richfaces.component.state.TreeState class, line 156, "return expandedNodes.contains(rowKey);" is ignoring expanded nodes queued in line 72 "queuedExpandedNodes.add(nextKey);" since it runs before line 278 "expandedNodes.addAll(queuedExpandedNodes);".