svn commit: r1095467 - /tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/translator/BigTypesFormatter.java

[hl...@apache.org]

Author: hlship
Date: Wed Apr 20 17:41:59 2011
New Revision: 1095467

URL: http://svn.apache.org/viewvc?rev=1095467&view=rev
Log:
TAP5-875: BigTypesFormatter should allow for plain space as a grouping separator since FF2 replaces non-breaking space character \u00A0 (grouping separator for Sweden, Finland , etc) with normal space \u0020

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/translator/BigTypesFormatter.java

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/translator/BigTypesFormatter.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/translator/BigTypesFormatter.java?rev=1095467&r1=1095466&r2=1095467&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/translator/BigTypesFormatter.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/translator/BigTypesFormatter.java Wed Apr 20 17:41:59 2011
@@ -47,7 +47,8 @@ public abstract class BigTypesFormatter 
 
     public Number parse(String clientValue) throws ParseException
     {
-        String noGroups = clientValue.replace(toString(symbols.getGroupingSeparator()), "");
+		// Allow for " " here since FF2/IE replaces \u00A0 (non breaking space) with \u0020 (normal space)
+        String noGroups = clientValue.replace(toString(symbols.getGroupingSeparator()), "").replace(" ", "");
         String fixedNeg = noGroups.replace(symbols.getMinusSign(), '-');
         String fixedDec = fixedNeg.replace(symbols.getDecimalSeparator(), '.');