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

[Mauricio Noda (JIRA)]

Tree component doesn´t work with adviseNodeOpened, immediate="true" and switchType="ajax" or switchType="server"
----------------------------------------------------------------------------------------------------------------

                 Key: RF-3899
                 URL: http://jira.jboss.com/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


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);".

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