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


Reply via email to