Changeset 17823

Timestamp:

27 Jun 2024, 09:18:42 (3 months ago)

Author:

Henrik Bettermann

Message:

Make _set_exporter_values customizable.

Location:

main/waeup.kofa/trunk

Files:

• CHANGES.txt (modified) (1 diff)
• src/waeup/kofa/students/browser.py (modified) (1 diff)
• src/waeup/kofa/utils/utils.py (modified) (2 diffs)

main/waeup.kofa/trunk/CHANGES.txt

@@ -4 +4 @@
 1.8.2.dev0 (unreleased)
 =======================
+
+* Make `_set_exporter_values` customizable.
 
 * Limit the file name length to 65 characters.

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

@@ -4344 +4344 @@
 
     def _set_exporter_values(self):
-        # We provide all student exporters, nothing else, yet.
-        # Bursary, Department or Accommodation Officers don't
-        # have the general exportData
-        # permission and are only allowed to export bursary, payments
-        # overview or accommodation data respectively.
-        # This is the only place where waeup.exportAccommodationData,
-        # waeup.exportBursaryData and waeup.exportPaymentsOverview
-        # are used.
+        kofa_utils = getUtility(IKofaUtils)
+        exporters = kofa_utils.collect_exporters(self.context)
+        if exporters:
+            self.exporters = exporters
+            return
         exporters = []
-        if not checkPermission('waeup.exportData', self.context):
-            if checkPermission('waeup.exportBursaryData', self.context):
-                exporters += [('Bursary Data', 'bursary')]
-            if checkPermission('waeup.exportPaymentsOverview', self.context):
-                exporters += [('School Fee Payments Overview',
-                               'sfpaymentsoverview'),
-                              ('Session Payments Overview',
-                               'sessionpaymentsoverview')]
-            if checkPermission('waeup.exportAccommodationData', self.context):
-                exporters += [('Bed Tickets', 'bedtickets'),
-                              ('Accommodation Payments',
-                               'accommodationpayments')]
-            self.exporters = exporters
-            return
         STUDENT_EXPORTER_NAMES = getUtility(
             IStudentsUtils).STUDENT_EXPORTER_NAMES

main/waeup.kofa/trunk/src/waeup/kofa/utils/utils.py

@@ -26 +26 @@
 from random import SystemRandom as r
 from zope.i18n import translate
+from zope.security import checkPermission
 from waeup.kofa.interfaces import IKofaUtils
 from waeup.kofa.interfaces import MessageFactory as _
@@ -567 +568 @@
         value = int(pow(10, prec)*value) / (1.0*pow(10, prec))
         return '{:{width}.{prec}f}'.format(value, width=0, prec=prec)
+
+    def collect_exporters(self, context):
+        # Used for `ExportJobContainerJobConfig`.
+        # We provide all student exporters, nothing else, yet.
+        # Bursary, Department or Accommodation Officers don't
+        # have the general exportData
+        # permission and are only allowed to export bursary, payments
+        # overview or accommodation data respectively.
+        # This is the only place where waeup.exportAccommodationData,
+        # waeup.exportBursaryData and waeup.exportPaymentsOverview
+        # are used.
+        exporters = []
+        if not checkPermission('waeup.exportData', context):
+            if checkPermission('waeup.exportBursaryData', context):
+                exporters += [('Bursary Data', 'bursary')]
+            if checkPermission('waeup.exportPaymentsOverview', context):
+                exporters += [('School Fee Payments Overview',
+                               'sfpaymentsoverview'),
+                              ('Session Payments Overview',
+                               'sessionpaymentsoverview')]
+            if checkPermission('waeup.exportAccommodationData', context):
+                exporters += [('Bed Tickets', 'bedtickets'),
+                              ('Accommodation Payments',
+                               'accommodationpayments')]
+            return exporters