#3548 - Table.setColumnCollapsed should not throw IllegalAccessException

svn changeset:15536/svn branch:6.4

diff --git a/src/com/vaadin/ui/Table.java b/src/com/vaadin/ui/Table.java
index 775470e04833bdcc0fdde2a5c59fd68ed0e23909..11420e9e993083540086f74a51fd0ecd3fb90e7a 100644 (file)
--- a/src/com/vaadin/ui/Table.java
+++ b/src/com/vaadin/ui/Table.java
@@ -24,9 +24,9 @@ import com.vaadin.data.Property;
 import com.vaadin.data.util.ContainerOrderedWrapper;
 import com.vaadin.data.util.IndexedContainer;
 import com.vaadin.event.Action;
-import com.vaadin.event.Action.Handler;
 import com.vaadin.event.DataBoundTransferable;
 import com.vaadin.event.ItemClickEvent;
+import com.vaadin.event.Action.Handler;
 import com.vaadin.event.ItemClickEvent.ItemClickListener;
 import com.vaadin.event.ItemClickEvent.ItemClickSource;
 import com.vaadin.event.MouseEvents.ClickEvent;
@@ -1108,12 +1108,12 @@ public class Table extends AbstractSelect implements Action.Container,
      *            the propertyID identifying the column.
      * @param collapsed
      *            the desired collapsedness.
-     * @throws IllegalAccessException
+     * @throws IllegalStateException
+     *             if column collapsing is not allowed
      */
-    public void setColumnCollapsed(Object propertyId, boolean collapsed)
-            throws IllegalAccessException {
+    public void setColumnCollapsed(Object propertyId, boolean collapsed) {
         if (!isColumnCollapsingAllowed()) {
-            throw new IllegalAccessException("Column collapsing not allowed!");
+            throw new IllegalStateException("Column collapsing is not allowed!");
         }
 
         if (collapsed) {
@@ -1855,7 +1855,7 @@ public class Table extends AbstractSelect implements Action.Container,
         HashSet<Object> ids = new HashSet<Object>();
         for (int i = 0; i < length; i++) {
             assert itemId != null; // should not be null unless client-server
-                                   // are out of sync
+            // are out of sync
             ids.add(itemId);
             itemId = nextItemId(itemId);
         }