#12780: Provide a hook for compound form/formset validation in ModelAdmin
-------------------------------------------+--------------------------------
Reporter: mrts | Owner: nobody
Status: new | Milestone:
Component: django.contrib.admin | Version: SVN
Resolution: | Keywords:
Stage: Accepted | Has_patch: 1
Needs_docs: 0 | Needs_tests: 1
Needs_better_patch: 0 |
-------------------------------------------+--------------------------------
Changes (by russellm):
* needs_tests: 0 => 1
* stage: Unreviewed => Accepted
Comment:
Patch needs tests.
Also - regarding the prototype for formsets_are_valid():
* For consistency with other parts of Django (e.g.,
inlineformset_factory), I'd say the argument order should be request,
new_object, form, formset
* Is there a reason that form_validated is required as an argument? I can
understand the "provide everything just in case" approach, but given that
if form_validated=False, it doesn't matter what you return from
formsets_are_valid, it doesn't make much sense to me to pass it in as an
argument. Am I missing something?
--
Ticket URL: <http://code.djangoproject.com/ticket/12780#comment:3>
Django <http://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
