summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLouise Grandjonc <louve.grandjonc@gmail.com>2019-10-01 16:25:40 -0700
committerMariusz Felisiak <felisiak.mariusz@gmail.com>2019-10-11 12:01:42 +0200
commita843a9ba8da40ffae49c9ec13e7b5c41d5e40ca5 (patch)
tree36b28ba4565fbb0fd7ea257852bcde38f7ee962b
parentcf2b475aaba8e854cafa99575c8082d844e5cb09 (diff)
downloaddjango-a843a9ba8da40ffae49c9ec13e7b5c41d5e40ca5.tar.gz
[1.11.x] Fixed #30826 -- Fixed crash of many JSONField lookups when one hand side is key transform.
Regression in 6c3dfba89215fc56fc27ef61829a6fff88be4abb. Backport of 7d1bf29977bb368d7c28e7c6eb146db3b3009ae7 from master.
-rw-r--r--django/contrib/postgres/lookups.py2
-rw-r--r--docs/releases/1.11.26.txt5
-rw-r--r--tests/postgres_tests/test_json.py30
3 files changed, 33 insertions, 4 deletions
diff --git a/django/contrib/postgres/lookups.py b/django/contrib/postgres/lookups.py
index 53a62eacd1..466b8cdf56 100644
--- a/django/contrib/postgres/lookups.py
+++ b/django/contrib/postgres/lookups.py
@@ -8,7 +8,7 @@ class PostgresSimpleLookup(Lookup):
def as_sql(self, qn, connection):
lhs, lhs_params = self.process_lhs(qn, connection)
rhs, rhs_params = self.process_rhs(qn, connection)
- params = lhs_params + rhs_params
+ params = tuple(lhs_params) + tuple(rhs_params)
return '%s %s %s' % (lhs, self.operator, rhs), params
diff --git a/docs/releases/1.11.26.txt b/docs/releases/1.11.26.txt
index a0c39b4168..1a54b47e1b 100644
--- a/docs/releases/1.11.26.txt
+++ b/docs/releases/1.11.26.txt
@@ -9,4 +9,7 @@ Django 1.11.26 fixes a regression in 1.11.25.
Bugfixes
========
-* ...
+* Fixed a crash when using a ``contains``, ``contained_by``, ``has_key``,
+ ``has_keys``, or ``has_any_keys`` lookup on
+ :class:`~django.contrib.postgres.fields.JSONField`, if the right or left hand
+ side of an expression is a key transform (:ticket:`30826`).
diff --git a/tests/postgres_tests/test_json.py b/tests/postgres_tests/test_json.py
index b8b5dd481f..faefcd18a3 100644
--- a/tests/postgres_tests/test_json.py
+++ b/tests/postgres_tests/test_json.py
@@ -21,7 +21,9 @@ from .models import JSONModel
try:
from django.contrib.postgres import forms
from django.contrib.postgres.fields import JSONField
- from django.contrib.postgres.fields.jsonb import KeyTransform
+ from django.contrib.postgres.fields.jsonb import (
+ KeyTextTransform, KeyTransform
+ )
except ImportError:
pass
@@ -130,7 +132,12 @@ class TestQuerying(PostgreSQLTestCase):
'k': True,
'l': False,
}),
- JSONModel.objects.create(field={'foo': 'bar'}),
+ JSONModel.objects.create(field={
+ 'foo': 'bar',
+ 'baz': {'a': 'b', 'c': 'd'},
+ 'bar': ['foo', 'bar'],
+ 'bax': {'foo': 'bar'},
+ }),
]
def test_exact(self):
@@ -305,6 +312,25 @@ class TestQuerying(PostgreSQLTestCase):
queries[0]['sql'],
)
+ def test_lookups_with_key_transform(self):
+ tests = (
+ ('field__d__contains', 'e'),
+ ('field__baz__contained_by', {'a': 'b', 'c': 'd', 'e': 'f'}),
+ ('field__baz__has_key', 'c'),
+ ('field__baz__has_keys', ['a', 'c']),
+ ('field__baz__has_any_keys', ['a', 'x']),
+ ('field__contains', KeyTransform('bax', 'field')),
+ (
+ 'field__contained_by',
+ KeyTransform('x', RawSQL('%s::jsonb', ['{"x": {"a": "b", "c": 1, "d": "e"}}'])),
+ ),
+ ('field__has_key', KeyTextTransform('foo', 'field')),
+ )
+ for lookup, value in tests:
+ self.assertTrue(JSONModel.objects.filter(
+ **{lookup: value}
+ ).exists())
+
@skipUnlessDBFeature('has_jsonb_datatype')
class TestSerialization(PostgreSQLTestCase):