You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@labs.apache.org by si...@apache.org on 2009/12/14 14:49:59 UTC

svn commit: r890326 - /labs/magma/trunk/foundation-beans/src/main/java/org/apache/magma/conversion/string/BooleanConverter.java

Author: simoneg
Date: Mon Dec 14 13:49:58 2009
New Revision: 890326

URL: http://svn.apache.org/viewvc?rev=890326&view=rev
Log:
Empty string is null, not false

Modified:
    labs/magma/trunk/foundation-beans/src/main/java/org/apache/magma/conversion/string/BooleanConverter.java

Modified: labs/magma/trunk/foundation-beans/src/main/java/org/apache/magma/conversion/string/BooleanConverter.java
URL: http://svn.apache.org/viewvc/labs/magma/trunk/foundation-beans/src/main/java/org/apache/magma/conversion/string/BooleanConverter.java?rev=890326&r1=890325&r2=890326&view=diff
==============================================================================
--- labs/magma/trunk/foundation-beans/src/main/java/org/apache/magma/conversion/string/BooleanConverter.java (original)
+++ labs/magma/trunk/foundation-beans/src/main/java/org/apache/magma/conversion/string/BooleanConverter.java Mon Dec 14 13:49:58 2009
@@ -33,7 +33,7 @@
 	}
 
 	public Boolean from(String value) {
-		if (value == null) return onNative ? Boolean.FALSE : null;
+		if (value == null || value.length() == 0) return onNative ? Boolean.FALSE : null;
 		value = value.toLowerCase();
 		return value.equals("t") || value.equals("true") || value.equals("yes") || value.equals("y") || value.equals("1");
 	}



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@labs.apache.org
For additional commands, e-mail: commits-help@labs.apache.org