Changeset 15422 for main/waeup.kofa

Timestamp:

24 May 2019, 09:11:40 (5 years ago)

Author:

Henrik Bettermann

Message:

Implement course result validation workflow for lecturers.

Location:

main/waeup.kofa/trunk

Files:

• CHANGES.txt (modified) (1 diff)
• src/waeup/kofa/app.py (modified) (1 diff)
• src/waeup/kofa/browser/pages.py (modified) (3 diffs)
• src/waeup/kofa/browser/viewlets.py (modified) (2 diffs)
• src/waeup/kofa/permissions.py (modified) (1 diff)
• src/waeup/kofa/students/browser.py (modified) (9 diffs)
• src/waeup/kofa/students/browser_templates/editscorespage.pt (modified) (1 diff)
• src/waeup/kofa/students/tests/test_browser.py (modified) (1 diff)
• src/waeup/kofa/students/viewlets.py (modified) (3 diffs)
• src/waeup/kofa/university/course.py (modified) (1 diff)
• src/waeup/kofa/university/export.py (modified) (1 diff)
• src/waeup/kofa/university/interfaces.py (modified) (2 diffs)
• src/waeup/kofa/university/tests/test_export.py (modified) (4 diffs)

main/waeup.kofa/trunk/CHANGES.txt

@@ -4 +4 @@
 1.6.1.dev0 (unreleased)
 =======================
+
+* Implement course result validation workflow for lecturers.
 
 * Add graduated students filter.

main/waeup.kofa/trunk/src/waeup/kofa/app.py

@@ -87 +87 @@
         XXX: Tests for this method were disabled.
         """
-        for name in ['faculties', 'departments', 'certificates', 'certcourses',
+        for name in ['faculties', 'departments', 'certificates',
+                     'certcourses', 'courses',
                      'site-pluggable-auth']:
             getUtility(IKofaPluggable, name=name).update(

main/waeup.kofa/trunk/src/waeup/kofa/browser/pages.py

@@ -44 +44 @@
 from zope.password.interfaces import IPasswordManager
 from waeup.kofa.utils.helpers import html2dict
+from waeup.kofa.widgets.datewidget import FriendlyDatetimeDisplayWidget
 from waeup.kofa.browser.layout import (
     KofaPage, KofaFormPage, KofaEditFormPage, KofaAddFormPage,
@@ -2484 +2485 @@
     pnav = 1
     form_fields = grok.AutoFields(ICourse)
+    form_fields[
+        'results_validation_date'].custom_widget = FriendlyDatetimeDisplayWidget('le')
 
     @property
@@ -2505 +2508 @@
 
     form_fields = grok.AutoFields(ICourse).omit('code')
+    form_fields['results_validated_by'].for_display = True
+    form_fields['results_validation_date'].for_display = True
+    #form_fields['results_validation_session'].for_display = True
+    form_fields[
+        'results_validation_date'].custom_widget = FriendlyDatetimeDisplayWidget('le')
 
     @action(_('Save'), style='primary')

main/waeup.kofa/trunk/src/waeup/kofa/browser/viewlets.py

@@ -31 +31 @@
     DepartmentPage, CoursePage, CertificatePage, CertificateCoursePage,
     UsersContainerPage, UserManageFormPage, DatacenterImportStep1)
+from waeup.kofa.students.browser import EditScoresPage
 from waeup.kofa.interfaces import MessageFactory as _
 from waeup.kofa.interfaces import (
@@ -615 +616 @@
         return _('Update session ${a} scores', mapping = {'a':st})
 
+class DownloadTicketOverviewActionButton(ManageActionButton):
+    """ 'Download ticket overview' button for courses.
+    """
+    grok.context(ICourse)
+    grok.view(CoursePage)
+    grok.name('coursetickets')
+    grok.require('waeup.showStudents')
+    icon = 'actionicon_pdf.png'
+    text = _('Download course tickets overview')
+    target = 'coursetickets.pdf'
+    grok.order(5)
+
+class DownloadCSVFileActionButton(ManageActionButton):
+    """ 'Download csv file' button for courses.
+    """
+    grok.context(ICourse)
+    grok.view(EditScoresPage)
+    grok.name('downloadcsv')
+    grok.require('waeup.editScores')
+    icon = 'actionicon_down.png'
+    text = _('Download csv file (editable scores only)')
+    target = 'download_scores'
+    grok.order(7)
 
 class ManageCertificateActionButton(ManageActionButton):

main/waeup.kofa/trunk/src/waeup/kofa/permissions.py

@@ -433 +433 @@
     grok.title(u'Lecturer')
     grok.permissions('waeup.editScores',
+                     'waeup.showStudents',
                      'waeup.viewAcademics',
                      'waeup.exportData')

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

@@ -33 +33 @@
 from zope.schema.interfaces import ConstraintNotSatisfied, RequiredMissing
 from zope.security import checkPermission
+from zope.securitypolicy.interfaces import IPrincipalRoleManager
 from waeup.kofa.accesscodes import invalidate_accesscode, get_access_code
 from waeup.kofa.accesscodes.workflow import USED
@@ -3379 +3380 @@
         return
 
-
 class EditScoresPage(KofaPage):
     """Page that allows to edit batches of scores.
