From 60777a28254702b18fa129065cb6515cce468f4a Mon Sep 17 00:00:00 2001 From: Pedro Amorim Date: Thu, 16 Jan 2025 15:49:20 +0000 Subject: [PATCH] Bug 36197: (QA follow-up): Make ConfirmAuto::get_priority_backends respect opac_available_backends if OPAC This makes it so that the Auto ILL stage on OPAC only queries opac_available_backends. Fix is submitted here and not in bug 35604 because it's here that opac_available_backends is implemented. --- Koha/ILL/Request/Workflow/ConfirmAuto.pm | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/Koha/ILL/Request/Workflow/ConfirmAuto.pm b/Koha/ILL/Request/Workflow/ConfirmAuto.pm index fe257020114..45adbe02577 100644 --- a/Koha/ILL/Request/Workflow/ConfirmAuto.pm +++ b/Koha/ILL/Request/Workflow/ConfirmAuto.pm @@ -78,7 +78,7 @@ sub confirm_auto_template_params { $params->{method} = 'confirmautoill' if $self->{ui_context} eq 'staff'; delete $params->{stage} if $self->{ui_context} eq 'staff'; - my @backends = $self->get_priority_backends(); + my @backends = $self->get_priority_backends( $self->{ui_context} ); return ( whole => $params, metadata => $self->prep_metadata($params), @@ -101,11 +101,15 @@ Returns backends ordered by AutoILLBackendPriority =cut sub get_priority_backends { - my ( $self, $params ) = @_; + my ( $self, $ui_context ) = @_; + + my $opac_backends; + $opac_backends = Koha::ILL::Backends->opac_available_backends() if $ui_context eq 'opac'; my @backends; my @priority_enabled_backends = split ",", C4::Context->preference('AutoILLBackendPriority'); foreach my $backend (@priority_enabled_backends) { + next if $ui_context eq 'opac' && !grep { $_ eq $backend } @$opac_backends; my $loaded_backend = Koha::ILL::Request->new->load_backend($backend); my $availability_check_info = $loaded_backend->_backend->availability_check_info( $self->{metadata} ); push @backends, $availability_check_info if $availability_check_info; -- 2.39.5