Opened 7 years ago

Closed 7 years ago

#6926 closed (fixed)

newforms-admin ValidationError in BaseFormSet

Reported by: msundstr Owned by: nobody
Component: Forms Version: newforms-admin
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: UI/UX:

Description

The formset management_form is created with the passed-in prefix instead of self.prefix. This causes a ValidationError (ManagementForm data is missing or has been tampered with) when data or files is passed in, since in that case the management_form is created with self.prefix. This looks to be an oversight from a few revs back.

Attachments (1)

formsets.diff (655 bytes) - added by msundstr 7 years ago.
path formsets.py to fix the ValidationError

Download all attachments as: .zip

Change History (3)

Changed 7 years ago by msundstr

path formsets.py to fix the ValidationError

comment:1 Changed 7 years ago by jkocherhans

  • Needs documentation unset
  • Needs tests set
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 7 years ago by jkocherhans

  • Resolution set to fixed
  • Status changed from new to closed

(In [7391]) newforms-admin: Fixed #6926. Formset management forms now use the proper prefix. Thanks, msundstr.

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