Fix for #6588

svn changeset:19527/svn branch:6.6

diff --git a/src/com/vaadin/ui/AbstractTextField.java b/src/com/vaadin/ui/AbstractTextField.java
index 4ed76d367bdf8a3573d3e3cb42edde37a23a42f4..622b8d18d9fdedc05c040a547135a4bcd198e4da 100644 (file)
--- a/src/com/vaadin/ui/AbstractTextField.java
+++ b/src/com/vaadin/ui/AbstractTextField.java
@@ -1,4 +1,4 @@
-/* 
+/*
 @ITMillApache2LicenseForJavaFiles@
  */
 
@@ -123,7 +123,20 @@ public abstract class AbstractTextField extends AbstractField implements
             throw new IllegalStateException(
                     "Null values are not allowed if the null-representation is null");
         }
-        target.addVariable(this, "text", value);
+
+        if (requestRepaintInTextChangeEvent && !valueChangeInTextChangeEvent) {
+            /*
+             * If the repaint occurred in a text change event then we do not
+             * want to send back the old value since it will just overwrite the
+             * typed value so we send the last known value instead which is
+             * updated by the text change event.
+             */
+            target.addVariable(this, "text", lastKnownTextContent);
+        } else {
+            target.addVariable(this, "text", value);
+        }
+        requestRepaintInTextChangeEvent = false;
+        valueChangeInTextChangeEvent = false;
 
         if (selectionPosition != -1) {
             target.addAttribute("selpos", selectionPosition);
@@ -171,6 +184,22 @@ public abstract class AbstractTextField extends AbstractField implements
         }
     }
 
+    /**
+     * Flag for monitoring if a repaint gets requested in a text change event
+     */
+    private boolean requestRepaintInTextChangeEvent = false;
+
+    @Override
+    public void requestRepaint() {
+        if (textChangeEventPending) {
+            /*
+             * Textchange event listener triggered this repaint
+             */
+            requestRepaintInTextChangeEvent = true;
+        }
+        super.requestRepaint();
+    }
+
     @Override
     public void changeVariables(Object source, Map<String, Object> variables) {
         changingVariables = true;
@@ -438,13 +467,26 @@ public abstract class AbstractTextField extends AbstractField implements
 
     private void firePendingTextChangeEvent() {
         if (textChangeEventPending) {
-            textChangeEventPending = false;
             fireEvent(new TextChangeEventImpl(this));
+            textChangeEventPending = false;
         }
     }
 
+    /**
+     * Flag for monitoring if the value got changed in a TextChangeEvent
+     * listener
+     */
+    protected boolean valueChangeInTextChangeEvent = false;
+
     @Override
     protected void setInternalValue(Object newValue) {
+        if (textChangeEventPending) {
+            /*
+             * Value changed in a TextChangeEvent listener
+             */
+            valueChangeInTextChangeEvent = true;
+        }
+
         if (changingVariables && !textChangeEventPending) {
             /*
              * Fire a "simulated" text change event before value change event if