From 7ea4fde27bc801af0523b1243e20e03f2f0b3162 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Tue, 21 Apr 2020 13:47:59 +0000 Subject: [PATCH] Bug 25229: Return authid of record rather than 001 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Frédéric Demians Document _id field, in authorities ES index, is always authid --- Koha/SearchEngine/Elasticsearch/Search.pm | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/Koha/SearchEngine/Elasticsearch/Search.pm b/Koha/SearchEngine/Elasticsearch/Search.pm index a5b410e3a6..0bc8c3ebc6 100644 --- a/Koha/SearchEngine/Elasticsearch/Search.pm +++ b/Koha/SearchEngine/Elasticsearch/Search.pm @@ -207,12 +207,10 @@ sub search_auth_compat { my $record = $es_record->{'_source'}; my %result; - # I wonder if these should be real values defined in the mapping - # rather than hard-coded conversions. - #my $record = $_[0]; - # Handle legacy nested arrays indexed with splitting enabled. - my $authid = $record->{ 'local-number' }[0]; - $authid = @$authid[0] if (ref $authid eq 'ARRAY'); + # We are using the authid to create links, we should honor the authid as stored in the db, not + # the 001 which, in some circumstances, can contain other data + my $authid = $es_record->{_id}; + $result{authid} = $authid; -- 2.17.1