Opened 9 years ago

Last modified 9 years ago

#24933 new Cleanup/optimization

Make FormSet management forms optional

Reported by: Patryk Zawadzki Owned by: nobody
Component: Forms Version: dev
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 by Patryk Zawadzki, 9 years ago

Has patch: set

comment:2 by Tim Graham, 9 years ago

Description: modified (diff)

comment:3 by Tim Graham, 9 years ago

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