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

Timestamp:

12 Dec 2011, 14:11:21 (13 years ago)

Author:

Henrik Bettermann

Message:

Implement local CourseAdviser? roles. These roles can be assigned in departments and certificates. There are 6 different roles, one for each study level. getRolesForPrincipal grants the additional waeup.StudentsCourseAdviser? role only if the current level of a student corresponds with the level number in the external role name.

To do: Assign local roles on CertificateManageFormPage?. Add browser tests.

Location:

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

Files:

• browser.py (modified) (4 diffs)
• dynamicroles.py (modified) (3 diffs)
• permissions.py (modified) (2 diffs)

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

@@ -594 +594 @@
 
     def render(self):
-        self.redirect(self.url(self.context, 'view_clearance'))
         return
 
@@ -615 +614 @@
         else:
             self.flash('Student is in the wrong state.')
+            self.redirect(self.url(self.context,'view_clearance'))
             return
         args = {'subject':message}
@@ -857 +857 @@
     target = 'manage'
 
+class StudentValidateCoursesActionButton(ManageActionButton):
+    grok.order(3)
+    grok.context(IStudentStudyLevel)
+    grok.view(StudyLevelDisplayFormPage)
+    grok.require('waeup.validateStudent')
+    text = 'Validate courses'
+    target = 'validate_courses'
+    icon = 'actionicon_accept.png'
+
+    @property
+    def target_url(self):
+        if self.context.getStudent().state != REGISTERED or \
+            str(self.context.__parent__.current_level) != self.context.__name__:
+            return ''
+        return self.view.url(self.view.context, self.target)
+
+class StudentRejectCoursesActionButton(ManageActionButton):
+    grok.order(4)
+    grok.context(IStudentStudyLevel)
+    grok.view(StudyLevelDisplayFormPage)
+    grok.require('waeup.validateStudent')
+    text = 'Reject courses'
+    target = 'reject_courses'
+    icon = 'actionicon_reject.png'
+
+    @property
+    def target_url(self):
+        if self.context.getStudent().state not in (VALIDATED, REGISTERED) or \
+            str(self.context.__parent__.current_level) != self.context.__name__:
+            return ''
+        return self.view.url(self.view.context, self.target)
+
 class StudyLevelManageFormPage(SIRPEditFormPage):
     """ Page to edit the student study level data
@@ -915 +947 @@
             self.flash('Successfully removed: %s' % ', '.join(deleted))
         self.redirect(self.url(self.context, u'@@manage')+'#tab-2')
+        return
+
+class ValidateCoursesPage(grok.View):
+    """ Validate course list by course adviser
+    """
+    grok.context(IStudentStudyLevel)
+    grok.name('validate_courses')
+    grok.require('waeup.validateStudent')
+
+    def update(self):
+        if str(self.context.__parent__.current_level) != self.context.__name__:
+            self.flash('This level does not correspond current level.')
+        elif self.context.getStudent().state == REGISTERED:
+            IWorkflowInfo(self.context.getStudent()).fireTransition('validate_courses')
+            self.flash('Course list has been registered.')
+        else:
+            self.flash('Student is in the wrong state.')
+        self.redirect(self.url(self.context))
+        return
+
+    def render(self):
+        return
+
+class RejectCoursesPage(grok.View):
+    """ Reject course list by course adviser
+    """
+    grok.context(IStudentStudyLevel)
+    grok.name('reject_courses')
+    grok.require('waeup.validateStudent')
+
+    def update(self):
+        if str(self.context.__parent__.current_level) != self.context.__name__:
+            self.flash('This level does not correspond current level.')
+            self.redirect(self.url(self.context))
+            return
+        elif self.context.getStudent().state == VALIDATED:
+            IWorkflowInfo(self.context.getStudent()).fireTransition('reset8')
+            message = 'Course list request has been annulled'
+            self.flash(message)
+        elif self.context.getStudent().state == REGISTERED:
+            IWorkflowInfo(self.context.getStudent()).fireTransition('reset7')
+            message = 'Course list request has been rejected'
+            self.flash(message)
+        else:
+            self.flash('Student is in the wrong state.')
+            self.redirect(self.url(self.context))
+            return
+        args = {'subject':message}
+        self.redirect(self.url(self.context.getStudent()) +
+            '/contactstudent?%s' % urlencode(args))
+        return
+
+    def render(self):
         return
 

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

@@ -46 +46 @@
     rolename_mapping = {
         'waeup.local.ClearanceOfficer':'waeup.StudentsClearanceOfficer',
-        'waeup.local.CourseAdviser100':'waeup.StudentsCourseAdviser100',
-        'waeup.local.CourseAdviser200':'waeup.StudentsCourseAdviser200',
-        'waeup.local.CourseAdviser300':'waeup.StudentsCourseAdviser300',
-        'waeup.local.CourseAdviser400':'waeup.StudentsCourseAdviser400',
-        'waeup.local.CourseAdviser500':'waeup.StudentsCourseAdviser500',
-        'waeup.local.CourseAdviser600':'waeup.StudentsCourseAdviser600'
+        'waeup.local.CourseAdviser100':'waeup.StudentsCourseAdviser',
+        'waeup.local.CourseAdviser200':'waeup.StudentsCourseAdviser',
+        'waeup.local.CourseAdviser300':'waeup.StudentsCourseAdviser',
+        'waeup.local.CourseAdviser400':'waeup.StudentsCourseAdviser',
+        'waeup.local.CourseAdviser500':'waeup.StudentsCourseAdviser',
+        'waeup.local.CourseAdviser600':'waeup.StudentsCourseAdviser'
         }
 
@@ -100 +100 @@
             obj = getattr(
                 self._context[self.subcontainer], self.extra_attrib, None)
+            current_level = getattr(
+                self._context[self.subcontainer], 'current_level', None)
         else:
             obj = getattr(self._context, self.extra_attrib, None)
+            current_level = None
         # lookup local roles for connected course and all parent
         # objects. This way we fake 'role inheritance'.
@@ -113 +116 @@
                     # permissions (allow, deny or unset) for the
                     # passed in principal id.
-                    result.append(
-                        (self.rolename_mapping[role_id], setting))
+                    # Make an exception for Course Advisers:
+                    # Grant additional role only if external role corresponds
+                    # with current_level of student.
+                    if not current_level or \
+                        'CourseAdviser' not in  role_id or \
+                        str(current_level) in role_id:
+                        result.append(
+                            (self.rolename_mapping[role_id], setting))
                     return result
             obj = getattr(obj, '__parent__', None)

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

@@ -53 +53 @@
     grok.name('waeup.clearStudent')
 
+class ValidateStudent(grok.Permission):
+    grok.name('waeup.validateStudent')
+
 # Local role
 class StudentRecordOwner(grok.Role):
@@ -83 +86 @@
     grok.title(u'Clearance Officer (all students)')
     grok.permissions('waeup.clearStudent','waeup.viewStudent')
+
+class StudentsCourseAdviser(grok.Role):
+    grok.name('waeup.StudentsCourseAdviser')
+    grok.title(u'Course Adviser (all students)')
+    grok.permissions('waeup.validateStudent','waeup.viewStudent')