From b22429f2d64a0c62d1f6517d61e21273d8a5b588 Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 22 Sep 2022 12:55:21 +0000 Subject: [PATCH] Bug 31600: Holds to pull displaying the patron for the last lowest priority hold instead of the first highest priority hold If there are holds on a record for patrons A, B and C with priorities 1, 2 and 3, the holds to pull report will display patron C even though patron A is most likely the patron the item will trap for. Test Plan: 1) Place some holds on a record 2) View the holds to pull report 3) Note the patron listed is not the top priority hold 4) Apply this patch 5) Restart all the things! 6) Reload the page 7) You should now see the top priority patron listed! --- circ/pendingreserves.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/circ/pendingreserves.pl b/circ/pendingreserves.pl index b14c5ab8e9..8960cb470f 100755 --- a/circ/pendingreserves.pl +++ b/circ/pendingreserves.pl @@ -218,6 +218,7 @@ my $holds_biblios_map = { { join => ['itembib', 'biblio'], select => ['me.biblionumber', 'me.reserve_id'], + order_by => { '-desc' => 'priority' }, } )->unblessed } -- 2.30.2