From 7b065c41e406fb22d39d1f7b4cfed96bf06dc591 Mon Sep 17 00:00:00 2001 From: Simon Charette Date: Sat, 21 Dec 2019 23:22:49 -0500 Subject: [3.0.x] Fixed #31109 -- Disabled grouping by aliases on QuerySet.exists(). Clearing the SELECT clause in Query.has_results was orphaning GROUP BY references to it. Thanks Thierry Bastian for the report and Baptiste Mispelon for the bisect. Regression in fb3f034f1c63160c0ff13c609acd01c18be12f80. Backport of 720de4d0441fcfdb543051389c70efbe66ed962a from master --- django/db/models/sql/query.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'django/db/models/sql') diff --git a/django/db/models/sql/query.py b/django/db/models/sql/query.py index cc1d4b1d38..7429939cf9 100644 --- a/django/db/models/sql/query.py +++ b/django/db/models/sql/query.py @@ -527,7 +527,9 @@ class Query(BaseExpression): if not q.distinct: if q.group_by is True: q.add_fields((f.attname for f in self.model._meta.concrete_fields), False) - q.set_group_by() + # Disable GROUP BY aliases to avoid orphaning references to the + # SELECT clause which is about to be cleared. + q.set_group_by(allow_aliases=False) q.clear_select_clause() q.clear_ordering(True) q.set_limits(high=1) @@ -1920,7 +1922,7 @@ class Query(BaseExpression): if force_empty: self.default_ordering = False - def set_group_by(self): + def set_group_by(self, allow_aliases=True): """ Expand the GROUP BY clause required by the query. @@ -1943,6 +1945,8 @@ class Query(BaseExpression): warnings.warn(msg, category=RemovedInDjango40Warning) group_by_cols = annotation.get_group_by_cols() else: + if not allow_aliases: + alias = None group_by_cols = annotation.get_group_by_cols(alias=alias) group_by.extend(group_by_cols) self.group_by = tuple(group_by) -- cgit v1.3