From 0973197b8292b2aec79b6f1a960f70ec99ef497f Mon Sep 17 00:00:00 2001
From: Matti Tahvonen <matti.tahvonen@itmill.com>
Date: Thu, 14 Aug 2008 07:52:11 +0000
Subject: [PATCH] Table: optimizing one extra cache build on container change

svn changeset:5182/svn branch:trunk
---
 src/com/itmill/toolkit/ui/Table.java | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/src/com/itmill/toolkit/ui/Table.java b/src/com/itmill/toolkit/ui/Table.java
index d5ab6120c3..fb11b165c7 100644
--- a/src/com/itmill/toolkit/ui/Table.java
+++ b/src/com/itmill/toolkit/ui/Table.java
@@ -398,7 +398,7 @@ public class Table extends AbstractSelect implements Action.Container,
 
         // Removes alignments, icons and headers from hidden columns
         if (this.visibleColumns != null) {
-            disableContentRefreshing();
+            boolean disabledHere = disableContentRefreshing();
             try {
                 for (final Iterator i = this.visibleColumns.iterator(); i
                         .hasNext();) {
@@ -410,7 +410,9 @@ public class Table extends AbstractSelect implements Action.Container,
                     }
                 }
             } finally {
-                enableContentRefreshing(false);
+                if (disabledHere) {
+                    enableContentRefreshing(false);
+                }
             }
         }
 
@@ -1495,6 +1497,8 @@ public class Table extends AbstractSelect implements Action.Container,
      */
     public void setContainerDataSource(Container newDataSource) {
 
+        disableContentRefreshing();
+
         if (newDataSource == null) {
             newDataSource = new IndexedContainer();
         }
@@ -1538,7 +1542,8 @@ public class Table extends AbstractSelect implements Action.Container,
 
         // Assure visual refresh
         resetPageBuffer();
-        refreshRenderedCells();
+        enableContentRefreshing(true);
+
     }
 
     /* Component basics ***************************************************** */
@@ -1690,9 +1695,12 @@ public class Table extends AbstractSelect implements Action.Container,
      * bypass expensive content for some reason (like when we know we may have
      * other content changes on their way).
      * 
+     * @return true if content refresh flag was enabled prior this call
      */
-    protected void disableContentRefreshing() {
+    protected boolean disableContentRefreshing() {
+        boolean wasDisabled = isContentRefreshesEnabled;
         isContentRefreshesEnabled = false;
+        return wasDisabled;
     }
 
     /**
-- 
2.39.5