From: jia.shao.peng <jia.shao.peng@ee073f10-1060-11df-b6a4-87a95322a99c>
Date: Tue, 25 May 2010 13:13:26 +0000 (+0000)
Subject: Fix a bug in AsYouTypeFormatter where the rememberedPosition is incorrect when a... 
X-Git-Tag: upstream/5.3.2~473
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=44ad231c0c4284bb78859472946502b28bb982cb;p=platform%2Fupstream%2Flibphonenumber.git

Fix a bug in AsYouTypeFormatter where the rememberedPosition is incorrect when a number is entered with more digits than correct.

git-svn-id: http://libphonenumber.googlecode.com/svn/trunk@27 ee073f10-1060-11df-b6a4-87a95322a99c
---

diff --git a/java/src/com/google/i18n/phonenumbers/AsYouTypeFormatter.java b/java/src/com/google/i18n/phonenumbers/AsYouTypeFormatter.java
index 95fcecdb..32310fde 100644
--- a/java/src/com/google/i18n/phonenumbers/AsYouTypeFormatter.java
+++ b/java/src/com/google/i18n/phonenumbers/AsYouTypeFormatter.java
@@ -192,6 +192,7 @@ public class AsYouTypeFormatter {
    */
   public String inputDigit(char nextChar) {
     accruedInput.append(nextChar);
+    rememberPosition();
     if (UNSUPPORTED_SYNTAX.matcher(Character.toString(nextChar)).matches()) {
       ableToFormat = false;
     }
@@ -200,7 +201,6 @@ public class AsYouTypeFormatter {
         positionRemembered = originalPosition;
         currentOutput.setLength(0);
       }
-      rememberPosition();
       return accruedInput.toString();
     }
 
@@ -215,19 +215,20 @@ public class AsYouTypeFormatter {
       case 3:
       case 4:
       case 5:
-        rememberPosition();
         return accruedInput.toString();
       case 6:
         if (!extractIddAndValidCountryCode()) {
           ableToFormat = false;
-          rememberPosition();
           return accruedInput.toString();
         }
         removeNationalPrefixFromNationalNumber();
         return attemptToChooseFormattingPattern();
       default:
         if (nationalNumber.length() > 4) {  // The formatting pattern is already chosen.
-          return prefixBeforeNationalNumber + inputDigitHelper(nextChar);
+          String temp = inputDigitHelper(nextChar);
+          return ableToFormat
+              ? prefixBeforeNationalNumber + temp
+              : temp;
         } else {
           return attemptToChooseFormattingPattern();
         }
@@ -288,8 +289,10 @@ public class AsYouTypeFormatter {
           positionRemembered = temp.length();
         }
       }
-      return prefixBeforeNationalNumber +
-          inputDigitHelper(nationalNumber.charAt(lengthOfNationalNumber - 1));
+      String temp = inputDigitHelper(nationalNumber.charAt(lengthOfNationalNumber - 1));
+      return ableToFormat
+          ? prefixBeforeNationalNumber + temp
+          : temp;
     } else {
       if (rememberPosition) {
         positionRemembered = prefixBeforeNationalNumber.length();
@@ -395,10 +398,11 @@ public class AsYouTypeFormatter {
     } else {  // More digits are entered than we could handle.
       currentOutput.append(nextChar);
       ableToFormat = false;
-      if (rememberPosition) {
-        positionRemembered = prefixBeforeNationalNumber.length() + currentOutput.length();
+      if (positionRemembered > 0) {
+        positionRemembered = originalPosition;
+        currentOutput.setLength(0);
       }
-      return currentOutput.toString();
+      return accruedInput.toString();
     }
   }
 }
diff --git a/java/test/com/google/i18n/phonenumbers/AsYouTypeFormatterTest.java b/java/test/com/google/i18n/phonenumbers/AsYouTypeFormatterTest.java
index fda9c798..6a039df4 100644
--- a/java/test/com/google/i18n/phonenumbers/AsYouTypeFormatterTest.java
+++ b/java/test/com/google/i18n/phonenumbers/AsYouTypeFormatterTest.java
@@ -85,6 +85,10 @@ public class AsYouTypeFormatterTest extends TestCase {
     assertEquals(13, formatter.getRememberedPosition());
     assertEquals("1 650 253 2222", formatter.inputDigit('2'));
     assertEquals(13, formatter.getRememberedPosition());
+    assertEquals("165025322222", formatter.inputDigit('2'));
+    assertEquals(10, formatter.getRememberedPosition());
+    assertEquals("1650253222222", formatter.inputDigit('2'));
+    assertEquals(10, formatter.getRememberedPosition());
 
     formatter.clear();
     assertEquals("6", formatter.inputDigit('6'));