You are viewing a plain text version of this content. The canonical link for it is here.
Posted to ojb-dev@db.apache.org by br...@apache.org on 2006/03/04 13:20:38 UTC

svn commit: r383100 - /db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/accesslayer/sql/SqlSelectStatement.java

Author: brj
Date: Sat Mar  4 04:20:38 2006
New Revision: 383100

URL: http://svn.apache.org/viewcvs?rev=383100&view=rev
Log:
fix for InheritanceMultipleTableTest#testLookupByIdentity_2

Modified:
    db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/accesslayer/sql/SqlSelectStatement.java

Modified: db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/accesslayer/sql/SqlSelectStatement.java
URL: http://svn.apache.org/viewcvs/db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/accesslayer/sql/SqlSelectStatement.java?rev=383100&r1=383099&r2=383100&view=diff
==============================================================================
--- db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/accesslayer/sql/SqlSelectStatement.java (original)
+++ db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/accesslayer/sql/SqlSelectStatement.java Sat Mar  4 04:20:38 2006
@@ -451,6 +451,16 @@
 
     public boolean isUseOjbClassColumn()
     {
+        // BRJ: force generation of sql-string.
+        //
+        // this is a workaraoundd for the problem described 
+        // in InheritanceMultipleTableTest#testLookupByIdentity_2
+        // the problem is caused by JdbcAccessImpl#materializeObject not caching
+        // the SqlSelectStatement and passing an 'empty' SqlSelectStatement to ResultSetAndStatement
+        //
+        // TODO: remove this workaround for ojb 1.x
+        getStatement();  
+
         return useOjbClassColumn;
     }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: ojb-dev-unsubscribe@db.apache.org
For additional commands, e-mail: ojb-dev-help@db.apache.org