From d82ee32aac323b7260cc95278c69b81d98dbd00a Mon Sep 17 00:00:00 2001 From: David Szotten Date: Thu, 2 Mar 2017 15:25:05 +0000 Subject: [PATCH] Fixed #27889 -- Fixed incorrect check error if ModelAdmin.ordering refers to 'pk'. --- django/contrib/admin/checks.py | 3 ++- tests/modeladmin/test_checks.py | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/django/contrib/admin/checks.py b/django/contrib/admin/checks.py index dad809a826..07d3341569 100644 --- a/django/contrib/admin/checks.py +++ b/django/contrib/admin/checks.py @@ -465,7 +465,8 @@ class BaseModelAdminChecks: else: if field_name.startswith('-'): field_name = field_name[1:] - + if field_name == 'pk': + return [] try: model._meta.get_field(field_name) except FieldDoesNotExist: diff --git a/tests/modeladmin/test_checks.py b/tests/modeladmin/test_checks.py index c90ed64958..decddee591 100644 --- a/tests/modeladmin/test_checks.py +++ b/tests/modeladmin/test_checks.py @@ -794,7 +794,7 @@ class OrderingCheckTests(CheckTestCase): def test_valid_case(self): class TestModelAdmin(ModelAdmin): - ordering = ('name',) + ordering = ('name', 'pk') self.assertIsValid(TestModelAdmin, ValidationTestModel)