https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28782

--- Comment #54 from David Cook <[email protected]> ---
(In reply to Jonathan Druart from comment #53)
> This introduces a warning in the log
> 
> Use of uninitialized value $bad_bibs_param in split at
> /kohadevbox/koha/reserve/placerequest.pl line 55.
> 
> We could fix it with:
> 
> diff --git a/reserve/placerequest.pl b/reserve/placerequest.pl
> index 0a8d6dd64e6..e85c6ad292a 100755
> --- a/reserve/placerequest.pl
> +++ b/reserve/placerequest.pl
> @@ -51,7 +51,7 @@ my $non_priority   = $input->param('non_priority');
>  
>  my $patron = Koha::Patrons->find( $borrowernumber );
>  
> -my $bad_bibs_param = $input->param('bad_bibs');
> +my $bad_bibs_param = $input->param('bad_bibs') || '';
>  my @bad_bibs = split '/', $bad_bibs_param;
>  my $holds_to_place_count = $input->param('holds_to_place_count') || 1;

Alternatively, @bad_bibs could be initialised ahead of time, and the split
function call could only be done if 'bad_bibs' is passed to the server.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to