Changeset 15344 for main/kofacustom.edopoly

Timestamp:

6 Mar 2019, 21:21:38 (6 years ago)

Author:

Henrik Bettermann

Message:

Customize increaseMatricInteger.

Location:

main/kofacustom.edopoly/trunk/src/kofacustom/edopoly/students

Files:

• tests/test_utils.py (modified) (4 diffs)
• utils.py (modified) (2 diffs)

main/kofacustom.edopoly/trunk/src/kofacustom/edopoly/students/tests/test_utils.py

@@ -36 +36 @@
         site['configuration'].next_matric_integer_2 = 1
         site['configuration'].next_matric_integer_3 = 1
-        site['configuration'].next_matric_integer_4 = 1
         self.student.matric_number = None
         # nd_ft
@@ -42 +41 @@
         msg, mnumber = utils.setMatricNumber(self.student)
         self.assertEqual(self.student.matric_number, 'fac1/ND/04/00001')
+        self.assertEqual(site['configuration'].next_matric_integer, 2)
         self.assertEqual(msg, None)
         # hnd_ft
@@ -48 +48 @@
         msg, mnumber = utils.setMatricNumber(self.student)
         self.assertEqual(self.student.matric_number, 'fac1/HD/04/00001')
+        self.assertEqual(site['configuration'].next_matric_integer_2, 2)
         self.assertEqual(msg, None)
         # nd_pt
@@ -54 +55 @@
         msg, mnumber = utils.setMatricNumber(self.student)
         self.assertEqual(self.student.matric_number, 'fac1/PT/04/00001')
+        self.assertEqual(site['configuration'].next_matric_integer_3, 2)
         self.assertEqual(msg, None)
         return

main/kofacustom.edopoly/trunk/src/kofacustom/edopoly/students/utils.py

@@ -93 +93 @@
         req_payments_titles = 'ICT and Logbook'
         if len(student['payments']):
-            # All ND and HND part time do not pay LOGBOOK
+            # All ND and HND part time do not pay for LOGBOOK
             if student.current_mode.endswith('_pt'):
                 req_payments = ('ict_entre',)
                 req_payments_titles = 'ICT'
-            # HND2 full time do not pay for SIWES
-            #elif student.current_mode == 'hnd_ft' and student.state == RETURNING:
-            #    req_payments = ('ict_entre', 'logbook_combo')
-            #    req_payments_titles = 'ICT and Logbook'
-            # HND1 full time do not pay both LOGBOOK
-            elif student.current_mode == 'hnd_ft' and student.state == CLEARED:
+            # HND1 and HND2 full time do not pay for LOGBOOK
+            elif student.current_mode == 'hnd_ft' and student.state in (
+                CLEARED, RETURNING):
                 req_payments = ('ict_entre',)
                 req_payments_titles = 'ICT'
-            # ND2 FULL TIME Do not pay LOGBOOK
+            # ND2 FULL TIME do not pay LOGBOOK
             elif student.current_mode == 'nd_ft' and student.state == RETURNING:
                 req_payments = ('ict_entre',)
                 req_payments_titles = 'ICT'
-            # ND1 full time do not pay SIWES
-            #elif student.current_mode == 'nd_ft' and student.state == CLEARED:
-            #    req_payments = ('ict_entre', 'logbook_combo')
-            #    req_payments_titles = 'ICT and Logbook'
             num = 0
             for ticket in student['payments'].values():
@@ -265 +258 @@
         return _('Matriculation number cannot be set.'), None
 
+
+    def increaseMatricInteger(self, student):
+        """Increase counter for matric numbers.
+        """
+        if student.current_mode == 'nd_ft':
+            grok.getSite()['configuration'].next_matric_integer += 1
+            return
+        elif student.current_mode == 'hnd_ft':
+            grok.getSite()['configuration'].next_matric_integer_2 += 1
+            return
+        elif student.current_mode in ('nd_pt', 'hnd_pt'):
+            grok.getSite()['configuration'].next_matric_integer_3 += 1
+            return
+        return
+
     def getAccommodationDetails(self, student):
         """Determine the accommodation data of a student.