From 66b1aa488099e8e79c3fb2fb1f0236c29f15f848 Mon Sep 17 00:00:00 2001
From: Artur Signell <artur@vaadin.com>
Date: Wed, 21 Nov 2012 17:06:57 +0200
Subject: Account for that converter can modify value (#8191)

Change-Id: Iee81b46c985bbb37b98d23433e42d8d007c30d6a
---
 server/src/com/vaadin/ui/AbstractField.java | 10 ++++++++++
 1 file changed, 10 insertions(+)

(limited to 'server')

diff --git a/server/src/com/vaadin/ui/AbstractField.java b/server/src/com/vaadin/ui/AbstractField.java
index 5c4a3468bb..b52d733632 100644
--- a/server/src/com/vaadin/ui/AbstractField.java
+++ b/server/src/com/vaadin/ui/AbstractField.java
@@ -451,6 +451,16 @@ public abstract class AbstractField<T> extends AbstractComponent implements
             if (isReadOnly()) {
                 throw new Property.ReadOnlyException();
             }
+            try {
+                T doubleConvertedFieldValue = convertFromDataSource(convertToModel(newFieldValue));
+                if (!equals(newFieldValue, doubleConvertedFieldValue)) {
+                    newFieldValue = doubleConvertedFieldValue;
+                    repaintIsNotNeeded = false;
+                }
+            } catch (Throwable t) {
+                // Ignore exceptions in the conversion at this stage. Any
+                // conversion error will be handled later by validate().
+            }
 
             // Repaint is needed even when the client thinks that it knows the
             // new state if validity of the component may change
-- 
cgit v1.2.3