Opened 8 years ago

Closed 8 years ago

Last modified 5 years ago

#7772 closed (fixed)

NFA: Fields/Fieldsets validation should be moved to new validation system

Reported by: Julien Phalip Owned by: nobody
Component: contrib.admin Version: master
Severity: Keywords: validation
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


There's a piece of validation that is still hanging around in

Attachments (1)

fields_and_fields.diff (2.1 KB) - added by Julien Phalip 8 years ago.
Patch + tests

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by Julien Phalip

Attachment: fields_and_fields.diff added

Patch + tests

comment:1 Changed 8 years ago by Julien Phalip

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset

Also check #4305 for another NFA-validation issue.

comment:2 Changed 8 years ago by Brian Rosner

Resolution: fixed
Status: newclosed

(In [7932]) newforms-admin: Fixed #7772 -- Moved the validation check for when both fields and fieldsets are specified on a ModelAdmin to django/contrib/admin/ Thanks Julien Phalip for catching this.

comment:3 Changed 5 years ago by Jacob

milestone: 1.0 alpha

Milestone 1.0 alpha deleted

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