Browse Source

Remove AutoStripWhitespacesMixin, as Django already does it now for CharField/TextField unless strip=False is specified

TheKit 9 years ago
parent
commit
b0db148745
2 changed files with 3 additions and 19 deletions
  1. 2 17
      misago/core/forms.py
  2. 1 2
      misago/core/tests/test_forms.py

+ 2 - 17
misago/core/forms.py

@@ -97,24 +97,9 @@ class IsoDateTimeField(DateTimeField):
 """
 Forms
 """
-class AutoStripWhitespacesMixin(object):
-    autostrip_exclude = []
-
-    def full_clean(self):
-        self.data = self.data.copy()
-        for name, field in self.fields.iteritems():
-            if (field.__class__ in TEXT_BASED_FIELDS and
-                    not name in self.autostrip_exclude):
-                try:
-                    self.data[name] = (self.data[name] or '').strip()
-                except KeyError:
-                    pass
-        return super(AutoStripWhitespacesMixin, self).full_clean()
-
-
-class Form(AutoStripWhitespacesMixin, BaseForm):
+class Form(BaseForm):
     pass
 
 
-class ModelForm(AutoStripWhitespacesMixin, BaseModelForm):
+class ModelForm(BaseModelForm):
     pass

+ 1 - 2
misago/core/tests/test_forms.py

@@ -4,8 +4,7 @@ from misago.core import forms
 
 class MockForm(forms.Form):
     stripme = forms.CharField(required=False)
-    autostrip_exclude = ['dontstripme']
-    dontstripme = forms.CharField(required=False)
+    dontstripme = forms.CharField(required=False, strip=False)
 
 
 class MisagoFormsTests(TestCase):