Changeset 2480

Timestamp:

30 Oct 2007, 15:23:25 (17 years ago)

Author:

joachim

Message:

check for existing app_doc, avoid accessviolation in getPaymentsFolderInfo (in custom)

Location:

WAeUP_SRP/base/skins

Files:

• cps_custom/logged_in.py (modified) (3 diffs)
• waeup_epayment/getPaymentsFolderInfo.py (modified) (1 diff)

WAeUP_SRP/base/skins/cps_custom/logged_in.py

@@ -69 +69 @@
 
         student_app = getattr(student,'application',None)
-
+            
         #########################################################
 
@@ -90 +90 @@
         p_review_state = wftool.getInfoFor(student_per,'review_state',None)
         logger.info('%s logged in, review_state %s' % (member,s_review_state))
-        app_doc = student_app.getContent()
+        if student_app is None:
+            logger.info('%s logged in, without application object' % (member))
+        else:
+            app_doc = student_app.getContent()
 
         #########################################################
@@ -96 +99 @@
         # save email and phone of returning students after all objects have been created
 
-        if request.has_key('returning'):
+        if request.has_key('returning') and student_app:
             email=request.get("email")
             phone=request.get("phone_nr")

WAeUP_SRP/base/skins/waeup_epayment/getPaymentsFolderInfo.py

@@ -52 +52 @@
 info['query_url'] = "https://webpay.interswitchng.com/paydirect/services/TransactionQueryURL.aspx"
 review_state = info['review_state'] = s_brain.review_state
-parttime = s_brain.mode.endswith('_pt')
-fulltime = s_brain.mode.endswith('_ft')
+parttime = s_brain.mode and s_brain.mode.endswith('_pt')
+fulltime = s_brain.mode and s_brain.mode.endswith('_ft')
 
 #if review_state == 'cleared_and_validated' or (parttime and review_state == 'returning'):