Changeset 5655

Timestamp:

21 Jan 2011, 15:02:44 (14 years ago)

Author:

Henrik Bettermann

Message:

Extend login procedure by copying passport pictures of transfer students after first-time login.

Location:

WAeUP_SRP/trunk

Files:

• WAeUPTool.py (modified) (1 diff)
• skins/cps_custom/logged_in.py (modified) (8 diffs)

WAeUP_SRP/trunk/WAeUPTool.py

@@ -653 +653 @@
     ###)
 
+    security.declarePublic('loadTransferStudentFoto') ###(
+    def loadTransferStudentFoto(self,student_old,student_new):
+        "return a student passport picture"
+        images_dir_new = getImagesDir(student_new)
+        images_dir_old = getImagesDir(student_old)        
+        if not os.path.exists(images_dir_new):
+            os.mkdir(images_dir_new)                
+        image_name_new = os.path.join(images_dir_new,"passport_%(student_new)s.jpg" % vars())
+        image_name_old = os.path.join(images_dir_old,"passport_%(student_old)s" % vars())
+        #import pdb;pdb.set_trace()
+        for extension in ('.jpg','.JPG'):
+            fullname = "%(image_name_old)s%(extension)s" % vars()
+            if os.path.exists(fullname):
+                copy2(fullname,image_name_new)
+                return "successfully copied passport picture"
+        return "passport picture not found: %s.jpg or .JPG" % image_name_old
+    ###)
 
     security.declareProtected(ModifyPortalContent,'createOne') ###(

WAeUP_SRP/trunk/skins/cps_custom/logged_in.py

@@ -19 +19 @@
         pass
 import DateTime
+import os
 current = DateTime.DateTime()
 import logging
@@ -36 +37 @@
 is_anon = mtool.isAnonymousUser()
 member = mtool.getAuthenticatedMember()
+member_id = str(member)
 
 if context.isStaff():
-    is_unsecure = context.waeup_tool.checkGenericPassword(str(member))
+    is_unsecure = context.waeup_tool.checkGenericPassword(member_id)
     if is_unsecure:
         response.expireCookie('__ac', path='/')
@@ -47 +49 @@
 if not is_anon:
     if "Student" in member.getRoles():
-        to_waeup_student_home = True
         students = context.portal_url.getPortalObject().campus.students
-        student = getattr(students,str(member))
+        student = getattr(students,member_id)
         student_app = getattr(student,'application',None)
         student_per = getattr(student,'personal',None)
+      
 
         #########################################################
@@ -61 +63 @@
             email=request.get("email")
             phone=request.get("phone_nr")
-            context.waeup_tool.makeStudentData(str(member),
+            context.waeup_tool.makeStudentData(member_id,
                                                    email=email,
                                                    phone_nr=phone,
                                                    )
             #d = {}
-            #d['id'] = str(member)
+            #d['id'] = member_id
             #d['email'] = email
             #d['phone'] = phone
             #context.students_catalog.modifyRecord(**d)
         elif student_per is None:
-            context.waeup_tool.makeStudentData(str(member))
-
-        #########################################################
-
-        student_app = getattr(student,'application',None)
+            context.waeup_tool.makeStudentData(member_id)
 
         #########################################################
@@ -101 +99 @@
         else:
             app_doc = student_app.getContent()
+        
+        res = context.students_catalog(id = member_id)
+        matric_no = res[0].matric_no
+        jamb_reg_no = res[0].jamb_reg_no
+        entry_mode = res[0].entry_mode
+        verdict = res[0].verdict
+        level = res[0].level
+        has_level = level
+        has_verdict = verdict and not verdict == 'N/A'      
 
         #########################################################
@@ -124 +131 @@
         # look for passport pictures of returning students
 
-        #if s_review_state == 'returning' and 'passport' not in app_doc.objectIds():
-        # use new test
         if s_review_state == 'returning' and\
-           not context.waeup_tool.picturesExist(('passport',), student.getId()):
+           not context.waeup_tool.picturesExist(('passport',), member_id):
             folder = 'pictures_returning'
-            res = context.students_catalog(id = str(member))
+            #res = context.students_catalog(id = member_id)
             filename = res[0].matric_no.upper()
             msg = context.waeup_tool.loadStudentFoto(student,filename,folder)
             logger.info('%s (%s), %s' % (member,s_review_state,msg))
 
-
-        #########################################################
+        #########################################################
+        
+        # look for passport pictures of transfer students
+
+        if entry_mode == 'transfer' and\
+           s_review_state in ('returning','school_fee_paid',) and\
+           not context.waeup_tool.picturesExist(('passport',), member_id):
+            res_both = context.students_catalog(jamb_reg_no = jamb_reg_no)
+            #set_trace()
+            if len(res_both) == 2:
+                for ts in res_both:
+                    if ts.id != member_id:
+                        msg = context.waeup_tool.loadTransferStudentFoto(ts.id,member_id)
+                        logger.info('%s (%s on transfer), %s' % (member,s_review_state,msg))
+
+        #########################################################        
 
         # look for passport pictures of new students
@@ -144 +163 @@
                               'clearance_requested'
                              )  and not context.waeup_tool.picturesExist(('passport',),
-                                                                         student.getId()):
+                                                                         member_id):
             folder = 'pictures_admitted_latest'
             filename = app_doc.jamb_reg_no.replace('/','_')
@@ -220 +239 @@
             wftool.doActionFor(student_per,'open')
 
-        #########################################################
-
-        res = context.students_catalog(id = str(member))
-        matric_no = res[0].matric_no
-        verdict = res[0].verdict
-        level = res[0].level
-        has_level = level
-        has_verdict = verdict and not verdict == 'N/A'
 
         #########################################################