Bug 33247: (QA follow-up) Remove pending deletion from results array

Both Elastic and Zebra search_auth_compat return a hash key
authid for a result.

Test plan:
Remove an authority that should be in your results list.
Note that it is no longer marked as pending, but just not displayed.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
This commit is contained in:
Marcel de Rooy 2023-06-02 08:17:45 +00:00 committed by Tomas Cohen Arazi
parent 61de402012
commit 1f0574dd91
Signed by: tomascohen
GPG key ID: 0A272EA1B2F3C15F
2 changed files with 4 additions and 2 deletions

View file

@ -201,6 +201,10 @@ if ( $op eq "do_search" ) {
}
}
if( $pending_deletion_authid && $results ) {
$results = [ grep { $_->{authid} != $pending_deletion_authid } @$results ];
}
$template->param( result => $results ) if $results;
my $max_result_window = $searcher->max_result_window;
@ -241,7 +245,6 @@ my $servers = Koha::Z3950Servers->search(
$template->param(
servers => $servers,
authority_types => $authority_types,
pending_deletion_authid => $pending_deletion_authid,
op => $op,
);

View file

@ -73,7 +73,6 @@
[% ELSE %]
[% PROCESS authresult summary=resul.summary authid=resul.authid %]
[% END %]
[% IF pending_deletion_authid == resul.authid %]<span style="color: red;">Pending deletion</span>[% END %]
</td>
<td>[% resul.authtype | html %]</td>
[% UNLESS ( resul.isEDITORS ) %]