Problems with ModelAdmin.get_list_display()
|Reported by:||julien||Owned by:||nobody|
|Cc:||Triage Stage:||Ready for checkin|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
- There is an inconsistency with list_display. If you set list_display to ('bacon', 'cheese',) then get_list_display() will return ('action_checkbox', 'bacon', 'cheese',).
- The presence (or absence) of the action checkbox should be dictated solely by the presence (or absence) of ModelAdmin.actions, not by what is returned by get_list_display().
The attached patch rectifies this behavior by injecting 'action_checkbox' further downstream after 'get_list_display()' is called. This would be a minor backwards-incompatibility for those who have been using the still-unreleased 'get_list_display()' method.