Attachment 117469 Details for Bug 27125 – Bug 27125: Show authority type also for UNIMARC in authority search result display

[patch] Bug 27125: Show authority type also for UNIMARC in authority search result display

Bug-27125-Show-authority-type-also-for-UNIMARC-in-.patch (text/plain), 1.81 KB, created by Andreas Roussos on 2021-03-02 02:15:34 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andreas Roussos

Created: 2021-03-02 02:15:34 UTC

Size: 1.81 KB

patch

obsolete

>From 4bc670bef1f025e56782d42058ed3352c0e5007a Mon Sep 17 00:00:00 2001
>From: Fridolin Somers <fridolin.somers@biblibre.com>
>Date: Tue, 1 Dec 2020 14:28:24 +0100
>Subject: [PATCH] Bug 27125: Show authority type also for UNIMARC in authority
> search result display
>
>Bug 12533 added authority type in authority search result display.
>But it does not work for UNIMARC, due to :
>https://git.koha-community.org/Koha-community/Koha/src/commit/be9f64b638335cac7465ca302e187c7914534fbe/C4/AuthoritiesMarc.pm#L798
>
>I propose to use result->{authtype}
>
>Test plan :
>1) Use UNIMARC catalogue
>2) Search for autorities
>3) Check you see autority type in column
>4) Same for a MARC21 catalogue
>
>Signed-off-by: Andreas Roussos <a.roussos@dataly.gr>
>I followed the test plan, and the patch works as expected for both
>UNIMARC and MARC21. There is actually an improvement in the MARC21 case:
>previously, the 'Authority type' column for records in the Default
>framework would not display anything, whereas now it displays 'Default'.
>---
> .../prog/en/modules/authorities/searchresultlist.tt             | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist.tt
>index 65536c9cb1..c684bfe5a6 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/searchresultlist.tt
>@@ -42,7 +42,7 @@
>     [% FOREACH resul IN result %]
>     <tr data-authid="[% resul.authid | html %]">
>       <td>[% PROCESS authresult summary=resul.summary authid=resul.authid %]</td>
>-      <td>[% resul.summary.label | html %]</td>
>+      <td>[% resul.authtype | html %]</td>
>     [% UNLESS ( resul.isEDITORS ) %]
>       <td>
>       [% IF resul.used > 0 %]
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 27125: 114072 | 117469 | 119732