Changeset 16000 for main/waeup.kofa/trunk/src/waeup/kofa/students/batching.py

Timestamp:

11 Feb 2020, 12:15:26 (5 years ago)

Author:

Henrik Bettermann

Message:

Add unlock_score field to CourseTicketProcessor and extend checkUpdateRequirements.

File:

• main/waeup.kofa/trunk/src/waeup/kofa/students/batching.py (modified) (4 diffs)

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

@@ -637 +637 @@
     Like all other student data importers, this processor also requires
     either `student_id`, `reg_number` or `matric_number` to find the student.
-    Then  it needs `level` and `code` to localize the course ticket.
+    Then it needs `level` and `code` to localize the course ticket.
 
     `checkConversion` first searches the courses catalog for the imported
@@ -650 +650 @@
     nor by import. Other values can be overwritten by import.
 
-    `ticket_session` is an additional attribute which can be used to store the 
+    `ticket_session` is an additional field which can be used to store the
     session of the course when it was taken. Usually this information is 
     redundant because the parent study level object already contains this
     information, except for the study level zero container which can be used to
     store 'orphaned' course tickets.
+
+    `checkUpdateRequirements` ensures that the `score` attribute can't
+    be accidentally overwritten by import in update mode. The `score`
+    attribute can be unlocked by setting the boolean field
+    `unlock_score` = 1.
     """
     grok.implements(IBatchProcessor)
@@ -675 +680 @@
             'mandatory', 'score', 'carry_over', 'automatic',
             'outstanding', 'course_category', 'level_session',
-            'title', 'credits', 'passmark', 'semester', 'ticket_session'
+            'title', 'credits', 'passmark', 'semester', 'ticket_session',
+            'unlock_score'
             ] + self.additional_fields
         return sorted(fields)
@@ -735 +741 @@
         if obj.student.studycourse_locked:
             return 'Studycourse is locked.'
+        if row.get('score',None) and obj.score and not row.get(
+            'unlock_score',None):
+            return 'Score attribute is locked.'
         return None