From f7be9bb1724c848e285dc588557c89b7f3373fbf Mon Sep 17 00:00:00 2001
From: Artur Signell <artur.signell@itmill.com>
Date: Fri, 13 Feb 2009 07:30:24 +0000
Subject: [PATCH] Fix for #2585 - NullPointerException when detaching an item
 in HierarchicalContainer

svn changeset:6826/svn branch:trunk
---
 src/com/itmill/toolkit/data/util/HierarchicalContainer.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/com/itmill/toolkit/data/util/HierarchicalContainer.java b/src/com/itmill/toolkit/data/util/HierarchicalContainer.java
index c702c14795..ce46da362a 100644
--- a/src/com/itmill/toolkit/data/util/HierarchicalContainer.java
+++ b/src/com/itmill/toolkit/data/util/HierarchicalContainer.java
@@ -167,7 +167,7 @@ public class HierarchicalContainer extends IndexedContainer implements
 
         // Checks if no change is necessary
         if ((newParentId == null && oldParentId == null)
-                || newParentId.equals(oldParentId)) {
+                || ((newParentId != null) && newParentId.equals(oldParentId))) {
             return true;
         }
 
-- 
2.39.5