Attachment 103780 Details for Bug 25229 – Bug 25229: Return authid of record rather than 001

[patch] Bug 25229: Return authid of record rather than 001

Bug-25229-Return-authid-of-record-rather-than-001.patch (text/plain), 1.54 KB, created by Jonathan Druart on 2020-04-27 12:49:20 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2020-04-27 12:49:20 UTC

Size: 1.54 KB

patch

obsolete

>From af2b504a852e298ac874b3d13a7aba840d60eb8d 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=UTF-8
>Content-Transfer-Encoding: 8bit
>
>Signed-off-by: FrÃ©dÃ©ric Demians <f.demians@tamil.fr>
>Document _id field, in authorities ES index, is always authid
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> 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.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 25229: 103376 | 103738 | 103759 | 103760 | 103761 | 103779 | 103780 | 103781