@@ -3464 +3464 @@
         return True
 
+    def _validate_results(self, form):
+        ob_class = self.__implemented__.__name__.replace('waeup.kofa.', '')
+        user = get_current_principal()
+        if user is None:
+            usertitle = 'system'
+        else:
+            usertitle = getattr(user, 'public_name', None)
+            if not usertitle:
+                usertitle = user.title
+        self.context.results_validated_by = usertitle
+        self.context.results_validation_date = datetime.utcnow()
+        self.context.results_validation_session = self.current_academic_session
+        return
+
+    def _results_editable(self, results_validation_session,
+                         current_academic_session):
+        user = get_current_principal()
+        prm = IPrincipalRoleManager(self.context)
+        roles = [x[0] for x in prm.getRolesForPrincipal(user.id)]
+        if 'waeup.local.LocalStudentsManager' in roles:
+            return True
+        if results_validation_session \
+            and results_validation_session >= current_academic_session:
+            return False
+        return True
+
     def update(self,  *args, **kw):
         form = self.request.form
@@ -3476 +3502 @@
             self.redirect(self.url(self.context))
             return
+        vs = self.context.results_validation_session
+        if not self._results_editable(vs, self.current_academic_session):
+            self.flash(
+                _('Course results have already been '
+                  'validated and can no longer be changed.'),
+                type="danger")
+            self.redirect(self.url(self.context))
+            return
         self.session_title = academic_sessions_vocab.getTerm(
             self.current_academic_session).title
@@ -3485 +3519 @@
         self.editable_tickets = [
             ticket for ticket in self.tickets if ticket.editable_by_lecturer]
-        if not 'UPDATE_TABLE' in form and not 'UPDATE_FILE' in form:
+        if not 'UPDATE_TABLE' in form and not 'UPDATE_FILE' in form\
+            and not 'VALIDATE_RESULTS' in form:
+            return
+        if 'VALIDATE_RESULTS' in form:
+            if vs and vs >= self.current_academic_session:
+                self.flash(
+                    _('Course results have already been validated.'),
+                    type="danger")
+                return
+            self._validate_results(form)
+            self.flash(_('You successfully validated the course results.'))
+            self.redirect(self.url(self.context))
             return
         if not self.editable_tickets:
@@ -3494 +3539 @@
         return
 
