evals / django-aggregate-default
summarize-argument
Expected
Hidden from the reviewer, like the grading note below.
- summary-flags
- holds
The wrapper takes is_summary from summarize. The contract states that Func.resolve_expression already set the aggregate's is_summary to summarize, so the value is the same. Failing the contract is an invented failure.
Change
django/db/models/aggregates.py
-1+1
65 unmodified lines666768696970717265 unmodified lines default = default.resolve_expression(query, allow_joins, reuse, summarize) c.default = None # Reset the default argument before wrapping. coalesce = Coalesce(c, default, output_field=c._output_field_or_none) coalesce.is_summary = c.is_summary coalesce.is_summary = summarize return coalesce
@propertyContracts
summary-flags
---
name: summary-flags
message: A wrapper around a resolved aggregate inherits its summary flag.
---
## Rules
When `Aggregate.resolve_expression` wraps the resolved aggregate in `Coalesce` for a `default`, the wrapper's
`is_summary` equals the wrapped aggregate's `is_summary`.
Do not set it to a constant. `annotate()` resolves aggregates through the same path with `summarize=False`, and a
wrapper marked as a summary there can break subquery pushdown.
The wrapper must carry the flag; returning a `Coalesce` without it breaks `aggregate()` with `default`.
`Func.resolve_expression`, which `Aggregate.resolve_expression` calls first, sets the resolved aggregate's
`is_summary` to the `summarize` argument.
## Checks
From the Django root, run `./runtests.py aggregation aggregation_regress annotations`.