From 729f5e61603ccdab7c0e3192b2e70ece6047b5ce Mon Sep 17 00:00:00 2001
From: Automerge <automerge@vaadin.com>
Date: Wed, 29 Feb 2012 14:14:27 +0000
Subject: [PATCH] [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
---
 src/com/vaadin/ui/Table.java | 16 +---------------
 1 file changed, 1 insertion(+), 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();
     }
 
-- 
2.39.5