Opened 2 years ago
Closed 2 years ago
#19991 closed New feature (wontfix)
Introduce `saved_forms` property to `BaseModelFormSet`
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.)
Change History (1)
comment:1 Changed 2 years ago by jacob
- Needs documentation unset
- Needs tests unset
- Patch needs improvement unset
- Resolution set to wontfix
- Status changed from new to closed
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?