Conversation
|
Ik zie dat de checks weer gemaild zijn, maar ik ben verder gegaan vanaf de development op Mac branch, dus ik zie niet zo goed hoe ik dat niet goed gedaan kon hebben @tiptenbrink |
|
@jefryelb Nee je hebt het goed gedaan, hij klaagt gewoon over de nieuwe code. Als je tegen je agent zegt: "Fix het totdat De error is: |
|
@tiptenbrink Thanks hahah, was alweer vergeten dat hier ruff checks op zaten |
tiptenbrink
left a comment
There was a problem hiding this comment.
Rebase nog even op main zodat je de aanpassingen niet weer undo'd.
git fetch
en daarna
git rebase origin/main
en als je conflicts krijgt, gewoon resolven zodat het weer was zoals op main voor die files
|
Maar daarna gewoon lekker zelf mergen! |
- Strip BOM and whitespace from CSV content consistently - Unwrap outer quotes from data rows to allow proper field parsing - Preserve inner quotes by converting escaped quotes back to single quotes - Fixes parsing errors when CSV exports wrap each row in outer quotes
|
@tiptenbrink thanks voor je hulp man! |
Fix CSV import failing for exports with quoted rows
The CSV export wraps each data row in outer quotes (e.g. "1234,Jan,,Jansen,M,...") which caused csv.DictReader to treat the entire line as a single field. This resulted in all columns except the first being None, crashing with 'NoneType' object has no attribute 'strip'.
Added a pre-processing step in parse_csv that strips outer quotes from data rows and unescapes doubled quotes before passing the content to DictReader.