#15212 closed (fixed)
get_actions() always needs to return SortedDict, not list
Description ¶
All docs state that get_actions() returns a dict. From documentation of ModelAdmin.get_actions():
""" Return a __dictionary__ mapping the names of all actions... """
However:
if self.actions is None or IS_POPUP_VAR in request.GET: return [] # list!
This can lead to problems when you extend that method and expect a dict, and treat its return value as such trying to modify its elements. My patch returns an empty SortedDict.
Change History (3)
by , 14 years ago
Attachment: | fix_get_actions.diff added |
---|
comment:1 by , 14 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
In [15393]: