Applied patch provided for #8600 (hasChildren() broken if item used to have children) and added junit test. Fixes #8600

svn changeset:23624/svn branch:6.8

src/com/vaadin/data/util/ContainerHierarchicalWrapper.java

@@ -294,7 +294,8 @@ public class ContainerHierarchicalWrapper implements Container.Hierarchical,
             return ((Container.Hierarchical) container).hasChildren(itemId);
         }
 
-        return children.get(itemId) != null;
+        LinkedList<Object> list = children.get(itemId);
+        return (list != null && !list.isEmpty());
     }
 
     /*

tests/server-side/com/vaadin/tests/server/component/tree/TestHasChildren.java

@@ -0,0 +1,25 @@
+package com.vaadin.tests.server.component.tree;
+
+import junit.framework.TestCase;
+
+import com.vaadin.ui.Tree;
+
+public class TestHasChildren extends TestCase {
+
+    private Tree tree;
+
+    @Override
+    protected void setUp() {
+        tree = new Tree();
+        tree.addItem("parent");
+        tree.addItem("child");
+        tree.setChildrenAllowed("parent", true);
+        tree.setParent("child", "parent");
+    }
+
+    public void testRemoveChildren() {
+        assertTrue(tree.hasChildren("parent"));
+        tree.removeItem("child");
+        assertFalse(tree.hasChildren("parent"));
+    }
+}