diff options
| author | David Sanders <shang.xiao.sanders@gmail.com> | 2023-11-13 00:45:52 +1100 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2023-11-13 12:01:40 +0100 |
| commit | b863c5ffde0bafa5eaa9f262103eaeb71877787c (patch) | |
| tree | 3e0930b96daeddee486a2f011fd4598db3fa7e6b /django/db | |
| parent | 1b56b24f81a2e64b4bd3059abad9b6fd0c801c66 (diff) | |
Fixed #34967 -- Fixed queryset crash when grouping by constants on SQLite < 3.39.
On SQLite < 3.39, this forces a GROUP BY clause with a HAVING clause
when no grouping is specified.
Co-authored-by: Simon Charette <charette.s@gmail.com>
Diffstat (limited to 'django/db')
| -rw-r--r-- | django/db/backends/base/operations.py | 7 | ||||
| -rw-r--r-- | django/db/backends/sqlite3/operations.py | 5 | ||||
| -rw-r--r-- | django/db/models/sql/compiler.py | 2 |
3 files changed, 14 insertions, 0 deletions
diff --git a/django/db/backends/base/operations.py b/django/db/backends/base/operations.py index b1d818d90e..d5a40eb46e 100644 --- a/django/db/backends/base/operations.py +++ b/django/db/backends/base/operations.py @@ -231,6 +231,13 @@ class BaseDatabaseOperations: ) return "%s" + def force_group_by(self): + """ + Return a GROUP BY clause to use with a HAVING clause when no grouping + is specified. + """ + return [] + def force_no_ordering(self): """ Return a list used in the "ORDER BY" clause to force no ordering at diff --git a/django/db/backends/sqlite3/operations.py b/django/db/backends/sqlite3/operations.py index 85ad804348..dfc9857b84 100644 --- a/django/db/backends/sqlite3/operations.py +++ b/django/db/backends/sqlite3/operations.py @@ -14,6 +14,8 @@ from django.utils import timezone from django.utils.dateparse import parse_date, parse_datetime, parse_time from django.utils.functional import cached_property +from .base import Database + class DatabaseOperations(BaseDatabaseOperations): cast_char_field_without_max_length = "text" @@ -439,3 +441,6 @@ class DatabaseOperations(BaseDatabaseOperations): update_fields, unique_fields, ) + + def force_group_by(self): + return ["GROUP BY TRUE"] if Database.sqlite_version_info < (3, 39) else [] diff --git a/django/db/models/sql/compiler.py b/django/db/models/sql/compiler.py index b28dc925ba..7cec040cee 100644 --- a/django/db/models/sql/compiler.py +++ b/django/db/models/sql/compiler.py @@ -877,6 +877,8 @@ class SQLCompiler: if self._meta_ordering: order_by = None if having: + if not grouping: + result.extend(self.connection.ops.force_group_by()) result.append("HAVING %s" % having) params.extend(h_params) |
