Ticket #16479: 16479-formset-forms-error_class-with-tests.diff

tests/regressiontests/forms/tests/formsets.py

@@ -1 +1 @@
 # -*- coding: utf-8 -*-
 from django.forms import Form, CharField, IntegerField, ValidationError, DateField
 from django.forms.formsets import formset_factory, BaseFormSet
+from django.forms.util import ErrorList
 from django.utils.unittest import TestCase
 
 
@@ -805 +806 @@
         self.assertEqual(str(reverse_formset[1]), str(forms[-2]))
         self.assertEqual(len(reverse_formset), len(forms))
 
+    def test_formset_error_class(self):
+        # Regression tests for #16479 -- formsets form use ErrorList instead of supplied error_class
+        class CustomErrorList(ErrorList):
+            pass
+        
+        formset = FavoriteDrinksFormSet(error_class=CustomErrorList)
+        self.assertEqual(formset.forms[0].error_class, CustomErrorList)
+
 data = {
     'choices-TOTAL_FORMS': '1', # the number of forms rendered
     'choices-INITIAL_FORMS': '0', # the number of forms with initial data

django/forms/formsets.py

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