Ticket #17877: t17877.diff

django/db/models/sql/where.py

@@ -281 +281 @@
         self.params = params
 
     def as_sql(self, qn=None, connection=None):
-        return " AND ".join(self.sqls), tuple(self.params or ())
+        sqls = ["(%s)" % e for e in self.sqls]
+        return " AND ".join(sqls), tuple(self.params or ())
 
 class Constraint(object):
     """

docs/ref/models/querysets.txt

@@ -968 +968 @@
 
   Example::
 
-      Entry.objects.extra(where=['id IN (3, 4, 5, 20)'])
+      Entry.objects.extra(where=["foo='a' OR bar = 'a'", "baz = 'a'"])
 
   ...translates (roughly) into the following SQL::
 
-      SELECT * FROM blog_entry WHERE id IN (3, 4, 5, 20);
+      SELECT * FROM blog_entry WHERE (foo='a' OR bar='a') AND (baz='a')
 
   Be careful when using the ``tables`` parameter if you're specifying
   tables that are already used in the query. When you add extra tables

tests/regressiontests/extra_regress/tests.py

@@ -313 +313 @@
                TestObject.objects.extra(where=["id > %s"], params=[obj.pk]),
             ['<TestObject: TestObject: first,second,third>']
         )
+
+    def test_regression_17877(self):
+        """
+        Regression for #17877: When chaining several elements including OR
+        operations, the result was wrong due to higher priority for AND
+        operatoins. Fix is to wrap all element in parenthesis
+        """
+        # Test Case 1: should appear in queryset.
+        t = TestObject(first='a', second='a', third='a')
+        t.save()
+        # Test Case 2: should appear in queryset.
+        t = TestObject(first='b', second='a', third='a')
+        t.save()
+        # Test Case 3: should not appear in queryset, bug case.
+        t = TestObject(first='a', second='a', third='b')
+        t.save()
+        # Test Case 4: should not appear in queryset.
+        t = TestObject(first='b', second='a', third='b')
+        t.save()
+        # Test Case 5: should not appear in queryset.
+        t = TestObject(first='b', second='b', third='a')
+        t.save()
+        # Test Case 6: should not appear in queryset, bug case.
+        t = TestObject(first='a', second='b', third='b')
+        t.save()
+
+        self.assertQuerysetEqual(
+            TestObject.objects.extra(
+                where=["first = 'a' OR second = 'a'", "third = 'a'"],
+            ),
+            ['<TestObject: TestObject: a,a,a>', '<TestObject: TestObject: b,a,a>']
+        )