You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@myfaces.apache.org by ma...@apache.org on 2007/10/18 15:54:52 UTC

svn commit: r585962 - /myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js

Author: matzew
Date: Thu Oct 18 06:54:52 2007
New Revision: 585962

URL: http://svn.apache.org/viewvc?rev=585962&view=rev
Log:
fixing percentage parsing issue

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

Modified: myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js?rev=585962&r1=585961&r2=585962&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js (original)
+++ myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js Thu Oct 18 06:54:52 2007
@@ -290,11 +290,12 @@
 /**
  * Formats a percent string into a number.
  */
-TrNumberFormat.prototype.stringToPercentage = function(numberString)
+TrNumberFormat.prototype.stringToPercentage = function(percentString)
 {
-  numberString = numberString.replace(/\%/g, '');
+  var isPercentage = (percentString.indexOf('%') != -1);
+  var numberString = percentString.replace(/\%/g, '');
   numberString = parseFloat(numberString);
-  if(isNaN(numberString))
+  if(!isPercentage || isNaN(numberString))
   {
     throw new TrParseException("not able to parse number");
   }