Skip to content


Subversion checkout URL

You can clone with
Download ZIP
Browse files

Fixed #14366 -- Model.objects.none().values() now correctly returns a…

… QuerySet with no items, rather than raising an Exception. Thanks to Carl Meyer for the patch.

git-svn-id: bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
commit 560fedde4cf26eac23d393ab6a64a1ad6d8513bd 1 parent 2314fad
@alex alex authored
2  django/db/models/
@@ -1021,7 +1021,7 @@ def delete(self):
def _clone(self, klass=None, setup=False, **kwargs):
- c = super(EmptyQuerySet, self)._clone(klass, **kwargs)
+ c = super(EmptyQuerySet, self)._clone(klass, setup=setup, **kwargs)
c._result_cache = []
return c
8 tests/regressiontests/queries/
@@ -4,7 +4,7 @@
from django.db.models import Count
from django.test import TestCase
-from models import Tag, Annotation, DumbCategory, Note, ExtraInfo
+from models import Tag, Annotation, DumbCategory, Note, ExtraInfo, Number
class QuerysetOrderedTests(unittest.TestCase):
@@ -81,3 +81,9 @@ def test_evaluated_queryset_as_argument(self):
self.assertEquals(ExtraInfo.objects.filter(note__in=n_list)[0].info, 'good')
except:'Query should be clonable')
+class EmptyQuerySetTests(TestCase):
+ def test_emptyqueryset_values(self):
+ "#14366 -- calling .values() on an EmptyQuerySet and then cloning that should not cause an error"
+ self.assertEqual(list(Number.objects.none().values('num').order_by('num')), [])

0 comments on commit 560fedd

Please sign in to comment.
Something went wrong with that request. Please try again.