Bug 6983

Summary: Item type filter on overdues report does not respect item-level_itypes preference
Product: Koha Reporter: Owen Leonard <oleonard>
Component: CirculationAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chris, gmcharlt, julian.maurice, koha.sekjal
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:
Attachments: Proposed fix

Description Owen Leonard 2011-10-06 19:10:46 UTC
The item type filter on the overdues report (overdue.pl) only checks the biblio-level itemtype (biblioitems.itemtype). The script should check the value of the item-level_itypes preference and alternately use items.itype.
Comment 1 Owen Leonard 2011-10-06 19:18:38 UTC
Created attachment 5749 [details] [review]
Proposed fix

To test: identify or modify records contained in your overdues report so that some have different item-level item types and some have different biblio-level item types:

Title 1 - Item-level item type A
Title 2 - Item-level item type B
Title 3 - Biblio-level item type C
Title 4 - Biblio-level item type D

With item-level_itypes ON your report should correctly filter on item types A and B. With itme-level_itypes OFF your report should correctly filter on item types C and D.
Comment 2 Julian Maurice 2011-10-07 14:29:41 UTC
Works fine.
Signed off.
Comment 3 Ian Walls 2011-10-17 19:21:26 UTC
Simple SQL change, consistent with the many other such switches we have for this kind of test.  Marking as Passed QA
Comment 4 Chris Cormack 2011-10-18 02:56:10 UTC
Pushed, please test