Changeset 2466 for WAeUP_SRP/base/skins/waeup_student

Timestamp:

28 Oct 2007, 20:58:41 (17 years ago)

Author:

Henrik Bettermann

Message:

rebuild getNextInfo.py (was completely wrong)
fix makeStudentData
and other changes

Location:

WAeUP_SRP/base/skins/waeup_student

Files:

• getNextInfo.py (modified) (1 diff)
• getSessionResults.py (modified) (3 diffs)
• getStudyLevelInfo.py (modified) (2 diffs)
• session_results_anon_view.pt (modified) (1 diff)
• session_results_view.pt (modified) (1 diff)

WAeUP_SRP/base/skins/waeup_student/getNextInfo.py

@@ -20 +20 @@
 info = {}
 
+reasons = ''
+
 session_id = student_brain.session
 level_id = student_brain.level
 review_state = student_brain.review_state
 verdict = student_brain.verdict
+
+has_verdict = verdict and verdict != 'N/A'
+has_session = session_id and session_id != 'N/A'
+has_level = level_id and level_id != 'N/A'
+
 info['next_transition'] = None
 info['next_session_id'] = next_session_id = session_id
 info['next_level_id'] = next_level_id = level_id
-reasons = []
 verdicts_voc = context.portal_vocabularies.verdicts
 if review_state == 'cleared_and_validated':
     info['next_transition'] = "pay_school_fee"
-elif review_state == 'returning' and verdict and level_id and student_brain.session:
+elif review_state == 'returning' and has_verdict and has_level and has_session:
+    info['next_session_id'] = next_session_id = "%02d" % (int(session_id)+1)
     if int(session_id) > int(context.getSessionId()[0]) -2:
-        if verdict not in ('A','B',):
+        info['next_transition'] = "pay_school_fee"
+        if verdict in ('A','B',):
+            info['next_level_id'] = "%s" % (int(level_id) + 100)
+        else:
             info['next_level_id'] = level_id
-            if not verdict:
-                reasons += "There is no verdict.",
-            else:
-                reasons += "Your are not a %s" % verdicts_voc.get('A'),
-                reasons += "or not a %s" % verdicts_voc.get('B'),
-        else:
-            info['next_level_id'] = "%s" % (int(level_id) + 100)
-            info['next_session_id'] = next_session_id = "%02d" % (int(session_id)+1)
-            info['next_transition'] = "pay_school_fee"
+    else:
+        info['next_level_id'] = "%s" % (int(level_id) + 100)
+elif not (has_session and has_level and has_verdict):
+    reasons = "Your data record is incomplete. School fee payment has been disabled, please contact your faculty!"
+
 info['reasons'] = reasons
 info['next_session_str'] = context.sessions_voc(next_session_id)

WAeUP_SRP/base/skins/waeup_student/getSessionResults.py

@@ -56 +56 @@
     res = results(matric_no = matric_no)
     results = res
-else: 
+else:
     results = []
 
@@ -67 +67 @@
 course_count = 0
 session = None
+level = None
 
 for r in results:
     if session is None:
         session = r.Session
+    if level is None:
+        level = r.Level        
     result = {}
     for field in fields:
@@ -110 +113 @@
     info['verdict'] = verdict = 'N/A'
 
-if student_from_cat.level:
-    if student_from_cat.sex:
-        info['sex'] = 'F'
-    else:
-        info['sex'] = 'M'
-    if verdict in ('A','B'):
-        info['level'] = int(student_from_cat.level)-100
-    else:
-        info['level'] = int(student_from_cat.level)
+if session:
+    info['session'] = session
 else:
-    info['sex'] = 'N/A'
+    info['session'] = 'N/A'
+if level:
+    info['level'] = level
+else:
     info['level'] = 'N/A'
-
-info['session'] = session
 info['gpa'] = gpa
 info['results'] = (sem1, sem2)

WAeUP_SRP/base/skins/waeup_student/getStudyLevelInfo.py

@@ -17 +17 @@
 if mtool.isAnonymousUser():
     return None
+member = mtool.getAuthenticatedMember()
+member_id = str(member)
 
 try:
@@ -92 +94 @@
 if context.objectIds():
     course_results.moveResultsHere(context,student_id)
-    logger.info("%s moved courseresults and deleted objects in level %s" % (student_id,level_id))
+    logger.info("%s initiated moveResultsHere for %s in level %s" % (member_id,student_id,level_id))
 credits_total,carry_overs,normal = course_results.getCourses(student_id,level_id)
 info['credits_total'] = credits_total

WAeUP_SRP/base/skins/waeup_student/session_results_anon_view.pt

@@ -12 +12 @@
         <tr>
           <th width="150px">Matriculation Number:</th><td tal:content="student/matric_no" />                     
-        </tr>
-        <tr>
-          <th>Sex:</th><td tal:content="student/Sex" />                     
         </tr>
         <tr>

WAeUP_SRP/base/skins/waeup_student/session_results_view.pt

@@ -26 +26 @@
         <tr>
           <th width="150px">Matriculation Number:</th><td tal:content="info/student/matric_no" />                     
-        </tr>
-        <tr>
-          <th>Sex:</th><td tal:content="info/sex" />                     
-        </tr>
-        <tr>
-          <th valign="top">Course of Study:</th><td tal:content="info/student/course" />                     
         </tr>
         <tr>