Opened 5 years ago

Last modified 5 years ago

#24933 new Cleanup/optimization

Make FormSet management forms optional

Reported by: Patryk Zawadzki Owned by: nobody
Component: Forms Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description (last modified by Tim Graham)

They are only required if the form class used consists of nothing but checkbox fields. Dealing with management forms makes it unnecessarily hard to dynamically alter the list of forms in front-end and forgetting to render one will result in the FormSet.__init__() raising an exception.

I've submitted a discussion to the mailing list but I'm creating a ticket here to provide a pull request for people to play with to make an informed decision.

Change History (3)

comment:1 Changed 5 years ago by Patryk Zawadzki

Has patch: set

comment:2 Changed 5 years ago by Tim Graham

Description: modified (diff)

comment:3 Changed 5 years ago by Tim Graham

Patch needs improvement: set
Triage Stage: UnreviewedAccepted
Type: UncategorizedCleanup/optimization
Note: See TracTickets for help on using tickets.
Back to Top