Ticket #15305: 15305.1.diff

django/contrib/gis/db/models/sql/compiler.py

@@ -171 +171 @@
         """
         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

django/contrib/gis/tests/relatedapp/tests.py

@@ -245 +245 @@
         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.
     @no_oracle
     def test13_select_related_null_fk(self):