Opened 16 years ago
Closed 11 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: | dev |
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)
Change History (7)
by , 16 years ago
Attachment: | inline-fieldsets.diff added |
---|
comment:1 by , 16 years ago
Cc: | added |
---|
comment:2 by , 16 years ago
Has patch: | set |
---|
comment:3 by , 16 years ago
Triage Stage: | Unreviewed → Design decision needed |
---|
comment:4 by , 14 years ago
Severity: | → Normal |
---|---|
Type: | → New feature |
comment:5 by , 13 years ago
Easy pickings: | unset |
---|---|
Needs tests: | set |
Patch needs improvement: | set |
Triage Stage: | Design decision needed → Accepted |
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 by , 11 years ago
Resolution: | → duplicate |
---|---|
Status: | new → closed |
Duplicate of #18681, I believe.
inline form fieldsets retrieved from 'get_fieldsets' method