Changeset 13535 for main/waeup.kwarapoly/trunk/src/waeup/kwarapoly/students/utils.py

Timestamp:

7 Dec 2015, 11:44:46 (9 years ago)

Author:

Henrik Bettermann

Message:

Implement new matric number construction method.

Temporarily deactivate the method.

File:

• main/waeup.kwarapoly/trunk/src/waeup/kwarapoly/students/utils.py (modified) (4 diffs)

main/waeup.kwarapoly/trunk/src/waeup/kwarapoly/students/utils.py

@@ -19 +19 @@
 import random
 from time import time
-from zope.component import createObject, getUtility
+from zope.component import createObject, getUtility, queryUtility
 from zope.catalog.interfaces import ICatalog
 from waeup.kofa.interfaces import CLEARED, RETURNING, PAID
@@ -313 +313 @@
 
     def increaseMatricInteger(self, student):
-        """Increase counter for matric numbers.
-        """
-        cert = getattr(student.get('studycourse', None), 'certificate', None)
-        entry_session = getattr(
-            student.get('studycourse', None), 'entry_session', None)
-        dep = cert.__parent__.__parent__
-        next_matric_dict = getattr(dep, 'next_matric_dict', None)
-        if not next_matric_dict:
-            # add next_matric_dict attribute
-            dep.next_matric_dict = {}
-        if not dep.next_matric_dict.get(entry_session, None):
-            # initialize counter element with 11
-            dep.next_matric_dict[entry_session] = 11
-            dep._p_changed = True
-            return
-        # increase counter value of entry_session
-        dep.next_matric_dict[entry_session] += 1
-        dep._p_changed = True
+        """We don't use a persistent counter in Kwarapoly.
+        """
         return
 
@@ -343 +327 @@
         STA = Department Code
         FT = Study Mode Suffix
-        015 = Serial Number (Every department starts from "001" every
+        015 = Serial Number (Every programme starts from "001" every
         session and the numbers build up arithmetically)
         """
+
+        return _('Matriculation assgnment deactivated.'), None
+
         cert = getattr(student.get('studycourse', None), 'certificate', None)
         entry_session = getattr(
@@ -362 +349 @@
         except ValueError:
             return _('Matriculation number cannot be set.'), None
-        dep = cert.__parent__.__parent__
-        next_integer = getattr(
-            dep, 'next_matric_dict', {}).get(entry_session, 10)
-
-        #############################################
-        # self-repair, can be removed after a while
-        if next_integer == 2:
-            next_integer == 10
-            dep.next_matric_dict[entry_session] = 10
-            dep._p_changed = True
-        #############################################
-
         entry_year = entry_session - 100*(entry_session/100)
-        matric_number = "%s/%s/%s/%s/%03d" % (
-            prefix, entry_year, student.depcode, suffix, next_integer)
+        part1 =  "%s/%s/%s/%s/" % (
+            prefix, entry_year, student.depcode, suffix)
+        cat = queryUtility(ICatalog, name='students_catalog')
+        results = cat.searchResults(matric_number=(part1+'0000', part1+'9999'))
+        if not len(results):
+            return None, part1 + '001'
+        try:
+            numbers = [int(i.matric_number.replace(part1, ''))
+                       for i in results if i.matric_number]
+        except ValueError:
+            return _('Matriculation number cannot be determined.'), None
+        counter = max(numbers) + 1
+        matric_number = "%s%03d" % (part1, counter)
         return None, matric_number