From 83e97ecf885676f29170ee036ca210d28dfe19fb Mon Sep 17 00:00:00 2001 From: Malcolm Tredinnick Date: Tue, 22 Jul 2008 07:37:09 +0000 Subject: [PATCH] Fixed #7791 -- Fixed a really silly error I introduced in [7926]. :-( git-svn-id: http://code.djangoproject.com/svn/django/trunk@8052 bcc190cf-cafb-0310-a4f2-bffc1f526a37 --- django/db/models/sql/query.py | 5 +++-- tests/regressiontests/queries/models.py | 5 +++++ 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/django/db/models/sql/query.py b/django/db/models/sql/query.py index 9639ff4956..ce3b49cd42 100644 --- a/django/db/models/sql/query.py +++ b/django/db/models/sql/query.py @@ -1616,8 +1616,9 @@ class Query(object): if self.ordering_aliases: result = order_modified_iter(cursor, len(self.ordering_aliases), self.connection.features.empty_fetchmany_value) - result = iter((lambda: cursor.fetchmany(GET_ITERATOR_CHUNK_SIZE)), - self.connection.features.empty_fetchmany_value) + else: + result = iter((lambda: cursor.fetchmany(GET_ITERATOR_CHUNK_SIZE)), + self.connection.features.empty_fetchmany_value) if not self.connection.features.can_use_chunked_reads: # If we are using non-chunked reads, we return the same data # structure as normally, but ensure it is all read into memory diff --git a/tests/regressiontests/queries/models.py b/tests/regressiontests/queries/models.py index 847d515422..51cae7cde8 100644 --- a/tests/regressiontests/queries/models.py +++ b/tests/regressiontests/queries/models.py @@ -822,6 +822,11 @@ Bug #7759 -- count should work with a partially read result set. ... break True +Bug #7791 -- there were "issues" when ordering and distinct-ing on fields +related via ForeignKeys. +>>> Note.objects.order_by('extrainfo__info').distinct() +[, , ] + """} # In Python 2.3, exceptions raised in __len__ are swallowed (Python issue