Better fix for #5770 - Clearing a Table column footer/header does not repaint Table

svn changeset:15487/svn branch:6.4

diff --git a/src/com/vaadin/ui/Table.java b/src/com/vaadin/ui/Table.java
index 3c2611b0cc836554f8b9081c913483c5ad4ef25d..775470e04833bdcc0fdde2a5c59fd68ed0e23909 100644 (file)
--- a/src/com/vaadin/ui/Table.java
+++ b/src/com/vaadin/ui/Table.java
@@ -1031,10 +1031,9 @@ public class Table extends AbstractSelect implements Action.Container,
 
         if (header == null) {
             columnHeaders.remove(propertyId);
-            requestRepaint();
-            return;
+        } else {
+            columnHeaders.put(propertyId, header);
         }
-        columnHeaders.put(propertyId, header);
 
         // Assures the visual refresh
         // FIXME: Is this really needed? Header captions should not affect
@@ -3997,10 +3996,9 @@ public class Table extends AbstractSelect implements Action.Container,
     public void setColumnFooter(Object propertyId, String footer) {
         if (footer == null) {
             columnFooters.remove(propertyId);
-            requestRepaint();
-            return;
+        } else {
+            columnFooters.put(propertyId, footer);
         }
-        columnFooters.put(propertyId, footer);
 
         requestRepaint();
     }