You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by fa...@apache.org on 2008/12/19 03:17:37 UTC

svn commit: r727895 - /openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java

Author: fancy
Date: Thu Dec 18 18:17:36 2008
New Revision: 727895

URL: http://svn.apache.org/viewvc?rev=727895&view=rev
Log:
OPENJPA-835 Loading nested toMany EAGER relation resuled in PersistenceException

Modified:
    openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java

Modified: openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java?rev=727895&r1=727894&r2=727895&view=diff
==============================================================================
--- openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java (original)
+++ openjpa/trunk/openjpa-jdbc/src/main/java/org/apache/openjpa/jdbc/kernel/JDBCStoreManager.java Thu Dec 18 18:17:36 2008
@@ -355,7 +355,9 @@
                     getMappedByFieldMapping();
                 Object mappedByObject = info.result.getMappedByValue();
                 if (mappedByFieldMapping != null && mappedByObject != null)
-                    if (mappedByObject instanceof OpenJPAId)
+                    if (mappedByObject instanceof OpenJPAId &&
+                        mapping.getExtraFieldDataIndex(mappedByFieldMapping.
+                        getIndex()) != -1)
                         // The inverse relation can not be set since
                         // we are eagerly loading this sm for
                         // a sm owner that is still in the process of