Changeset 13201 for main/waeup.kofa/trunk/src/waeup/kofa/students

Timestamp:

16 Aug 2015, 05:06:02 (9 years ago)

Author:

Henrik Bettermann

Message:

Remove duplicate code.

File:

• main/waeup.kofa/trunk/src/waeup/kofa/students/browser.py (modified) (4 diffs)

main/waeup.kofa/trunk/src/waeup/kofa/students/browser.py

@@ -39 +39 @@
 from waeup.kofa.browser.breadcrumbs import Breadcrumb
 from waeup.kofa.browser.pages import (
-    ContactAdminFormPage, ExportCSVView, doll_up, running_jobs)
+    ContactAdminFormPage, ExportCSVView, doll_up, exports_not_allowed)
 from waeup.kofa.browser.interfaces import ICaptchaManager
 from waeup.kofa.hostels.hostel import NOT_OCCUPIED
@@ -3236 +3236 @@
         if START is None:
             return
-        utils = queryUtility(IKofaUtils)
-        if utils.export_disabled_message():
-            self.flash(utils.export_disabled_message(), type='danger')
-            self.entries = doll_up(self, user=None)
-            return
-        if not utils.expensive_actions_allowed():
-            self.flash(_(
-                "Currently, exporters cannot be started due to high "
-                "system load. Please try again later."), type='danger')
-            return
-        if running_jobs(self):
-            self.flash(_(
-                "Sorry, another export job is running. "
-                "Please try again later."), type='danger')
-            self.entries = doll_up(self, user=None)
+        ena = exports_not_allowed(self)
+        if ena:
+            self.flash(ena, type='danger')
             return
         if payments_start or payments_end:
@@ -3346 +3334 @@
         if START is None:
             return
-        utils = queryUtility(IKofaUtils)
-        if utils.export_disabled_message():
-            self.flash(utils.export_disabled_message(), type='danger')
-            self.entries = doll_up(self, user=None)
-            return
-        if not utils.expensive_actions_allowed():
-            self.flash(_(
-                "Currently, exporters cannot be started due to high "
-                "system load. Please try again later."), type='danger')
-            return
-        if running_jobs(self):
-            self.flash(_(
-                "Sorry, another export job is running. "
-                "Please try again later."), type='danger')
-            self.entries = doll_up(self, user=None)
+        ena = exports_not_allowed(self)
+        if ena:
+            self.flash(ena, type='danger')
             return
         try:
@@ -3442 +3418 @@
         if START is None:
             return
-        utils = queryUtility(IKofaUtils)
-        if utils.export_disabled_message():
-            self.flash(utils.export_disabled_message(), type='danger')
-            self.entries = doll_up(self, user=None)
-            return
-        if not utils.expensive_actions_allowed():
-            self.flash(_(
-                "Currently, exporters cannot be started due to high "
-                "system load. Please try again later."), type='danger')
-            return
-        if running_jobs(self):
-            self.flash(_(
-                "Sorry, another export job is running. "
-                "Please try again later."), type='danger')
-            self.entries = doll_up(self, user=None)
+        ena = exports_not_allowed(self)
+        if ena:
+            self.flash(ena, type='danger')
             return
         if session == 'all':