New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

mysqldump_to_csv: edgecase where last row starts with bracket #73

Merged

mtmail merged 1 commit into osm-search:master from mtmail:mysqldump-fix

Jul 21, 2023

Contributor

mtmail commented Jul 21, 2023

After the #67 change there seem to be an edge-case when the first character of a value starts with '('.

     if latest_row[-1][-1] == ")":
 IndexError: string index out of range

Example data from German redirect file below. Previously we didn't see an error because the previous value was filled with ^A instead of empty.

4588104,3,Ärsäbät,,
4588112,2,Kaizen77/Farbe1,,
4588129,0,Bishop_Lloyd’s_House,,
4588148,12,Benutzer,,(Automatisch) bestätigter Benutzer
4588150,0,Profos,,
4588152,0,Universidade_Estadual_Paulista,,
4588153,0,Kloster_St._Walburg,,


          mysqldump_to_csv: edgecase where last row starts with bracket

cafe285

mtmail merged commit a757a2e into osm-search:master

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet