Attachment 103376 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.26 KB, created by Nick Clemens (kidclamp) on 2020-04-21 13:48:48 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nick Clemens (kidclamp)

Created: 2020-04-21 13:48:48 UTC

Size: 1.26 KB

patch

obsolete

>From f001a82762b49110ac311d01371f1c410873c837 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
>
>---
> 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.11.0

Actions: View | Diff | Splinter Review

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