From 31baccbba68e31ae1a9fd4808d50c9a82f03c0e8 Mon Sep 17 00:00:00 2001 From: Chris Cormack Date: Tue, 2 Dec 2014 14:16:40 +1300 Subject: [PATCH] [SIGNED-OFF] Bug 13368 Holds priority messed up on checkout Content-Type: text/plain; charset="utf-8" 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. --- C4/Reserves.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Reserves.pm b/C4/Reserves.pm index 5058c51..22a9ecb 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 }); } } -- 1.7.9.5