From b34500197a833080ae7ab25898207fa96ee016e6 Mon Sep 17 00:00:00 2001
From: Mika Murtojärvi <mika@vaadin.com>
Date: Wed, 17 Jun 2015 07:56:55 +0000
Subject: Revert "Prevent field from updating when removing text change
 listener. (#16270)"

This reverts commit 4af793d06a0f4a6577aad13403ca7982c6fce224.

Test ConverterThatEnforcesAFormatTest.checkElaborating is broken by this
change.

Change-Id: I08dba8e47386dfb001caafae562304858ad02348
---
 server/src/com/vaadin/ui/AbstractTextField.java | 33 ++++++++++++++-----------
 1 file changed, 18 insertions(+), 15 deletions(-)

(limited to 'server/src')

diff --git a/server/src/com/vaadin/ui/AbstractTextField.java b/server/src/com/vaadin/ui/AbstractTextField.java
index 14c135228c..93025ac0fd 100644
--- a/server/src/com/vaadin/ui/AbstractTextField.java
+++ b/server/src/com/vaadin/ui/AbstractTextField.java
@@ -126,22 +126,25 @@ public abstract class AbstractTextField extends AbstractField<String> implements
             selectionPosition = -1;
         }
 
-        target.addAttribute(TextFieldConstants.ATTR_TEXTCHANGE_EVENTMODE,
-                getTextChangeEventMode().toString());
-        target.addAttribute(TextFieldConstants.ATTR_TEXTCHANGE_TIMEOUT,
-                getTextChangeTimeout());
-        if (lastKnownTextContent != null) {
-            /*
-             * The field has be repainted for some reason (e.g. caption, size,
-             * stylename), but the value has not been changed since the last
-             * text change event. Let the client side know about the value the
-             * server side knows. Client side may then ignore the actual value,
-             * depending on its state.
-             */
-            target.addAttribute(
-                    TextFieldConstants.ATTR_NO_VALUE_CHANGE_BETWEEN_PAINTS,
-                    true);
+        if (hasListeners(TextChangeEvent.class)) {
+            target.addAttribute(TextFieldConstants.ATTR_TEXTCHANGE_EVENTMODE,
+                    getTextChangeEventMode().toString());
+            target.addAttribute(TextFieldConstants.ATTR_TEXTCHANGE_TIMEOUT,
+                    getTextChangeTimeout());
+            if (lastKnownTextContent != null) {
+                /*
+                 * The field has be repainted for some reason (e.g. caption,
+                 * size, stylename), but the value has not been changed since
+                 * the last text change event. Let the client side know about
+                 * the value the server side knows. Client side may then ignore
+                 * the actual value, depending on its state.
+                 */
+                target.addAttribute(
+                        TextFieldConstants.ATTR_NO_VALUE_CHANGE_BETWEEN_PAINTS,
+                        true);
+            }
         }
+
     }
 
     @Override
-- 
cgit v1.2.3