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 12:44:45 UTC
svn commit: r585921 -
/myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js
Author: matzew
Date: Thu Oct 18 03:44:44 2007
New Revision: 585921
URL: http://svn.apache.org/viewvc?rev=585921&view=rev
Log:
added check for null or empty string inside of getAsObject
Modified:
myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js
Modified: myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js?rev=585921&r1=585920&r2=585921&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js (original)
+++ myfaces/trinidad/branches/1.2.3-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js Thu Oct 18 03:44:44 2007
@@ -212,6 +212,16 @@
label
)
{
+ // The following are from the javadoc for Number and DateTimeConverter.
+ // If the specified String is null, return a null. Otherwise, trim leading and trailing whitespace before proceeding.
+ // If the specified String - after trimming - has a zero length, return null.
+ if (numberString== null)
+ return null;
+
+ numberString= TrUIUtils.trim(numberString);
+ if (numberString.length == 0)
+ return null
+
if(this._isConvertible())
{
var parsedValue;