From 6879393681f89739970d0cb6bd30cbac3926bd19 Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
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=utf8
Content-Transfer-Encoding: 8bit

Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
---
 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 635d6b5687..12c8b81082 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.39.5