You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@jackrabbit.apache.org by th...@apache.org on 2010/03/11 16:20:03 UTC

svn commit: r921892 - /jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java

Author: thomasm
Date: Thu Mar 11 15:20:03 2010
New Revision: 921892

URL: http://svn.apache.org/viewvc?rev=921892&view=rev
Log:
JCR-2564 SQL2 query: QOMFormatter creates incorrect NOT conditions

Modified:
    jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java

Modified: jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java?rev=921892&r1=921891&r2=921892&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java (original)
+++ jackrabbit/trunk/jackrabbit-jcr-commons/src/main/java/org/apache/jackrabbit/commons/query/sql2/QOMFormatter.java Thu Mar 11 15:20:03 2010
@@ -178,7 +178,7 @@ public class QOMFormatter implements Que
                 constraint.getConstraint2())) {
             append(and);
             and = " AND ";
-            boolean paren = c instanceof Or;
+            boolean paren = c instanceof Or || c instanceof Not;
             if (paren) {
                 append("(");
             }