Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#21006 closed Cleanup/optimization (fixed)

Add an example of the formset argument for inlineformset_factory

Reported by: treysta@… Owned by: Tianyi Wang
Component: Documentation Version: 1.5
Severity: Normal Keywords: afraid-to-commit
Cc: treysta@…, Daniele Procida Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Tim Graham)

https://docs.djangoproject.com/en/1.5/topics/forms/modelforms/#overriding-clean-on-an-inlineformset

It looks like there is a somewhat relevant piece of information missing from this section. It leaves out the fact that you also must specify which formset to use when creating your inlineformset in the view:

Currently:

"See Overriding clean() on a ModelFormSet, but subclass BaseInlineFormSet rather than BaseModelFormSet."

Suggestion:

"See Overriding clean() on a ModelFormSet, but subclass BaseInlineFormSet rather than BaseModelFormSet. You must also specify which formset to use when creating your inlineformset in your view:"

>>> from django.forms.models import inlineformset_factory
>>> BookFormSet = inlineformset_factory(Author, Book, formset= MyInlineFormSet)
>>> author = Author.objects.get(name=u'Mike Royko')
>>> formset = BookFormSet(instance=author)

Change History (12)

comment:1 by Tim Graham, 11 years ago

Description: modified (diff)
Summary: Overriding clean() on an InlineFormSetAdd an example of the formset argument for inlineformset_factory
Triage Stage: UnreviewedAccepted
Type: UncategorizedCleanup/optimization

There's an example for modelformset_factory that uses the formset argument and while you might be able to infer a similar technique will work for inlineformset_factory since the docs say "Inline formsets is a small abstraction layer on top of model formsets", I agree an example could be helpful. I'm not sure it should be specific to the "Overriding clean()" section though, since the formset argument is useful for other cases as well.

If you could offer a patch, I'll be happy to review and commit it. Thanks!

comment:2 by Daniele Procida, 11 years ago

Cc: Daniele Procida added
Keywords: afraid-to-commit added

comment:3 by Daniele Procida, 11 years ago

I've marked this ticket as especially suitable for first-time committers or people following the Don't be afraid to commit tutorial. If you're tackling this ticket, please don't hesitate to ask me for guidance if you'd like any, either here or on the Django IRC channels, where I can be found as EvilDMP.

comment:4 by Tianyi Wang, 11 years ago

Owner: changed from nobody to Tianyi Wang
Status: newassigned

comment:6 by Tim Graham <timograham@…>, 11 years ago

Resolution: fixed
Status: assignedclosed

In 9bfe66164e0e214abc6063a1a60ce729094d0632:

Fixed #21006 -- Added example code for overriding methods on an InlineFormSet.

Thanks treysta at gmail.com for the suggestion.

comment:7 by Tim Graham <timograham@…>, 11 years ago

In ecfd958c84a2b224ff1b71c42cfe481da98c14e6:

[1.6.x] Fixed #21006 -- Added example code for overriding methods on an InlineFormSet.

Thanks treysta at gmail.com for the suggestion.

Backport of 9bfe66164e from master

comment:8 by Tim Graham <timograham@…>, 11 years ago

In 1acd5fc9d293ac07adbb38d11c415dbadd431353:

[1.5.x] Fixed #21006 -- Added example code for overriding methods on an InlineFormSet.

Thanks treysta at gmail.com for the suggestion.

Backport of 9bfe66164e from master

comment:9 by Tianyi Wang <wty52133@…>, 11 years ago

In 944a2bb7c1ffdbf577c3b76acd76b7df7f7e11c0:

Improvement on InlineFormSet doc, refs #21006

comment:10 by Daniele Procida <daniele@…>, 11 years ago

In ec0e780e7ff2f6d4ac22fa8d41040e45b519e80e:

Merge pull request #1751 from tmaster/ticket_21006_improvement

Improvement on InlineFormSet doc, refs #21006

comment:11 by Tim Graham <timograham@…>, 11 years ago

In 312ca5e9cb89af790fa8af04811a3e4c5e621758:

[1.5.x] Improvement on InlineFormSet doc, refs #21006

Backport of 944a2bb7c1 from master

comment:12 by Tim Graham <timograham@…>, 11 years ago

In 1241a2cc104d092ff0f03a16a664e9e4ab801dd1:

[1.6.x] Improvement on InlineFormSet doc, refs #21006

Backport of 944a2bb7c1 from master

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