Bug 14640: 'Cancel Hold' check box on check-out confirmation does not cancel the...
commit4da312f724a26a74a8bea25fe302610ec8e695f1
authorKyle M Hall <kyle@bywatersolutions.com>
Tue, 25 Aug 2015 14:35:28 +0000 (25 10:35 -0400)
committerLiz Rea <wizzyrea@gmail.com>
Wed, 23 Sep 2015 03:08:29 +0000 (23 15:08 +1200)
tree4912f48eeade10a51b85cd8f91c3c5560df85668
parent4fd13caf84b18b5f0a025f8507f0aa159703808c
Bug 14640: 'Cancel Hold' check box on check-out confirmation does not cancel the hold when item is checked out.

This bug is dealing with the situation where an item is checked out to a
patron that is not the next in line hold-wise for an item. In this case,
Koha will warn the librarian that there are holds on the item and
show the first person in line. Again, I want to stress that this
is the case where the item *is not waiting* for a patron. The
hold for the patron listed will just have a priority of 1.

The only situation where the "Cancel hold" checkbox will function
is when the priority 1 hold is an item level hold. This is due to
the fact that CancelReserve is being passed the trio of
biblionumber, borrowernumber, and itemnumber rather than the
singular reserve_id.

1) place biblio level hold on a book to borrower A.
2) check out an item of the book to borrower B.
3) When confirming checkout, check the 'Cancel hold' check-box, and
   click the "Yes, check out" button.
4) Note the hold was not canceled
5) Apply this patch
6) Repeat steps 1 through 3
7) Note the hold was indeed canceled

Signed-off-by: Joonas Kylmälä <j.kylmala@gmail.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Tomas Cohen Arazi <tomascohen@unc.edu.ar>
(cherry picked from commit a37b3bb7f76b5ddea7400f5268c592c0cc578a8c)
Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
(cherry picked from commit b565d3a62a0ae7980ac58f653182e0e6c0ccf914)
Signed-off-by: Liz Rea <wizzyrea@gmail.com>
C4/Reserves.pm