Skip to content


Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Fixed #15305 -- Made `Count` aggregate and `.values()` play nice toge…

…ther on `GeoQuerySets`. Thanks, vrehak for the bug report and milosu for initial patch.

git-svn-id: bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
commit a25413bf86e5e65129f8582a713a29308296c374 1 parent 2918c3d
@jbronn jbronn authored
4 django/contrib/gis/db/models/sql/
@@ -171,10 +171,6 @@ def resolve_columns(self, row, fields=()):
values = []
aliases = self.query.extra_select.keys()
- if self.query.aggregates:
- # If we have an aggregate annotation, must extend the aliases
- # so their corresponding row values are included.
- aliases.extend([None for i in xrange(len(self.query.aggregates))])
# Have to set a starting row number offset that is used for
# determining the correct starting row index -- needed for
9 django/contrib/gis/tests/relatedapp/
@@ -245,6 +245,13 @@ def test12b_count(self):
self.assertEqual(1, len(vqs))
self.assertEqual(3, vqs[0]['num_books'])
+ def test13c_count(self):
+ "Testing `Count` aggregate with `.values()`. See #15305."
+ qs = Location.objects.filter(id=5).annotate(num_cities=Count('city')).values('id', 'point', 'num_cities')
+ self.assertEqual(1, len(qs))
+ self.assertEqual(2, qs[0]['num_cities'])
+ self.assertTrue(isinstance(qs[0]['point'], GEOSGeometry))
# TODO: The phantom model does appear on Oracle.
def test13_select_related_null_fk(self):
@@ -284,7 +291,7 @@ def test15_invalid_select_related(self):
def test16_annotated_date_queryset(self):
"Ensure annotated date querysets work if spatial backend is used. See #14648."
- birth_years = [dt.year for dt in
+ birth_years = [dt.year for dt in
list(Author.objects.annotate(num_books=Count('books')).dates('dob', 'year'))]
self.assertEqual([1950, 1974], birth_years)
Please sign in to comment.
Something went wrong with that request. Please try again.