fixes #2933, regression from #2823

svn changeset:7750/svn branch:6.0

diff --git a/src/com/vaadin/ui/Table.java b/src/com/vaadin/ui/Table.java
index de1e744da265239724289894d1388a9db608ed37..426560c4a889af824d3d7e97079bf271b7b5905f 100644 (file)
--- a/src/com/vaadin/ui/Table.java
+++ b/src/com/vaadin/ui/Table.java
@@ -1422,6 +1422,26 @@ public class Table extends AbstractSelect implements Action.Container,
 
                                 } else if (iscomponent[j]) {
                                     value = p.getValue();
+                                    if (p instanceof Property.ValueChangeNotifier) {
+                                        /*
+                                         * Component in property may change ->
+                                         * listen value changes events.
+                                         */
+                                        if (oldListenedProperties == null
+                                                || !oldListenedProperties
+                                                        .contains(p)) {
+                                            ((Property.ValueChangeNotifier) p)
+                                                    .addListener(this);
+                                        }
+                                        /*
+                                         * register listened properties, so we
+                                         * can do proper cleanup to free memory.
+                                         * Essential if table has loads of data
+                                         * and it is used for a long time.
+                                         */
+                                        listenedProperties.add(p);
+
+                                    }
                                 } else if (p != null) {
                                     value = getPropertyValue(id, colids[j], p);
                                     /*