Changeset 13200 for main

Timestamp:

15 Aug 2015, 20:20:15 (9 years ago)

Author:

Henrik Bettermann

Message:

Add the same restrictions to student data exporters.

File:

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

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

@@ -38 +38 @@
     NullValidator, jsaction, action, UtilityView)
 from waeup.kofa.browser.breadcrumbs import Breadcrumb
-from waeup.kofa.browser.pages import ContactAdminFormPage, ExportCSVView, doll_up
+from waeup.kofa.browser.pages import (
+    ContactAdminFormPage, ExportCSVView, doll_up, running_jobs)
 from waeup.kofa.browser.interfaces import ICaptchaManager
 from waeup.kofa.hostels.hostel import NOT_OCCUPIED
@@ -3245 +3246 @@
                 "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)
+            return
         if payments_start or payments_end:
             date_format = '%d/%m/%Y'
@@ -3340 +3347 @@
             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)
             return
         try:
@@ -3426 +3443 @@
             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)
             return
         if session == 'all':