You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cayenne.apache.org by jo...@apache.org on 2013/05/07 16:39:56 UTC

svn commit: r1479928 - /cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/reflect/ConverterFactory.java

Author: johnthuss
Date: Tue May  7 14:39:56 2013
New Revision: 1479928

URL: http://svn.apache.org/r1479928
Log:
Enhance PropertyUtils.setProperty to convert 0/1 to Booleans if needed

Modified:
    cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/reflect/ConverterFactory.java

Modified: cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/reflect/ConverterFactory.java
URL: http://svn.apache.org/viewvc/cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/reflect/ConverterFactory.java?rev=1479928&r1=1479927&r2=1479928&view=diff
==============================================================================
--- cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/reflect/ConverterFactory.java (original)
+++ cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/java/org/apache/cayenne/reflect/ConverterFactory.java Tue May  7 14:39:56 2013
@@ -64,6 +64,11 @@ class ConverterFactory {
 
                 if (object instanceof Boolean) {
                     return object;
+                } else if (object instanceof Integer || object instanceof Long || object instanceof Short) {
+                	if (((Number)object).longValue() == 0)
+                		return Boolean.FALSE;
+                	else if (((Number)object).longValue() == 1)
+                		return Boolean.TRUE;
                 }
 
                 return "true".equalsIgnoreCase(object.toString())