Commit b188c8e7 by Victor Shnayder

fix bug in string splitting.

parent 9c7f2cf6
...@@ -2,6 +2,7 @@ ...@@ -2,6 +2,7 @@
from collections import defaultdict from collections import defaultdict
import csv import csv
import itertools
import json import json
import logging import logging
import os import os
...@@ -798,14 +799,15 @@ def _split_by_comma_and_whitespace(s): ...@@ -798,14 +799,15 @@ def _split_by_comma_and_whitespace(s):
Split a string both by on commas and whitespice. Split a string both by on commas and whitespice.
""" """
# Note: split() with no args removes empty strings from output # Note: split() with no args removes empty strings from output
return [x.split() for x in s.split(',')] lists = [x.split() for x in s.split(',')]
# return all of them
return itertools.chain(*lists)
def _do_enroll_students(course, course_id, students, overload=False): def _do_enroll_students(course, course_id, students, overload=False):
"""Do the actual work of enrolling multiple students, presented as a string """Do the actual work of enrolling multiple students, presented as a string
of emails separated by commas or returns""" of emails separated by commas or returns"""
ns = _split_by_comma_and_whitespace(students) new_students = _split_by_comma_and_whitespace(students)
new_students = [item for sublist in ns for item in sublist]
new_students = [str(s.strip()) for s in new_students] new_students = [str(s.strip()) for s in new_students]
new_students_lc = [x.lower() for x in new_students] new_students_lc = [x.lower() for x in new_students]
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment