From: Matti Tahvonen <matti.tahvonen@itmill.com>
Date: Tue, 13 Jan 2009 13:17:00 +0000 (+0000)
Subject: fixes #2456
X-Git-Tag: 6.7.0.beta1~3344
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=9d0c94a56d7e1c26a3e39cebc61b2c3bc635c8ef;p=vaadin-framework.git

fixes #2456

svn changeset:6520/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/ui/Table.java b/src/com/itmill/toolkit/ui/Table.java
index d5b8b81ce8..2b528780ca 100644
--- a/src/com/itmill/toolkit/ui/Table.java
+++ b/src/com/itmill/toolkit/ui/Table.java
@@ -2484,7 +2484,13 @@ public class Table extends AbstractSelect implements Action.Container,
                     "Can not add the same GeneratedColumn twice, id:" + id);
         } else {
             columnGenerators.put(id, generatedColumn);
-            visibleColumns.add(id);
+            /*
+             * add to visible column list unless already there (overriding
+             * column from DS)
+             */
+            if (!visibleColumns.contains(id)) {
+                visibleColumns.add(id);
+            }
             resetPageBuffer();
             refreshRenderedCells();
         }