You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by kn...@apache.org on 2006/11/03 16:52:33 UTC

svn commit: r470863 - /incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/tree/AbstractTree.java

Author: knopp
Date: Fri Nov  3 07:52:32 2006
New Revision: 470863

URL: http://svn.apache.org/viewvc?view=rev&rev=470863
Log:
Fixed bug where tree nodes were unselected when parent was rebuilt

Modified:
    incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/tree/AbstractTree.java

Modified: incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/tree/AbstractTree.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/tree/AbstractTree.java?view=diff&rev=470863&r1=470862&r2=470863
==============================================================================
--- incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/tree/AbstractTree.java (original)
+++ incubator/wicket/trunk/wicket/src/main/java/wicket/markup/html/tree/AbstractTree.java Fri Nov  3 07:52:32 2006
@@ -760,6 +760,8 @@
 						public void visitItem(TreeItem item)
 						{
 							removeItem(item);
+							// unselect the node
+							getTreeState().selectNode(item.getModelObject(), false);
 						}
 					});
 
@@ -1317,9 +1319,6 @@
 		// remove the id
 		// note that this doesn't update item's parent's children list
 		item.remove();
-
-		// unselect the node
-		getTreeState().selectNode(item.getModelObject(), false);
 	}
 
 	/**