You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by mp...@apache.org on 2006/10/04 02:33:46 UTC
svn commit: r452684 -
/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/jpql/JPQLExpressionBuilder.java
Author: mprudhom
Date: Tue Oct 3 17:33:46 2006
New Revision: 452684
URL: http://svn.apache.org/viewvc?view=rev&rev=452684
Log:
Removed special logic for forcing an inner join for projection clauses, since that logic is duplicated elsewhere.
Modified:
incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/jpql/JPQLExpressionBuilder.java
Modified: incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/jpql/JPQLExpressionBuilder.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/jpql/JPQLExpressionBuilder.java?view=diff&rev=452684&r1=452683&r2=452684
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/jpql/JPQLExpressionBuilder.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/main/java/org/apache/openjpa/kernel/jpql/JPQLExpressionBuilder.java Tue Oct 3 17:33:46 2006
@@ -313,16 +313,6 @@
Value proj = getValue(node);
exps.projections[i] = proj;
exps.projectionAliases[i] = nextAlias();
-
- // projections along PCs in JPQL imply inner join semantics:
- // e.g. "select x.y.z.someField from Entity x" implies
- // "where y is not null and z is not null"
- if (proj instanceof Path && node.id == JJTPATH) {
- Path path = getPCPath(node);
- if (path.last() != null
- && path.last().getDeclaredTypeMetaData() != null)
- exp = addJoin(path, null, true, exp);
- }
}
return exp;
}