Changeset 7652 for main/waeup.sirp/trunk/src/waeup/sirp/applicants/applicant.py

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.

File:

• main/waeup.sirp/trunk/src/waeup/sirp/applicants/applicant.py (modified) (2 diffs)

main/waeup.sirp/trunk/src/waeup/sirp/applicants/applicant.py

@@ -33 +33 @@
     IExtFileStore, IPDF, IUserAccount)
 from waeup.sirp.students.vocabularies import RegNumNotInSource
-from waeup.sirp.utils.helpers import attrs_to_fields, get_current_principal
+from waeup.sirp.utils.helpers import attrs_to_fields
 from waeup.sirp.applicants.interfaces import IApplicant, IApplicantEdit
 
@@ -314 +314 @@
     comment = 'Applicant record removed'
     target = applicant.applicant_id
-    # In some tests we don't have a principal
     try:
-        user = get_current_principal().id
-    except (TypeError, AttributeError):
-        return
-    try:
-        grok.getSite()['applicants'].logger.info('%s - %s - %s' % (
-            user, target, comment))
+        grok.getSite()['applicants'].logger.info('%s - %s' % (
+            target, comment))
     except KeyError:
         # If we delete an entire university instance there won't be