Bug 6169

Summary: Inventory tool fails when ignoreissued set
Product: Koha Reporter: Jared Camins-Esakov <jcamins>
Component: ToolsAssignee: Jared Camins-Esakov <jcamins>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chris, colin.campbell
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: proposed patch

Description Jared Camins-Esakov 2011-04-11 18:05:34 UTC
Due to incorrect parameters passed to C4::Items::GetItemsForInventory, checking the "Skip copies on loan" box results in the inventory failing to run.
Comment 1 Jared Camins-Esakov 2011-04-11 18:09:45 UTC
Created attachment 3882 [details] [review]
proposed patch
Comment 2 Colin Campbell 2011-04-12 10:09:17 UTC
A good example of why subroutines with lots of positional parameters are problematic. Use named parameters if creating them
Comment 3 Chris Cormack 2011-04-12 10:20:25 UTC
Patch pushed, please test
Comment 4 Jared Camins-Esakov 2011-07-08 14:42:19 UTC
Seems to be working now. Closing.