Changeset 3903 for WAeUP_SRP/trunk/patches/PatchCPSUserFolderUserFolder.py

Timestamp:

3 Feb 2009, 08:19:08 (16 years ago)

Author:

Henrik Bettermann

Message:

resolve #605

File:

• WAeUP_SRP/trunk/patches/PatchCPSUserFolderUserFolder.py (modified) (2 diffs)

WAeUP_SRP/trunk/patches/PatchCPSUserFolderUserFolder.py

@@ -125 +125 @@
                     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
-            certificate_obj = getCertificateObject(self,st_entry)
-            if certificate_obj is None:
-                #from pdb import set_trace;set_trace()
-                break
-            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=st_entry.department)
@@ -149 +135 @@
         if intersect:
             roles.extend(list(intersect))
+        #from pdb import set_trace;set_trace()            
+
+        if real_object.portal_type == "Student" and "CourseAdvisers" in groups:
+            #we need some special processing since CourseAdvisers are only
+            #specified per StudyLevel
+            certificate_obj = getCertificateObject(self,st_entry)
+            if certificate_obj is None:
+                #from pdb import set_trace;set_trace()
+                break
+            allowed = set(('CourseAdviser', 'SectionManager'))
+            for context_obj in certificate_obj.objectValues():
+                dynamic_roles = set(self.getRolesInContext(context_obj))
+                intersect2 = dynamic_roles & allowed
+                if intersect2:
+                    roles.extend(list(intersect2))
+
         break
     return roles