Fix for #3948 - Document that Container.Hierarchical.removeItem should not remove...

svn changeset:10652/svn branch:6.2

diff --git a/src/com/vaadin/ui/AbstractSelect.java b/src/com/vaadin/ui/AbstractSelect.java
index b8ef6bc2067b55c507eb3ece62d8fae12ce892d3..4b5ef4a0a8bca40a425da9ac41efedafb42c45f7 100644 (file)
--- a/src/com/vaadin/ui/AbstractSelect.java
+++ b/src/com/vaadin/ui/AbstractSelect.java
@@ -793,12 +793,9 @@ public abstract class AbstractSelect extends AbstractField implements
         return retval;
     }
 
-    /**
-     * Removes the item identified by Id from the container. This functionality
-     * is optional. If the function is not implemented, the functions allways
-     * returns false.
+    /*
+     * (non-Javadoc)
      * 
-     * @return True if the operation succeeded.
      * @see com.vaadin.data.Container#removeItem(java.lang.Object)
      */
     public boolean removeItem(Object itemId)

diff --git a/src/com/vaadin/ui/Tree.java b/src/com/vaadin/ui/Tree.java
index 509d83a1532bcbdaccd837efa56414438fc0212b..28a410fa137f851ff715f53339def3161c09c461 100644 (file)
--- a/src/com/vaadin/ui/Tree.java
+++ b/src/com/vaadin/ui/Tree.java
@@ -1068,4 +1068,11 @@ public class Tree extends AbstractSelect implements Container.Hierarchical,
         public abstract String getStyle(Object itemId);
     }
 
+    // Overriden so javadoc comes from Container.Hierarchical
+    @Override
+    public boolean removeItem(Object itemId)
+            throws UnsupportedOperationException {
+        return super.removeItem(itemId);
+    }
+
 }