http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3638 Ian Walls <koha.sekjal@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|PATCH-Sent (DO NOT USE) |P2 Status|Signed Off |Passed QA --- Comment #17 from Ian Walls <koha.sekjal@gmail.com> --- First patch: modifies some subroutines in SIP/ILS/Item.pm to take borrowernumber/branchcode params, and set them for the hold. Moves hold handling to do_checkin instead of handle_checkin, and adds the processing from C4::Reserves we'd normally get in the staff client. Also strips out some commented lines. Marking as Passed QA. Second patch: Better handling of conditional (separate lines and shift instead of a Perl one-liner). Adds return_date to do_checkin call. Adds a call to ModItemTransfer where applicable. Bulk of work is to set a default current location, as the self check machine may not. Also marked Passed QA. -- You are receiving this mail because: You are the QA Contact for the bug. You are watching all bug changes.