Cannot specify 'pk' in model default ordering

[anonymous]

I wanted to make an explicit default ordering over a model's primary key (it does make sense in my app), but manage.py validate complains:
"ordering" refers to "pk", a field that doesn't exist.

Specifying 'id' is fine with it.

[Russell Keith-Magee]

Looks like a case where the 'pk' alias isn't being expanded correctly. I'm not sure we specifically document that this *should* be possible, but it would certainly be consistent.

[Filip Gruszczyński]

I can either change get_field in options to accept pk:

def get_field(self, name, many_to_many=True):

"""
Returns the requested field by name. Raises FieldDoesNotExist on error.
"""
if name == 'pk':

return self.pk

to_search = many_to_many and (self.fields + self.many_to_many) or self.fields
for f in to_search:

if f.name == name:

return f

raise FieldDoesNotExist('%s has no field named %r' % (self.object_name, name))

or check this inside get_validation_errors in django.core.management.validation. I don't know django core that well to decide, which way is better, but it seems to me, that get_field should return some good value also for 'pk'. If someone could make this decision, I will gladly make a patch and write some tests.

[Filip Gruszczyński]

Proper formatting of the code.

    def get_field(self, name, many_to_many=True):
        """
        Returns the requested field by name. Raises FieldDoesNotExist on error.
        """
        if name == 'pk':
            return self.pk
        to_search = many_to_many and (self.fields + self.many_to_many) or self.fields
        for f in to_search:
            if f.name == name:
                return f
        raise FieldDoesNotExist('%s has no field named %r' % (self.object_name, name))

[Filip Gruszczyński]

I am adding two patches. First changes get_field in options, second changes get_validation_errors in validation. Both also have test (which is only a model with ordering by pk specified). I hope one of them is good, if not, I will be happy to provide something better, if only I am advised what can be fixed in this case.

[Karen Tracey]

This was already reported in the still-open #8291.