From 924e987ad808525d85122bbd14786ebb1a1f0a47 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Tue, 13 Mar 2018 14:57:47 +0000 Subject: [PATCH] Bug 20385: Don't try to get info for bad authtypecodes TO test: 0 - Have a koha with ES enabled and some auth records 1 - backup your db 2 - sudo koha-mysql <> 3 - UPDATE auth_types SET authtypecode="PERSO_NAMB" WHERE authtypecode="PERSO_NAME" (or adjust as necessary) 4 - Search for some authorities where a PERSO_NAME record will be returned 5 - KA_BOOM! 6 - Apply patch 7 - Results are returned, they do not display a code, viewing record you won't see code Signed-off-by: Nicolas Legrand Signed-off-by: Alex Arnaud Signed-off-by: Jonathan Druart --- Koha/SearchEngine/Elasticsearch/Search.pm | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/Koha/SearchEngine/Elasticsearch/Search.pm b/Koha/SearchEngine/Elasticsearch/Search.pm index 4d584f7906..2ddd165d24 100644 --- a/Koha/SearchEngine/Elasticsearch/Search.pm +++ b/Koha/SearchEngine/Elasticsearch/Search.pm @@ -217,7 +217,7 @@ sub search_auth_compat { # with the record. It's not documented why this is the case, so # it's not reproduced here yet. my $authtype = $rs->single; - my $auth_tag_to_report = $authtype->auth_tag_to_report; + my $auth_tag_to_report = $authtype ? $authtype->auth_tag_to_report : $authtypecode; my $marc = $self->json2marc($marc_json); my $mainentry = $marc->field($auth_tag_to_report); my $reported_tag; @@ -227,11 +227,7 @@ sub search_auth_compat { } } # Turn the resultset into a hash - my %authtype_cols; - foreach my $col ($authtype->result_source->columns) { - $authtype_cols{$col} = $authtype->get_column($col); - } - $result{authtype} = $authtype->authtypetext; + $result{authtype} = $authtype ? $authtype->authtypetext : $authtypecode; $result{reported_tag} = $reported_tag; # Reimplementing BuildSummary is out of scope because it'll be hard -- 2.39.5