Ticket #8291: 8291_last_patch_evar.diff

django/core/management/validation.py

@@ -283 +283 @@
                 # this format would be nice, but it's a little fiddly).
                 if '__' in field_name:
                     continue
+                # Skip ordering on pk, this is always a valid order_by field
+                # but is an alias and therefore won't be found by
+                # opts.get_field.
+                if field_name == 'pk':
+                    continue
                 try:
                     opts.get_field(field_name, many_to_many=False)
                 except models.FieldDoesNotExist:

tests/modeltests/invalid_models/invalid_models/models.py

@@ -233 +233 @@
 class PrimaryKeyNull(models.Model):
     my_pk_field = models.IntegerField(primary_key=True, null=True)
 
+class OrderByPKModel(models.Model):
+    """
+    Model to test that ordering by pk passes validation.
+    Included due to https://code.djangoproject.com/ticket/8291
+    """
+    name = models.CharField(max_length=100, blank=True)
+
+    class Meta:
+        ordering = ('pk',)
 
 model_errors = """invalid_models.fielderrors: "charfield": CharFields require a "max_length" attribute that is a positive integer.
 invalid_models.fielderrors: "charfield2": CharFields require a "max_length" attribute that is a positive integer.

tests/modeltests/ordering/models.py

@@ -24 +24 @@
 
     def __unicode__(self):
         return self.headline
+
+class ArticlePKOrdering(models.Model):
+    headline = models.CharField(max_length=100)
+    pub_date = models.DateTimeField()
+    class Meta:
+        ordering = ('-pk',)
+
+    def __unicode__(self):
+        return self.headline
+

tests/modeltests/ordering/tests.py

@@ -5 +5 @@
 
 from django.test import TestCase
 
-from .models import Article
+from .models import Article, ArticlePKOrdering
 
 
 class OrderingTests(TestCase):
@@ -137 +137 @@
             ],
             attrgetter("headline")
         )
+
+
+        a1 = ArticlePKOrdering.objects.create(
+            pk=1, headline="Article 1", pub_date=datetime(2005, 7, 26)
+        )
+        a2 = ArticlePKOrdering.objects.create(
+            pk=2, headline="Article 2", pub_date=datetime(2005, 7, 27)
+        )
+        a3 = ArticlePKOrdering.objects.create(
+            pk=3, headline="Article 3", pub_date=datetime(2005, 7, 27)
+        )
+        a4 = ArticlePKOrdering.objects.create(
+            pk=4, headline="Article 4", pub_date=datetime(2005, 7, 28)
+        )
+
+        self.assertQuerysetEqual(
+            ArticlePKOrdering.objects.all(), [
+                "Article 4",
+                "Article 3",
+                "Article 2",
+                "Article 1",
+            ],
+            attrgetter("headline")
+        )
+