diff options
author | Étienne Loks <etienne.loks@iggdrasil.net> | 2016-09-08 12:47:29 +0200 |
---|---|---|
committer | Étienne Loks <etienne.loks@iggdrasil.net> | 2016-09-08 12:47:29 +0200 |
commit | 5b6460ca816302f5b967f1939d55a2ed1f9f0ea4 (patch) | |
tree | e12dec02dc9c32164e0e965f54a50fccf589834b /ishtar_common/forms_common.py | |
parent | 4d1c74baf591f6e021f7820b381fa962e3317bcd (diff) | |
download | Ishtar-5b6460ca816302f5b967f1939d55a2ed1f9f0ea4.tar.bz2 Ishtar-5b6460ca816302f5b967f1939d55a2ed1f9f0ea4.zip |
Imports: explicit error when conservative is check and no unicity key is defined (refs #3181)
Diffstat (limited to 'ishtar_common/forms_common.py')
-rw-r--r-- | ishtar_common/forms_common.py | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/ishtar_common/forms_common.py b/ishtar_common/forms_common.py index 560a1abb3..e288745e2 100644 --- a/ishtar_common/forms_common.py +++ b/ishtar_common/forms_common.py @@ -107,6 +107,16 @@ class NewImportForm(forms.ModelForm): fields = ('importer_type', 'imported_file', 'imported_images', 'conservative_import', 'encoding', 'skip_lines') + def clean(self): + data = self.cleaned_data + if data.get('conservative_import', None) \ + and data.get('importer_type') \ + and not data.get('importer_type').unicity_keys: + raise forms.ValidationError( + _(u"This import type have no unicity type defined. " + u"Conservative import is not possible.")) + return data + def save(self, user, commit=True): self.instance.user = user return super(NewImportForm, self).save(commit) |