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;
     }