Ticket #6790: error_dict_list_fixes_and_overrides_4.diff

django/newforms/forms.py

@@ -64 +64 @@
     # information. Any improvements to the form API should be made to *this*
     # class, not to the Form class.
     def __init__(self, data=None, files=None, auto_id='id_%s', prefix=None,
-                 initial=None, error_class=ErrorList, label_suffix=':'):
+                 initial=None, error_class=ErrorDict, label_suffix=':'):
         self.is_bound = data is not None or files is not None
         self.data = data or {}
         self.files = files or {}
@@ -126 +126 @@
         output, hidden_fields = [], []
         for name, field in self.fields.items():
             bf = BoundField(self, field, name)
-            bf_errors = self.error_class([escape(error) for error in bf.errors]) # Escape and cache in local variable.
+            bf_errors = bf.errors # Escape and cache in local variable.
             if bf.is_hidden:
                 if bf_errors:
                     top_errors.extend([u'(Hidden field %s) %s' % (name, force_unicode(e)) for e in bf_errors])
@@ -182 +182 @@
         field -- i.e., from Form.clean(). Returns an empty ErrorList if there
         are none.
         """
-        return self.errors.get(NON_FIELD_ERRORS, self.error_class())
+        return self.errors.get(NON_FIELD_ERRORS)
 
     def full_clean(self):
         """
         Cleans all of self.data and populates self._errors and
         self.cleaned_data.
         """
-        self._errors = ErrorDict()
+        self._errors = self.error_class()
         if not self.is_bound: # Stop further processing.
             return
         self.cleaned_data = {}
@@ -269 +269 @@
         Returns an ErrorList for this field. Returns an empty ErrorList
         if there are none.
         """
-        return self.form.errors.get(self.name, self.form.error_class())
+        return self.form.errors.get(self.name)
     errors = property(_errors)
 
     def as_widget(self, widget=None, attrs=None):

django/newforms/util.py

@@ -12 +12 @@
     """
     return u''.join([u' %s="%s"' % (k, escape(v)) for k, v in attrs.items()])
 
-class ErrorDict(dict, StrAndUnicode):
+class ErrorList(list, StrAndUnicode):
     """
     A collection of errors that knows how to display itself in various formats.
-
-    The dictionary keys are the field names, and the values are the errors.
     """
     def __unicode__(self):
         return self.as_ul()
@@ -24 +22 @@
     def as_ul(self):
         if not self: return u''
         return mark_safe(u'<ul class="errorlist">%s</ul>'
-                % ''.join([u'<li>%s%s</li>' % (k, force_unicode(v))
-                    for k, v in self.items()]))
+                % ''.join([u'<li>%s</li>' % escape(e) for e in self]))
 
     def as_text(self):
-        return u'\n'.join([u'* %s\n%s' % (k, u'\n'.join([u'  * %s' % force_unicode(i) for i in v])) for k, v in self.items()])
+        if not self: return u''
+        return u'\n'.join([u'* %s' % force_unicode(e) for e in self])
 
-class ErrorList(list, StrAndUnicode):
+    def __repr__(self):
+        return repr([force_unicode(e) for e in self])
+
+class ErrorDict(dict, StrAndUnicode):
     """
     A collection of errors that knows how to display itself in various formats.
+
+    The dictionary keys are the field names, and the values are the errors.
     """
+    error_class = ErrorList
+
     def __unicode__(self):
         return self.as_ul()
 
+    def __getitem__(self, key):
+        value = super(ErrorDict, self).__getitem__(key)
+        if value:
+            return self.error_class(value)
+        return self.error_class()
+
+    def get(self, key, default=None):
+        value = super(ErrorDict, self).get(key, default)
+        if value:
+            return self.error_class(value)
+        return self.error_class()
+
     def as_ul(self):
         if not self: return u''
         return mark_safe(u'<ul class="errorlist">%s</ul>'
-                % ''.join([u'<li>%s</li>' % force_unicode(e) for e in self]))
+                % ''.join([u'<li>%s%s</li>' % (k, force_unicode(v))
+                    for k, v in self.items()]))
 
     def as_text(self):
-        if not self: return u''
-        return u'\n'.join([u'* %s' % force_unicode(e) for e in self])
-
-    def __repr__(self):
-        return repr([force_unicode(e) for e in self])
+        return u'\n'.join([u'* %s\n%s' % (k, u'\n'.join([u'  * %s' % force_unicode(i) for i in v])) for k, v in self.items()])
 
 class ValidationError(Exception):
     def __init__(self, message):

django/test/_doctest.py

@@ -503 +503 @@
         self.globs = globs.copy()
         self.name = name
         self.filename = filename
-        self.lineno = lineno
+        self.lineno = lineno or 0
 
     def __repr__(self):
         if len(self.examples) == 0:

tests/regressiontests/forms/extra.py

@@ -357 +357 @@
 # Test overriding ErrorList in a form #
 #######################################
 
->>> from django.newforms.util import ErrorList
+>>> from django.newforms.util import ErrorDict, ErrorList
 >>> class DivErrorList(ErrorList):
 ...     def __unicode__(self):
 ...         return self.as_divs()
 ...     def as_divs(self):
 ...         if not self: return u''
 ...         return u'<div class="errorlist">%s</div>' % ''.join([u'<div class="error">%s</div>' % force_unicode(e) for e in self])
+>>> class DivErrorDict(ErrorDict):
+...     error_class = DivErrorList
 >>> class CommentForm(Form):
 ...     name = CharField(max_length=50, required=False)
 ...     email = EmailField()
 ...     comment = CharField()
 >>> data = dict(email='invalid')
->>> f = CommentForm(data, auto_id=False, error_class=DivErrorList)
+>>> f = CommentForm(data, auto_id=False, error_class=DivErrorDict)
 >>> print f.as_p()
 <p>Name: <input type="text" name="name" maxlength="50" /></p>
 <div class="errorlist"><div class="error">Enter a valid e-mail address.</div></div>