diff options
| -rw-r--r-- | django/db/models/base.py | 6 | ||||
| -rw-r--r-- | tests/invalid_models_tests/test_models.py | 12 |
2 files changed, 17 insertions, 1 deletions
diff --git a/django/db/models/base.py b/django/db/models/base.py index fe3d84677a..0b8425aa85 100644 --- a/django/db/models/base.py +++ b/django/db/models/base.py @@ -1708,7 +1708,11 @@ class Model(metaclass=ModelBase): fld = None for part in field.split(LOOKUP_SEP): try: - fld = _cls._meta.get_field(part) + # pk is an alias that won't be found by opts.get_field. + if part == 'pk': + fld = _cls._meta.pk + else: + fld = _cls._meta.get_field(part) if fld.is_relation: _cls = fld.get_path_info()[-1].to_opts.model else: diff --git a/tests/invalid_models_tests/test_models.py b/tests/invalid_models_tests/test_models.py index 02db3ea54a..60b89b6f2e 100644 --- a/tests/invalid_models_tests/test_models.py +++ b/tests/invalid_models_tests/test_models.py @@ -844,6 +844,18 @@ class OtherModelTests(SimpleTestCase): with register_lookup(models.CharField, Lower): self.assertEqual(Model.check(), []) + def test_ordering_pointing_to_related_model_pk(self): + class Parent(models.Model): + pass + + class Child(models.Model): + parent = models.ForeignKey(Parent, models.CASCADE) + + class Meta: + ordering = ('parent__pk',) + + self.assertEqual(Child.check(), []) + def test_ordering_pointing_to_foreignkey_field(self): class Parent(models.Model): pass |