From: Marc Englund <marc.englund@itmill.com>
Date: Wed, 12 Mar 2008 07:26:00 +0000 (+0000)
Subject: One more: Abstract select assumed Property is always ValueChangeNotifier (bug in... 
X-Git-Tag: 6.7.0.beta1~4941
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=dd13fff6aa69982bb1cfc470d20ef85188d383ac;p=vaadin-framework.git

One more: Abstract select assumed Property is always ValueChangeNotifier (bug in if -clause)

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 9c0523e02e..bda24b1ecf 100644
--- 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);