Changeset 4949

Timestamp:

06/24/08 14:21:11 (5 months ago)

Author:

marc.englund@…

Message:

Table now refreshes explicitly when adding and removing stuff and the contained does not implement the appropriate notifier interface. Fixes #1461.

Files:

• trunk/src/com/itmill/toolkit/ui/Table.java (modified) (5 diffs)

trunk/src/com/itmill/toolkit/ui/Table.java

@@ -1450 +1450 @@
         }
 
+        if (!(items instanceof Container.ItemSetChangeNotifier)) {
+            resetPageBuffer();
+            refreshRenderedCells();
+        }
+
         return itemId;
     }
@@ -2193 +2198 @@
             currentPageFirstItemId = nextItemId;
         }
+        if (!(items instanceof Container.ItemSetChangeNotifier)) {
+            resetPageBuffer();
+            refreshRenderedCells();
+        }
         return ret;
     }
@@ -2239 +2248 @@
             }
             return false;
+        }
+        if (!(items instanceof Container.PropertySetChangeNotifier)) {
+            resetPageBuffer();
+            refreshRenderedCells();
         }
         return true;
@@ -2480 +2493 @@
     public Object addItemAfter(Object previousItemId)
             throws UnsupportedOperationException {
-        return ((Container.Ordered) items).addItemAfter(previousItemId);
+        Object itemId = ((Container.Ordered) items)
+                .addItemAfter(previousItemId);
+        if (!(items instanceof Container.ItemSetChangeNotifier)) {
+            resetPageBuffer();
+            refreshRenderedCells();
+        }
+        return itemId;
     }
 
@@ -2491 +2510 @@
     public Item addItemAfter(Object previousItemId, Object newItemId)
             throws UnsupportedOperationException {
-        return ((Container.Ordered) items).addItemAfter(previousItemId,
+        Item item = ((Container.Ordered) items).addItemAfter(previousItemId,
                 newItemId);
+        if (!(items instanceof Container.ItemSetChangeNotifier)) {
+            resetPageBuffer();
+            refreshRenderedCells();
+        }
+        return item;
     }