From 7ad70ef214928f3aa9c33feee4a644f11ffe4eda Mon Sep 17 00:00:00 2001 From: Chris Cormack Date: Tue, 2 Dec 2014 14:16:40 +1300 Subject: [PATCH] Bug 13368 Holds priority messed up on checkout To Test 1/ Create 3 (or more holds) on one biblionumber, make sure at least one item is not on loan 2/ Check out the not on loan item to a borrower (maybe number 2 in the queue) 3/ Look in the database (or on the holds tab on the moredetail.pl) notice the priorities have not been reordered 4/ Apply patch and try again, notice now they have Signed-off-by: Owen Leonard Confirmed the problem without the patch, and confirmed that the patch corrects it. Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi (cherry picked from commit ef6bc21b2c4ebe04f6d22d081e447f6bb2955029) Signed-off-by: Chris Cormack --- C4/Reserves.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Reserves.pm b/C4/Reserves.pm index 5058c51ebb..22a9ecb438 100644 --- a/C4/Reserves.pm +++ b/C4/Reserves.pm @@ -1309,7 +1309,7 @@ sub ModReserveFill { # now fix the priority on the others (if the priority wasn't # already sorted!).... unless ( $priority == 0 ) { - _FixPriority({ reserve_id => $reserve_id }); + _FixPriority({ reserve_id => $reserve_id, biblionumber => $biblionumber }); } } -- 2.20.1