[Mpuls-commits] r3791 - in wasko/trunk: . waskoweb/controllers

[scm-commit@wald.intevation.org]

Author: bh
Date: 2010-09-27 16:48:26 +0200 (Mon, 27 Sep 2010)
New Revision: 3791

Modified:
   wasko/trunk/ChangeLog
   wasko/trunk/waskoweb/controllers/formularpage.py
Log:
* waskoweb/controllers/formularpage.py
(FormularpageController.save): Use the base class method
_check_case_validity for the validity chec.  The inline code for
that feature was identical to the contents of that base class
method.


Modified: wasko/trunk/ChangeLog
===================================================================
--- wasko/trunk/ChangeLog	2010-09-27 14:46:37 UTC (rev 3790)
+++ wasko/trunk/ChangeLog	2010-09-27 14:48:26 UTC (rev 3791)
@@ -2,6 +2,14 @@
 
 	* waskoweb/controllers/formularpage.py
 	(FormularpageController.save): Use the base class method
+	_check_case_validity for the validity chec.  The inline code for
+	that feature was identical to the contents of that base class
+	method.
+
+2010-09-27  Bernhard Herzog  <bh at intevation.de>
+
+	* waskoweb/controllers/formularpage.py
+	(FormularpageController.save): Use the base class method
 	_handle_automatic_phase_change for the automatic phase change
 	feature.  The inline code for that feature was identical to the
 	contents of that base class method.

Modified: wasko/trunk/waskoweb/controllers/formularpage.py
===================================================================
--- wasko/trunk/waskoweb/controllers/formularpage.py	2010-09-27 14:46:37 UTC (rev 3790)
+++ wasko/trunk/waskoweb/controllers/formularpage.py	2010-09-27 14:48:26 UTC (rev 3791)
@@ -114,9 +114,7 @@
                 page_id = session['navigation.tree'].getTreeItem(page_id).nextKey().key
 
             # Check consistence of case and retag
-            if g.mpuls_config.is_enabled ('case-module', 'checktags'):
-                case = self._loadCase(session['case'].id)
-                case.checkValidity()
+            self._check_case_validity(session['case'].id)
 
         except SetDataException, err:
             # Do nothing other than setting the status message. In the future the