From 8e4b31ab7a677dd2406854be3b36b72c69b6bc36 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Mon, 12 Apr 2021 16:01:03 -0300 Subject: [PATCH] Bug 28118: Default to current branch when placing hold During bug 27071 development, this line got inadvertedly changed. This patch restores the original behaviour. To test: 1. Search for a title 2. Go try place a hold => FAIL: Look at the HTML, there's no pickup location with selected="selected" 3. Switch branch and repeat 2 => FAIL: Still the same 4. Apply this patch 5. Repeat 2 and 3 => SUCCESS: Branch is selected, chosen current branch is picked 6. Sign off :-D Signed-off-by: Tomas Cohen Arazi --- reserve/request.pl | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/reserve/request.pl b/reserve/request.pl index dffe1da4364..565c9b093a4 100755 --- a/reserve/request.pl +++ b/reserve/request.pl @@ -551,7 +551,7 @@ foreach my $biblionumber (@biblionumbers) { $item->{'holdallowed'} = $branchitemrule->{'holdallowed'}; - my $reserves_control_branch = $pickup // C4::Reserves::GetReservesControlBranch( $item, $patron_unblessed ); + my $reserves_control_branch = $pickup || C4::Reserves::GetReservesControlBranch( $item, $patron_unblessed ); my $can_item_be_reserved = CanItemBeReserved( $patron->borrowernumber, $itemnumber, $reserves_control_branch )->{status}; $item->{not_holdable} = $can_item_be_reserved unless ( $can_item_be_reserved eq 'OK' ); @@ -732,6 +732,8 @@ $template->param( biblionumbers => $biblionumbers ); $template->param( exceeded_maxreserves => $exceeded_maxreserves ); $template->param( exceeded_holds_per_record => $exceeded_holds_per_record ); $template->param( subscriptionsnumber => CountSubscriptionFromBiblionumber($biblionumber)); +# pass the userenv branch to be picked as default +$pickup ||= C4::Context->userenv->{branch}; $template->param( pickup => $pickup ); if ( C4::Context->preference( 'AllowHoldDateInFuture' ) ) { -- 2.31.1