From 29ae8a7a77a5b23753c15cec1b5a7d651e13876b Mon Sep 17 00:00:00 2001 From: David Cook Date: Mon, 12 Feb 2024 23:20:35 +0000 Subject: [PATCH] Bug 15565: Tidy reserve/placerequest.pl Signed-off-by: Lisette Scheer Signed-off-by: Nick Clemens Signed-off-by: Katrin Fischer --- reserve/placerequest.pl | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/reserve/placerequest.pl b/reserve/placerequest.pl index 48dd0636c5..44dd3f24dd 100755 --- a/reserve/placerequest.pl +++ b/reserve/placerequest.pl @@ -44,7 +44,7 @@ my $branch = $input->param('pickup'); my $startdate = $input->param('reserve_date') || ''; my @rank = $input->multi_param('rank-request'); my $title = $input->param('title'); -my @checkitems = $input->multi_param('checkitem'); +my @checkitems = $input->multi_param('checkitem'); my $item_group_id = $input->param('item_group_id'); my $expirationdate = $input->param('expiration_date'); my $itemtype = $input->param('itemtype') || undef; @@ -72,7 +72,7 @@ if ( $op eq 'cud-placerequest' && $patron ) { my $can_override = C4::Context->preference('AllowHoldPolicyOverride'); if ( @checkitems ) { - for ( my $i = 0; $i < scalar @checkitems; $i++ ){ + for ( my $i = 0 ; $i < scalar @checkitems ; $i++ ) { my $checkitem = $checkitems[$i]; if ( my $item_pickup_location = $input->param("item_pickup_$checkitem") ) { @@ -82,9 +82,11 @@ if ( $op eq 'cud-placerequest' && $patron ) { $biblionumber = $item->biblionumber; } - my $can_item_be_reserved = CanItemBeReserved($patron, $item, $item_pickup_location)->{status}; + my $can_item_be_reserved = CanItemBeReserved( $patron, $item, $item_pickup_location )->{status}; - if ( $can_item_be_reserved eq 'OK' || ( $can_item_be_reserved ne 'itemAlreadyOnHold' && $can_override ) ) { + if ( $can_item_be_reserved eq 'OK' + || ( $can_item_be_reserved ne 'itemAlreadyOnHold' && $can_override ) ) + { AddReserve( { branchcode => $item_pickup_location, @@ -102,8 +104,7 @@ if ( $op eq 'cud-placerequest' && $patron ) { } ); - } - else { + } else { $failed_holds{$can_item_be_reserved} = 1; } } @@ -157,11 +158,12 @@ if ( $op eq 'cud-placerequest' && $patron ) { my $redirect_url = URI->new("request.pl"); my @failed_hold_msgs = (); + #NOTE: Deduplicate failed hold reason statuses/codes - foreach my $msg (keys %failed_holds){ - push(@failed_hold_msgs,$msg); + foreach my $msg ( keys %failed_holds ) { + push( @failed_hold_msgs, $msg ); } - $redirect_url->query_form( biblionumber => [@biblionumbers], failed_holds => \@failed_hold_msgs); + $redirect_url->query_form( biblionumber => [@biblionumbers], failed_holds => \@failed_hold_msgs ); print $input->redirect($redirect_url); } elsif ( $borrowernumber eq '' ) { -- 2.39.5