# HG changeset patch # User BenoƮt Allard # Date 1412770516 -7200 # Node ID b4fb652484b441d70965299aca5bca226bfb9386 # Parent ce49bd1512dd0c5e2bd106f841ecb363c2c7ea96 Define a split_fields function to split fields, and use it diff -r ce49bd1512dd -r b4fb652484b4 farol/controller.py --- a/farol/controller.py Wed Oct 08 12:55:40 2014 +0200 +++ b/farol/controller.py Wed Oct 08 14:15:16 2014 +0200 @@ -31,6 +31,11 @@ from farolluz.cvrf import CVRFNote, CVRFReference, CVRFAcknowledgment +def split_fields(field, separator=','): + if not field: + return [] + return [f.strip() for f in field.split(separator)] + def update_note_from_request(note): note._type = request.form['type'] note._ordinal = int(request.form['ordinal']) @@ -55,15 +60,15 @@ request.form['type'] or None) def update_acknowledgment_from_request(ack): - ack._name = request.form['name'] or None - ack._organization = request.form['organization'] or None + ack._names = split_fields(request.form['names']) + ack._organizations = split_fields(request.form['organizations']) ack._description = request.form['description'] or None ack._url = request.form['url'] or None def create_acknowledgment_from_request(): ack = CVRFAcknowledgment() - ack._name = request.form['name'] or None - ack._organization = request.form['organization'] or None + ack._names = split_fields(request.form['names']) + ack._organizations = split_fields(request.form['organizations']) ack._description = request.form['description'] or None ack._url = request.form['url'] or None return ack diff -r ce49bd1512dd -r b4fb652484b4 farol/document.py --- a/farol/document.py Wed Oct 08 12:55:40 2014 +0200 +++ b/farol/document.py Wed Oct 08 14:15:16 2014 +0200 @@ -33,7 +33,8 @@ from .controller import (update_note_from_request, create_note_from_request, update_reference_from_request, create_reference_from_request, - update_acknowledgment_from_request, create_acknowledgment_from_request) + update_acknowledgment_from_request, create_acknowledgment_from_request, + split_fields) from .session import document_required, get_current @@ -83,9 +84,7 @@ return render_template('document/edit_tracking.j2', tracking=tracking, version='.'.join('%s'%v for v in tracking._version), generator=generator, now=utcnow(), statuses=tracking.STATUSES) tracking._identification._id = request.form['id'] - aliases = [] - if request.form['id_aliases']: - aliases = [a.strip() for a in request.form['id_aliases'].split(',')] + aliases = split_fields(request.form['id_aliases']) tracking._identification._aliases = aliases tracking._status = request.form['status'] tracking._version = parseVersion(request.form['version'])