From 96a82db48c2ac872d1b15dc952dee2a4f420e196 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Mon, 2 May 2022 10:59:23 +0200
Subject: [PATCH] Bug 29897: Don't use $6 for linkage but build the link
 instead

See comment 26:
$6 is a very specialized field that will never contain a URL. Linkage here means 'linking MARC fields within the record'. For example it is used to link an original script tag with the corresponding tag in transliteration. See: https://www.loc.gov/marc/authority/ecadcntf.html

Sponsored-by: Orex Digital
---
 .../bootstrap/en/modules/opac-auth-detail.tt  | 23 +++++++++++--------
 .../bootstrap/en/modules/opac-detail.tt       | 23 +++++++++++--------
 2 files changed, 26 insertions(+), 20 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-auth-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-auth-detail.tt
index fe0d2e7120a..db14deb23ea 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-auth-detail.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-auth-detail.tt
@@ -184,16 +184,19 @@
                             [% FOR identifier IN author_identifiers %]
                                 <li>
                                 [% SWITCH identifier.source.lower %]
-                                [% CASE 'orcid' %]ORCID:
-                                [% CASE 'scopus' %]ScopusID:
-                                [% CASE 'loop' %]loop:
-                                [% CASE 'rid' %]Publons:
-                                [% CASE %][% identifier.source | html %]:
-                                [% END %]
-                                [% IF identifier.linkage %]
-                                    <a href="[% identifier.linkage | url %]">[% identifier.number | html %]</a>
-                                [% ELSE %]
-                                    [% identifier.number | html %]
+                                [% CASE 'orcid' %]
+                                    ORCID:
+                                    <a href="https://orcid.org/[% identifier.number | url %]">[% identifier.number | html %]</a>
+                                [% CASE 'scopus' %]
+                                    ScopusID:
+                                    <a href="https://www.scopus.com/authid/detail.uri?authorId=[% identifier.number | url %]">[% identifier.number | html %]</a>
+                                [% CASE 'loop' %]
+                                    loop:
+                                    <a href="https://loop.frontiersin.org/people/[% identifier.number | url %]">[% identifier.number | html %]</a>
+                                [% CASE 'rid' %]
+                                    Publons:
+                                    <a href="https://publons.com/researcher/[% identifier.number | url %]">[% identifier.number | html %]</a>
+                                [% CASE %][% identifier.source | html %]: [% identifier.number | html %]
                                 [% END %]
                                 </li>
                             [% END %]
diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
index ef234cd5ca2..84dac115ed3 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
@@ -940,16 +940,19 @@
                                         [% FOR identifier IN author.identifiers %]
                                             <li>
                                             [% SWITCH identifier.source.lower %]
-                                            [% CASE 'orcid' %]ORCID:
-                                            [% CASE 'scopus' %]ScopusID:
-                                            [% CASE 'loop' %]loop:
-                                            [% CASE 'rid' %]Publons:
-                                            [% CASE %][% identifier.source | html %]:
-                                            [% END %]
-                                            [% IF identifier.linkage %]
-                                                <a href="[% identifier.linkage | url %]">[% identifier.number | html %]</a>
-                                            [% ELSE %]
-                                                [% identifier.number | html %]
+                                            [% CASE 'orcid' %]
+                                                ORCID:
+                                                <a href="https://orcid.org/[% identifier.number | url %]">[% identifier.number | html %]</a>
+                                            [% CASE 'scopus' %]
+                                                ScopusID:
+                                                <a href="https://www.scopus.com/authid/detail.uri?authorId=[% identifier.number | url %]">[% identifier.number | html %]</a>
+                                            [% CASE 'loop' %]
+                                                loop:
+                                                <a href="https://loop.frontiersin.org/people/[% identifier.number | url %]">[% identifier.number | html %]</a>
+                                            [% CASE 'rid' %]
+                                                Publons:
+                                                <a href="https://publons.com/researcher/[% identifier.number | url %]">[% identifier.number | html %]</a>
+                                            [% CASE %][% identifier.source | html %]: [% identifier.number | html %]
                                             [% END %]
                                             </li>
                                         [% END %]
-- 
2.25.1