Ticket #11418: formset.diff

django/forms/formsets.py

@@ -92 +92 @@
         Instantiates and returns the i-th form instance in a formset.
         """
         defaults = {'auto_id': self.auto_id, 'prefix': self.add_prefix(i)}
-        if self.data or self.files:
+        if self.is_bound:
             defaults['data'] = self.data
             defaults['files'] = self.files
         if self.initial:

tests/regressiontests/forms/formsets.py

@@ -5 +5 @@
 FormSet allows us to use multiple instance of the same form on 1 page. For now,
 the best way to create a FormSet is by using the formset_factory function.
 
->>> from django.forms import Form, CharField, IntegerField, ValidationError
+>>> from django.forms import Form, CharField, IntegerField, ValidationError, DateField
 >>> from django.forms.formsets import formset_factory, BaseFormSet
 
 >>> class Choice(Form):
@@ -598 +598 @@
 >>> formset.management_form.prefix
 'form'
 
+
+# Regression test for #11418 #################################################
+
+>>> class ArticleForm(Form):
+...     title = CharField()
+...     pub_date = DateField()
+
+>>> ArticleFormSet = formset_factory(ArticleForm)
+>>> formset = ArticleFormSet({})
+>>> formset.initial_form_count()
+0
+>>> formset.total_form_count()
+1
+>>> formset.is_bound
+True
+>>> formset.forms[0].is_bound
+True
+>>> formset.is_valid()
+True
+>>> formset.forms[0].is_valid()
+True
+>>> formset.cleaned_data
+[{}]
+>>> data = {
+...     'form-TOTAL_FORMS': u'2',
+...     'form-INITIAL_FORMS': u'0',
+...     'form-0-title': u'Test',
+...     'form-0-pub_date': u'16 June 1904',
+...     'form-1-title': u'Test',
+...     'form-1-pub_date': u'', # <-- this date is missing but required
+... }
+>>> formset = ArticleFormSet(data)
+>>> formset.is_valid()
+False
+>>> formset.errors
+[{}, {'pub_date': [u'This field is required.']}]
+
 """