gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 40590: Prevent OPACAuthorIdentifiersAndInformation from showing empty list elements for unknown 024$2 

This patch modifies the template which is used to display additional
author information if OPACAuthorIdentifiersAndInformation is displayed.
The change prevents an empty list item from appearing.

To test, apply the patch and go to system preferences in the staff
interface.

- Search for OPACAuthorIdentifiersAndInformation and make sure that
  "Identifiers (024$2$a)" is checked.
- Have a record with at least 2 persons (100, 700) linked to an
  authority record
  - One of those authority records should not have an 024 tag
  - The other authority record should have two 024 tags:
    - 024 $a someidyoumakeup $2orcid
    - 024 $a someidyoumakeup $2somethingelse
- Check the detail page in OPAC -> "Author information" tab
- Check the authority detail page in OPAC, under "Author information"
  in the right-hand column.

Before the patch, '$2somethingelse' is not "supported" and will create
the empty list element. After the patch there should be no empty list
item.

Sponsored-by: Athens County Public Libraries
Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit 43dfb04536)
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
This commit is contained in:
Owen Leonard 2025-08-05 16:00:24 +00:00 committed by Paul Derscheid
parent ae5df2bc75
commit df6768d616
No known key found for this signature in database
GPG Key ID: C25F9FD44CED595B
1 changed files with 23 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
koha-tmpl/opac-tmpl/bootstrap/en/includes/authority-information.inc
View File

@ -1,31 +1,43 @@
[% FOREACH info IN information %]
[% SWITCH info_type %]
[% CASE 'identifiers' %]
<li class="identifier_[% info.source.lower | html %]">
[% SWITCH info.source.lower %]
[% CASE 'orcid' %]
[% SWITCH info.source.lower %]
[% CASE 'orcid' %]
<li class="identifier_[% info.source.lower | html %]">
<span>ORCID: </span>
<a href="https://orcid.org/[% info.number | url %]">[% info.number | html %]</a>
[% CASE 'scopus' %]
</li>
[% CASE 'scopus' %]
<li class="identifier_[% info.source.lower | html %]">
<span>ScopusID: </span>
<a href="https://www.scopus.com/authid/detail.uri?authorId=[% info.number | url %]">[% info.number | html %]</a>
[% CASE 'loop' %]
</li>
[% CASE 'loop' %]
<li class="identifier_[% info.source.lower | html %]">
<span>Loop ID: </span>
<a href="https://loop.frontiersin.org/people/[% info.number | url %]">[% info.number | html %]</a>
[% CASE 'rid' %]
</li>
[% CASE 'rid' %]
<li class="identifier_[% info.source.lower | html %]">
<span>Publons ID: </span>
<a href="https://publons.com/researcher/[% info.number | url %]">[% info.number | html %]</a>
[% CASE 'viaf' %]
</li>
[% CASE 'viaf' %]
<li class="identifier_[% info.source.lower | html %]">
<span>VIAF ID: </span>
<a href="https://viaf.org/viaf/[% info.number | url %]">[% info.number | html %]</a>
[% CASE 'isni' %]
</li>
[% CASE 'isni' %]
<li class="identifier_[% info.source.lower | html %]">
<span>ISNI ID: </span>
<a href="https://isni.org/isni/[% info.number | url %]">[% info.number | html %]</a>
[% CASE 'wikidata' %]
</li>
[% CASE 'wikidata' %]
<li class="identifier_[% info.source.lower | html %]">
<span>WIKIDATA ID: </span>
<a href="http://www.wikidata.org/entity/[% info.number | url %]">[% info.number | html %]</a>
[% END %]
</li>
</li>
[% END %]
[% CASE 'activity' %]
[% IF info.field_of_activity %]
<li>