Opened 13 years ago

Closed 13 years ago

#17128 closed Bug (fixed)

Python 2.5-incompatible change in r16319 -- ModelAdmin.list_display.index method called

Reported by: Simon Meers Owned by: Simon Meers
Component: contrib.admin Version: dev
Severity: Normal Keywords: python 2.5, tuple, index, list_display
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

r16319 introduced a python 2.5-incompatible change in which ModelAdmin.list_display's index method is called despite the fact that it is likely a tuple (which only has an index method in 2.6+).

I imagine this managed to hide for so long because:

  • No tests failed -- some use lists for list_display, and most don't have ordering set on the ModelAdmin or Model.Meta
  • ChangeLists for which actions were present (which they are by default) have their list_display listified when action_checkbox is prepended (bypassed if rendering the result_list without using changelist_view)
  • The number of people using Django trunk with Python 2.5 are no doubt a minority

I've attached a patch which addresses the issue and includes regression tests, which I'll commit unless someone thinks it could be done better.

Attachments (1)

17128.diff (8.7 KB ) - added by Simon Meers 13 years ago.

Download all attachments as: .zip

Change History (3)

by Simon Meers, 13 years ago

Attachment: 17128.diff added

comment:1 by Aymeric Augustin, 13 years ago

Triage Stage: UnreviewedReady for checkin

This looks good to me.

comment:2 by Jannis Leidel, 13 years ago

Resolution: fixed
Status: newclosed

In [17102]:

Fixed #17128 -- Fixed a Python 2.5 incompatibility. Thanks, Simon Meers.

Note: See TracTickets for help on using tickets.
Back to Top