setContainerDataSource() now accepts null. Fixes #1971.

svn changeset:5224/svn branch:trunk

diff --git a/src/com/itmill/toolkit/ui/Tree.java b/src/com/itmill/toolkit/ui/Tree.java
index b833930be736b24231bf8ebe4a67cc2798a8e508..35c5f1dacacb3a11b952263d4c92aad095aa738e 100644 (file)
--- a/src/com/itmill/toolkit/ui/Tree.java
+++ b/src/com/itmill/toolkit/ui/Tree.java
@@ -19,6 +19,7 @@ import java.util.StringTokenizer;
 
 import com.itmill.toolkit.data.Container;
 import com.itmill.toolkit.data.util.ContainerHierarchicalWrapper;
+import com.itmill.toolkit.data.util.IndexedContainer;
 import com.itmill.toolkit.event.Action;
 import com.itmill.toolkit.terminal.KeyMapper;
 import com.itmill.toolkit.terminal.PaintException;
@@ -659,6 +660,12 @@ public class Tree extends AbstractSelect implements Container.Hierarchical,
      * @see com.itmill.toolkit.data.Container.Viewer#setContainerDataSource(Container)
      */
     public void setContainerDataSource(Container newDataSource) {
+        if (newDataSource == null) {
+            // Note: using wrapped IndexedContainer to match constructor (super
+            // creates an IndexedContainer, which is then wrapped).
+            newDataSource = new ContainerHierarchicalWrapper(
+                    new IndexedContainer());
+        }
 
         // Assure that the data source is ordered by making unordered
         // containers ordered by wrapping them