Changeset 13224 for main/waeup.kofa/trunk/src/waeup

Timestamp:

24 Aug 2015, 15:21:21 (9 years ago)

Author:

Henrik Bettermann

Message:

Remove global roles after deletion of applicants and students.

Location:

main/waeup.kofa/trunk/src/waeup/kofa

Files:

• applicants/applicant.py (modified) (2 diffs)
• students/student.py (modified) (1 diff)
• students/utils.py (modified) (1 diff)

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

@@ -391 +391 @@
 @grok.subscribe(IApplicant, grok.IObjectRemovedEvent)
 def handle_applicant_removed(applicant, event):
-    """If an applicant is removed a message is logged.
+    """If an applicant is removed a message is logged, passport images are
+    deleted and the global role is unset.
     """
     comment = 'Application record removed'
@@ -402 +403 @@
         # an applicants subcontainer
         return
-    # Remove also any passport image.
+    # Remove any passport image.
     file_store = getUtility(IExtFileStore)
     file_store.deleteFileByContext(applicant)
+    # Remove global role
+    role_manager = IPrincipalRoleManager(grok.getSite())
+    role_manager.unsetRoleForPrincipal(
+        'waeup.Applicant', applicant.applicant_id)
     return

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

@@ -486 +486 @@
     # update CSV files
     update_student_deletion_csvs(student, del_dir)
+
+    # remove global role
+    role_manager = IPrincipalRoleManager(grok.getSite())
+    role_manager.unsetRoleForPrincipal('waeup.Student', student.student_id)
     return
 

main/waeup.kofa/trunk/src/waeup/kofa/students/utils.py

@@ -560 +560 @@
             student.matric_number = matric_number
         except MatNumNotInSource:
-            return _('Matriculation number exists.'), None
+            return _('Matriculation number %s exists.' % matric_number), None
         notify(grok.ObjectModifiedEvent(student))
         self.increaseMatricInteger(student)