Attachment 99468 Details for Bug 24699 – Bug 24699: Split items.uri in OPAC detail view

[patch] Bug 24699: Split items.uri in OPAC detail view

Bug-24699-Split-itemsuri-in-OPAC-detail-view.patch (text/plain), 6.15 KB, created by Bernardo Gonzalez Kriegel on 2020-02-23 15:43:07 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Bernardo Gonzalez Kriegel

Created: 2020-02-23 15:43:07 UTC

Size: 6.15 KB

patch

obsolete

>From 1c7ad912533dd3f9c2fa843b9088e56d91fae35b Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Thu, 20 Feb 2020 14:43:43 +0000
>Subject: [PATCH] Bug 24699: Split items.uri in OPAC detail view
>
>Same change as in bug 24697 for staff. If there are more URIs in 952u,
>we split them and show the real URL instead of Link to resource or the
>value of pref URLLinkText.
>
>I introduced a few template variables here to make the code a bit more
>compact and readable.
>
>Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>Work as described, no errors.
>---
> .../bootstrap/en/modules/opac-detail.tt       | 43 ++++++-------------
> 1 file changed, 14 insertions(+), 29 deletions(-)
>
>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 d3b449351d..46b3f8a27c 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
>@@ -1351,37 +1351,22 @@
>                     [% END %]
>                     [% IF ( itemdata_uri ) %]
>                         <td class="url">
>-                            [% IF ITEM_RESULT.uri %]
>-                                [% IF Koha.Preference('URLLinkText') %]
>-                                    [% IF trackclicks == 'track' || trackclicks == 'anonymous' %]
>-                                        [% IF Koha.Preference("OPACURLOpenInNewWindow") %]
>-                                        <a target="_blank" rel="noreferrer" href="/cgi-bin/koha/tracklinks.pl?uri=[% ITEM_RESULT.uri | uri %]&amp;biblionumber=[% biblio.biblionumber | html %]&amp;itemnumber=[% ITEM_RESULT.itemnumber | url %]" property="url">[% Koha.Preference('URLLinkText') | html %]</a>
>-                                        [% ELSIF ITEM_RESULT.uri %]
>-                                        <a href="/cgi-bin/koha/tracklinks.pl?uri=[% ITEM_RESULT.uri | uri %]&amp;biblionumber=[% biblio.biblionumber | uri %]&amp;itemnumber=[% ITEM_RESULT.itemnumber | url %]" property="url">[% Koha.Preference('URLLinkText') | html %]</a>
>-                                        [% END %]
>-                                    [% ELSE %]
>-                                        [% IF Koha.Preference("OPACURLOpenInNewWindow") %]
>-                                        <a target="_blank" rel="noreferrer" href="[% ITEM_RESULT.uri | url %]" property="url">[% Koha.Preference('URLLinkText') | html %]</a>
>-                                        [% ELSIF ITEM_RESULT.uri %]
>-                                        <a href="[% ITEM_RESULT.uri | url %]" property="url">[% Koha.Preference('URLLinkText') | html %]</a>
>-                                        [% END %]
>-                                    [% END %]
>-                                [% ELSE %]
>-                                    [% IF trackclicks == 'track' || trackclicks == 'anonymous' %]
>-                                        [% IF Koha.Preference("OPACURLOpenInNewWindow") %]
>-                                        <a target="_blank" rel="noreferrer" href="/cgi-bin/koha/tracklinks.pl?uri=[% ITEM_RESULT.uri | uri %]&amp;biblionumber=[% biblio.biblionumber | html %]&amp;itemnumber=[% ITEM_RESULT.itemnumber | url %]" property="url">Link to resource</a>
>-                                        [% ELSIF ITEM_RESULT.uri %]
>-                                        <a href="/cgi-bin/koha/tracklinks.pl?uri=[% ITEM_RESULT.uri | uri %]&amp;biblionumber=[% biblio.biblionumber | uri %]&amp;itemnumber=[% ITEM_RESULT.itemnumber | url %]" property="url">Link to resource</a>
>-                                        [% END %]
>-                                    [% ELSE %]
>-                                        [% IF Koha.Preference("OPACURLOpenInNewWindow") %]
>-                                        <a target="_blank" rel="noreferrer" href="[% ITEM_RESULT.uri | url %]" property="url">Link to resource</a>
>-                                        [% ELSIF ITEM_RESULT.uri %]
>-                                        <a href="[% ITEM_RESULT.uri | url %]" property="url">Link to resource</a>
>-                                        [% END %]
>-                                    [% END %]
>+                        [% IF ITEM_RESULT.uri %]
>+                            [%# Initialize three helper vars %]
>+                            [% SET target_attr = ''; SET rel_attr = ''; SET href_attr = '' %]
>+                            [% IF Koha.Preference('OPACURLOpenInNewWindow') %][% SET target_attr = '_blank'; SET rel_attr = 'noreferrer' %][% END %]
>+
>+                            [%# Check if there are multiple URIs; in the single case we only use pref URLLinkText or Link to resource %]
>+                            [% IF ITEM_RESULT.uri.split(' \| ').size > 1 %]
>+                                [% FOREACH uri IN ITEM_RESULT.uri.split(' \| ') %]
>+                                    [% IF trackclicks == 'track' || trackclicks == 'anonymous' %][% SET href_attr = "/cgi-bin/koha/tracklinks.pl?uri=" _ uri _ "&amp;biblionumber=" _ biblio.biblionumber _ "&amp;itemnumber=" _ ITEM_RESULT.itemnumber %] [% ELSE %][% SET href_attr = uri %][% END %]
>+                                    <a target="[% target_attr | html %]" rel="[% rel_attr | html %]" href="[% href_attr | url %]" property="url">[% uri | html %]</a><br>
>                                 [% END %]
>+                            [% ELSE %]
>+                                [% IF trackclicks == 'track' || trackclicks == 'anonymous' %][% SET href_attr = "/cgi-bin/koha/tracklinks.pl?uri=" _ ITEM_RESULT.uri _ "&amp;biblionumber=" _ biblio.biblionumber _ "&amp;itemnumber=" _ ITEM_RESULT.itemnumber %] [% ELSE %][% SET href_attr = ITEM_RESULT.uri %][% END %]
>+                                <a target="[% target_attr | html %]" rel="[% rel_attr | html %]" href="[% href_attr | url %]" property="url">[% IF Koha.Preference('URLLinkText') %][% Koha.Preference('URLLinkText') | html %][% ELSE %]Link to resource[% END %]</a>
>                             [% END %]
>+                        [% END %]
>                         </td>
>                     [% END %]
>                     [% IF ( itemdata_copynumber ) %]<td class="copynumber">[% ITEM_RESULT.copynumber | html %]</td>[% END %]
>-- 
>2.17.1

Actions: View | Diff | Splinter Review

Attachments on bug 24699: 99355 | 99468 | 102004