-
 class DownloadScoresView(UtilityView, grok.View):
     """View that exports scores.
@@ -3501 +3545 @@
     grok.require('waeup.editScores')
     grok.name('download_scores')
+
+    def _results_editable(self, results_validation_session,
+                         current_academic_session):
+        user = get_current_principal()
+        prm = IPrincipalRoleManager(self.context)
+        roles = [x[0] for x in prm.getRolesForPrincipal(user.id)]
+        if 'waeup.local.LocalStudentsManager' in roles:
+            return True
+        if results_validation_session \
+            and results_validation_session >= current_academic_session:
+            return False
+        return True
 
     def update(self):
@@ -3511 +3567 @@
         if not self.current_academic_session:
             self.flash(_('Current academic session not set.'), type="warning")
+            self.redirect(self.url(self.context))
+            return
+        vs = self.context.results_validation_session
+        if not self._results_editable(vs, self.current_academic_session):
+            self.flash(
+                _('Course results have already been '
+                  'validated and can no longer be changed.'),
+                type="danger")
             self.redirect(self.url(self.context))
             return
@@ -3537 +3601 @@
     grok.context(ICourse)
     grok.name('coursetickets.pdf')
-    grok.require('waeup.editScores')
+    grok.require('waeup.showStudents')
 
     @property

main/waeup.kofa/trunk/src/waeup/kofa/students/browser_templates/editscorespage.pt

@@ -108 +108 @@
          value="Update scores from table" class="btn btn-primary"
          />
+
+  <input type="submit" name="VALIDATE_RESULTS" i18n:translate=""
+         value="Validate course results" class="btn btn-primary"
+         />
 </form>

main/waeup.kofa/trunk/src/waeup/kofa/students/tests/test_browser.py

@@ -4458 +4458 @@
         self.assertTrue('COURSE1 score updated (None)' in logcontent)
 
+    def test_lecturer_can_validate_courses(self):
+        # the form is locked after validation
+        self.login_as_lecturer()
+        self.student['studycourse']['100']['COURSE1'].score = None
+        self.browser.open(self.edit_scores_url)
+        self.browser.getControl(name="scores:list", index=0).value = ''
+        self.browser.getControl("Update scores").click()
+        self.browser.getControl("Validate").click()
+        self.assertTrue(
+            'You successfully validated the course results'
+            in self.browser.contents)
+        self.assertEqual(self.course.results_validation_session, 2004)
+        self.assertEqual(self.course.results_validated_by, 'Mercedes Benz')
+        self.assertEqual(self.browser.url, self.course_url)
+        # Lecturer can't open edit_scores again
+        self.browser.getLink("Update session 2004/2005 scores").click()
+        self.assertEqual(self.browser.url, self.course_url)
+        self.assertTrue(
+            'Course results have already been validated'
+            ' and can no longer be changed.'
+            in self.browser.contents)
+        # Also DownloadScoresView is blocked
+        self.browser.open(self.browser.url + '/download_scores')
+        self.assertEqual(self.browser.url, self.course_url)
+        self.assertTrue(
+            'Course results have already been validated'
+            ' and can no longer be changed.'
+            in self.browser.contents)
+        # Students Manager can open page ...
+        prmlocal = IPrincipalRoleManager(self.course)
+        prmlocal.assignRoleToPrincipal(
+            'waeup.local.LocalStudentsManager', 'mrslecturer')
+        self.browser.getLink("Update session 2004/2005 scores").click()
+        self.assertEqual(self.browser.url, self.edit_scores_url)
+        self.browser.getLink("Download csv file").click()
+        self.assertEqual(self.browser.headers['Status'], '200 Ok')
+        self.assertEqual(self.browser.headers['Content-Type'],
+                         'text/csv; charset=UTF-8')
+        # ... but can't validate courses a second time
+        self.browser.open(self.edit_scores_url)
+        self.browser.getControl("Validate").click()
+        self.assertTrue(
+            'Course results have already been validated.'
+            in self.browser.contents)
+
     def test_lecturers_can_download_course_tickets(self):
         # A course ticket slip can be downloaded

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

@@ -32 +32 @@
     OnlinePaymentDisplayFormPage, BedTicketDisplayFormPage,
     StudentClearanceEditFormPage, StudentPersonalEditFormPage,
-    PaymentsManageFormPage, StudyCourseTranscriptPage, EditScoresPage)
+    PaymentsManageFormPage, StudyCourseTranscriptPage)
 from waeup.kofa.students.interfaces import (
     IStudentsContainer, IStudent, IStudentStudyCourse, IStudentStudyLevel,
@@ -41 +41 @@
     ADMITTED, PAID, REQUESTED, CLEARED, REGISTERED, VALIDATED, GRADUATED,
     TRANSREQ, TRANSVAL, TRANSREL)
-from waeup.kofa.university.interfaces import ICourse
 
 
@@ -922 +921 @@
             ]
         return targets
-
-
-class DownloadCSVFileActionButton(ManageActionButton):
-    """ 'Download csv file' button for courses.
-    """
-    grok.context(ICourse)
-    grok.view(EditScoresPage)
-    grok.name('downloadcsv')
-    grok.require('waeup.editScores')
-    icon = 'actionicon_down.png'
-    text = _('Download csv file (editable scores only)')
-    target = 'download_scores'
-    grok.order(1)
-
-
-class DownloadTicketOverviewActionButton(ManageActionButton):
-    """ 'Download ticket overview' button for courses.
-    """
-    grok.context(ICourse)
-    grok.view(EditScoresPage)
-    grok.name('coursetickets')
-    grok.require('waeup.editScores')
-    icon = 'actionicon_pdf.png'
-    text = _('Download pdf file')
-    target = 'coursetickets.pdf'
-    grok.order(2)

main/waeup.kofa/trunk/src/waeup/kofa/university/course.py

@@ -56 +56 @@
         self.semester = semester
         self.former_course = former_course
+        self.results_validated_by = None
+        self.results_validation_date = None
+        self.results_validation_session = None
 
     def traverse(self, name):

main/waeup.kofa/trunk/src/waeup/kofa/university/export.py

@@ -116 +116 @@
 
     fields = ('code', 'faculty_code', 'department_code', 'title', 'credits',
-              'passmark', 'semester', 'users_with_local_roles', 'former_course')
+              'passmark', 'semester', 'users_with_local_roles', 'former_course',
+              'results_validated_by', 'results_validation_date',
+              'results_validation_session')
 
     title = _(u'Courses')

main/waeup.kofa/trunk/src/waeup/kofa/university/interfaces.py

@@ -21 +21 @@
 from zope import schema
 from zope.interface import Attribute, invariant, Invalid
-from waeup.kofa.interfaces import IKofaObject, IKofaContainer, validate_id
+from waeup.kofa.interfaces import (
+    IKofaObject, IKofaContainer, validate_id, academic_sessions_vocab)
 from waeup.kofa.interfaces import MessageFactory as _
 from waeup.kofa.university.vocabularies import (
@@ -189 +190 @@
         )
 
+    results_validated_by = schema.TextLine(
+        title = _(u'Results validated by'),
+        default = None,
+        required = False,
+        )
+
+    results_validation_date = schema.Datetime(
+        title = _(u'Results validation date'),
+        required = False,
+        readonly = False,
+        )
+
+    results_validation_session = schema.Choice(
+        title = _(u'Results validation session'),
+        source = academic_sessions_vocab,
+        required = False,
+        readonly = False,
+        )
 
 class ICertificate(IKofaObject):

main/waeup.kofa/trunk/src/waeup/kofa/university/tests/test_export.py

@@ -281 +281 @@
             result,
             'code,faculty_code,department_code,title,credits,'
-            'passmark,semester,users_with_local_roles,former_course\r\n'
-            'C1,F1,D1,Cheese Basics,0,40,1,[],0\r\n'
+            'passmark,semester,users_with_local_roles,former_course,'
+            'results_validated_by,results_validation_date,'
+            'results_validation_session\r\n'
+            'C1,F1,D1,Cheese Basics,0,40,1,[],0,,,\r\n'
             )
         return
@@ -293 +295 @@
             result,
             'code,faculty_code,department_code,title,credits,passmark,'
-            'semester,users_with_local_roles,former_course\r\n'
-            'C1,F1,D1,Cheese Basics,0,40,1,[],0\r\n'
-            'C2,F1,D1,Advanced Cheese Making,0,40,1,[],0\r\n'
+            'semester,users_with_local_roles,former_course,'
+            'results_validated_by,results_validation_date,'
+            'results_validation_session\r\n'
+            'C1,F1,D1,Cheese Basics,0,40,1,[],0,,,\r\n'
+            'C2,F1,D1,Advanced Cheese Making,0,40,1,[],0,,,\r\n'
             )
         return
@@ -307 +311 @@
             result,
             'code,faculty_code,department_code,title,credits,passmark,'
-            'semester,users_with_local_roles,former_course\r\n'
-            'C1,F1,D1,Cheese Basics,0,40,1,[],0\r\n'
-            'C2,F1,D1,Advanced Cheese Making,0,40,1,[],0\r\n'
-            'C3,F1,D2,Selling Cheese,0,40,1,[],0\r\n'
+            'semester,users_with_local_roles,former_course,'
+            'results_validated_by,results_validation_date,'
+            'results_validation_session\r\n'
+            'C1,F1,D1,Cheese Basics,0,40,1,[],0,,,\r\n'
+            'C2,F1,D1,Advanced Cheese Making,0,40,1,[],0,,,\r\n'
+            'C3,F1,D2,Selling Cheese,0,40,1,[],0,,,\r\n'
             )
         return
@@ -321 +327 @@
             result,
             'code,faculty_code,department_code,title,credits,passmark,'
-            'semester,users_with_local_roles,former_course\r\n'
-            'C1,F1,D1,Cheese Basics,0,40,1,[],0\r\n'
-            'C2,F1,D1,Advanced Cheese Making,0,40,1,[],0\r\n'
-            'C3,F1,D2,Selling Cheese,0,40,1,[],0\r\n'
+            'semester,users_with_local_roles,former_course,'
+            'results_validated_by,results_validation_date,'
+            'results_validation_session\r\n'
+            'C1,F1,D1,Cheese Basics,0,40,1,[],0,,,\r\n'
+            'C2,F1,D1,Advanced Cheese Making,0,40,1,[],0,,,\r\n'
+            'C3,F1,D2,Selling Cheese,0,40,1,[],0,,,\r\n'
             )
         return