Attachment 140846 Details for Bug 31600 – Bug 31600: Holds to pull displaying the patron for the last lowest priority hold instead of the first highest priority hold

[patch] Bug 31600: Holds to pull displaying the patron for the last lowest priority hold instead of the first highest priority hold

Bug-31600-Holds-to-pull-displaying-the-patron-for-.patch (text/plain), 1.24 KB, created by Kyle M Hall (khall) on 2022-09-22 12:58:19 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2022-09-22 12:58:19 UTC

Size: 1.24 KB

patch

obsolete

>From b22429f2d64a0c62d1f6517d61e21273d8a5b588 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>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

Actions: View | Diff | Splinter Review

Attachments on bug 31600: 140846