From 2eb053603d9df3b03f4ee0ec54f84a697aab8716 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 Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi --- C4/Circulation.pm | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/C4/Circulation.pm b/C4/Circulation.pm index 2d5fe90f45..0677258c28 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.39.5