From 5b57798513066bb6d5ea54225d0bd030a089c501 Mon Sep 17 00:00:00 2001 From: Jon Dufresne Date: Thu, 8 Aug 2019 13:34:25 -0700 Subject: [PATCH] Removed unnecessary StatAggregate.resolve_expression(). This method only calls the parent method, but without the for_save argument. The parent class, Aggregate, already ignores the for_save argument so there is no need for special handling. Unnecessary since its introduction in e4cf8c8420634d6f2dc8ce873246256ce635972d. --- django/contrib/postgres/aggregates/statistics.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/django/contrib/postgres/aggregates/statistics.py b/django/contrib/postgres/aggregates/statistics.py index db6c8e6dc5..2df8f95cb0 100644 --- a/django/contrib/postgres/aggregates/statistics.py +++ b/django/contrib/postgres/aggregates/statistics.py @@ -15,9 +15,6 @@ class StatAggregate(Aggregate): raise ValueError('Both y and x must be provided.') super().__init__(y, x, output_field=output_field, filter=filter) - def resolve_expression(self, query=None, allow_joins=True, reuse=None, summarize=False, for_save=False): - return super().resolve_expression(query, allow_joins, reuse, summarize) - class Corr(StatAggregate): function = 'CORR'