This will produce:
Leno, Charles, Jr.' -> 'Leno', 'Charles Jr.'
name.split(',', 1) -> only splits on the first occurrence of ',' ensuring not more than 2 items
s.strip().replace(',', '') -> removes any remaining commas so that it doesn't affect CSVs
The other PRs would produce the following names:
277: 'Leno', 'Charles, Jr.'
266: 'Leno', 'Charles'
Regardless of which fix is best, one of these should really be merged.
Fixes #316 and #214
This will produce:
Leno, Charles, Jr.'
->'Leno'
,'Charles Jr.'
name.split(',', 1)
-> only splits on the first occurrence of ',' ensuring not more than 2 itemss.strip().replace(',', '')
-> removes any remaining commas so that it doesn't affect CSVsThe other PRs would produce the following names:
277:
'Leno'
,'Charles, Jr.'
266:
'Leno'
,'Charles'
Regardless of which fix is best, one of these should really be merged.