http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7896
Katrin Fischer <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] Patch complexity|--- |Medium patch QA Contact|[email protected] |[email protected] --- Comment #11 from Katrin Fischer <[email protected]> --- Hi Julian, I looked at the patch and there are lots of smaller things I noted while testing: 1) Terms and capitalization - Budget/All budgets is labelled wrong, as it really shows the funds - All Item Types should be 'All item types' - All Suppliers should be 'All vendors' 2) Untranslatable terms - $select{''} = "All budgets"; - $select{''} = "All Item Types"; 3) POD - Old copy & paste error: plugin that shows a stats on borrowers 4) Item type - biblio level only? - Looking at the code it seems like only bibio level itypes are taken into account. I wonder if the stats should use 'item-level itypes'. If that can't be done, a little note would be helpful. 5) Error logs (can be reproduced by sending the form with preselected options) [Sun Dec 02 10:24:14 2012] [error] [client 127.0.0.1] [Sun Dec 2 10:24:14 2012] acquisitions_stats.pl: Use of uninitialized value in addition (+) at /home/katrin/kohaclone/reports/acquisitions_stats.pl line 582., referer: http://localhost:8080/cgi-bin/koha/reports/acquisitions_stats.pl That all said, perlcritic passes, all tests pass, and most problems are preexistant to your patches. But I would love to see a follow up addressing some of the other problems. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
