Summary: | itemtypes.plugin report: not handling item-level_itypes syspref | ||
---|---|---|---|
Product: | Koha | Reporter: | Victor Grousset/tuxayo <victor> |
Component: | Reports | Assignee: | helene <hickeyhe> |
Status: | CLOSED FIXED | QA Contact: | Tomás Cohen Arazi <tomascohen> |
Severity: | minor | ||
Priority: | P5 - low | CC: | fridolin.somers, hickeyhe, lucas, martin.renvoize, nick, tomascohen |
Version: | Main | Keywords: | Academy |
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: |
Bug 20274 When the system preference for item-level_itypes changed, it gave a zero result.
Bug 20274 When the system preference for item-level_itypes changed, it gives a zero result. Bug 20274: Itemtypes report gives zero results if item-level_itypes set to bibliographic records |
Description
Victor Grousset/tuxayo
2018-02-22 16:00:03 UTC
Created attachment 84032 [details] [review] Bug 20274 When the system preference for item-level_itypes changed, it gave a zero result. 1) Go to reports then catalog by item types 2) Pick a location and look at table 3) Change the system prefernce of item-level_itypes 4) Verify a zero result 5) Apply patch and retry 6) Verify that result does not change to zero on either preference Created attachment 84138 [details] [review] Bug 20274 When the system preference for item-level_itypes changed, it gives a zero result. 1) Go to reports then catalog by item types 2) Pick a location and look at table 3) Change the system prefernce of item-level_itypes 4) Verify a zero result 5) Apply patch and retry 6) Verify that result does not change to zero on either preference Signed-off-by: David Nind <david@davidnind.com> Created attachment 84144 [details] [review] Bug 20274: Itemtypes report gives zero results if item-level_itypes set to bibliographic records 1) Go to reports then catalog by item types 2) Pick a location and look at table 3) Change the system prefernce of item-level_itypes 4) Verify a zero result 5) Apply patch and retry 6) Verify that result does not change to zero on either preference Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Good fix! For next patch: please make sure the commit subject (the first line) follows the guidelines. You can run the qa script yourself before submitting so you can fix that! Awesome work all! Pushed to master for 19.05 Pushed to 18.11.x for 18.11.03 pushed to 18.05.x for 18.05.09 Ok for this patch. But I think this file needs more work. I see : while (($itemtype, $description,$biblioitems,$total) = $sth->fetchrow) So first var is the item type. $biblioitems seems unused. This code is confusing. Pushed to 17.11.x for 17.11.15 |