diff options
| author | David Wobrock <david.wobrock@gmail.com> | 2019-12-28 22:42:46 +0100 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2019-12-31 10:35:43 +0100 |
| commit | 2f565f84aca136d9cc4e4d061f3196ddf9358ab8 (patch) | |
| tree | b084853444500fa3eeaa8a1fe07df961015f46e4 /django | |
| parent | 7d44aeb388a33ac56353a658b41a6119e93931ff (diff) | |
Fixed #31097 -- Fixed crash of ArrayAgg and StringAgg with filter when used in Subquery.
Diffstat (limited to 'django')
| -rw-r--r-- | django/contrib/postgres/aggregates/mixins.py | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/django/contrib/postgres/aggregates/mixins.py b/django/contrib/postgres/aggregates/mixins.py index 4625738beb..3a43ca1a63 100644 --- a/django/contrib/postgres/aggregates/mixins.py +++ b/django/contrib/postgres/aggregates/mixins.py @@ -40,16 +40,7 @@ class OrderableAggMixin: return super().set_source_expressions(exprs[:self._get_ordering_expressions_index()]) def get_source_expressions(self): - return self.source_expressions + self.ordering - - def get_source_fields(self): - # Filter out fields contributed by the ordering expressions as - # these should not be used to determine which the return type of the - # expression. - return [ - e._output_field_or_none - for e in self.get_source_expressions()[:self._get_ordering_expressions_index()] - ] + return super().get_source_expressions() + self.ordering def _get_ordering_expressions_index(self): """Return the index at which the ordering expressions start.""" |
