Changeset 15998 for main/waeup.kofa

Timestamp:

11 Feb 2020, 07:26:29 (5 years ago)

Author:

Henrik Bettermann

Message:

Add UpdateCourseTicketsView.

Location:

main/waeup.kofa/trunk

Files:

• CHANGES.txt (modified) (1 diff)
• src/waeup/kofa/browser/pages.py (modified) (2 diffs)
• src/waeup/kofa/browser/viewlets.py (modified) (1 diff)
• src/waeup/kofa/students/interfaces.py (modified) (1 diff)
• src/waeup/kofa/students/tests/test_browser.py (modified) (1 diff)
• src/waeup/kofa/students/utils.py (modified) (2 diffs)

main/waeup.kofa/trunk/CHANGES.txt

@@ -4 +4 @@
 1.6.1.dev0 (unreleased)
 =======================
+
+* Add `UpdateCourseTicketsView`.
 
 * Don't show 'Base Data' header on slips if no form fields do

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

@@ -2042 +2042 @@
         return result
 
-class SwitchMaintModePage(UtilityView, grok.View):
+class SwitchMaintModeView(UtilityView, grok.View):
     """Import managers must disable authentication for all other users
     before starting an import. This pages switches maintenance mode
@@ -2553 +2553 @@
     def delLocalRoles(self, **data):
         return del_local_roles(self,2,**data)
+
+class UpdateCourseTicketsView(UtilityView, grok.View):
+    """Udate course tickets.
+    """
+    grok.context(ICourse)
+    grok.name('update_coursetickets')
+    grok.require('waeup.editScores')
+
+    def update(self):
+        ob_class = self.__implemented__.__name__.replace('waeup.kofa.','')
+        grok.getSite().logger.info(
+            '%s - course tickets updated: %s' % (
+                ob_class, self.context.code))
+        number = getUtility(IStudentsUtils).updateCourseTickets(self.context)
+        if number is None:
+            self.flash(_('Current academic session not set.'))
+        elif number is 0:
+            self.flash(_('No course ticket found.'))
+        else:
+            self.flash(
+                _('%s course ticket(s) updated.' % number))
+        self.redirect(self.url(self.context))
+        return
+
+    def render(self):
+        return
 
 #

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

@@ -640 +640 @@
     grok.order(6)
 
+class UpdateCourseTicketsActionButton(ManageActionButton):
+    """ 'Update credits in course tickets' button for courses.
+    """
+    grok.context(ICourse)
+    grok.view(CoursePage)
+    grok.name('updatecoursetickets')
+    grok.require('waeup.editScores')
+    icon = 'actionicon_reload.png'
+    target = 'update_coursetickets'
+    grok.order(7)
+
+    @property
+    def target_url(self):
+        ca = grok.getSite()[
+            'configuration'].current_academic_session
+        if not ca:
+            return ''
+        return self.view.url(self.view.context, self.target)
+
+    @property
+    def text(self):
+        ca = grok.getSite()[
+            'configuration'].current_academic_session
+        st = academic_sessions_vocab.getTerm(ca).title
+        return _('Update session ${a} credits', mapping = {'a':st})
+
+    @property
+    def onclick(self):
+        current_academic_session = grok.getSite()[
+            'configuration'].current_academic_session
+        if not current_academic_session:
+            return
+        message = _("All current session (%s/%s) course tickets of students, "
+                    "who paid school fee but not yet registered their courses, "
+                    "will be updated.\\n\\n"
+                    "You really want to update?") % (
+                        current_academic_session, current_academic_session+1)
+        return "return window.confirm('%s');" % message
+
 class DownloadCSVFileActionButton(ManageActionButton):
     """ 'Download csv file' button for courses.

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

@@ -180 +180 @@
         """
 
+    def updateCourseTickets(course):
+        """Udate registered course tickets if course
+        attributes were changed.
+        """
+
 class IStudentsContainer(IKofaObject):
     """A students container contains university students.

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

@@ -2375 +2375 @@
             in self.browser.contents)
 
+    def test_update_coursetickets(self):
+        IWorkflowState(self.student).setState('school fee paid')
+        studylevel = createObject(u'waeup.StudentStudyLevel')
+        studylevel.level = 100
+        studylevel.level_session = 2015
+        self.student['studycourse'].entry_mode = 'ug_ft'
+        self.student['studycourse'].addStudentStudyLevel(
+            self.certificate, studylevel)
+        self.browser.addHeader('Authorization', 'Basic mgr:mgrpw')
+        self.browser.open(
+            'http://localhost/app/faculties/fac1/dep1/courses/COURSE1/')
+        self.assertFalse(
+            'Update session 2015/2016 credits' in self.browser.contents)
+        self.app['configuration'].current_academic_session = 2015
+        self.browser.open(
+            'http://localhost/app/faculties/fac1/dep1/courses/COURSE1/')
+        self.browser.getLink("Update session 2015/2016 credits").click()
+        self.assertTrue(
+            'No course ticket found.' in self.browser.contents)
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'main.log')
+        logcontent = open(logfile).read()
+        self.assertTrue(
+            'zope.mgr - browser.pages.UpdateCourseTicketsView - '
+            'course tickets updated: COURSE1' in logcontent)
+        studylevel['COURSE1'].credits = 12
+        self.browser.getLink("Update session 2015/2016 credits").click()
+        self.assertTrue(
+            'No course ticket found.' in self.browser.contents)
+        studylevel.level_session = 2015
+        self.student['studycourse'].current_session = 2015
+        self.browser.getLink("Update session 2015/2016 credits").click()
+        self.assertTrue(
+            '1 course ticket(s) updated.' in self.browser.contents)
+        logfile = os.path.join(
+            self.app['datacenter'].storage, 'logs', 'students.log')
+        logcontent = open(logfile).read()
+        self.assertTrue(
+            'zope.mgr - students.utils.StudentsUtils - '
+            'K1000000 100/COURSE1 credits updated (10->12)' in logcontent)
+        return
+
 
 class StudentUITests(StudentsFullSetup):

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

@@ -32 +32 @@
 from zope.event import notify
 from zope.schema.interfaces import ConstraintNotSatisfied
-from zope.component import getUtility, createObject
+from zope.component import getUtility, createObject, queryUtility
+from zope.catalog.interfaces import ICatalog
 from zope.formlib.form import setUpEditWidgets
 from zope.i18n import translate
@@ -1148 +1149 @@
         return pdf_stream
 
+    def updateCourseTickets(self, course):
+        """Udate course tickets if course attributes were changed.
+        """
+        current_academic_session = grok.getSite()[
+            'configuration'].current_academic_session
+        if not current_academic_session:
+            return
+        cat = queryUtility(ICatalog, name='coursetickets_catalog')
+        coursetickets = cat.searchResults(
+            code=(course.code, course.code),
+            session=(current_academic_session,current_academic_session))
+        number = 0
+        ob_class = self.__implemented__.__name__.replace('waeup.kofa.', '')
+        for ticket in coursetickets:
+            if ticket.credits == course.credits:
+                continue
+            if ticket.student.current_session != current_academic_session:
+                continue
+            if ticket.student.state not in (PAID,):
+                continue
+            number += 1
+            ticket.student.__parent__.logger.info(
+                '%s - %s %s/%s credits updated (%s->%s)' % (
+                    ob_class, ticket.student.student_id,
+                    ticket.level, ticket.code, course.credits,
+                    ticket.credits))
+            ticket.credits = course.credits
+        return number
+
     #: A dictionary which maps widget names to headlines. The headline
     #: is rendered in forms and on pdf slips above the respective