Bug 16654

Summary: The output from reports/cat_issues_top.pl is not proper CSV
Product: Koha Reporter: Barton Chittenden <barton>
Component: ReportsAssignee: Galen Charlton <gmcharlt>
Status: NEW --- QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: blawlor
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

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.