[7191] | 1 | ## $Id: batching.py 14053 2016-08-05 14:06:31Z henrik $ |
---|
| 2 | ## |
---|
| 3 | ## Copyright (C) 2011 Uli Fouquet & Henrik Bettermann |
---|
| 4 | ## This program is free software; you can redistribute it and/or modify |
---|
| 5 | ## it under the terms of the GNU General Public License as published by |
---|
| 6 | ## the Free Software Foundation; either version 2 of the License, or |
---|
| 7 | ## (at your option) any later version. |
---|
| 8 | ## |
---|
| 9 | ## This program is distributed in the hope that it will be useful, |
---|
| 10 | ## but WITHOUT ANY WARRANTY; without even the implied warranty of |
---|
| 11 | ## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
---|
| 12 | ## GNU General Public License for more details. |
---|
| 13 | ## |
---|
| 14 | ## You should have received a copy of the GNU General Public License |
---|
| 15 | ## along with this program; if not, write to the Free Software |
---|
| 16 | ## Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA |
---|
| 17 | ## |
---|
[7433] | 18 | """Batch processing components for student objects. |
---|
[6821] | 19 | |
---|
| 20 | Batch processors eat CSV files to add, update or remove large numbers |
---|
| 21 | of certain kinds of objects at once. |
---|
| 22 | |
---|
[7261] | 23 | Here we define the processors for students specific objects like |
---|
| 24 | students, studycourses, payment tickets and accommodation tickets. |
---|
[6821] | 25 | """ |
---|
| 26 | import grok |
---|
[10028] | 27 | import unicodecsv as csv # XXX: csv ops should move to dedicated module. |
---|
[8884] | 28 | from time import time |
---|
[9295] | 29 | from datetime import datetime |
---|
[9296] | 30 | from zope.i18n import translate |
---|
[6821] | 31 | from zope.interface import Interface |
---|
[6825] | 32 | from zope.schema import getFields |
---|
[9960] | 33 | from zope.component import queryUtility, getUtility, createObject |
---|
[7429] | 34 | from zope.event import notify |
---|
[6825] | 35 | from zope.catalog.interfaces import ICatalog |
---|
[7951] | 36 | from hurry.workflow.interfaces import IWorkflowState, IWorkflowInfo |
---|
[7811] | 37 | from waeup.kofa.interfaces import ( |
---|
[7522] | 38 | IBatchProcessor, FatalCSVError, IObjectConverter, IUserAccount, |
---|
[9284] | 39 | IObjectHistory, VALIDATED, REGISTERED, IGNORE_MARKER) |
---|
[12623] | 40 | from waeup.kofa.interfaces import IKofaUtils, DuplicationError |
---|
[7959] | 41 | from waeup.kofa.interfaces import MessageFactory as _ |
---|
[7811] | 42 | from waeup.kofa.students.interfaces import ( |
---|
[9960] | 43 | IStudent, IStudentStudyCourse, IStudentStudyCourseTransfer, |
---|
[7536] | 44 | IStudentUpdateByRegNo, IStudentUpdateByMatricNo, |
---|
[9420] | 45 | IStudentStudyLevel, ICourseTicketImport, |
---|
[8174] | 46 | IStudentOnlinePayment, IStudentVerdictUpdate) |
---|
[13871] | 47 | from waeup.kofa.payments.interfaces import IPayer |
---|
[8309] | 48 | from waeup.kofa.students.workflow import ( |
---|
[9028] | 49 | IMPORTABLE_STATES, IMPORTABLE_TRANSITIONS, |
---|
| 50 | FORBIDDEN_POSTGRAD_TRANS, FORBIDDEN_POSTGRAD_STATES) |
---|
[7811] | 51 | from waeup.kofa.utils.batching import BatchProcessor |
---|
[6821] | 52 | |
---|
| 53 | class StudentProcessor(BatchProcessor): |
---|
[12872] | 54 | """The Student Processor imports student base data. |
---|
| 55 | |
---|
| 56 | In create mode no locator is required. If no `student_id` is given, |
---|
| 57 | the portal automatically assigns a new student id. |
---|
| 58 | |
---|
| 59 | In update or remove mode the processor uses |
---|
| 60 | either the `student_id`, `reg_number` or `matric_number` to localize the |
---|
| 61 | student object, exactly in this order. If `student_id` is given and an |
---|
| 62 | object can be found, `reg_number` and `matric_number` will be overwritten |
---|
| 63 | by the values provided in the import file. If `student_id` is missing, |
---|
| 64 | `reg_number` is used to localize the object and only `matric_number` |
---|
| 65 | will be overwritten. `matric_number` is used as locator only if both |
---|
| 66 | `student_id` and `reg_number` are missing. `student_id` can't be changed |
---|
| 67 | by the batch processor. |
---|
| 68 | |
---|
[13024] | 69 | There are two ways to change the workflow state of the student, |
---|
[12872] | 70 | an unsafe and a safe way. The safe way makes use of workflow transitions. |
---|
| 71 | Transitions are only possible between allowed workflow states. Only |
---|
| 72 | transitions ensure that the registration workflow is maintained. |
---|
| 73 | |
---|
| 74 | **Always prefer the safe way!** |
---|
[6821] | 75 | """ |
---|
| 76 | grok.implements(IBatchProcessor) |
---|
| 77 | grok.provides(IBatchProcessor) |
---|
| 78 | grok.context(Interface) |
---|
[7933] | 79 | util_name = 'studentprocessor' |
---|
[6821] | 80 | grok.name(util_name) |
---|
| 81 | |
---|
[11891] | 82 | name = _('Student Processor') |
---|
[6821] | 83 | iface = IStudent |
---|
[8581] | 84 | iface_byregnumber = IStudentUpdateByRegNo |
---|
| 85 | iface_bymatricnumber = IStudentUpdateByMatricNo |
---|
[6821] | 86 | |
---|
| 87 | factory_name = 'waeup.Student' |
---|
| 88 | |
---|
| 89 | @property |
---|
[6849] | 90 | def available_fields(self): |
---|
[8176] | 91 | fields = getFields(self.iface) |
---|
[6849] | 92 | return sorted(list(set( |
---|
[7513] | 93 | ['student_id','reg_number','matric_number', |
---|
[8309] | 94 | 'password', 'state', 'transition'] + fields.keys()))) |
---|
[6821] | 95 | |
---|
[6849] | 96 | def checkHeaders(self, headerfields, mode='create'): |
---|
[8309] | 97 | if 'state' in headerfields and 'transition' in headerfields: |
---|
| 98 | raise FatalCSVError( |
---|
[12180] | 99 | "State and transition can't be imported at the same time!") |
---|
[6854] | 100 | if not 'reg_number' in headerfields and not 'student_id' \ |
---|
| 101 | in headerfields and not 'matric_number' in headerfields: |
---|
[6849] | 102 | raise FatalCSVError( |
---|
[6854] | 103 | "Need at least columns student_id or reg_number " + |
---|
| 104 | "or matric_number for import!") |
---|
[6849] | 105 | if mode == 'create': |
---|
| 106 | for field in self.required_fields: |
---|
| 107 | if not field in headerfields: |
---|
| 108 | raise FatalCSVError( |
---|
| 109 | "Need at least columns %s for import!" % |
---|
| 110 | ', '.join(["'%s'" % x for x in self.required_fields])) |
---|
| 111 | # Check for fields to be ignored... |
---|
| 112 | not_ignored_fields = [x for x in headerfields |
---|
| 113 | if not x.startswith('--')] |
---|
| 114 | if len(set(not_ignored_fields)) < len(not_ignored_fields): |
---|
| 115 | raise FatalCSVError( |
---|
| 116 | "Double headers: each column name may only appear once.") |
---|
| 117 | return True |
---|
| 118 | |
---|
[6821] | 119 | def parentsExist(self, row, site): |
---|
| 120 | return 'students' in site.keys() |
---|
| 121 | |
---|
[6849] | 122 | def getLocator(self, row): |
---|
[8232] | 123 | if row.get('student_id',None) not in (None, IGNORE_MARKER): |
---|
[6849] | 124 | return 'student_id' |
---|
[8232] | 125 | elif row.get('reg_number',None) not in (None, IGNORE_MARKER): |
---|
[6849] | 126 | return 'reg_number' |
---|
[8232] | 127 | elif row.get('matric_number',None) not in (None, IGNORE_MARKER): |
---|
[6849] | 128 | return 'matric_number' |
---|
| 129 | else: |
---|
| 130 | return None |
---|
| 131 | |
---|
[6821] | 132 | # The entry never exists in create mode. |
---|
| 133 | def entryExists(self, row, site): |
---|
[7267] | 134 | return self.getEntry(row, site) is not None |
---|
| 135 | |
---|
| 136 | def getParent(self, row, site): |
---|
| 137 | return site['students'] |
---|
| 138 | |
---|
| 139 | def getEntry(self, row, site): |
---|
[6846] | 140 | if not 'students' in site.keys(): |
---|
[6849] | 141 | return None |
---|
| 142 | if self.getLocator(row) == 'student_id': |
---|
[6846] | 143 | if row['student_id'] in site['students']: |
---|
| 144 | student = site['students'][row['student_id']] |
---|
| 145 | return student |
---|
[6849] | 146 | elif self.getLocator(row) == 'reg_number': |
---|
[6846] | 147 | reg_number = row['reg_number'] |
---|
| 148 | cat = queryUtility(ICatalog, name='students_catalog') |
---|
| 149 | results = list( |
---|
| 150 | cat.searchResults(reg_number=(reg_number, reg_number))) |
---|
| 151 | if results: |
---|
| 152 | return results[0] |
---|
[6849] | 153 | elif self.getLocator(row) == 'matric_number': |
---|
[6846] | 154 | matric_number = row['matric_number'] |
---|
| 155 | cat = queryUtility(ICatalog, name='students_catalog') |
---|
| 156 | results = list( |
---|
| 157 | cat.searchResults(matric_number=(matric_number, matric_number))) |
---|
| 158 | if results: |
---|
| 159 | return results[0] |
---|
[6849] | 160 | return None |
---|
[6821] | 161 | |
---|
| 162 | def addEntry(self, obj, row, site): |
---|
| 163 | parent = self.getParent(row, site) |
---|
| 164 | parent.addStudent(obj) |
---|
[8491] | 165 | # Reset _curr_stud_id if student_id has been imported |
---|
| 166 | if self.getLocator(row) == 'student_id': |
---|
| 167 | parent._curr_stud_id -= 1 |
---|
[8287] | 168 | # We have to log this if state is provided. If not, |
---|
[7959] | 169 | # logging is done by the event handler handle_student_added |
---|
[9701] | 170 | if 'state' in row: |
---|
[7959] | 171 | parent.logger.info('%s - Student record created' % obj.student_id) |
---|
[7656] | 172 | history = IObjectHistory(obj) |
---|
[7959] | 173 | history.addMessage(_('Student record created')) |
---|
[6821] | 174 | return |
---|
| 175 | |
---|
| 176 | def delEntry(self, row, site): |
---|
[7267] | 177 | student = self.getEntry(row, site) |
---|
[7263] | 178 | if student is not None: |
---|
[6846] | 179 | parent = self.getParent(row, site) |
---|
[7656] | 180 | parent.logger.info('%s - Student removed' % student.student_id) |
---|
[6846] | 181 | del parent[student.student_id] |
---|
[6821] | 182 | pass |
---|
[6825] | 183 | |
---|
[8309] | 184 | def checkUpdateRequirements(self, obj, row, site): |
---|
| 185 | """Checks requirements the object must fulfill when being updated. |
---|
| 186 | |
---|
| 187 | This method is not used in case of deleting or adding objects. |
---|
| 188 | |
---|
| 189 | Returns error messages as strings in case of requirement |
---|
| 190 | problems. |
---|
| 191 | """ |
---|
| 192 | transition = row.get('transition', IGNORE_MARKER) |
---|
| 193 | if transition not in (IGNORE_MARKER, ''): |
---|
| 194 | allowed_transitions = IWorkflowInfo(obj).getManualTransitionIds() |
---|
| 195 | if transition not in allowed_transitions: |
---|
| 196 | return 'Transition not allowed.' |
---|
[9028] | 197 | if transition in FORBIDDEN_POSTGRAD_TRANS and \ |
---|
| 198 | obj.is_postgrad: |
---|
| 199 | return 'Transition not allowed (pg student).' |
---|
| 200 | state = row.get('state', IGNORE_MARKER) |
---|
| 201 | if state not in (IGNORE_MARKER, ''): |
---|
| 202 | if state in FORBIDDEN_POSTGRAD_STATES and \ |
---|
| 203 | obj.is_postgrad: |
---|
| 204 | return 'State not allowed (pg student).' |
---|
[8309] | 205 | return None |
---|
| 206 | |
---|
[9706] | 207 | def updateEntry(self, obj, row, site, filename): |
---|
[7497] | 208 | """Update obj to the values given in row. |
---|
| 209 | """ |
---|
[8221] | 210 | items_changed = '' |
---|
| 211 | |
---|
[7643] | 212 | # Remove student_id from row if empty |
---|
[9701] | 213 | if 'student_id' in row and row['student_id'] in (None, IGNORE_MARKER): |
---|
[7643] | 214 | row.pop('student_id') |
---|
[8221] | 215 | |
---|
| 216 | # Update password |
---|
[9701] | 217 | # XXX: Take DELETION_MARKER into consideration |
---|
| 218 | if 'password' in row: |
---|
[8498] | 219 | passwd = row.get('password', IGNORE_MARKER) |
---|
| 220 | if passwd not in ('', IGNORE_MARKER): |
---|
| 221 | if passwd.startswith('{SSHA}'): |
---|
| 222 | # already encrypted password |
---|
| 223 | obj.password = passwd |
---|
| 224 | else: |
---|
| 225 | # not yet encrypted password |
---|
| 226 | IUserAccount(obj).setPassword(passwd) |
---|
| 227 | items_changed += ('%s=%s, ' % ('password', passwd)) |
---|
[8221] | 228 | row.pop('password') |
---|
| 229 | |
---|
| 230 | # Update registration state |
---|
[9701] | 231 | if 'state' in row: |
---|
[8498] | 232 | state = row.get('state', IGNORE_MARKER) |
---|
| 233 | if state not in (IGNORE_MARKER, ''): |
---|
| 234 | value = row['state'] |
---|
| 235 | IWorkflowState(obj).setState(value) |
---|
| 236 | msg = _("State '${a}' set", mapping = {'a':value}) |
---|
| 237 | history = IObjectHistory(obj) |
---|
| 238 | history.addMessage(msg) |
---|
| 239 | items_changed += ('%s=%s, ' % ('state', state)) |
---|
[8287] | 240 | row.pop('state') |
---|
[8498] | 241 | |
---|
[9701] | 242 | if 'transition' in row: |
---|
[8498] | 243 | transition = row.get('transition', IGNORE_MARKER) |
---|
| 244 | if transition not in (IGNORE_MARKER, ''): |
---|
| 245 | value = row['transition'] |
---|
| 246 | IWorkflowInfo(obj).fireTransition(value) |
---|
| 247 | items_changed += ('%s=%s, ' % ('transition', transition)) |
---|
[8309] | 248 | row.pop('transition') |
---|
[8221] | 249 | |
---|
| 250 | # apply other values... |
---|
[8498] | 251 | items_changed += super(StudentProcessor, self).updateEntry( |
---|
[9706] | 252 | obj, row, site, filename) |
---|
[8221] | 253 | |
---|
| 254 | # Log actions... |
---|
[7656] | 255 | parent = self.getParent(row, site) |
---|
| 256 | if hasattr(obj,'student_id'): |
---|
[8995] | 257 | # Update mode: the student exists and we can get the student_id. |
---|
| 258 | # Create mode: the record contains the student_id |
---|
[7656] | 259 | parent.logger.info( |
---|
[9706] | 260 | '%s - %s - %s - updated: %s' |
---|
| 261 | % (self.name, filename, obj.student_id, items_changed)) |
---|
[7656] | 262 | else: |
---|
| 263 | # Create mode: the student does not yet exist |
---|
[9706] | 264 | # XXX: It seems that this never happens because student_id |
---|
| 265 | # is always set. |
---|
| 266 | parent.logger.info( |
---|
| 267 | '%s - %s - %s - imported: %s' |
---|
| 268 | % (self.name, filename, obj.student_id, items_changed)) |
---|
[8221] | 269 | return items_changed |
---|
[7497] | 270 | |
---|
[6849] | 271 | def getMapping(self, path, headerfields, mode): |
---|
| 272 | """Get a mapping from CSV file headerfields to actually used fieldnames. |
---|
| 273 | """ |
---|
| 274 | result = dict() |
---|
| 275 | reader = csv.reader(open(path, 'rb')) |
---|
| 276 | raw_header = reader.next() |
---|
| 277 | for num, field in enumerate(headerfields): |
---|
[8221] | 278 | if field not in ['student_id', 'reg_number', 'matric_number' |
---|
| 279 | ] and mode == 'remove': |
---|
[6849] | 280 | continue |
---|
| 281 | if field == u'--IGNORE--': |
---|
| 282 | # Skip ignored columns in failed and finished data files. |
---|
| 283 | continue |
---|
| 284 | result[raw_header[num]] = field |
---|
| 285 | return result |
---|
| 286 | |
---|
| 287 | def checkConversion(self, row, mode='create'): |
---|
| 288 | """Validates all values in row. |
---|
| 289 | """ |
---|
[7643] | 290 | iface = self.iface |
---|
[6849] | 291 | if mode in ['update', 'remove']: |
---|
| 292 | if self.getLocator(row) == 'reg_number': |
---|
[8581] | 293 | iface = self.iface_byregnumber |
---|
[6849] | 294 | elif self.getLocator(row) == 'matric_number': |
---|
[8581] | 295 | iface = self.iface_bymatricnumber |
---|
[6849] | 296 | converter = IObjectConverter(iface) |
---|
| 297 | errs, inv_errs, conv_dict = converter.fromStringDict( |
---|
[8214] | 298 | row, self.factory_name, mode=mode) |
---|
[12883] | 299 | # We cannot import both state and transition. |
---|
| 300 | if 'transition' in row and 'state' in row: |
---|
| 301 | if row['transition'] not in (IGNORE_MARKER, '') and \ |
---|
| 302 | row['state'] not in (IGNORE_MARKER, ''): |
---|
| 303 | errs.append(('workflow','not allowed')) |
---|
| 304 | return errs, inv_errs, conv_dict |
---|
[9701] | 305 | if 'transition' in row: |
---|
[9028] | 306 | if row['transition'] not in IMPORTABLE_TRANSITIONS: |
---|
| 307 | if row['transition'] not in (IGNORE_MARKER, ''): |
---|
| 308 | errs.append(('transition','not allowed')) |
---|
[9701] | 309 | if 'state' in row: |
---|
[9028] | 310 | if row['state'] not in IMPORTABLE_STATES: |
---|
| 311 | if row['state'] not in (IGNORE_MARKER, ''): |
---|
| 312 | errs.append(('state','not allowed')) |
---|
| 313 | else: |
---|
| 314 | # State is an attribute of Student and must not |
---|
| 315 | # be changed if empty. |
---|
| 316 | conv_dict['state'] = IGNORE_MARKER |
---|
[8490] | 317 | try: |
---|
| 318 | # Correct stud_id counter. As the IConverter for students |
---|
| 319 | # creates student objects that are not used afterwards, we |
---|
| 320 | # have to fix the site-wide student_id counter. |
---|
| 321 | site = grok.getSite() |
---|
| 322 | students = site['students'] |
---|
| 323 | students._curr_stud_id -= 1 |
---|
| 324 | except (KeyError, TypeError, AttributeError): |
---|
| 325 | pass |
---|
[6849] | 326 | return errs, inv_errs, conv_dict |
---|
| 327 | |
---|
[8232] | 328 | |
---|
| 329 | class StudentProcessorBase(BatchProcessor): |
---|
| 330 | """A base for student subitem processor. |
---|
| 331 | |
---|
| 332 | Helps reducing redundancy. |
---|
[6825] | 333 | """ |
---|
[8232] | 334 | grok.baseclass() |
---|
[6825] | 335 | |
---|
[9420] | 336 | # additional available fields |
---|
[8884] | 337 | # beside 'student_id', 'reg_number' and 'matric_number' |
---|
[8232] | 338 | additional_fields = [] |
---|
[6825] | 339 | |
---|
[12869] | 340 | # additional required fields (subset of additional_fields) |
---|
| 341 | additional_fields_required = [] |
---|
[6849] | 342 | |
---|
[6825] | 343 | @property |
---|
| 344 | def available_fields(self): |
---|
[8232] | 345 | fields = ['student_id','reg_number','matric_number' |
---|
| 346 | ] + self.additional_fields |
---|
| 347 | return sorted(list(set(fields + getFields( |
---|
[6843] | 348 | self.iface).keys()))) |
---|
[6825] | 349 | |
---|
[6837] | 350 | def checkHeaders(self, headerfields, mode='ignore'): |
---|
[6854] | 351 | if not 'reg_number' in headerfields and not 'student_id' \ |
---|
| 352 | in headerfields and not 'matric_number' in headerfields: |
---|
[6825] | 353 | raise FatalCSVError( |
---|
[6854] | 354 | "Need at least columns student_id " + |
---|
| 355 | "or reg_number or matric_number for import!") |
---|
[12869] | 356 | for name in self.additional_fields_required: |
---|
[8232] | 357 | if not name in headerfields: |
---|
| 358 | raise FatalCSVError( |
---|
| 359 | "Need %s for import!" % name) |
---|
| 360 | |
---|
[6834] | 361 | # Check for fields to be ignored... |
---|
[6825] | 362 | not_ignored_fields = [x for x in headerfields |
---|
| 363 | if not x.startswith('--')] |
---|
| 364 | if len(set(not_ignored_fields)) < len(not_ignored_fields): |
---|
| 365 | raise FatalCSVError( |
---|
| 366 | "Double headers: each column name may only appear once.") |
---|
| 367 | return True |
---|
| 368 | |
---|
[8232] | 369 | def _getStudent(self, row, site): |
---|
[8225] | 370 | NON_VALUES = ['', IGNORE_MARKER] |
---|
[6846] | 371 | if not 'students' in site.keys(): |
---|
[6849] | 372 | return None |
---|
[8225] | 373 | if row.get('student_id', '') not in NON_VALUES: |
---|
[6825] | 374 | if row['student_id'] in site['students']: |
---|
| 375 | student = site['students'][row['student_id']] |
---|
| 376 | return student |
---|
[8225] | 377 | elif row.get('reg_number', '') not in NON_VALUES: |
---|
[6825] | 378 | reg_number = row['reg_number'] |
---|
| 379 | cat = queryUtility(ICatalog, name='students_catalog') |
---|
| 380 | results = list( |
---|
| 381 | cat.searchResults(reg_number=(reg_number, reg_number))) |
---|
| 382 | if results: |
---|
| 383 | return results[0] |
---|
[8225] | 384 | elif row.get('matric_number', '') not in NON_VALUES: |
---|
[6843] | 385 | matric_number = row['matric_number'] |
---|
| 386 | cat = queryUtility(ICatalog, name='students_catalog') |
---|
| 387 | results = list( |
---|
| 388 | cat.searchResults(matric_number=(matric_number, matric_number))) |
---|
| 389 | if results: |
---|
| 390 | return results[0] |
---|
[6849] | 391 | return None |
---|
[6825] | 392 | |
---|
[7267] | 393 | def parentsExist(self, row, site): |
---|
| 394 | return self.getParent(row, site) is not None |
---|
| 395 | |
---|
[6825] | 396 | def entryExists(self, row, site): |
---|
[7534] | 397 | return self.getEntry(row, site) is not None |
---|
[6825] | 398 | |
---|
[8232] | 399 | def checkConversion(self, row, mode='ignore'): |
---|
| 400 | """Validates all values in row. |
---|
| 401 | """ |
---|
| 402 | converter = IObjectConverter(self.iface) |
---|
| 403 | errs, inv_errs, conv_dict = converter.fromStringDict( |
---|
| 404 | row, self.factory_name, mode=mode) |
---|
| 405 | return errs, inv_errs, conv_dict |
---|
| 406 | |
---|
[8885] | 407 | def getMapping(self, path, headerfields, mode): |
---|
| 408 | """Get a mapping from CSV file headerfields to actually used fieldnames. |
---|
| 409 | """ |
---|
| 410 | result = dict() |
---|
| 411 | reader = csv.reader(open(path, 'rb')) |
---|
| 412 | raw_header = reader.next() |
---|
| 413 | for num, field in enumerate(headerfields): |
---|
[8888] | 414 | if field not in ['student_id', 'reg_number', 'matric_number', |
---|
| 415 | 'p_id', 'code', 'level' |
---|
[8885] | 416 | ] and mode == 'remove': |
---|
| 417 | continue |
---|
| 418 | if field == u'--IGNORE--': |
---|
| 419 | # Skip ignored columns in failed and finished data files. |
---|
| 420 | continue |
---|
| 421 | result[raw_header[num]] = field |
---|
| 422 | return result |
---|
[8232] | 423 | |
---|
[8885] | 424 | |
---|
[8232] | 425 | class StudentStudyCourseProcessor(StudentProcessorBase): |
---|
[12872] | 426 | """The Student Study Course Processor imports data which refer |
---|
| 427 | to the student's course of study. The study course container data |
---|
| 428 | describe the current state of the course of study and it stores the |
---|
| 429 | entry conditions, i.e. when the student started the course. |
---|
| 430 | |
---|
| 431 | Most important is the `certificate` attribute which tells us which course |
---|
| 432 | the student is studying. The terms 'study course' and 'course of study' |
---|
| 433 | are used synonymously. The 'certificate' is the study programme described |
---|
| 434 | in the acadmic section. The study course object stores a referrer to a |
---|
| 435 | certificate in the acadmic section. |
---|
| 436 | |
---|
| 437 | When importing a new certificate code, `checkConversion` does not only |
---|
| 438 | check whether a certificate with the same code exists, it also |
---|
| 439 | proves if `current_level` is inside the level range of the certificate. |
---|
| 440 | For example, some study programmes start at level 200. The imported |
---|
| 441 | current level must thus be 200 or higher. |
---|
| 442 | |
---|
| 443 | `checkUpdateRequirements` looks up if the imported values match the |
---|
| 444 | certificate already stored with the study course object. The imported |
---|
| 445 | `current_level` must be in the range of the certificate already |
---|
| 446 | stored. |
---|
| 447 | |
---|
| 448 | .. note:: |
---|
| 449 | |
---|
| 450 | The processor does only offer an update mode. An 'empty' study course |
---|
| 451 | object is automatically created when the student object is added. So this |
---|
| 452 | object always exists. It can neither be added a second time nor |
---|
| 453 | be removed. |
---|
| 454 | |
---|
| 455 | Students can be transferred by import. A transfer is initialized if the |
---|
| 456 | `entry_mode` value is ``transfer``. In this case `checkConversion` uses a |
---|
| 457 | different interface for data validation and `checkUpdateRequirements` |
---|
| 458 | ensures that a student can only be transferred twice. The student transfer |
---|
| 459 | process is described elsewhere. |
---|
[8232] | 460 | """ |
---|
| 461 | grok.implements(IBatchProcessor) |
---|
| 462 | grok.provides(IBatchProcessor) |
---|
| 463 | grok.context(Interface) |
---|
| 464 | util_name = 'studycourseupdater' |
---|
| 465 | grok.name(util_name) |
---|
| 466 | |
---|
[11891] | 467 | name = _('StudentStudyCourse Processor (update only)') |
---|
[8232] | 468 | iface = IStudentStudyCourse |
---|
[9960] | 469 | iface_transfer = IStudentStudyCourseTransfer |
---|
[8232] | 470 | factory_name = 'waeup.StudentStudyCourse' |
---|
| 471 | |
---|
| 472 | def getParent(self, row, site): |
---|
| 473 | return self._getStudent(row, site) |
---|
| 474 | |
---|
[6825] | 475 | def getEntry(self, row, site): |
---|
[7534] | 476 | student = self.getParent(row, site) |
---|
[7536] | 477 | if student is None: |
---|
[6825] | 478 | return None |
---|
| 479 | return student.get('studycourse') |
---|
[7429] | 480 | |
---|
[9706] | 481 | def updateEntry(self, obj, row, site, filename): |
---|
[7429] | 482 | """Update obj to the values given in row. |
---|
| 483 | """ |
---|
[9960] | 484 | entry_mode = row.get('entry_mode', None) |
---|
| 485 | certificate = row.get('certificate', None) |
---|
[9962] | 486 | current_session = row.get('current_session', None) |
---|
| 487 | student = self.getParent(row, site) |
---|
[9960] | 488 | if entry_mode == 'transfer': |
---|
[9962] | 489 | # We do not expect any error here since we |
---|
| 490 | # checked all constraints in checkConversion and |
---|
| 491 | # in checkUpdateRequirements |
---|
| 492 | student.transfer( |
---|
| 493 | certificate=certificate, current_session=current_session) |
---|
[9960] | 494 | obj = student['studycourse'] |
---|
[8221] | 495 | items_changed = super(StudentStudyCourseProcessor, self).updateEntry( |
---|
[9706] | 496 | obj, row, site, filename) |
---|
[9962] | 497 | student.__parent__.logger.info( |
---|
[9706] | 498 | '%s - %s - %s - updated: %s' |
---|
[9962] | 499 | % (self.name, filename, student.student_id, items_changed)) |
---|
[7429] | 500 | # Update the students_catalog |
---|
[9962] | 501 | notify(grok.ObjectModifiedEvent(student)) |
---|
[7429] | 502 | return |
---|
| 503 | |
---|
[7532] | 504 | def checkConversion(self, row, mode='ignore'): |
---|
| 505 | """Validates all values in row. |
---|
| 506 | """ |
---|
[9960] | 507 | # We have to use the correct interface. Transfer |
---|
| 508 | # updates have different constraints. |
---|
| 509 | entry_mode = row.get('entry_mode', None) |
---|
| 510 | if entry_mode == 'transfer': |
---|
| 511 | converter = IObjectConverter(self.iface_transfer) |
---|
| 512 | else: |
---|
| 513 | converter = IObjectConverter(self.iface) |
---|
| 514 | errs, inv_errs, conv_dict = converter.fromStringDict( |
---|
| 515 | row, self.factory_name, mode=mode) |
---|
| 516 | |
---|
[7532] | 517 | # We have to check if current_level is in range of certificate. |
---|
[9701] | 518 | if 'certificate' in conv_dict and 'current_level' in conv_dict: |
---|
[8940] | 519 | cert = conv_dict['certificate'] |
---|
| 520 | level = conv_dict['current_level'] |
---|
| 521 | if level < cert.start_level or level > cert.end_level+120: |
---|
| 522 | errs.append(('current_level','not in range')) |
---|
[7532] | 523 | return errs, inv_errs, conv_dict |
---|
| 524 | |
---|
[9028] | 525 | def checkUpdateRequirements(self, obj, row, site): |
---|
| 526 | """Checks requirements the object must fulfill when being updated. |
---|
| 527 | Returns error messages as strings in case of requirement |
---|
| 528 | problems. |
---|
| 529 | """ |
---|
[9960] | 530 | certificate = getattr(obj, 'certificate', None) |
---|
| 531 | entry_session = getattr(obj, 'entry_session', None) |
---|
[9028] | 532 | current_level = row.get('current_level', None) |
---|
[9960] | 533 | entry_mode = row.get('entry_mode', None) |
---|
| 534 | # We have to ensure that the student can be transferred. |
---|
| 535 | if entry_mode == 'transfer': |
---|
| 536 | if certificate is None or entry_session is None: |
---|
| 537 | return 'Former study course record incomplete.' |
---|
| 538 | if 'studycourse_1' in obj.__parent__.keys() and \ |
---|
| 539 | 'studycourse_2' in obj.__parent__.keys(): |
---|
| 540 | return 'Maximum number of transfers exceeded.' |
---|
[9735] | 541 | if current_level: |
---|
| 542 | if current_level == 999 and \ |
---|
| 543 | obj.__parent__.state in FORBIDDEN_POSTGRAD_STATES: |
---|
| 544 | return 'Not a pg student.' |
---|
| 545 | cert = row.get('certificate', None) |
---|
| 546 | if certificate is None and cert is None: |
---|
| 547 | return 'No certificate to check level.' |
---|
[9960] | 548 | if certificate is not None and cert is None and ( |
---|
[9735] | 549 | current_level < certificate.start_level or \ |
---|
| 550 | current_level > certificate.end_level+120): |
---|
| 551 | return 'current_level not in range.' |
---|
[9028] | 552 | return None |
---|
| 553 | |
---|
[8232] | 554 | class StudentStudyLevelProcessor(StudentProcessorBase): |
---|
[12872] | 555 | """The Student Study Level Processor imports study level data. |
---|
| 556 | It overwrites the container attributes but not the content of the container, |
---|
| 557 | i.e. the course tickets stored inside the container. There is nothing |
---|
| 558 | special about this processor. |
---|
[7536] | 559 | """ |
---|
| 560 | grok.implements(IBatchProcessor) |
---|
| 561 | grok.provides(IBatchProcessor) |
---|
| 562 | grok.context(Interface) |
---|
[7933] | 563 | util_name = 'studylevelprocessor' |
---|
[7536] | 564 | grok.name(util_name) |
---|
| 565 | |
---|
[11891] | 566 | name = _('StudentStudyLevel Processor') |
---|
[7536] | 567 | iface = IStudentStudyLevel |
---|
| 568 | factory_name = 'waeup.StudentStudyLevel' |
---|
| 569 | |
---|
[9690] | 570 | @property |
---|
| 571 | def available_fields(self): |
---|
| 572 | fields = super(StudentStudyLevelProcessor, self).available_fields |
---|
| 573 | fields.remove('total_credits') |
---|
[10479] | 574 | fields.remove('gpa') |
---|
[9690] | 575 | return fields |
---|
| 576 | |
---|
[7536] | 577 | def getParent(self, row, site): |
---|
[8232] | 578 | student = self._getStudent(row, site) |
---|
| 579 | if student is None: |
---|
[7536] | 580 | return None |
---|
[8232] | 581 | return student['studycourse'] |
---|
[7536] | 582 | |
---|
| 583 | def getEntry(self, row, site): |
---|
| 584 | studycourse = self.getParent(row, site) |
---|
| 585 | if studycourse is None: |
---|
| 586 | return None |
---|
[12873] | 587 | return studycourse.get(str(row['level'])) |
---|
[7536] | 588 | |
---|
[9301] | 589 | def delEntry(self, row, site): |
---|
| 590 | studylevel = self.getEntry(row, site) |
---|
| 591 | parent = self.getParent(row, site) |
---|
| 592 | if studylevel is not None: |
---|
| 593 | student = self._getStudent(row, site) |
---|
| 594 | student.__parent__.logger.info('%s - Level removed: %s' |
---|
| 595 | % (student.student_id, studylevel.__name__)) |
---|
| 596 | del parent[studylevel.__name__] |
---|
| 597 | return |
---|
| 598 | |
---|
[9706] | 599 | def updateEntry(self, obj, row, site, filename): |
---|
[8626] | 600 | """Update obj to the values given in row. |
---|
| 601 | """ |
---|
| 602 | items_changed = super(StudentStudyLevelProcessor, self).updateEntry( |
---|
[9706] | 603 | obj, row, site, filename) |
---|
[8626] | 604 | student = self.getParent(row, site).__parent__ |
---|
| 605 | student.__parent__.logger.info( |
---|
[9706] | 606 | '%s - %s - %s - updated: %s' |
---|
| 607 | % (self.name, filename, student.student_id, items_changed)) |
---|
[8626] | 608 | return |
---|
| 609 | |
---|
[7536] | 610 | def addEntry(self, obj, row, site): |
---|
| 611 | parent = self.getParent(row, site) |
---|
[12873] | 612 | parent[str(row['level'])] = obj |
---|
[7536] | 613 | return |
---|
| 614 | |
---|
[8232] | 615 | class CourseTicketProcessor(StudentProcessorBase): |
---|
[12882] | 616 | """The Course Ticket Processor imports course tickets, the subobjects |
---|
| 617 | of student study levels (= course lists). |
---|
| 618 | |
---|
| 619 | An imported course ticket contains a copy of the original course data. |
---|
| 620 | During import only a few attributes can be set/overwritten. |
---|
| 621 | |
---|
| 622 | Like all other student data importers, this processor also requires |
---|
| 623 | either `student_id`, `reg_number` or `matric_number` to find the student. |
---|
| 624 | Then it needs `level` and `code` to localize the course ticket. |
---|
| 625 | |
---|
| 626 | `checkConversion` first searches the courses catalog for the imported |
---|
| 627 | `code` and ensures that a course with such a code really exists |
---|
| 628 | in the academic section. It furthermore checks if `level_session` in |
---|
| 629 | the row corresponds with the session of the parent student |
---|
| 630 | study level object. It fails if one of the conditions is not met. |
---|
| 631 | |
---|
| 632 | In create mode `fcode`, `dcode`, `title`, `credits`, `passmark` and |
---|
| 633 | `semester` are taken from the course found in the academic section. |
---|
| 634 | These attributes can nevermore be changed, neither via the user interface |
---|
| 635 | nor by import. |
---|
[7548] | 636 | """ |
---|
| 637 | grok.implements(IBatchProcessor) |
---|
| 638 | grok.provides(IBatchProcessor) |
---|
| 639 | grok.context(Interface) |
---|
[7933] | 640 | util_name = 'courseticketprocessor' |
---|
[7548] | 641 | grok.name(util_name) |
---|
| 642 | |
---|
[11891] | 643 | name = _('CourseTicket Processor') |
---|
[9420] | 644 | iface = ICourseTicketImport |
---|
[7548] | 645 | factory_name = 'waeup.CourseTicket' |
---|
| 646 | |
---|
[9316] | 647 | additional_fields = ['level', 'code'] |
---|
[12869] | 648 | additional_fields_required = additional_fields |
---|
[7548] | 649 | |
---|
[9420] | 650 | @property |
---|
| 651 | def available_fields(self): |
---|
| 652 | fields = [ |
---|
| 653 | 'student_id','reg_number','matric_number', |
---|
| 654 | 'mandatory', 'score', 'carry_over', 'automatic', |
---|
| 655 | 'level_session' |
---|
| 656 | ] + self.additional_fields |
---|
| 657 | return sorted(fields) |
---|
| 658 | |
---|
[7548] | 659 | def getParent(self, row, site): |
---|
[8232] | 660 | student = self._getStudent(row, site) |
---|
| 661 | if student is None: |
---|
[7548] | 662 | return None |
---|
[12873] | 663 | return student['studycourse'].get(str(row['level'])) |
---|
[7548] | 664 | |
---|
| 665 | def getEntry(self, row, site): |
---|
| 666 | level = self.getParent(row, site) |
---|
| 667 | if level is None: |
---|
| 668 | return None |
---|
| 669 | return level.get(row['code']) |
---|
| 670 | |
---|
[9706] | 671 | def updateEntry(self, obj, row, site, filename): |
---|
[8626] | 672 | """Update obj to the values given in row. |
---|
| 673 | """ |
---|
| 674 | items_changed = super(CourseTicketProcessor, self).updateEntry( |
---|
[9706] | 675 | obj, row, site, filename) |
---|
[8888] | 676 | parent = self.getParent(row, site) |
---|
[8626] | 677 | student = self.getParent(row, site).__parent__.__parent__ |
---|
| 678 | student.__parent__.logger.info( |
---|
[9706] | 679 | '%s - %s - %s - %s - updated: %s' |
---|
| 680 | % (self.name, filename, student.student_id, parent.level, items_changed)) |
---|
[8626] | 681 | return |
---|
| 682 | |
---|
[7548] | 683 | def addEntry(self, obj, row, site): |
---|
| 684 | parent = self.getParent(row, site) |
---|
| 685 | catalog = getUtility(ICatalog, name='courses_catalog') |
---|
| 686 | entries = list(catalog.searchResults(code=(row['code'],row['code']))) |
---|
| 687 | obj.fcode = entries[0].__parent__.__parent__.__parent__.code |
---|
| 688 | obj.dcode = entries[0].__parent__.__parent__.code |
---|
| 689 | obj.title = entries[0].title |
---|
[9420] | 690 | #if getattr(obj, 'credits', None) is None: |
---|
| 691 | obj.credits = entries[0].credits |
---|
| 692 | #if getattr(obj, 'passmark', None) is None: |
---|
| 693 | obj.passmark = entries[0].passmark |
---|
[7548] | 694 | obj.semester = entries[0].semester |
---|
| 695 | parent[row['code']] = obj |
---|
| 696 | return |
---|
| 697 | |
---|
[8888] | 698 | def delEntry(self, row, site): |
---|
| 699 | ticket = self.getEntry(row, site) |
---|
| 700 | parent = self.getParent(row, site) |
---|
| 701 | if ticket is not None: |
---|
| 702 | student = self._getStudent(row, site) |
---|
| 703 | student.__parent__.logger.info('%s - Course ticket in %s removed: %s' |
---|
| 704 | % (student.student_id, parent.level, ticket.code)) |
---|
| 705 | del parent[ticket.code] |
---|
| 706 | return |
---|
| 707 | |
---|
[7548] | 708 | def checkConversion(self, row, mode='ignore'): |
---|
| 709 | """Validates all values in row. |
---|
| 710 | """ |
---|
[8232] | 711 | errs, inv_errs, conv_dict = super( |
---|
| 712 | CourseTicketProcessor, self).checkConversion(row, mode=mode) |
---|
[9420] | 713 | if mode == 'remove': |
---|
| 714 | return errs, inv_errs, conv_dict |
---|
| 715 | # In update and create mode we have to check if course really exists. |
---|
[7548] | 716 | # This is not done by the converter. |
---|
| 717 | catalog = getUtility(ICatalog, name='courses_catalog') |
---|
| 718 | entries = catalog.searchResults(code=(row['code'],row['code'])) |
---|
| 719 | if len(entries) == 0: |
---|
| 720 | errs.append(('code','non-existent')) |
---|
| 721 | return errs, inv_errs, conv_dict |
---|
[9420] | 722 | # If level_session is provided in row we have to check if |
---|
| 723 | # the parent studylevel exists and if its level_session |
---|
| 724 | # attribute corresponds with the expected value in row. |
---|
[9421] | 725 | level_session = conv_dict.get('level_session', IGNORE_MARKER) |
---|
| 726 | if level_session not in (IGNORE_MARKER, None): |
---|
[9420] | 727 | site = grok.getSite() |
---|
| 728 | studylevel = self.getParent(row, site) |
---|
| 729 | if studylevel is not None: |
---|
| 730 | if studylevel.level_session != level_session: |
---|
| 731 | errs.append(('level_session','does not match %s' |
---|
| 732 | % studylevel.level_session)) |
---|
| 733 | else: |
---|
[13001] | 734 | errs.append(('level object','does not exist')) |
---|
[7623] | 735 | return errs, inv_errs, conv_dict |
---|
| 736 | |
---|
[8232] | 737 | class StudentOnlinePaymentProcessor(StudentProcessorBase): |
---|
[13871] | 738 | """The Student Online Payment Processor imports student payment tickets. |
---|
| 739 | The tickets are located in the ``payments`` subfolder of the student |
---|
[12882] | 740 | container. The only additional locator is `p_id`, the object id. |
---|
| 741 | |
---|
| 742 | The `checkConversion` method checks the format of the payment identifier. |
---|
| 743 | In create mode it does also ensures that same p_id does not exist |
---|
| 744 | elsewhere. It must be portal-wide unique. |
---|
| 745 | |
---|
| 746 | When adding a payment ticket, the `addEntry` method checks if the same |
---|
| 747 | payment has already been made. It compares `p_category` and `p_session` |
---|
| 748 | in the row with the corresponding attributes of existing payment |
---|
| 749 | tickets in state ``paid``. If they match, a `DuplicationError` is raised. |
---|
[7623] | 750 | """ |
---|
| 751 | grok.implements(IBatchProcessor) |
---|
| 752 | grok.provides(IBatchProcessor) |
---|
| 753 | grok.context(Interface) |
---|
[7933] | 754 | util_name = 'paymentprocessor' |
---|
[7623] | 755 | grok.name(util_name) |
---|
| 756 | |
---|
[11891] | 757 | name = _('StudentOnlinePayment Processor') |
---|
[8174] | 758 | iface = IStudentOnlinePayment |
---|
[7623] | 759 | factory_name = 'waeup.StudentOnlinePayment' |
---|
| 760 | |
---|
[8232] | 761 | additional_fields = ['p_id'] |
---|
[7623] | 762 | |
---|
[13871] | 763 | @property |
---|
| 764 | def available_fields(self): |
---|
| 765 | af = super( |
---|
| 766 | StudentOnlinePaymentProcessor, self).available_fields |
---|
| 767 | af.remove('display_item') |
---|
| 768 | return af |
---|
| 769 | |
---|
[8884] | 770 | def checkHeaders(self, headerfields, mode='ignore'): |
---|
| 771 | super(StudentOnlinePaymentProcessor, self).checkHeaders(headerfields) |
---|
[8885] | 772 | if mode in ('update', 'remove') and not 'p_id' in headerfields: |
---|
[8884] | 773 | raise FatalCSVError( |
---|
[8885] | 774 | "Need p_id for import in update and remove modes!") |
---|
[8884] | 775 | return True |
---|
| 776 | |
---|
[8232] | 777 | def parentsExist(self, row, site): |
---|
| 778 | return self.getParent(row, site) is not None |
---|
[7623] | 779 | |
---|
| 780 | def getParent(self, row, site): |
---|
[8232] | 781 | student = self._getStudent(row, site) |
---|
| 782 | if student is None: |
---|
[7623] | 783 | return None |
---|
[8232] | 784 | return student['payments'] |
---|
[7623] | 785 | |
---|
| 786 | def getEntry(self, row, site): |
---|
| 787 | payments = self.getParent(row, site) |
---|
| 788 | if payments is None: |
---|
| 789 | return None |
---|
[8884] | 790 | p_id = row.get('p_id', None) |
---|
[12996] | 791 | if p_id in (None, IGNORE_MARKER): |
---|
[8884] | 792 | return None |
---|
[7626] | 793 | # We can use the hash symbol at the end of p_id in import files |
---|
| 794 | # to avoid annoying automatic number transformation |
---|
| 795 | # by Excel or Calc |
---|
[8884] | 796 | p_id = p_id.strip('#') |
---|
[9467] | 797 | if len(p_id.split('-')) != 3 and not p_id.startswith('p'): |
---|
[8884] | 798 | # For data migration from old SRP only |
---|
| 799 | p_id = 'p' + p_id[7:] + '0' |
---|
| 800 | entry = payments.get(p_id) |
---|
[7623] | 801 | return entry |
---|
| 802 | |
---|
[9706] | 803 | def updateEntry(self, obj, row, site, filename): |
---|
[8626] | 804 | """Update obj to the values given in row. |
---|
| 805 | """ |
---|
| 806 | items_changed = super(StudentOnlinePaymentProcessor, self).updateEntry( |
---|
[9706] | 807 | obj, row, site, filename) |
---|
[8626] | 808 | student = self.getParent(row, site).__parent__ |
---|
| 809 | student.__parent__.logger.info( |
---|
[9706] | 810 | '%s - %s - %s - updated: %s' |
---|
| 811 | % (self.name, filename, student.student_id, items_changed)) |
---|
[8626] | 812 | return |
---|
| 813 | |
---|
[12623] | 814 | def samePaymentMade(self, student, category, p_session): |
---|
| 815 | for key in student['payments'].keys(): |
---|
| 816 | ticket = student['payments'][key] |
---|
| 817 | if ticket.p_state == 'paid' and\ |
---|
| 818 | ticket.p_category == category and \ |
---|
[14053] | 819 | ticket.p_session == p_session and \ |
---|
| 820 | ticket.p_item != 'Balance': |
---|
[12623] | 821 | return True |
---|
| 822 | return False |
---|
| 823 | |
---|
[7623] | 824 | def addEntry(self, obj, row, site): |
---|
| 825 | parent = self.getParent(row, site) |
---|
[12623] | 826 | student = parent.student |
---|
[7626] | 827 | p_id = row['p_id'].strip('#') |
---|
[12623] | 828 | # Requirement added on 19/02/2015: same payment must not exist. |
---|
| 829 | if self.samePaymentMade(student, obj.p_category, obj.p_session): |
---|
[13872] | 830 | student.__parent__.logger.info( |
---|
| 831 | '%s - %s - previous update cancelled' |
---|
| 832 | % (self.name, student.student_id)) |
---|
[12623] | 833 | raise DuplicationError('Same payment has already been made.') |
---|
[9467] | 834 | if len(p_id.split('-')) != 3 and not p_id.startswith('p'): |
---|
[7623] | 835 | # For data migration from old SRP |
---|
[8884] | 836 | obj.p_id = 'p' + p_id[7:] + '0' |
---|
[7623] | 837 | parent[obj.p_id] = obj |
---|
| 838 | else: |
---|
[7626] | 839 | parent[p_id] = obj |
---|
[7623] | 840 | return |
---|
| 841 | |
---|
[8885] | 842 | def delEntry(self, row, site): |
---|
| 843 | payment = self.getEntry(row, site) |
---|
| 844 | parent = self.getParent(row, site) |
---|
| 845 | if payment is not None: |
---|
| 846 | student = self._getStudent(row, site) |
---|
[8886] | 847 | student.__parent__.logger.info('%s - Payment ticket removed: %s' |
---|
[8885] | 848 | % (student.student_id, payment.p_id)) |
---|
| 849 | del parent[payment.p_id] |
---|
[8886] | 850 | return |
---|
[8885] | 851 | |
---|
[7623] | 852 | def checkConversion(self, row, mode='ignore'): |
---|
| 853 | """Validates all values in row. |
---|
| 854 | """ |
---|
[8232] | 855 | errs, inv_errs, conv_dict = super( |
---|
| 856 | StudentOnlinePaymentProcessor, self).checkConversion(row, mode=mode) |
---|
| 857 | |
---|
[7623] | 858 | # We have to check p_id. |
---|
[8884] | 859 | p_id = row.get('p_id', None) |
---|
[12996] | 860 | if mode == 'create' and p_id in (None, IGNORE_MARKER): |
---|
[8884] | 861 | timestamp = ("%d" % int(time()*10000))[1:] |
---|
| 862 | p_id = "p%s" % timestamp |
---|
| 863 | conv_dict['p_id'] = p_id |
---|
| 864 | return errs, inv_errs, conv_dict |
---|
[12996] | 865 | elif p_id in (None, IGNORE_MARKER): |
---|
| 866 | errs.append(('p_id','missing')) |
---|
| 867 | return errs, inv_errs, conv_dict |
---|
[8884] | 868 | else: |
---|
| 869 | p_id = p_id.strip('#') |
---|
[12996] | 870 | if p_id.startswith('p'): |
---|
| 871 | if not len(p_id) == 14: |
---|
| 872 | errs.append(('p_id','invalid length')) |
---|
| 873 | return errs, inv_errs, conv_dict |
---|
| 874 | elif len(p_id.split('-')) == 3: |
---|
| 875 | # The SRP used either pins as keys ... |
---|
| 876 | if len(p_id.split('-')[2]) not in (9, 10): |
---|
| 877 | errs.append(('p_id','invalid pin')) |
---|
| 878 | return errs, inv_errs, conv_dict |
---|
| 879 | else: |
---|
| 880 | # ... or order_ids. |
---|
| 881 | if not len(p_id) == 19: |
---|
| 882 | errs.append(('p_id','invalid format')) |
---|
| 883 | return errs, inv_errs, conv_dict |
---|
[12513] | 884 | # Requirement added on 24/01/2015: p_id must be portal-wide unique. |
---|
| 885 | if mode == 'create': |
---|
| 886 | cat = getUtility(ICatalog, name='payments_catalog') |
---|
| 887 | results = list(cat.searchResults(p_id=(p_id, p_id))) |
---|
| 888 | if len(results) > 0: |
---|
[13871] | 889 | sids = [IPayer(payment).id for payment in results] |
---|
[12513] | 890 | sids_string = '' |
---|
| 891 | for id in sids: |
---|
| 892 | sids_string += '%s ' % id |
---|
| 893 | errs.append(('p_id','p_id exists in %s' % sids_string)) |
---|
| 894 | return errs, inv_errs, conv_dict |
---|
[7623] | 895 | return errs, inv_errs, conv_dict |
---|
[7951] | 896 | |
---|
| 897 | class StudentVerdictProcessor(StudentStudyCourseProcessor): |
---|
[12882] | 898 | """The Student Verdict Processor inherits from the Student Study |
---|
| 899 | Course Processor. It's a pure updater. Import step 2 raises a warning |
---|
| 900 | message if a datacenter manager tries to select another mode. |
---|
| 901 | But it does more than only overwriting study course attributes. |
---|
[7951] | 902 | |
---|
[12882] | 903 | The Student Verdict Processor is the only processor which cannot be |
---|
| 904 | used for restoring data. Purpose is to announce a verdict at the end of |
---|
| 905 | each academic session. The processor does not only import a verdict, |
---|
| 906 | it also conditions the student data so that the student can pay for the |
---|
| 907 | next session and proceed to the next study level. |
---|
| 908 | |
---|
| 909 | The `checkUpdateRequirements` method ensures that the imported data |
---|
| 910 | really correspond to the actual state of the student. |
---|
| 911 | `current_level` and `current_session` in the row must be on par |
---|
| 912 | with the attributes of the study course object. Thus, the processor |
---|
| 913 | does not use these values to overwrite the attributes of the study course |
---|
| 914 | but to control that the verdict is really meant for the current session of |
---|
| 915 | the student. The verdict is only imported if a corresponding study level |
---|
| 916 | object exists and the student is in the right registration state, |
---|
| 917 | either ``courses validated`` or ``courses registered``. Course registration |
---|
| 918 | can be bypassed by setting `bypass_validation` to ``True``. |
---|
| 919 | |
---|
| 920 | The `updateEntry` method does not only update the current verdict of |
---|
| 921 | the student study course, it also updates the matching student study |
---|
| 922 | level object. It saves the current verdict as `level_verdict` and sets |
---|
| 923 | the `validated_by` and `validation_date` attributes, whereas `validated_by` |
---|
| 924 | is taken from the row of the import file and `validation_date` is set to the |
---|
| 925 | actual UTC datetime. Finally, the student is moved to state ``returning``. |
---|
[7951] | 926 | """ |
---|
| 927 | |
---|
| 928 | util_name = 'verdictupdater' |
---|
| 929 | grok.name(util_name) |
---|
| 930 | |
---|
[11891] | 931 | name = _('Verdict Processor (special processor, update only)') |
---|
[7951] | 932 | iface = IStudentVerdictUpdate |
---|
| 933 | factory_name = 'waeup.StudentStudyCourse' |
---|
| 934 | |
---|
| 935 | def checkUpdateRequirements(self, obj, row, site): |
---|
| 936 | """Checks requirements the studycourse and the student must fulfill |
---|
| 937 | before being updated. |
---|
| 938 | """ |
---|
| 939 | # Check if current_levels correspond |
---|
| 940 | if obj.current_level != row['current_level']: |
---|
| 941 | return 'Current level does not correspond.' |
---|
| 942 | # Check if current_sessions correspond |
---|
| 943 | if obj.current_session != row['current_session']: |
---|
| 944 | return 'Current session does not correspond.' |
---|
[9282] | 945 | # Check if new verdict is provided |
---|
| 946 | if row['current_verdict'] in (IGNORE_MARKER, ''): |
---|
[9293] | 947 | return 'No verdict in import file.' |
---|
[9631] | 948 | # Check if studylevel exists |
---|
[9293] | 949 | level_string = str(obj.current_level) |
---|
| 950 | if obj.get(level_string) is None: |
---|
| 951 | return 'Study level object is missing.' |
---|
[9284] | 952 | # Check if student is in state REGISTERED or VALIDATED |
---|
[9296] | 953 | if row.get('bypass_validation'): |
---|
[9284] | 954 | if obj.student.state not in (VALIDATED, REGISTERED): |
---|
| 955 | return 'Student in wrong state.' |
---|
| 956 | else: |
---|
| 957 | if obj.student.state != VALIDATED: |
---|
| 958 | return 'Student in wrong state.' |
---|
[7951] | 959 | return None |
---|
| 960 | |
---|
[9706] | 961 | def updateEntry(self, obj, row, site, filename): |
---|
[7951] | 962 | """Update obj to the values given in row. |
---|
| 963 | """ |
---|
[8221] | 964 | # Don't set current_session, current_level |
---|
| 965 | vals_to_set = dict((key, val) for key, val in row.items() |
---|
| 966 | if key not in ('current_session','current_level')) |
---|
[9706] | 967 | super(StudentVerdictProcessor, self).updateEntry( |
---|
| 968 | obj, vals_to_set, site, filename) |
---|
[7951] | 969 | parent = self.getParent(row, site) |
---|
[12882] | 970 | # Set current_verdict in corresponding studylevel |
---|
[9293] | 971 | level_string = str(obj.current_level) |
---|
| 972 | obj[level_string].level_verdict = row['current_verdict'] |
---|
[9296] | 973 | # Fire transition and set studylevel attributes |
---|
| 974 | # depending on student's state |
---|
[9284] | 975 | if obj.__parent__.state == REGISTERED: |
---|
[9296] | 976 | validated_by = row.get('validated_by', '') |
---|
| 977 | if validated_by in (IGNORE_MARKER, ''): |
---|
| 978 | portal_language = getUtility(IKofaUtils).PORTAL_LANGUAGE |
---|
| 979 | system = translate(_('System'),'waeup.kofa', |
---|
| 980 | target_language=portal_language) |
---|
| 981 | obj[level_string].validated_by = system |
---|
| 982 | else: |
---|
| 983 | obj[level_string].validated_by = validated_by |
---|
| 984 | obj[level_string].validation_date = datetime.utcnow() |
---|
[9284] | 985 | IWorkflowInfo(obj.__parent__).fireTransition('bypass_validation') |
---|
| 986 | else: |
---|
| 987 | IWorkflowInfo(obj.__parent__).fireTransition('return') |
---|
[7951] | 988 | # Update the students_catalog |
---|
| 989 | notify(grok.ObjectModifiedEvent(obj.__parent__)) |
---|
| 990 | return |
---|