@@ -, +, @@ $ kshell k$ prove t/db_dependent/Koha/Hold.t --- Koha/Hold.pm | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) --- a/Koha/Hold.pm +++ a/Koha/Hold.pm @@ -238,16 +238,16 @@ sub is_pickup_location_valid { Koha::Exceptions::MissingParameter->throw('The library_id parameter is mandatory') unless $params->{library_id}; - my @pickup_locations; + my $pickup_locations; if ( $self->itemnumber ) { # item-level - @pickup_locations = $self->item->pickup_locations({ patron => $self->patron }); + $pickup_locations = $self->item->pickup_locations({ patron => $self->patron }); } else { # biblio-level - @pickup_locations = $self->biblio->pickup_locations({ patron => $self->patron }); + $pickup_locations = $self->biblio->pickup_locations({ patron => $self->patron }); } - return any { $_->branchcode eq $params->{library_id} } @pickup_locations; + return any { $_->branchcode eq $params->{library_id} } $pickup_locations->as_list; } =head3 set_pickup_location --