Sorting on ForeignKey field whose model uses desc ordering breaks admin change_list

[django@…]

On line 709 of contrib.admin.views.main.ChangeList , the lookup assumes that f.rel.to._meta.ordering[0] will return a valid column name. Not so in the case of '-fieldname'.

[ben.fordnz@…]

Here's a quick hack that solves this problem for me. It's not tested, but it seems to work in my case.
709,713c709
< o_field = f.rel.to._meta.ordering[0]
< if o_field[0] == '-':
< self.order_type = 'desc'
< o_field = o_field[1:]
< rel_ordering = f.rel.to._meta.ordering and o_field or f.rel.to._meta.pk.column
---

rel_ordering = f.rel.to._meta.ordering and f.rel.to._meta.ordering[0] or f.rel.to._meta.pk.column

[Ben Ford <ben.fordnz@…>]

diff of the necessary changes to django.contrib.admin.views.main.py

[Sergey Kirillov <rushman@…>]

Patch from this ticket breaks ability to change sorting of rows by clicking on column header. With this patch clicking on column header has no effect.

[Sergey Kirillov <rushman@…>]

Updated patch

[Gary Wilson <gary.wilson@…>]

It has been mentioned that this ticket might be fixed by #3002.

[Gary Wilson]

Note, this was fixed in newforms-admin branch in [5122].

[James Bennett]

#6127 was a duplicate.

[Brian Rosner]

This is no longer a problem since the merge of newforms-admin in [7967].