Ticket #6241: formset_refactor_7125.diff

a/django/contrib/admin/options.py

@@ -1 +1 @@
-from django import oldforms, template
-from django import newforms as forms
-from django.newforms.formsets import all_valid
+from django.newforms.models import modelform_for_model, inline_formset
-from django.contrib.contenttypes.models import ContentType
-from django.contrib.admin import widgets
-from django.contrib.admin.util import get_deleted_objects
@@ -342 +343 @@
-            fields = flatten_fieldsets(self.declared_fieldsets)
-        else:
-            fields = None
-forms.
+model
-
-    def form_change(self, request, obj):
-        """
@@ -352 +353 @@
-            fields = flatten_fieldsets(self.declared_fieldsets)
-        else:
-            fields = None
-forms.form_for_instance(obj
+modelform_for_model(self.model
-
-    def save_add(self, request, model, form, formsets, post_url_continue):
-        """
@@ -492 +493 @@
-            # Object list will give 'Permission Denied', so go back to admin home
-            post_url = '../../../'
-
-Model
+
-        inline_formsets = []
-        obj = self.model()
-        if request.method == 'POST':
-Model
+
-            for FormSet in self.formsets_add(request):
-obj, data=request.POST, files=
+request.POST, 
-                inline_formsets.append(inline_formset)
-            if all_valid(inline_formsets) and form.is_valid():
-                return self.save_add(request, model, form, inline_formsets, '../%s/')
-        else:
-Model
+
-            for FormSet in self.formsets_add(request):
-obj
+
-                inline_formsets.append(inline_formset)
-
-        adminForm = AdminForm(form, list(self.fieldsets_add(request)), self.prepopulated_fields)
@@ -552 +553 @@
-        if request.POST and request.POST.has_key("_saveasnew"):
-            return self.add_view(request, form_url='../../add/')
-
-Model
+
-        inline_formsets = []
-        if request.method == 'POST':
-ModelForm(request.POST, request.FILES
+Form(request.POST, request.FILES, instance=obj
-            for FormSet in self.formsets_change(request, obj):
-obj, request.POST, request.FILES
+request.POST, request.FILES, instance=obj
-                inline_formsets.append(inline_formset)
-
-            if all_valid(inline_formsets) and form.is_valid():
-                return self.save_change(request, model, form, inline_formsets)
-        else:
-ModelForm(
+Form(instance=obj, initial=request.GET
-            for FormSet in self.formsets_change(request, obj):
-
+instance=
-                inline_formsets.append(inline_formset)
-
-        ## Populate the FormWrapper.
@@ -742 +743 @@
-            fields = flatten_fieldsets(self.declared_fieldsets)
-        else:
-            fields = None
-forms.
+
-
-    def formset_change(self, request, obj):
-        """Returns an InlineFormSet class for use in admin change views."""
@@ -750 +751 @@
-            fields = flatten_fieldsets(self.declared_fieldsets)
-        else:
-            fields = None
-forms.
+
-
-    def fieldsets_add(self, request):
-        if self.declared_fieldsets:
-            return self.declared_fieldsets
- = self.formset_add(request).form_class
-
+set = self.formset_add(request)
+set
-
-    def fieldsets_change(self, request, obj):
-        if self.declared_fieldsets:
-            return self.declared_fieldsets
- = self.formset_change(request, obj).form_class
-
+set = self.formset_change(request, obj)
+set
-
-class StackedInline(InlineModelAdmin):
-    template = 'admin/edit_inline/stacked.html'
@@ -778 +779 @@
-        self.opts = inline
-        self.formset = formset
-        self.fieldsets = fieldsets
+        # place orderable and deletable here since _meta is inaccesible in the
+        # templates.
+        self.orderable = formset._meta.orderable
+        self.deletable = formset._meta.deletable
-
-    def __iter__(self):
-        for form, original in zip(self.formset.change_forms, self.formset.get_queryset()):
@@ -787 +792 @@
-
-    def fields(self):
-        for field_name in flatten_fieldsets(self.fieldsets):
-form_class.
+
-
-class InlineAdminForm(AdminForm):
-    """

a/django/contrib/admin/templates/admin/edit_inline/tabular.html

@@ -11 +11 @@
-         <th {% if forloop.first %}colspan="2"{% endif %}>{{ field.label|capfirst|escape }}</th>
-        {% endif %}
-     {% endfor %}
-formset.
+
-     </tr></thead>
-   
-     {% for inline_admin_form in inline_admin_formset %}
@@ -45 +45 @@
-          {% endfor %}
-        {% endfor %}
-                
-formset.
+
-        
-        </tr>
-

a/django/newforms/formsets.py

@@ -1 @@
-
-
+
+
+
+
+
+
+
+
+
-from widgets import HiddenInput, Media
-from util import ErrorList, ValidationError
-
-
+FormSet', '
-
-# special field names
-FORM_COUNT_FIELD_NAME = 'COUNT'
@@ -20 +27 @@
-        self.base_fields[FORM_COUNT_FIELD_NAME] = IntegerField(widget=HiddenInput)
-        super(ManagementForm, self).__init__(*args, **kwargs)
-
+class FormSetMetaclass(type):
+    def __new__(cls, name, bases, attrs, **options):
+        attrs["base_fields"] = get_declared_fields(bases, attrs)
+        attrs["_meta"] = FormSetOptions(options and options or attrs.get("Meta", None))
+        return type.__new__(cls, name, bases, attrs)
+
-class BaseFormSet(object):
-    """A collection of instances of the same Form class."""
-
@@ -37 +50 @@
-            self.management_form = ManagementForm(data, files, auto_id=self.auto_id, prefix=self.prefix)
-            if self.management_form.is_valid():
-                self.total_forms = self.management_form.cleaned_data[FORM_COUNT_FIELD_NAME]
-
-
+_meta.
+_meta.
-            else:
-                # not sure that ValidationError is the best thing to raise here
-                raise ValidationError('ManagementForm data is missing or has been tampered with')
-        elif initial:
-            self.change_form_count = len(initial)
-            self.required_forms = len(initial)
-
+_meta.
-            self.management_form = ManagementForm(initial={FORM_COUNT_FIELD_NAME: self.total_forms}, auto_id=self.auto_id, prefix=self.prefix)
-        else:
-            self.change_form_count = 0
-            self.required_forms = 0
-
+_meta.
-            self.management_form = ManagementForm(initial={FORM_COUNT_FIELD_NAME: self.total_forms}, auto_id=self.auto_id, prefix=self.prefix)
-
-    def _get_add_forms(self):
-        """Return a list of all the add forms in this ``FormSet``."""
-        FormClass = self.form_class
-        if not hasattr(self, '_add_forms'):
-            add_forms = []
-            for i in range(self.change_form_count, self.total_forms):
@@ -64 +76 @@
-                    kwargs['data'] = self.data
-                if self.files:
-                    kwargs['files'] = self.files
-FormClass
+self.get_form_class()
-                self.add_fields(add_form, i)
-                add_forms.append(add_form)
-            self._add_forms = add_forms
@@ -73 +85 @@
-
-    def _get_change_forms(self):
-        """Return a list of all the change forms in this ``FormSet``."""
-        FormClass = self.form_class
-        if not hasattr(self, '_change_forms'):
-            change_forms = []
-            for i in range(0, self.change_form_count):
@@ -84 +95 @@
-                    kwargs['files'] = self.files
-                if self.initial:
-                    kwargs['initial'] = self.initial[i]
-FormClass
+self.get_form_class(change=True)
-                self.add_fields(change_form, i)
-                change_forms.append(change_form)
-
+ 
-        return self._change_forms
-    change_forms = property(_get_change_forms)
-
@@ -117 +128 @@
-        # Process change forms
-        for form in self.change_forms:
-            if form.is_valid():
-
+_meta.
-                    self.deleted_data.append(form.cleaned_data)
-                else:
-                    self.cleaned_data.append(form.cleaned_data)
@@ -144 +155 @@
-        add_errors.reverse()
-        errors.extend(add_errors)
-        # Sort cleaned_data if the formset is orderable.
-
+_meta.
-            self.cleaned_data.sort(lambda x,y: x[ORDERING_FIELD_NAME] - y[ORDERING_FIELD_NAME])
-        # Give self.clean() a chance to do validation
-        try:
@@ -167 +178 @@
-        via formset.non_form_errors()
-        """
-        return self.cleaned_data
+    
+    def get_form_class(self, change=False):
+        """
+        A hook to change a form class object.
+        """
+        FormClass = self._meta.form
+        FormClass.base_fields = self.base_fields
+        return FormClass
-
-    def add_fields(self, form, index):
-        """A hook for adding extra fields on to each form instance."""
-
+_meta.
-            form.fields[ORDERING_FIELD_NAME] = IntegerField(label='Order', initial=index+1)
-
+_meta.
-            form.fields[DELETION_FIELD_NAME] = BooleanField(label='Delete', required=False)
-
-    def add_prefix(self, index):
@@ -192 +211 @@
-        else:
-            return Media()
-    media = property(_get_media)
-    
-def formset_for_form(form, formset=BaseFormSet, num_extra=1, orderable=False, deletable=False):
-    """Return a FormSet for the given form class."""
-    attrs = {'form_class': form, 'num_extra': num_extra, 'orderable': orderable, 'deletable': deletable}
-    return type(form.__name__ + 'FormSet', (formset,), attrs)
-
+class FormSet(BaseFormSet):
+    __metaclass__ = FormSetMetaclass
+
+def formset_for_form(form, formset=FormSet, num_extra=1, orderable=False,
+                     deletable=False):
+    """Return a FormSet for the given form class."""
+    warn("formset_for_form is deprecated, use FormSet instead.",
+         PendingDeprecationWarning,
+         stacklevel=3)
+    return BaseFormSetMetaclass(
+        form.__name__ + "FormSet", (formset,), form.base_fields,
+        form=form, num_extra=num_extra, orderable=orderable,
+        deletable=deletable)
+        
-def all_valid(formsets):
-    """Returns true if every formset in formsets is valid."""
-    valid = True

a/django/newforms/models.py

@@ -13 +13 @@
-from util import ValidationError, ErrorList
-from forms import BaseForm, get_declared_fields
-from fields import Field, ChoiceField, IntegerField, EMPTY_VALUES
-
+
+
-from widgets import Select, SelectMultiple, HiddenInput, MultipleHiddenInput
-
-__all__ = (
-    'ModelForm', 'BaseModelForm', 'model_to_dict', 'fields_for_model',
-    'save_instance', 'form_for_model', 'form_for_instance', 'form_for_fields',
-formset_for_model', 'inline_forms
+ModelFormSet', 'InlineFormS
-    'ModelChoiceField', 'ModelMultipleChoiceField',
-)
-
@@ -207 +208 @@
-            field_list.append((f.name, formfield))
-    return SortedDict(field_list)
-
-class ModelFormOptions(object):
-    def __init__(self, options=None):
-        self.model = getattr(options, 'model', None)
-        self.fields = getattr(options, 'fields', None)
-        self.exclude = getattr(options, 'exclude', None)
-
-
-class ModelFormMetaclass(type):
+    opts_class = ModelFormOptions
+
-    def __new__(cls, name, bases, attrs,
-
+, **options
-        try:
-
+
+
+
+
+
-        except NameError:
-            # We are defining ModelForm itself.
-            parents = None
@@ -228 +228 @@
-
-        new_class = type.__new__(cls, name, bases, attrs)
-        declared_fields = get_declared_fields(bases, attrs, False)
-ModelFormOptions(
+cls.opts_class(options and options or 
-        if opts.model:
-            # If a model is defined, extract form fields from it.
-            fields = fields_for_model(opts.model, opts.fields,
@@ -276 +276 @@
-class ModelForm(BaseModelForm):
-    __metaclass__ = ModelFormMetaclass
-
+def modelform_for_model(model, form=ModelForm,
+                        formfield_callback=lambda f: f.formfield(), **options):
+    opts = model._meta
+    options.update({"model": model})
+    return ModelFormMetaclass(opts.object_name + "ModelForm", (form,),
+                              {}, formfield_callback, **options)
+
-
-# Fields #####################################################################
-
@@ -390 +397 @@
-
-# Model-FormSet integration ###################################################
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
-
-class BaseModelFormSet(BaseFormSet):
-    """
-    A ``FormSet`` for editing a queryset and/or adding new objects to it.
-    """
-    model = None
-    queryset = None
-
-
+
+
-        kwargs = {'data': data, 'files': files, 'auto_id': auto_id, 'prefix': prefix}
-
-
+
+
+
+
+
+
+
+
+
-        super(BaseModelFormSet, self).__init__(**kwargs)
+    
+    def get_queryset(self, **kwargs):
+        """
+        Hook to returning a queryset for this model.
+        """
+        return self._meta.model._default_manager.all()
-
-    def save_new(self, form, commit=True):
-        """Saves and returns a new model instance for the given form."""
-
+_meta.
-
-    def save_instance(self, form, instance, commit=True):
-        """Saves and returns an existing model instance for the given form."""
@@ -445 +442 @@
-    def save_existing_objects(self, commit=True):
-        if not self.queryset:
-            return []
-get_
+
-        existing_objects = {}
+        opts = self._meta
-        for obj in self.queryset:
-            existing_objects[obj.pk] = obj
-        saved_instances = []
-        for form in self.change_forms:
-self
-self
+opts
+opts
-                obj.delete()
-            else:
-                saved_instances.append(self.save_instance(form, obj, commit=commit))
@@ -460 +458 @@
-
-    def save_new_objects(self, commit=True):
-        new_objects = []
+        opts = self._meta
-        for form in self.add_forms:
-            if form.is_empty():
-                continue
-            # If someone has marked an add form for deletion, don't save the
-            # object. At some point it would be nice if we didn't display
-            # the deletion widget for add forms.
-self
+opts
-                continue
-            new_objects.append(self.save_new(form, commit=commit))
-        return new_objects
-
-    def add_fields(self, form, index):
-        """Add a hidden field for the object's primary key."""
-
+_meta.
-        form.fields[self._pk_field_name] = IntegerField(required=False, widget=HiddenInput)
-        super(BaseModelFormSet, self).add_fields(form, index)
-
-
-
+
+
+
+
+
-    """
-    Returns a FormSet class for the given Django model class. This FormSet
-    will contain change forms for every instance of the given model as well
@@ -487 +489 @@
-    This is essentially the same as ``formset_for_queryset``, but automatically
-    uses the model's default manager to determine the queryset.
-    """
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
-
-InlineForms
+BaseInlineFormS
-    """A formset for child objects related to a parent."""
-instance, data=None, files=None
+*args, **kwargs
-        from django.db.models.fields.related import RelatedObject
-
+
+
-        # is there a better way to get the object descriptor?
-self.rel_name = RelatedObject(self.fk.rel.to, self
-qs = self.get_queryset()
-InlineFormset, self).__init__(qs, data, files, prefix=self.rel_name
+rel_name = RelatedObject(self.fk.rel.to, opts
+kwargs["prefix"] = rel_name
+BaseInlineFormSet, self).__init__(*args, **kwargs
-
-
+, **kwargs
-        """
-        Returns this FormSet's queryset, but restricted to children of 
-        self.instance
-        """
-
-
+
+
+
+
+
-
-    def save_new(self, form, commit=True):
-        kwargs = {self.fk.get_attname(): self.instance.pk}
-
+_meta.
-        return save_instance(form, new_obj, commit=commit)
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
-
-
+
+
-    """
-s
+S
-
-    You must provide ``fk_name`` if ``model`` has more than one ``ForeignKey``
-    to ``parent_model``.
-    """
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
+
+

a/tests/modeltests/model_formsets/models.py

@@ -16 +16 @@
-
-__test__ = {'API_TESTS': """
-
-
+
+
-
-
-
+
+
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
->>> for form in formset.forms:
-...     print form.as_p()
-<p><label for="id_form-0-name">Name:</label> <input id="id_form-0-name" type="text" name="form-0-name" maxlength="100" /><input type="hidden" name="form-0-id" id="id_form-0-id" /></p>
@@ -35 +59 @@
-...     'form-2-name': '',
-... }
-
-qs, data=
+
->>> formset.is_valid()
-True
-
@@ -49 +73 @@
-
-
-Gah! We forgot Paul Verlaine. Let's create a formset to edit the existing
-
-
-
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
->>> for form in formset.forms:
-...     print form.as_p()
-<p><label for="id_form-0-name">Name:</label> <input id="id_form-0-name" type="text" name="form-0-name" value="Arthur Rimbaud" maxlength="100" /><input type="hidden" name="form-0-id" value="2" id="id_form-0-id" /></p>
@@ -73 +104 @@
-...     'form-2-name': 'Paul Verlaine',
-... }
-
-qs, data=
+
->>> formset.is_valid()
-True
-
@@ -90 +121 @@
-This probably shouldn't happen, but it will. If an add form was marked for
-deltetion, make sure we don't save that form.
-
-
-
-
-
+
+
+
+
+
+
+
+
+
+
+
->>> for form in formset.forms:
-...     print form.as_p()
-<p><label for="id_form-0-name">Name:</label> <input id="id_form-0-name" type="text" name="form-0-name" value="Arthur Rimbaud" maxlength="100" /></p>
@@ -117 +155 @@
-...     'form-3-DELETE': 'on',
-... }
-
-qs, data=
+
->>> formset.is_valid()
-True
-
@@ -134 +172 @@
-We can also create a formset that is tied to a parent model. This is how the
-admin system's edit inline functionality works.
-
-
+
+
+
+
+
+
+
+
-
->>> AuthorBooksFormSet = inline_formset(Author, Book, deletable=False, extra=3)
->>> author = Author.objects.get(name='Charles Baudelaire')
-
-
+instance=
->>> for form in formset.forms:
-...     print form.as_p()
-<p><label for="id_book_set-0-title">Title:</label> <input id="id_book_set-0-title" type="text" name="book_set-0-title" maxlength="100" /><input type="hidden" name="book_set-0-id" id="id_book_set-0-id" /></p>
@@ -153 +197 @@
-...     'book_set-2-title': '',
-... }
-
-author, data=data
+data, instance=author
->>> formset.is_valid()
-True
-
@@ -169 +213 @@
-one. This time though, an edit form will be available for every existing
-book.
-
-
+
+
+
+
+
+
+
->>> author = Author.objects.get(name='Charles Baudelaire')
-
-
+instance=
->>> for form in formset.forms:
-...     print form.as_p()
-<p><label for="id_book_set-0-title">Title:</label> <input id="id_book_set-0-title" type="text" name="book_set-0-title" value="Les Fleurs du Mal" maxlength="100" /><input type="hidden" name="book_set-0-id" value="1" id="id_book_set-0-id" /></p>
@@ -187 +237 @@
-...     'book_set-2-title': '',
-... }
-
-author, data=data
+data, instance=author
->>> formset.is_valid()
-True
-

a/tests/regressiontests/forms/formsets.py

@@ -2 +2 @@
-formset_tests = """
-# Basic FormSet creation and usage ############################################
-
-For now,
-best way to create a FormSet is by using the formset_for_form function
+Create
+formset as you would a regular form by defining the fields declaratively
-
->>> from django.newforms import Form, CharField, IntegerField, ValidationError
-
+
+
-
-(Form
+FormSet(FormSet
-...     choice = CharField()
-...     votes = IntegerField()
-
->>> ChoiceFormSet = formset_for_form(Choice)
-
-
-A FormSet constructor takes the same arguments as Form. Let's create a FormSet
-for adding data. By default, it displays 1 blank form. It can display more,
@@ -145 +144 @@
->>> formset.errors
-[{'votes': [u'This field is required.'], 'choice': [u'This field is required.']}]
-
+# Subclassing a FormSet class #################################################
+
+We can subclass a FormSet to add addition fields to an already exisiting
+FormSet.
+
+>>> class SecondChoiceFormSet(ChoiceFormSet):
+...     is_public = BooleanField()
+
+>>> formset = SecondChoiceFormSet(auto_id=False, prefix="choices")
+>>> for form in formset.forms:
+...     print form.as_ul()
+<li>Choice: <input type="text" name="choices-0-choice" /></li>
+<li>Votes: <input type="text" name="choices-0-votes" /></li>
+<li>Is public: <input type="checkbox" name="choices-0-is_public" /></li>
-
-# Displaying more than 1 blank form ###########################################
-
-pass a
-num_extra argument to formset_for_form
+create an inner
+Meta class with an attribute num_extra
-
-
+
+
+
-
-
+NumExtra
->>> for form in formset.forms:
-...    print form.as_ul()
-<li>Choice: <input type="text" name="choices-0-choice" /&