Merge pull request #472 from Cadasta/bugfix/#315

Fixes #315 -- Set _errors when contact is removed

cadasta/organization/forms.py

@@ -7,6 +7,7 @@
 from core.util import slugify
 from django.utils.translation import ugettext as _
 from django.db import transaction
+from django.forms.utils import ErrorDict
 
 from leaflet.forms.widgets import LeafletWidget
 from tutelary.models import Role
@@ -63,6 +64,7 @@ def full_clean(self):
         if self.data.get(self.prefix + '-remove') != 'on':
             super().full_clean()
         else:
+            self._errors = ErrorDict()
             self.cleaned_data = {'remove': True}
 
     def clean(self):

cadasta/organization/tests/test_forms.py

@@ -4,6 +4,7 @@
 from zipfile import ZipFile
 
 from django.conf import settings
+from django.forms.utils import ErrorDict
 
 from buckets.test import utils as bucket_uitls
 from buckets.test.storage import FakeS3Storage
@@ -561,6 +562,7 @@ def test_full_clean_remove(self):
         form = forms.ContactsForm(data=data, prefix='contacts')
         form.full_clean()
         assert form.cleaned_data == {'remove': True}
+        assert isinstance(form._errors, ErrorDict)
 
     def test_validate_valid_form_with_email(self):
         data = {