[1.11.x] Fixed #27800 -- Fixed QuerySet.annotate(Length(...)).distinct() crash.

Backport of ac5f886c56 from master
This commit is contained in:
Lex Berezhny 2017-01-31 18:45:55 -05:00 committed by Tim Graham
parent 8afe790df9
commit d43f847847
3 changed files with 13 additions and 2 deletions

View File

@ -456,6 +456,7 @@ answer newbie questions, and generally made Django that much better:
Leo Shklovskii Leo Shklovskii
Leo Soto <leo.soto@gmail.com> Leo Soto <leo.soto@gmail.com>
lerouxb@gmail.com lerouxb@gmail.com
Lex Berezhny <lex@damoti.com>
Liang Feng <hutuworm@gmail.com> Liang Feng <hutuworm@gmail.com>
limodou limodou
Loek van Gent <loek@barakken.nl> Loek van Gent <loek@barakken.nl>

View File

@ -1345,7 +1345,12 @@ class Query(object):
"querying. If it is a GenericForeignKey, consider " "querying. If it is a GenericForeignKey, consider "
"adding a GenericRelation." % name "adding a GenericRelation." % name
) )
model = field.model._meta.concrete_model try:
model = field.model._meta.concrete_model
except AttributeError:
# QuerySet.annotate() may introduce fields that aren't
# attached to a model.
model = None
else: else:
# We didn't find the current field, so move position back # We didn't find the current field, so move position back
# one step. # one step.

View File

@ -8,7 +8,7 @@ from django.db.models import (
BooleanField, CharField, Count, DateTimeField, ExpressionWrapper, F, Func, BooleanField, CharField, Count, DateTimeField, ExpressionWrapper, F, Func,
IntegerField, NullBooleanField, Q, Sum, Value, IntegerField, NullBooleanField, Q, Sum, Value,
) )
from django.db.models.functions import Lower from django.db.models.functions import Length, Lower
from django.test import TestCase, skipUnlessDBFeature from django.test import TestCase, skipUnlessDBFeature
from django.utils import six from django.utils import six
@ -208,6 +208,11 @@ class NonAggregateAnnotationTestCase(TestCase):
).distinct('test_alias') ).distinct('test_alias')
self.assertEqual(len(people2), 1) self.assertEqual(len(people2), 1)
lengths = Employee.objects.annotate(
name_len=Length('first_name'),
).distinct('name_len').values_list('name_len', flat=True)
self.assertSequenceEqual(lengths, [3, 7, 8])
def test_filter_annotation(self): def test_filter_annotation(self):
books = Book.objects.annotate( books = Book.objects.annotate(
is_book=Value(1, output_field=IntegerField()) is_book=Value(1, output_field=IntegerField())