From: Artur Signell <artur.signell@itmill.com>
Date: Tue, 23 Feb 2010 08:30:01 +0000 (+0000)
Subject: Fix for #4215 and #4216 for ContainerHierarchicalWrapper
X-Git-Tag: 6.7.0.beta1~2026
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=40cddca975b102d8195704f63c4a841266f8179b;p=vaadin-framework.git

Fix for #4215 and #4216 for ContainerHierarchicalWrapper

svn changeset:11478/svn branch:6.3
---

diff --git a/src/com/vaadin/data/util/ContainerHierarchicalWrapper.java b/src/com/vaadin/data/util/ContainerHierarchicalWrapper.java
index 6420aaf838..72d6962736 100644
--- a/src/com/vaadin/data/util/ContainerHierarchicalWrapper.java
+++ b/src/com/vaadin/data/util/ContainerHierarchicalWrapper.java
@@ -217,7 +217,12 @@ public class ContainerHierarchicalWrapper implements Container.Hierarchical,
             return ((Container.Hierarchical) container)
                     .areChildrenAllowed(itemId);
         }
-        return !noChildrenAllowed.contains(itemId);
+
+        if (noChildrenAllowed.contains(itemId)) {
+            return false;
+        }
+
+        return containsId(itemId);
     }
 
     /*
@@ -281,7 +286,11 @@ public class ContainerHierarchicalWrapper implements Container.Hierarchical,
             return ((Container.Hierarchical) container).isRoot(itemId);
         }
 
-        return parent.get(itemId) == null;
+        if (parent.containsKey(itemId)) {
+            return false;
+        }
+
+        return containsId(itemId);
     }
 
     /*