[Proxool-cvs] proxool/src/java/org/logicalcobwebs/proxool ConnectionPoolDefinition.java,1.25,1.26

[<bil...@us...>]

Update of /cvsroot/proxool/proxool/src/java/org/logicalcobwebs/proxool
In directory sc8-pr-cvs1:/tmp/cvs-serv12404

Modified Files:
	ConnectionPoolDefinition.java 
Log Message:
Setting a property to a zero length String actually sets it to a null

Index: ConnectionPoolDefinition.java
===================================================================
RCS file: /cvsroot/proxool/proxool/src/java/org/logicalcobwebs/proxool/ConnectionPoolDefinition.java,v
retrieving revision 1.25
retrieving revision 1.26
diff -C2 -d -r1.25 -r1.26
*** ConnectionPoolDefinition.java	16 Oct 2003 18:54:49 -0000	1.25
--- ConnectionPoolDefinition.java	19 Oct 2003 13:31:57 -0000	1.26
***************
*** 223,226 ****
--- 223,232 ----
                  String key = (String) i.next();
                  String value = info.getProperty(key);
+                 // This allows us to set properties to a null, even though Properties
+                 // object doesn't allow null values. It means that you can't set a property
+                 // to be a zero length string but that's ok.
+                 if (value != null && value.length() == 0) {
+                     value = null;
+                 }
                  changed = changed | setAnyProperty(key, value, pretend);
                  if (!pretend) {
***************
*** 1129,1132 ****
--- 1135,1141 ----
   Revision history:
   $Log$
+  Revision 1.26  2003/10/19 13:31:57  billhorsman
+  Setting a property to a zero length String actually sets it to a null
+ 
   Revision 1.25  2003/10/16 18:54:49  billhorsman
   Fixed javadoc for update() and redefine() methods which were transposed. Also improved exception handling for