Opened 11 years ago

Closed 11 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.)

Change History (1)

comment:1 by Jacob, 11 years ago

Resolution: wontfix
Status: newclosed

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?

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