Bug 8124 has commented the option to download results items with no checkouts report into a file. But the perl code of this page uses the export into a file has default behavior since the input "output" is no more present. The consequence is that there are never results in page.
I believe this is fixed by my patch for Bug 7478.
Created attachment 20306 [details] [review] Bug 10718 - items with no checkouts report does not work Main patch, see commit message
Created attachment 20307 [details] [review] Bug 10718 - select user branch as default Followup 1 : select user branch as default
Created attachment 20308 [details] [review] Bug 10718 - items with no checkouts report permission is execute_reports Follow-up 2 : page permissions are too strong, set to reports/execute_reports
Created attachment 20309 [details] [review] Bug 10718 - redefine limit input Followup 3 : redefine limit input to avoid huge results
Created attachment 20310 [details] [review] Bug 10718 - perltidy on reports/catalogue_out.pl Followup 4 : perltidy on reports/catalogue_out.pl
(In reply to Owen Leonard from comment #1) > I believe this is fixed by my patch for Bug 7478. This is a correct solution, but isn't it better to remove the obsolete code ?
Created attachment 20570 [details] [review] Bug 10718 - items with no checkouts report does not work Bug 8124 has commented the option to download results items with no checkouts report into a file. But the perl code of this page uses the export into a file has default behavior since the input "output" is no more present. The consequence is that there are never results in page. This patch removes the code concerning file output from template and perl. Such a feature exists in guided reports. Test plan : Play with cgi-bin/koha/reports/catalogue_out.pl form and see if you get results Bug 10718 - select user branch as default In cgi-bin/koha/reports/catalogue_out.pl report, select in library filter the user logged-in. Test plan : - Log into intranet with a normal user - Got to "Items with no checkouts" reports => Your branch is selected - Run report => You see "Filtered on : Branch = <your branch>" - Come back to report - Select "Any library" - Run report => You do not see "Filtered on : Branch" Bug 10718 - items with no checkouts report permission is execute_reports This report only executes SQL to change reports/catalogue_out.pl permissions from all reports subpermissions to reports/execute_reports. Test plan : Test you can access and run this report with only execute_reports subpermission into reports permission. Bug 10718 - redefine limit input In reports/catalogue_out.pl form, the limit input is by defaut none. So by default the report queries all the catalogue. This may take a very long time (fall into timeout) and since now the results are displayed in page, the page may be huge. This patch modifies limit input to removes the "none" option. Also reduces the number of options (same options as guided reports results per page). Test plan: - Go to reports/catalogue_out.pl => first value "10" is selected - Launch report => You see "Filtered on: limit = 10" Bug 10718 - perltidy on reports/catalogue_out.pl Signed-off-by: Liz Rea <liz@catalyst.net.nz> Tested all of these things, all ok - squashed patch for neatness.
This patch doesn't seem to fix the translation problems fixed by 7478 - so I think we need both.
Created attachment 20600 [details] [review] [PASSED QA] Bug 10718 - items with no checkouts report does not work Bug 8124 has commented the option to download results items with no checkouts report into a file. But the perl code of this page uses the export into a file has default behavior since the input "output" is no more present. The consequence is that there are never results in page. This patch removes the code concerning file output from template and perl. Such a feature exists in guided reports. Test plan : Play with cgi-bin/koha/reports/catalogue_out.pl form and see if you get results Bug 10718 - select user branch as default In cgi-bin/koha/reports/catalogue_out.pl report, select in library filter the user logged-in. Test plan : - Log into intranet with a normal user - Got to "Items with no checkouts" reports => Your branch is selected - Run report => You see "Filtered on : Branch = <your branch>" - Come back to report - Select "Any library" - Run report => You do not see "Filtered on : Branch" Bug 10718 - items with no checkouts report permission is execute_reports This report only executes SQL to change reports/catalogue_out.pl permissions from all reports subpermissions to reports/execute_reports. Test plan : Test you can access and run this report with only execute_reports subpermission into reports permission. Bug 10718 - redefine limit input In reports/catalogue_out.pl form, the limit input is by defaut none. So by default the report queries all the catalogue. This may take a very long time (fall into timeout) and since now the results are displayed in page, the page may be huge. This patch modifies limit input to removes the "none" option. Also reduces the number of options (same options as guided reports results per page). Test plan: - Go to reports/catalogue_out.pl => first value "10" is selected - Launch report => You see "Filtered on: limit = 10" Bug 10718 - perltidy on reports/catalogue_out.pl Signed-off-by: Liz Rea <liz@catalyst.net.nz> Tested all of these things, all ok - squashed patch for neatness. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Passes all tests and QA script. Fixed a tab in one of the comments. Improves script, but more work needed. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
I'm going to have to fail this one. Some of the individual patches before the squash could have been accepted as is (for example, the one fixing the permissions), but the end result completely breaks the report output when the group option is enabled. I'll post screenshots momentarily.
Created attachment 20886 [details] screenshot of output grouping by library before patch Note that each group has a list of its items.
Created attachment 20887 [details] screenshot of output grouping by library afterpatch Note that only the first group has its items listed.
I apologize -- what I'm seeing is breakage of the report when grouping by item type that existed /before/ the patch. In my second screenshot, there is only one item of item type CR in the database, which *had* circulated. Consequently, I think it shouldn't be listed in the report at all, or at the very least, the third column should be more clearly labelled that it's counting /all/ items of that type, regardless of whether they have circulated or not. Consequently, I have pushed the patch to master, but echo Katrin's comment that more work is needed. Thanks, Fridolyn! Thanks, Katrin!
*** Bug 8833 has been marked as a duplicate of this bug. ***
This patch has been pushed to 3.12.x, will be in 3.12.7. Thanks Fridolyn!
This patch has been pushed to 3.8.x, and will be in 3.08.20. Thanks Fridolyn!