Changeset 7652 for main/waeup.sirp/trunk/src/waeup/sirp/students

Timestamp:

15 Feb 2012, 10:51:53 (13 years ago)

Author:

Henrik Bettermann

Message:

Do not retrieve current principal in derived logging components. This is now done by the parent Logger class.

Location:

main/waeup.sirp/trunk/src/waeup/sirp/students

Files:

• batching.py (modified) (2 diffs)
• container.py (modified) (2 diffs)
• student.py (modified) (2 diffs)
• workflow.py (modified) (2 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/students/batching.py

@@ -42 +42 @@
 from waeup.sirp.students.workflow import  IMPORTABLE_STATES
 from waeup.sirp.utils.batching import BatchProcessor
-from waeup.sirp.utils.helpers import get_current_principal
 
 class StudentProcessor(BatchProcessor):
@@ -135 +134 @@
         parent = self.getParent(row, site)
         parent.addStudent(obj)
-        # In some tests we don't have a students container or a user
+        # In some tests we don't have a students container
         try:
-            user = get_current_principal()
-            parent.logger.info('%s - %s - Student record imported' % (
-                user.id,obj.student_id))
+            parent.logger.info('%s - Student record imported' % (
+                obj.student_id))
             history = IObjectHistory(obj)
             history.addMessage('Student record imported')

main/waeup.sirp/trunk/src/waeup/sirp/students/container.py

@@ -22 +22 @@
 from waeup.sirp.students.interfaces import (
     IStudentsContainer, IStudent)
-from waeup.sirp.utils.helpers import get_current_principal
 from waeup.sirp.utils.logger import Logger
 
@@ -53 +52 @@
         """Get the logger's info method.
         """
-        user = get_current_principal()
-        if user is None:
-            # In tests we do not necessarily have a logged on principal
-            return
-        user = user.id
-        self.logger.info('%s - %s - %s - %s' % (
-                user, ob_class, target, comment))
+        self.logger.info('%s - %s - %s' % (
+                ob_class, target, comment))
         return

main/waeup.sirp/trunk/src/waeup/sirp/students/student.py

@@ -35 +35 @@
 from waeup.sirp.students.payments import StudentPaymentsContainer
 from waeup.sirp.students.accommodation import StudentAccommodation
-from waeup.sirp.utils.helpers import attrs_to_fields, get_current_principal
+from waeup.sirp.utils.helpers import attrs_to_fields
 from waeup.sirp.students.utils import generate_student_id
 
@@ -174 +174 @@
     comment = 'Student record removed'
     target = student.student_id
-    # In some tests we don't have a principal
     try:
-        user = get_current_principal().id
-    except (TypeError, AttributeError):
-        return
-    try:
-        grok.getSite()['students'].logger.info('%s - %s - %s' % (
-            user, target, comment))
+        grok.getSite()['students'].logger.info('%s - %s' % (
+            target, comment))
     except KeyError:
         # If we delete an entire university instance there won't be

main/waeup.sirp/trunk/src/waeup/sirp/students/workflow.py

@@ -10 +10 @@
     REGISTERED, VALIDATED)
 from waeup.sirp.workflow import SIRPWorkflow, SIRPWorkflowInfo
-from waeup.sirp.utils.helpers import get_current_principal
 from waeup.sirp.students.interfaces import IStudent, IStudentsUtils
 
@@ -197 +196 @@
     if event.transition.transition_id == 'pay_school_fee':
         getUtility(IStudentsUtils).setReturningData(obj)
-    # In some tests we don't have a students container or a user
+    # In some tests we don't have a students container
     try:
-        user = get_current_principal()
         students_container = grok.getSite()['students']
-        students_container.logger.info('%s - %s - %s' % (user.id,obj.student_id,msg))
+        students_container.logger.info('%s - %s' % (obj.student_id,msg))
     except (TypeError, AttributeError):
         pass