From dd13fff6aa69982bb1cfc470d20ef85188d383ac Mon Sep 17 00:00:00 2001
From: Marc Englund <marc.englund@itmill.com>
Date: Wed, 12 Mar 2008 07:26:00 +0000
Subject: [PATCH] One more: Abstract select assumed Property is always
 ValueChangeNotifier (bug in if -clause)

svn changeset:4029/svn branch:trunk
---
 src/com/itmill/toolkit/ui/AbstractSelect.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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);
-- 
2.39.5