Changeset 16223 for main/kofacustom.iuokada/trunk/src/kofacustom/iuokada/students/utils.py

Timestamp:

1 Sep 2020, 17:52:36 (4 years ago)

Author:

Henrik Bettermann

Message:

Puuh, there are different municipal fees for fresh and returning students.

File:

• main/kofacustom.iuokada/trunk/src/kofacustom/iuokada/students/utils.py (modified) (4 diffs)

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

@@ -45 +45 @@
     PORTRAIT_CHANGE_STATES = (ADMITTED, CLEARANCE,)
 
-    REQUIRED_PAYMENTS = {
+    REQUIRED_PAYMENTS_FRESH = {
         'registration': 'Registration Fee',
         'book': 'Book Deposit',
         'develop': 'Development Fee',
         'parentsconsult': 'Parents Consultative Forum (PCF) Fee',
-        'municipal': 'Municipal Fee',
+        'municipal_fresh': 'Fresh Students Municipal Fee',
+        }
+
+    REQUIRED_PAYMENTS_RETURNING = {
+        'registration': 'Registration Fee',
+        'book': 'Book Deposit',
+        'develop': 'Development Fee',
+        'parentsconsult': 'Parents Consultative Forum (PCF) Fee',
+        'municipal_returning': 'Returning Students Municipal Fee',
         }
 
@@ -64 +72 @@
 
     def _requiredPaymentsMissing(self, student, session):
-        cats_missing = deepcopy(self.REQUIRED_PAYMENTS)
+        if student.is_fresh:
+            rp = self.REQUIRED_PAYMENTS_FRESH
+        else:
+            rp = self.REQUIRED_PAYMENTS_RETURNING
+        for ticket in student['payments'].values():
+            if ticket.p_category == 'required_combi'and \
+                ticket.p_session == session and \
+                ticket.p_state == 'paid':
+                return
+        cats_missing = deepcopy(rp)
         if len(student['payments']):
-            for category in self.REQUIRED_PAYMENTS.keys():
+            for category in rp.keys():
                 for ticket in student['payments'].values():
-                    if ticket.p_category == 'required_combi' and \
-                        ticket.p_session == session and \
-                        ticket.p_state == 'paid':
-                        cats_missing = None
-                        break
                     if ticket.p_state == 'paid' and \
-                        ticket.p_category.startswith(category) and \
+                        ticket.p_category == category and \
                         ticket.p_session == session:
                         del cats_missing[category]
@@ -107 +119 @@
             p_level = student['studycourse'].current_level
             p_current = True
-            if category in self.REQUIRED_PAYMENTS.keys() + [
-                'schoolfee','schoolfee40','secondinstal'] \
+            if category in self.REQUIRED_PAYMENTS_FRESH.keys() \
+                + self.REQUIRED_PAYMENTS_RETURNING.keys() \
+                + ['schoolfee','schoolfee40','secondinstal'] \
                 and student.state == RETURNING:
                 # In case of school fee or required sundry fee payments the
@@ -200 +213 @@
             p_item = p_item.strip(' + ')
         elif category == 'required_combi':
-            rp = deepcopy(self.REQUIRED_PAYMENTS)
+            if student.is_fresh:
+                rp = deepcopy(self.REQUIRED_PAYMENTS_FRESH)
+            else:
+                rp = deepcopy(self.REQUIRED_PAYMENTS_RETURNING)
             for cat in rp:
                 fee_name = cat + '_fee'