From 6bd8547c92e74bc1a5632a07175cf7c0b2c60205 Mon Sep 17 00:00:00 2001 From: Shi Yao Wang Date: Wed, 29 Jun 2022 11:13:27 -0400 Subject: [PATCH] Bug 30935: Holds to pull shows wrong first patron When different patrons have item level holds on the same item, Holds to pull shows wrong first patron. Test plan: 1) Place a record-level hold on a biblio with an available item 2) Check the Holds to pull page (circ/pendingreserves.pl) --> The biblio should appear in the list and the patron should appear in the 'First patron' column 3) Place a second record-level hold on the same biblio 4) Go back to the Holds to pull page --> The biblio is still there, but the patron in the 'First patron' column is the second patron 5) Place a third record-level hold on the same biblio 6) Go back to the Holds to pull page --> The biblio is still there, but the patron in the 'First patron' column is the third patron 7) Apply the patch 8) Go back to the Holds to pull page --> The biblio is still there and the patron in the 'First patron' column is the first patron Signed-off-by: Sally Signed-off-by: Tomas Cohen Arazi (cherry picked from commit 87b985528edf0a714121fed13d2390fd57a27a36) Signed-off-by: Lucas Gass (cherry picked from commit c1cc0c07371f695646468862812fb82674480345) Signed-off-by: Arthur Suzuki --- circ/pendingreserves.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/circ/pendingreserves.pl b/circ/pendingreserves.pl index 314fd6d60d..d1e0f89d53 100755 --- a/circ/pendingreserves.pl +++ b/circ/pendingreserves.pl @@ -219,6 +219,7 @@ my $holds_biblios_map = { { join => ['itembib', 'biblio'], select => ['me.biblionumber', 'me.reserve_id'], + order_by => { -desc => 'priority' } } )->unblessed } -- 2.39.5