Defect #22583
CSV import delimiter detection broken
| Status: | Closed | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Jean-Philippe Lang | % Done: | 0% | |
| Category: | Issues | |||
| Target version: | 3.2.2 | |||
| Resolution: | Fixed | Affected version: | 3.2.1 |
Description
When uploading a CSV file to import, the delimiter used in the CSV is supposed to be auto-detected.
However, File.read seems to not be used properly: The third parameter may not be a string.
This is rescued in the block and therefore, the user does not see the error. However, the auto-detection does not work.
Attached is a simple patch that fixes this.
fix_csv_autodetection.patch 
(484 Bytes)
History
#1
Updated by Go MAEDA over 6 years ago
- Status changed from Needs feedback to New
- Target version changed from Candidate for next minor release to 3.2.2
Thanks for the patch. Setting target version to 3.2.2.
#2
Updated by Go MAEDA over 6 years ago
- Resolution deleted (
Fixed)
#3
Updated by Jean-Philippe Lang over 6 years ago
- Status changed from New to Closed
- Assignee set to Jean-Philippe Lang
- Resolution set to Fixed
Committed, thanks.