Changeset 15426 for main

Timestamp:

24 May 2019, 16:39:08 (5 years ago)

Author:

Henrik Bettermann

Message:

Catch traceback if current academic session is not set.

File:

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

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

@@ -3603 +3603 @@
     grok.require('waeup.showStudents')
 
+    def update(self):
+        self.current_academic_session = grok.getSite()[
+            'configuration'].current_academic_session
+        if not self.current_academic_session:
+            self.flash(_('Current academic session not set.'), type="danger")
+            self.redirect(self.url(self.context))
+            return
+
     @property
     def note(self):
@@ -3634 +3642 @@
 
     def render(self):
-        session = grok.getSite()['configuration'].current_academic_session
         lecturers = [i['user_title'] for i in self.getUsersWithLocalRoles()
                      if i['local_role'] == 'waeup.local.Lecturer']
@@ -3640 +3647 @@
         students_utils = getUtility(IStudentsUtils)
         return students_utils.renderPDFCourseticketsOverview(
-            self, 'coursetickets', session, self.data(session), lecturers,
+            self, 'coursetickets', self.current_academic_session,
+            self.data(self.current_academic_session), lecturers,
             'landscape', 90, self.note)
 
@@ -3650 +3658 @@
     grok.name('attendance.pdf')
     grok.require('waeup.showStudents')
+
+    def update(self):
+        self.current_academic_session = grok.getSite()[
+            'configuration'].current_academic_session
+        if not self.current_academic_session:
+            self.flash(_('Current academic session not set.'), type="danger")
+            self.redirect(self.url(self.context))
+            return
 
     @property
@@ -3682 +3698 @@
 
     def render(self):
-        session = grok.getSite()['configuration'].current_academic_session
         lecturers = [i['user_title'] for i in self.getUsersWithLocalRoles()
                      if i['local_role'] == 'waeup.local.Lecturer']
@@ -3688 +3703 @@
         students_utils = getUtility(IStudentsUtils)
         return students_utils.renderPDFCourseticketsOverview(
-            self, 'attendance', session, self.data(session),
+            self, 'attendance', self.current_academic_session,
+            self.data(self.current_academic_session),
             lecturers, 'landscape', 90, self.note)