Opened 12 years ago
Closed 12 years ago
#19991 closed New feature (wontfix)
Introduce `saved_forms` property to `BaseModelFormSet`
Reported by: | Ram Rachum | Owned by: | nobody |
---|---|---|---|
Component: | Forms | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Hi,
Here's something that I use in my code and I'm thinking it might be useful to include in Django by default:
@property def saved_forms(self): if not self.is_bound: return () return tuple(form for form in self.forms if (form.instance.pk is not None))
(This is a property of BaseModelFormSet
.)
This allows you to access the forms in the formset that were saved.
What do you think? Would this be useful to add to Django?
(I can easily make this code into a patch if you want.)
Note:
See TracTickets
for help on using tickets.
This seems somewhat domain-specific; I can't see a good reason to include this in Django. If you think I'm missing something, please start a thread on django-dev and maybe explain a bit more about your motivation?