You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by pc...@apache.org on 2006/08/01 01:13:01 UTC

svn commit: r427356 - /incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java

Author: pcl
Date: Mon Jul 31 16:13:00 2006
New Revision: 427356

URL: http://svn.apache.org/viewvc?rev=427356&view=rev
Log:
changed configuration value factory method to set a default for booleans by default. The normal pattern is to obtain a BooleanValue and then set the default to 'true' when desired; implicitly, this means that 'false' is the default otherwise. This change makes that explicit.

Modified:
    incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java

Modified: incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java?rev=427356&r1=427355&r2=427356&view=diff
==============================================================================
--- incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java (original)
+++ incubator/openjpa/trunk/openjpa-lib/src/main/java/org/apache/openjpa/lib/conf/ConfigurationImpl.java Mon Jul 31 16:13:00 2006
@@ -931,6 +931,7 @@
     public BooleanValue addBoolean(String property) {
         BooleanValue val = new BooleanValue(property);
         addValue(val);
+        val.setDefault("false");
         return val;
     }