Opened 17 years ago

Closed 17 years ago

#5353 closed (fixed)

FormSet validation hook(s)

Reported by: jkocherhans Owned by: Honza Král
Component: Forms Version: newforms-admin
Severity: Keywords: sprintsept14
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

We need a hook similar to the clean hook on newforms to do validation on all forms in a FormSet. This would allow min/max checks on the number of forms submitted and also allow a place to put unique_together style validation for inlines.

Attachments (2)

5353-BaseFormSet.clean.patch​ (2.7 KB ) - added by Honza Král 17 years ago.
take one
5353-BaseFormSet.clean_with_simple_tests.patch​ (9.9 KB ) - added by Honza Král 17 years ago.
added some tests and synced with 6245

Download all attachments as: .zip

Change History (7)

comment:1 by jkocherhans, 17 years ago

Status: new → assigned
Triage Stage: Unreviewed → Accepted

comment:2 by Honza Král, 17 years ago

Owner: changed from nobody to Honza Král
Status: assigned → new

by Honza Král, 17 years ago

Attachment: 5353-BaseFormSet.clean.patch​ added

take one

by Honza Král, 17 years ago

Attachment: 5353-BaseFormSet.clean_with_simple_tests.patch​ added

added some tests and synced with 6245

comment:3 by Honza Král, 17 years ago

Keywords: sprintsept14 added

comment:4 by Honza Král, 17 years ago

Has patch: set

added has patch

comment:5 by jkocherhans, 17 years ago

Resolution: → fixed
Status: new → closed

(In [6419]) newforms-admin: Fixed #5353. Added FormSet validation hook. Separated a few things out from the original patch and added more tests. Thanks, Honza Kral.

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