Changeset 1596 for WAeUP_SRP/branches/joachim-event-branch/PatchCPSUserFolderUserFolder.py

Timestamp:

19 Mar 2007, 21:45:44 (18 years ago)

Author:

joachim

Message:

merged changes from trunk to joachim-event-branch

File:

• WAeUP_SRP/branches/joachim-event-branch/PatchCPSUserFolderUserFolder.py (modified) (3 diffs)

WAeUP_SRP/branches/joachim-event-branch/PatchCPSUserFolderUserFolder.py

@@ -71 +71 @@
     ## patch to assign dynamic roles for WAeUP
     while 1:
+        if self.isStudent():
+            break
+        groups = self.portal_membership.getAuthenticatedMember().getGroups()
+        if not ("ClearanceOfficers" in groups or "CourseAdvisers" in groups):
+            break
         if callable(real_object) and hasattr(real_object,'im_self'):
             real_object = real_object.im_self
 
+        if real_object is None:
+            break
         if hasattr(real_object,'portal_type') and\
-                   real_object.portal_type not in ("Student","StudentClearance","StudentStudyLevel"):
+                   real_object.portal_type not in ("Student",
+                                                   "StudentClearance",
+                                                   "StudentStudyLevel"):
             break
-
-        #from pdb import set_trace;set_trace()
+        # don't test if it is not a proxy 
+        if real_object.portal_type == real_object.meta_type:
+            break
         # can be later simplified by replacing by students_catalog values - Henrik
         # getattr works always because of acquisition ?! Henrik
@@ -84 +94 @@
         if sc is None:
             break
+        #from pdb import set_trace;set_trace()
         sc_obj = sc.getContent()
         cert_id = sc_obj.study_course
@@ -110 +121 @@
                                                 )
         if real_object.portal_type == "StudentStudyLevel":
-            context_obj = getattr(certificate_obj,real_object.getId())
-            allowed = ('CourseAdviser', 'SectionManager')
+            context_obj = getattr(certificate_obj,real_object.getId(),None)
+            if context_obj is None:
+                break
+            allowed = set(('CourseAdviser', 'SectionManager'))
+        elif real_object.portal_type == "Student" and "CourseAdvisers" in groups:
+            #we need some special processing since CourseAdvisers are only 
+            #specified per StudyLevel
+            allowed = set(('CourseAdviser', 'SectionManager'))
+            for context_obj in certificate_obj.objectValues():
+                dynamic_roles = set(self.getRolesInContext(context_obj))
+                intersect = dynamic_roles & allowed
+                if intersect:
+                    roles.extend(list(intersect))
+            break
         else:
             res = self.portal_catalog(portal_type="Department",id=dep_id)
-            allowed = ('ClearanceOfficer', 'SectionManager')
+            allowed = set(('ClearanceOfficer', 'SectionManager'))
             if len(res) != 1:
                 break
             context_obj = res[0].getObject()
-        dynamic_roles = self.getRolesInContext(context_obj)
-        for dr in allowed:
-            if dr in dynamic_roles:
-                roles.append(dr)
+        dynamic_roles = set(self.getRolesInContext(context_obj))
+        intersect = dynamic_roles & allowed
+        if intersect:
+            roles.extend(list(intersect))
         break
     return roles