Bug 16654 - The output from reports/cat_issues_top.pl is not proper CSV
Summary: The output from reports/cat_issues_top.pl is not proper CSV
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: Reports (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor (vote)
Assignee: Galen Charlton
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-06-03 20:02 UTC by Barton Chittenden
Modified: 2024-03-25 16:19 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Barton Chittenden 2016-06-03 20:02:04 UTC
The output from reports/cat_issues_top.pl is *NOT* csv; it is DSV (delimiter separated values) with the option of using commas as delimiters... and the delimiters aren't properly escaped.

To illustrate the point

1/ Check out an item whose title contains a comma.
2/ Go to Home [34m~@� Reports [34m~@� Most-circulated items
3/ Select a date range containing today's date
4/ Select the "To a file" radio button, and use comma (,) as the delimiter.
5/ Click Submit

Open the file in the spreadsheet program of your choice. Note that the title containing the comma spans two columns, because it was not properly quoted.

If the report is going to use CSV as a file extension, it should be exported using Text::CSV_XS, which properly handles CSV export.
Comment 1 Katrin Fischer 2023-09-09 20:53:02 UTC
Still valid.