Merge pull request #631 from ostcar/Issue629

Fixed #629

openslides/utils/forms.py

@@ -85,9 +85,14 @@ class CleanHtmlFormMixin(object):
     def clean(self):
         cleaned_data = super(CleanHtmlFormMixin, self).clean()
         for field in self.get_clean_html_fields():
-            cleaned_data[field] = bleach.clean(cleaned_data[field],
-                                               tags=HTML_TAG_WHITELIST,
-                                               attributes=HTML_ATTRIBUTES_WHITELIST,
-                                               styles=HTML_STYLES_WHITELIST,
-                                               strip=True)
+            try:
+                cleaned_data[field] = bleach.clean(
+                    cleaned_data[field],
+                    tags=HTML_TAG_WHITELIST,
+                    attributes=HTML_ATTRIBUTES_WHITELIST,
+                    styles=HTML_STYLES_WHITELIST,
+                    strip=True)
+            except KeyError:
+                # The field 'field' is not pressent. Do not change cleaned_data
+                pass
         return cleaned_data

tests/motion/test_views.py

@@ -127,6 +127,10 @@ class TestMotionCreateView(MotionViewTestCase):
                                                      'identifier': 'foo'})
         self.assertFormError(response, 'form', 'identifier', 'The Identifier is not unique.')
 
+    def test_empty_text_field(self):
+        response = self.admin_client.post(self.url, {'title': 'foo',
+                                                     'submitter': self.admin})
+        self.assertFormError(response, 'form', 'text', 'This field is required.')
 
 
 class TestMotionUpdateView(MotionViewTestCase):