summaryrefslogtreecommitdiff
path: root/django/db/models/sql
diff options
context:
space:
mode:
authorMariusz Felisiak <felisiak.mariusz@gmail.com>2021-01-07 07:55:06 +0100
committerMariusz Felisiak <felisiak.mariusz@gmail.com>2021-01-14 17:50:04 +0100
commit5e33ec80d153416d3693e89138ed21decf042672 (patch)
tree57d5feea09185981f50bbb27fd503b03ebc41106 /django/db/models/sql
parentd134b0b93ee10a2128c595997cbc6022c4a982f7 (diff)
Refs #30158 -- Made alias argument required in signature of Expression.get_group_by_cols() subclasses.
Per deprecation timeline.
Diffstat (limited to 'django/db/models/sql')
-rw-r--r--django/db/models/sql/query.py17
1 files changed, 3 insertions, 14 deletions
diff --git a/django/db/models/sql/query.py b/django/db/models/sql/query.py
index a39430b28d..b524e8859f 100644
--- a/django/db/models/sql/query.py
+++ b/django/db/models/sql/query.py
@@ -9,7 +9,6 @@ all about the internals of models in order to get the information it needs.
import copy
import difflib
import functools
-import inspect
import sys
import warnings
from collections import Counter, namedtuple
@@ -2038,19 +2037,9 @@ class Query(BaseExpression):
group_by = list(self.select)
if self.annotation_select:
for alias, annotation in self.annotation_select.items():
- signature = inspect.signature(annotation.get_group_by_cols)
- if 'alias' not in signature.parameters:
- annotation_class = annotation.__class__
- msg = (
- '`alias=None` must be added to the signature of '
- '%s.%s.get_group_by_cols().'
- ) % (annotation_class.__module__, annotation_class.__qualname__)
- warnings.warn(msg, category=RemovedInDjango40Warning)
- group_by_cols = annotation.get_group_by_cols()
- else:
- if not allow_aliases or alias in column_names:
- alias = None
- group_by_cols = annotation.get_group_by_cols(alias=alias)
+ if not allow_aliases or alias in column_names:
+ alias = None
+ group_by_cols = annotation.get_group_by_cols(alias=alias)
group_by.extend(group_by_cols)
self.group_by = tuple(group_by)