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:57:21 UTC
svn commit: r585964 -
/myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js
Author: matzew
Date: Thu Oct 18 06:57:20 2007
New Revision: 585964
URL: http://svn.apache.org/viewvc?rev=585964&view=rev
Log:
fixing percentage parsing issue
(in trunk as well)
Modified:
myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js
Modified: myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js?rev=585964&r1=585963&r2=585964&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js (original)
+++ myfaces/trinidad/trunk/trinidad/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberFormat.js Thu Oct 18 06:57:20 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");
}