Attachment 78089 Details for Bug 19550 – Bug 19550: Add links to related authorities for UNIMARC

[patch] Bug 19550: Add links to related authorities for UNIMARC

Bug-19550-Add-links-to-related-authorities-for-UNI.patch (text/plain), 3.38 KB, created by Katrin Fischer on 2018-08-23 06:29:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2018-08-23 06:29:04 UTC

Size: 3.38 KB

patch

obsolete

>From 4f41715edbceff42b29be6536cad49938e88d93e Mon Sep 17 00:00:00 2001
>From: Julian Maurice <julian.maurice@biblibre.com>
>Date: Tue, 31 Oct 2017 12:25:17 +0000
>Subject: [PATCH] Bug 19550: Add links to related authorities for UNIMARC
>
>Links exist in search results for MARC21 but not for UNIMARC.
>This patch fixes that.
>
>Test plan:
>1. Create an authority with a field 550 that links to another authority
>   e.g. 550 $a Foo $9 42
>2. Reindex this authority
>3. Search for this authority
>4. See that you now have a link "Foo" to authorities/detail.pl?authid=42
>   in the summary
>
>Signed-off-by: delaye <stephane.delaye@biblibre.com>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> .../en/includes/authorities-search-results.inc  | 17 +++++++++++------
> 1 file changed, 11 insertions(+), 6 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc
>index 7deec643ea..be3c0abfe5 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/authorities-search-results.inc
>@@ -8,15 +8,20 @@
>     [% SET searchurl = '/cgi-bin/koha/authorities/authorities-home.pl?op=do_search&type=intranet&marclist=any&operator=contains&orderby=HeadingAsc&value=' %]
> 
>     [% IF marcflavour == 'UNIMARC' %]
>+        [% IF authid %]
>+            [% link = BLOCK %]<a href="[% authidurl _ authid %]">[% heading | html %]</a>[% END %]
>+        [% ELSE %]
>+            [% link = BLOCK %][% heading | html %][% END %]
>+        [% END %]
>         [% SWITCH type %]
>         [% CASE 'broader' %]
>-            <span class="BT"><abbr title="Broader Term">BT</abbr>: [% heading | html %]</span>
>+            <span class="BT"><abbr title="Broader Term">BT</abbr>: [% link %]</span>
>         [% CASE 'narrower' %]
>-            <span class="NT"><abbr title="Narrower Term">NT</abbr>: [% heading | html %]</span>
>+            <span class="NT"><abbr title="Narrower Term">NT</abbr>: [% link %]</span>
>         [% CASE 'seefrom' %]
>-            <span class="UF"><abbr title="Used For">UF</abbr>: [% heading | html %]</span>
>+            <span class="UF"><abbr title="Used For">UF</abbr>: [% link %]</span>
>         [% CASE 'seealso' %]
>-            <span class="RT"><abbr title="Related Term">RT</abbr>: [% heading | html %]</span>
>+            <span class="RT"><abbr title="Related Term">RT</abbr>: [% link %]</span>
>         [% END %]
>     [% ELSE %]
>         <span class="heading">
>@@ -72,7 +77,7 @@
>           [% IF summary.seefrom %]
>             <div class="authres_seefrom">
>             [% FOREACH see IN summary.seefrom %]
>-              [% PROCESS showreference heading=see.heading linkType="" type=see.type search='' %]
>+              [% PROCESS showreference heading=see.heading linkType="" type=see.type authid=see.authid %]
>               [% IF ! loop.last %] ; [% END %]
>             [% END %]
>             </div>
>@@ -80,7 +85,7 @@
>             [% IF summary.seealso %]
>               <div class="authres_seealso">
>               [% FOREACH see IN summary.seealso %]
>-                 [% PROCESS showreference heading=see.heading linkType="" type=see.type search='' %]
>+                 [% PROCESS showreference heading=see.heading linkType="" type=see.type authid=see.authid %]
>                  [% IF ! loop.last %] ; [% END %]
>               [% END %]
>               </div>
>-- 
>2.17.1

Actions: View | Diff | Splinter Review

Attachments on bug 19550: 68856 | 72871 | 78089 | 78490 | 78530