Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed use of deprecated Query.aggregates #4003

Merged
merged 1 commit into from Mar 19, 2016
Merged

Conversation

blaze33
Copy link
Contributor

@blaze33 blaze33 commented Mar 18, 2016

In Django 1.8 Query.aggregates is replaced by annotations.

Using aggregates currently raises a RemovedInDjango110Warning. See Django's source. This PR fixes it.

In Django 1.8 [`Query.aggregates` is replaced by `annotations`](https://docs.djangoproject.com/es/1.9/releases/1.8/#aggregate-methods-and-modules).

Using aggregates currently raises a `RemovedInDjango20Warning`. [Django's source](https://github.com/django/django/blob/1.8/django/db/models/sql/query.py#L192)
@jpadilla jpadilla self-assigned this Mar 19, 2016
@jpadilla
Copy link
Member

jpadilla commented Mar 19, 2016

@blaze33 good stuff, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants