You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by jg...@apache.org on 2013/12/05 18:52:00 UTC
svn commit: r1548222 -
/openjpa/branches/2.2.1.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/criteria/Joins.java
Author: jgrassel
Date: Thu Dec 5 17:52:00 2013
New Revision: 1548222
URL: http://svn.apache.org/r1548222
Log:
OPENJPA-2318: Left outer join is not generated when specified using Criteria API
Modified:
openjpa/branches/2.2.1.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/criteria/Joins.java
Modified: openjpa/branches/2.2.1.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/criteria/Joins.java
URL: http://svn.apache.org/viewvc/openjpa/branches/2.2.1.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/criteria/Joins.java?rev=1548222&r1=1548221&r2=1548222&view=diff
==============================================================================
--- openjpa/branches/2.2.1.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/criteria/Joins.java (original)
+++ openjpa/branches/2.2.1.x/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/criteria/Joins.java Thu Dec 5 17:52:00 2013
@@ -197,17 +197,18 @@ abstract class Joins {
if (c.isRegistered(_parent)) {
Value var = c.getRegisteredVariable(_parent);
path = factory.newPath(var);
- } else
+ } else {
path = factory.newPath(subQ);
+ }
path.setMetaData(meta);
- path.get(_member.fmd, false);
+ path.get(_member.fmd, allowNull);
path.setSchemaAlias(c.getAlias(_parent));
}
} else if (c.isRegistered(_parent)) {
Value var = c.getRegisteredVariable(_parent);
path = factory.newPath(var);
path.setMetaData(meta);
- path.get(_member.fmd, false);
+ path.get(_member.fmd, allowNull);
} else
path = (org.apache.openjpa.kernel.exps.Path)toValue(factory, c);
@@ -409,14 +410,14 @@ abstract class Joins {
path = factory.newPath(subQ);
}
path.setMetaData(meta);
- path.get(_member.fmd, false);
+ path.get(_member.fmd, allowNull);
path.setSchemaAlias(c.getAlias(_parent));
}
} else if (c.isRegistered(_parent)) {
Value var = c.getRegisteredVariable(_parent);
path = factory.newPath(var);
path.setMetaData(meta);
- path.get(_member.fmd, false);
+ path.get(_member.fmd, allowNull);
} else {
path = (org.apache.openjpa.kernel.exps.Path)toValue(factory, c);
}