One more: Abstract select assumed Property is always ValueChangeNotifier (bug in...

svn changeset:4029/svn branch:trunk

diff --git a/src/com/itmill/toolkit/ui/AbstractSelect.java b/src/com/itmill/toolkit/ui/AbstractSelect.java
index 9c0523e02ed88816746d571852b351bf8d4e6959..bda24b1ecf031157912df2394ddfcb3298635930 100644 (file)
--- a/src/com/itmill/toolkit/ui/AbstractSelect.java
+++ b/src/com/itmill/toolkit/ui/AbstractSelect.java
@@ -1621,7 +1621,7 @@ public abstract class AbstractSelect extends AbstractField implements
                     for (Iterator it = pids.iterator(); it.hasNext();) {
                         Property p = i.getItemProperty(it.next());
                         if (p != null
-                                || p instanceof Property.ValueChangeNotifier) {
+                                && p instanceof Property.ValueChangeNotifier) {
                             ((Property.ValueChangeNotifier) p)
                                     .addListener(getCaptionChangeListener());
                             captionChangeNotifiers.add(p);