If you download the results of a saved report Koha proposes a standard filename (reportresults.csv / reportresults.tab / reportresults.ods) It would be nice to get the name of the saved report as part of the download file name, e.g. MyReport-reportresults.csv OtherReport-reportresults.csv
Created attachment 38134 [details] [review] Bug 14013 - Use name of a saved report as part of the file name for download This patch propoeses the name of a saved report as part of the download file name. To test: - Apply patch - Got to Home > Reports > Guided reports wizard > Saved reports (or create a new report from sql) - Run a saved report - Download the report (as csv, tab, ods) Result: The downlaod file name proposed should be prepended by the report name, separated with a - - Bonus test: Try to downlaod saved reports which have special character in their name.
Created attachment 38135 [details] [review] Bug 14013 - Use name of a saved report as part of the file name for download This patch propoeses the name of a saved report as part of the download file name. To test: - Apply patch - Got to Home > Reports > Guided reports wizard > Saved reports (or create a new report from sql) - Run a saved report - Download the report (as csv, tab, ods) Result: The downlaod file name proposed should be prepended by the report name, separated with a - - Bonus test: Try to downlaod saved reports which have special character in their name. NOTE: Used "ñ/â is N/A" as a report name. The /'s were translated to underscored, but the letters were as expected. Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 38173 [details] [review] Bug 14013 - Use name of a saved report as part of the file name for download This patch propoeses the name of a saved report as part of the download file name. To test: - Apply patch - Got to Home > Reports > Guided reports wizard > Saved reports (or create a new report from sql) - Run a saved report - Download the report (as csv, tab, ods) Result: The downlaod file name proposed should be prepended by the report name, separated with a - - Bonus test: Try to downlaod saved reports which have special character in their name. NOTE: Used "ñ/â is N/A" as a report name. The /'s were translated to underscored, but the letters were as expected. Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Patch pushed to master. Thanks Marc!