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>
This commit is contained in:
Owen Leonard 2025-08-05 16:00:24 +00:00 committed by Lucas Gass
parent cf5359b90a
commit 43dfb04536
No known key found for this signature in database
GPG Key ID: 60B447398B0D36CA
1 changed files with 23 additions and 11 deletions

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>