From c378046be94dae66133122665ba30c09d0522e21 Mon Sep 17 00:00:00 2001
From: Artur Signell <artur.signell@itmill.com>
Date: Tue, 1 Sep 2009 08:25:54 +0000
Subject: [PATCH] Fixed #3177 - PropertyFormatter NPE

svn changeset:8604/svn branch:6.1
---
 src/com/vaadin/data/util/PropertyFormatter.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/com/vaadin/data/util/PropertyFormatter.java b/src/com/vaadin/data/util/PropertyFormatter.java
index 616c1beb0a..07a39d83f8 100644
--- a/src/com/vaadin/data/util/PropertyFormatter.java
+++ b/src/com/vaadin/data/util/PropertyFormatter.java
@@ -118,7 +118,7 @@ public abstract class PropertyFormatter implements Property,
         }
         String newVal = toString();
         if ((prevValue == null && newVal != null)
-                || !prevValue.equals(prevValue)) {
+                || (prevValue != null && !prevValue.equals(newVal))) {
             fireValueChange();
         }
     }
@@ -362,8 +362,7 @@ public abstract class PropertyFormatter implements Property,
      * 
      * This should not be called directly.
      */
-    public void valueChange(
-            com.vaadin.data.Property.ValueChangeEvent event) {
+    public void valueChange(com.vaadin.data.Property.ValueChangeEvent event) {
         fireValueChange();
     }
 
-- 
2.39.5