You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lu...@apache.org on 2013/10/16 00:19:15 UTC

svn commit: r1532564 - in /myfaces/core/trunk/api/src: main/java/javax/faces/convert/NumberConverter.java test/java/javax/faces/convert/NumberConverterTest.java

Author: lu4242
Date: Tue Oct 15 22:19:15 2013
New Revision: 1532564

URL: http://svn.apache.org/r1532564
Log:
MYFACES-3578 NumberConverter: implement workaround for http://bugs.sun.com/view_bug.do?bug_id=4510618 (thanks to Martin Koci for provide this patch)

Modified:
    myfaces/core/trunk/api/src/main/java/javax/faces/convert/NumberConverter.java
    myfaces/core/trunk/api/src/test/java/javax/faces/convert/NumberConverterTest.java

Modified: myfaces/core/trunk/api/src/main/java/javax/faces/convert/NumberConverter.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/api/src/main/java/javax/faces/convert/NumberConverter.java?rev=1532564&r1=1532563&r2=1532564&view=diff
==============================================================================
--- myfaces/core/trunk/api/src/main/java/javax/faces/convert/NumberConverter.java (original)
+++ myfaces/core/trunk/api/src/main/java/javax/faces/convert/NumberConverter.java Tue Oct 15 22:19:15 2013
@@ -139,6 +139,7 @@ public class NumberConverter
                 {
                   dfs.setGroupingSeparator(' ');
                   df.setDecimalFormatSymbols(dfs);
+                  value = value.replace('\u00a0', ' ');
                   changed = true;
                 }
                 

Modified: myfaces/core/trunk/api/src/test/java/javax/faces/convert/NumberConverterTest.java
URL: http://svn.apache.org/viewvc/myfaces/core/trunk/api/src/test/java/javax/faces/convert/NumberConverterTest.java?rev=1532564&r1=1532563&r2=1532564&view=diff
==============================================================================
--- myfaces/core/trunk/api/src/test/java/javax/faces/convert/NumberConverterTest.java (original)
+++ myfaces/core/trunk/api/src/test/java/javax/faces/convert/NumberConverterTest.java Tue Oct 15 22:19:15 2013
@@ -166,5 +166,21 @@ public class NumberConverterTest extends
         assertNotNull(number);
         assertEquals(testValue, number);        
     }
+    
+    @Test
+    public void testCzechLocaleWithNonBreakingSpace()
+    {
+        mock.setLocale(new Locale("cs"));
+        mock.setIntegerOnly(true);
+        mock.setGroupingUsed(true);
+        FacesContext.getCurrentInstance().getViewRoot().setLocale(new Locale("cs"));
+        UIInput input = new UIInput();
+        String stringValue = mock.getAsString(facesContext, input, new Long(7000));
+        assertEquals("must return 7&NBSP000", "7\u00a0000", stringValue);
+        
+        Number number = (Number) mock.getAsObject(FacesContext.getCurrentInstance(), input, stringValue);
+        assertNotNull(number);
+        assertEquals(new Long(7000), number);
+    }
 
 }