Opened 8 years ago

Closed 3 years ago

#10607 closed New feature (duplicate)

Use 'get_fieldset' to retieve fieldsets for inline related forms

Reported by: juriejan Owned by: juriejan
Component: contrib.admin Version: master
Severity: Normal Keywords: fieldsets inline related
Cc: juriejanbotha@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Previously the fieldsets for inline related forms where determined only once and then reused for the entire formset. This prevented me from overriding the 'get_fieldset' class in order to alter fieldsets as I wished for specific situations.

With my patch the 'get_fieldsets' method of the InlineModelAdmin class will be called for every form in the formset, allowing for an override that can customize behavior. (As I believe was the original intention with the 'get_fieldset' method)

Attachments (1)

inline-fieldsets.diff (3.5 KB) - added by juriejan 8 years ago.
inline form fieldsets retrieved from 'get_fieldsets' method

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by juriejan

Attachment: inline-fieldsets.diff added

inline form fieldsets retrieved from 'get_fieldsets' method

comment:1 Changed 8 years ago by juriejan

Cc: juriejanbotha@… added
Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset

comment:2 Changed 8 years ago by juriejan

Has patch: set

comment:3 Changed 8 years ago by Jacob

Triage Stage: UnreviewedDesign decision needed

comment:4 Changed 6 years ago by Chris Beaven

Severity: Normal
Type: New feature

comment:5 Changed 5 years ago by Julien Phalip

Easy pickings: unset
Needs tests: set
Patch needs improvement: set
Triage Stage: Design decision neededAccepted
UI/UX: unset

It'd make sense to allow more granularity in the control of inline fieldsets. The admin codebase has changed greatly since 3 years ago so the patch needs to be updated. Some tests would also be required.

comment:6 Changed 3 years ago by Tim Graham

Resolution: duplicate
Status: newclosed

Duplicate of #18681, I believe.

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