Attachment 138258 Details for Bug 29897 – Bug 29897: Move the identifier list to an include file

[patch] Bug 29897: Move the identifier list to an include file

Bug-29897-Move-the-identifier-list-to-an-include-f.patch (text/plain), 6.69 KB, created by Jonathan Druart on 2022-07-29 08:53:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2022-07-29 08:53:02 UTC

Size: 6.69 KB

patch

obsolete

>From a845a54850d9dc05e1cf200126e3dd0fe21baa3e Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 2 May 2022 11:23:05 +0200
>Subject: [PATCH] Bug 29897: Move the identifier list to an include file
>
>Sponsored-by: Orex Digital
>---
> .../en/includes/authority-identifiers.inc | 18 ++++++++++++++
> .../bootstrap/en/modules/opac-auth-detail.tt | 19 +--------------
> .../bootstrap/en/modules/opac-detail.tt | 24 +------------------
> 3 files changed, 20 insertions(+), 41 deletions(-)
> create mode 100644 koha-tmpl/opac-tmpl/bootstrap/en/includes/authority-identifiers.inc
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/includes/authority-identifiers.inc b/koha-tmpl/opac-tmpl/bootstrap/en/includes/authority-identifiers.inc
>new file mode 100644
>index 00000000000..726e736c50b
>--- /dev/null
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/includes/authority-identifiers.inc
>@@ -0,0 +1,18 @@
>+[% SWITCH identifier.source.lower %]
>+[% 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 'viaf' %]
>+ VIAF: 
>+ <a href="https://viaf.org/viaf/[% identifier.number | url %]">[% identifier.number | html %]</a>
>+[% CASE %][% identifier.source | html %]: [% identifier.number | html %]
>+[% END %]
>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 6d38f7ce3d8..36a53eeda29 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
>@@ -187,24 +187,7 @@
> <ul>
> [% FOR identifier IN author_identifiers %]
> <li>
>- [% SWITCH identifier.source.lower %]
>- [% 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 'viaf' %]
>- VIAF:
>- <a href="https://viaf.org/viaf/[% identifier.number | url %]">[% identifier.number | html %]</a>
>- [% CASE %][% identifier.source | html %]: [% identifier.number | html %]
>- [% END %]
>+ [% PROCESS "authority-identifiers.inc" identifier => identifier %]
> </li>
> [% END %]
> </ul>
>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 50b03827516..f86848d6c6f 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>@@ -966,29 +966,7 @@
> <ul>
> [% FOR identifier IN author.identifiers %]
> <li>
>- [% SWITCH identifier.source.lower %]
>- [% 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 'viaf' %]
>- VIAF:
>- <a href="https://viaf.org/viaf/[% identifier.number | url %]">[% identifier.number | html %]</a>
>- [% CASE %][% identifier.source | html %]: [% identifier.number | html %]
>- [% END %]
>- [% IF identifier.linkage %]
>- <a href="[% identifier.linkage | url %]">[% identifier.number | html %]</a>
>- [% ELSE %]
>- [% identifier.number | html %]
>- [% END %]
>+ [% PROCESS "authority-identifiers.inc" identifier => identifier %]
> </li>
> [% END %]
> </ul>
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 29897: 129816 | 129817 | 129818 | 129819 | 132961 | 132962 | 132963 | 132964 | 133081 | 133113 | 133114 | 133115 | 133116 | 133117 | 134459 | 134460 | 134461 | 134462 | 134463 | 134464 | 135983 | 135984 | 135985 | 135986 | 135987 | 135988 | 135989 | 135990 | 135991 | 135992 | 135993 | 138249 | 138250 | 138251 | 138252 | 138253 | 138254 | 138255 | 138256 | 138257 | 138258 | 138259 | 138260 | 138371 | 138372 | 138425 | 138426 | 138427 | 138428 | 138429 | 138430 | 138431 | 138432 | 138433 | 138434 | 138435 | 138486 | 138487 | 138490 | 138501 | 138664 | 138665 | 138666 | 138667 | 138668 | 138669 | 138670 | 138671 | 138672 | 138673 | 138674 | 138675 | 138676 | 138677 | 138678 | 138679