From 6c17f6c6a806435dbbf570e855d13a1dbfa434fc Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Fri, 19 Aug 2022 13:23:05 +0000 Subject: [PATCH] Bug 31112: (QA follow-up) Fetch patrons with reserves Previously we fetched all in a single call using biblionumbers Fetching each individually could be a performance hit on systems with large numbers of holds Signed-off-by: Nick Clemens --- C4/Circulation.pm | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index fd006ec2e1..606d83940b 100644 --- a/C4/Circulation.pm +++ b/C4/Circulation.pm @@ -2949,7 +2949,10 @@ sub CanBookBeRenewed { ) { my $biblio = Koha::Biblios->find($item->biblionumber); - my @possible_holds = $biblio->current_holds->unfilled->search({non_priority => 0})->as_list; + my @possible_holds = $biblio->current_holds->unfilled->search( + {non_priority => 0}, + { prefetch => 'patron' } + )->as_list; # Get all other items that could possibly fill reserves # FIXME We could join reserves (or more tables) here to eliminate some checks later -- 2.30.2