From 395dad4cd7a854314e42dca2eb9dd2879ba53de8 Mon Sep 17 00:00:00 2001
From: Anna Koskinen <anna@vaadin.com>
Date: Fri, 21 Dec 2012 16:53:54 +0200
Subject: [PATCH] Merge of (#9529) to Vaadin 7.

Limit unnecessary database queries.

Change-Id: Iae2ed72dd2d044f02abe775f2289126c5f5dbe1d
---
 .../data/util/sqlcontainer/SQLContainer.java  | 23 +++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/server/src/com/vaadin/data/util/sqlcontainer/SQLContainer.java b/server/src/com/vaadin/data/util/sqlcontainer/SQLContainer.java
index 12cb892090..065810a7b4 100644
--- a/server/src/com/vaadin/data/util/sqlcontainer/SQLContainer.java
+++ b/server/src/com/vaadin/data/util/sqlcontainer/SQLContainer.java
@@ -854,7 +854,21 @@ public class SQLContainer implements Container, Container.Filterable,
      * Does NOT remove sorting or filtering rules!
      */
     public void refresh() {
-        sizeDirty = true;
+        refresh(true);
+    }
+
+    /**
+     * Refreshes the container. If <code>setSizeDirty</code> is
+     * <code>false</code>, assumes that the current size is up to date. This is
+     * used in {@link #updateCount()} to refresh the contents when we know the
+     * size was just updated.
+     * 
+     * @param setSizeDirty
+     */
+    private void refresh(boolean setSizeDirty) {
+        if (setSizeDirty) {
+            sizeDirty = true;
+        }
         currentOffset = 0;
         cachedItems.clear();
         itemIndexes.clear();
@@ -1105,12 +1119,13 @@ public class SQLContainer implements Container, Container.Filterable,
                         "The query delegate doesn't support filtering", e);
             }
             int newSize = delegate.getCount();
+            sizeUpdated = new Date();
+            sizeDirty = false;
             if (newSize != size) {
                 size = newSize;
-                refresh();
+                // Size is up to date so don't set it back to dirty in refresh()
+                refresh(false);
             }
-            sizeUpdated = new Date();
-            sizeDirty = false;
             getLogger().log(Level.FINER,
                     "Updated row count. New count is: " + size);
         } catch (SQLException e) {
-- 
2.39.5