[cvs] expresso commit by mtraum: fix NPE which several junit tests were

JCorporate Ltd jcorp at jcorporate.com
Tue Feb 15 00:15:25 UTC 2005


Log Message:
-----------
fix NPE which several junit tests were catching

Modified Files:
--------------
    expresso/expresso-web/WEB-INF/src/com/jcorporate/expresso/core/dataobjects/jdbc:
        JDBCExecutor.java

Revision Data
-------------
Index: JDBCExecutor.java
===================================================================
RCS file: /home/javacorp/.cvs/expresso/expresso/expresso-web/WEB-INF/src/com/jcorporate/expresso/core/dataobjects/jdbc/JDBCExecutor.java,v
retrieving revision 1.65
retrieving revision 1.66
diff -Lexpresso-web/WEB-INF/src/com/jcorporate/expresso/core/dataobjects/jdbc/JDBCExecutor.java -Lexpresso-web/WEB-INF/src/com/jcorporate/expresso/core/dataobjects/jdbc/JDBCExecutor.java -u -r1.65 -r1.66
--- expresso-web/WEB-INF/src/com/jcorporate/expresso/core/dataobjects/jdbc/JDBCExecutor.java
+++ expresso-web/WEB-INF/src/com/jcorporate/expresso/core/dataobjects/jdbc/JDBCExecutor.java
@@ -745,7 +745,7 @@
                                     data = tmpData.toString();
                                 }
 
-                                if (data != null || data.length() != 0) {
+                                if (data != null && data.length() != 0) {
                                     lobUpdates.add(tmpData);
                                     lobFieldOrder.add(oneFieldName);
                                     lobFieldType.add(oneField.getTypeString());


More information about the cvs mailing list