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

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/browser/pages.py (modified) (2 diffs)

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

@@ -260 +260 @@
             return True
     return False
+
+def exports_not_allowed(view):
+    """Return flash message if exports are not allowed.
+    """
+    utils = queryUtility(IKofaUtils)
+    if utils.export_disabled_message():
+        view.entries = doll_up(view, user=None)
+        return utils.export_disabled_message()
+    if not utils.expensive_actions_allowed():
+        view.entries = doll_up(view, user=None)
+        return _("Currently, exporters cannot be started due to high "
+                 "system load. Please try again later.")
+    if running_jobs(view):
+        view.entries = doll_up(view, user=None)
+        return _("Sorry, another export job is running. "
+                 "Please try again later.")
+    return
 
 def getImporters(context):
@@ -1755 +1772 @@
             return
         if CREATE:
-            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')
-                self.entries = doll_up(self, user=None)
-                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)
-                return
+            ena = exports_not_allowed(self)
+            if ena:
+                self.flash(ena, type='danger')
+                return 
             job_id = self.context.start_export_job(
                 exporter, self.request.principal.id)