[merge from 6.7] Fixed #8077: removed a buggy optimization that prevented the client-side row cache from being updated when adding/removing rows that are inside the cache area. This only occurred with Container.Indexed containers.

svn changeset:23133/svn branch:6.8

1 files changed, 1 insertions, 15 deletions

diff --git a/src/com/vaadin/ui/Table.java b/src/com/vaadin/ui/Table.java
index 199a6805f6..0f6c1f3be8 100644
--- a/src/com/vaadin/ui/Table.java
+++ b/src/com/vaadin/ui/Table.java
@@ -3839,22 +3839,8 @@ public class Table extends AbstractSelect implements Action.Container,
     @Override
     public void containerItemSetChange(Container.ItemSetChangeEvent event) {
         super.containerItemSetChange(event);
-        if (event instanceof IndexedContainer.ItemSetChangeEvent) {
-            IndexedContainer.ItemSetChangeEvent evt = (IndexedContainer.ItemSetChangeEvent) event;
-            // if the event is not a global one and the added item is outside
-            // the visible/buffered area, no need to do anything
-            if (evt.getAddedItemIndex() != -1
-                    && (firstToBeRenderedInClient >= 0)
-                    && (lastToBeRenderedInClient >= 0)
-                    && (firstToBeRenderedInClient > evt.getAddedItemIndex() || lastToBeRenderedInClient < evt
-                            .getAddedItemIndex())) {
-                return;
-            }
-        }
-        // ensure that page still has first item in page, ignore buffer refresh
-        // (forced in this method)
-        setCurrentPageFirstItemIndex(getCurrentPageFirstItemIndex(), false);
 
+        setCurrentPageFirstItemIndex(getCurrentPageFirstItemIndex(), false);
         refreshRowCache();
     }