svn commit: r572746 - /myfaces/trinidad/branches/1.2.2-branch/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js

[ma...@apache.org]

Author: matzew
Date: Tue Sep  4 10:27:33 2007
New Revision: 572746

URL: http://svn.apache.org/viewvc?rev=572746&view=rev
Log:
a quick fix to have localized numbers working, with the NumberFormat AND the _decimalParse(...) see TRINIDAD-682 as well.

Modified:
    myfaces/trinidad/branches/1.2.2-branch/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js

Modified: myfaces/trinidad/branches/1.2.2-branch/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.2-branch/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js?rev=572746&r1=572745&r2=572746&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.2-branch/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js (original)
+++ myfaces/trinidad/branches/1.2.2-branch/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js Tue Sep  4 10:27:33 2007
@@ -219,7 +219,24 @@
     {
       try
       {
+        //TODO matzew - see TRINIDAD-682
+        // Remove the thousands separator - which Javascript doesn't want to see
+        var groupingSeparator = getLocaleSymbols().getGroupingSeparator();
+        var grouping = new RegExp("\\" + groupingSeparator,  "g");
+        numberString = numberString.replace(grouping, "");
+
+        // Then change the decimal separator into a period, the only
+        // decimal separator allowed by JS
+        var decimalSeparator = getLocaleSymbols().getDecimalSeparator()
+        var decimal = new RegExp("\\" + decimalSeparator,  "g");
+        numberString = numberString.replace(decimal, ".");
+
+        //parse the numberString
         numberString = this._numberFormat.parse(numberString)+"";
+
+        //to be able to pass the _decimalParse, we replace the decimal separator...
+        var jsSeparator = new RegExp("\\" + ".",  "g");
+        numberString = numberString.replace(jsSeparator, decimalSeparator);
       }
       catch(e)
       {