diff options
| author | Étienne Loks <etienne.loks@peacefrogs.net> | 2011-07-08 00:32:00 +0200 |
|---|---|---|
| committer | Étienne Loks <etienne.loks@peacefrogs.net> | 2011-07-08 00:32:00 +0200 |
| commit | 31b7b29906d8534ccc6318793750ad51650c12a2 (patch) | |
| tree | ded120f51f891ce3cca818e80b9b6e96928d2846 /ishtar/ishtar_base/forms.py | |
| parent | 88d4d40429ea080bf34f3f46764c40ba3982275f (diff) | |
| download | Ishtar-31b7b29906d8534ccc6318793750ad51650c12a2.tar.bz2 Ishtar-31b7b29906d8534ccc6318793750ad51650c12a2.zip | |
Tests to assure that operation code is unique (closes #307)
Diffstat (limited to 'ishtar/ishtar_base/forms.py')
| -rw-r--r-- | ishtar/ishtar_base/forms.py | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/ishtar/ishtar_base/forms.py b/ishtar/ishtar_base/forms.py index 9d106aed0..d5811f38e 100644 --- a/ishtar/ishtar_base/forms.py +++ b/ishtar/ishtar_base/forms.py @@ -332,6 +332,10 @@ class Wizard(NamedUrlSessionFormWizard): if k.startswith('pk'): continue setattr(obj, k, dct[k]) + try: + obj.full_clean() + except ValidationError, msg: + return self.render(request, storage, form_list[-1]) for dependant_item in other_objs: c_item = getattr(obj, dependant_item) # manage ManyToMany if only one associated @@ -368,8 +372,11 @@ class Wizard(NamedUrlSessionFormWizard): dct[dependant_item] = c_item if 'pk' in dct: dct.pop('pk') - print self.get_saved_model() obj = self.get_saved_model()(**dct) + try: + obj.full_clean() + except ValidationError, msg: + return self.render(request, storage, form_list[-1]) obj.save() for k in adds: getattr(obj, k).add(adds[k]) |
