Opened 3 years ago

Last modified 3 years ago

#33258 closed Bug

inconsistent use of () and [] for attributes in Admin class - Also class variables — at Version 1

Reported by: Martin Massera Owned by: nobody
Component: contrib.admin Version: 3.2
Severity: Normal Keywords: Admin
Cc: Claude Paroz Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Martin Massera)

In Django's Admin class, most "lists" are initialized as tuples () while two of them (inlines and actions)are initialized as lists []

`
list_display = ('str',)
list_display_links = ()
list_filter = ()
...
list_editable = ()
search_fields = ()
...
inlines = []
...
actions = []
`

This is inconsistent.

And also there is a Since these are declared in the class, they are shared among all instances if this value is not set.

I wanted to add an action to an admin subclass so I did MyAdmin.actions.append('some_action') which added this action to all my admins who had not set a new value to actions. While lists are more semantically correct than tuples, tuples have the advantage of being immutable, so they force users to reset the value every time.

So, two solutions:

  • declare all these attributes as instance variables in the constructor. Possibly setting them all as lists
  • declare all these attributes as tuples, keeping them where they are

Change History (1)

comment:1 by Martin Massera, 3 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.
Back to Top