Changeset 1834

Timestamp:

30 May 2007, 10:07:10 (18 years ago)

Author:

Henrik Bettermann

Message:

logging messages changed

A test run on my system shows that most of the students registered levels for their old study_course not for the study_course they are supposed to study after the transfer. More in ticket #173.

File:

• WAeUP_SRP/trunk/Students.py (modified) (5 diffs)

WAeUP_SRP/trunk/Students.py

@@ -474 +474 @@
             result['id'] = "None"
             if not res:
-                em = 'Student with Matric_no %s not found' % old_matric_no
+                em = 'Student with matric_no %s not found' % old_matric_no
                 logger.info(em)
-                result['Error'] = "Student not exists"
+                result['Error'] = "Student does not exist"
                 not_imported.append(format_error % result)
                 total_not_imported += 1
@@ -487 +487 @@
                 res = self.portal_catalog(meta_type = "Certificate",id = cert_id)
                 if not res:
-                    em = 'No Certificate with ID %s \n' % cert_id
+                    em = 'No certificate with ID %s \n' % cert_id
                     logger.info(em)
                     result['Error'] = "No Certificate %s" % cert_id
@@ -512 +512 @@
             old_current_level = study_course_doc.current_level
             current_level = result.get('current_level',None)
+            new_study_course = result.get('study_course',None)
             try:
                 icl = int(current_level)
@@ -522 +523 @@
                 continue
             if icl <= int(old_current_level) and study_course.objectIds():
-                em = 'Already registered level %s new %s' % (old_current_level,current_level)
+                em = 'Already registered level %s for %s, but is supposed to study %s at level %s' % (old_current_level,old_study_course,new_study_course,current_level)
                 logger.info(em)
                 result['Error'] = em
@@ -543 +544 @@
             study_course_history.append(old_study_course)
             dsc['study_course_history'] = study_course_history
-            dsc['study_course'] = result.get('study_course')
+            dsc['study_course'] = new_study_course
             dsc['current_level'] = current_level
             dsc['current_mode'] = result.get('current_mode')