Changeset 10041 for main/waeup.kofa/trunk/src/waeup/kofa/students/webservices.py

Timestamp:

21 Mar 2013, 10:57:48 (12 years ago)

Author:

Henrik Bettermann

Message:

Also matric or reg numbers can be used as student identifiers.

File:

• main/waeup.kofa/trunk/src/waeup/kofa/students/webservices.py (modified) (2 diffs)

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

@@ -22 +22 @@
 from waeup.kofa.interfaces import IUniversity
 
+def get_student(students, identifier):
+    if identifier is None:
+        return None
+    student = students.get(identifier, None)
+    if student is None:
+        cat = queryUtility(ICatalog, name='students_catalog')
+        results = list(
+            cat.searchResults(matric_number=(identifier, identifier)))
+        if len(results) == 1:
+            student = results[0]
+        else:
+            results = list(
+                cat.searchResults(reg_number=(identifier, identifier)))
+            if len(results) == 1:
+                student = results[0]
+    return student
 
 #class XMLRPCPermission(grok.Permission):
@@ -61 +77 @@
 
     @grok.require('waeup.xmlrpc')
-    def get_courses_by_session(self, student_id=None, session=None):
+    def get_courses_by_session(self, identifier=None, session=None):
         """What COURSES are registered by student X in session Y?
 
         """
-        if student_id is None:
+        students = self.context['students']
+        student = get_student(students, identifier)
+        if student is None:
             return None
-        student = self.context['students'].get(student_id, None)
-        if student is None:
-            return  None
         try:
             session = int(session)