Attachment 106988 Details for Bug 15851 – Bug 15851: Display analytics links for more cases

[patch] Bug 15851: Display analytics links for more cases

Bug-15851-Display-analytics-links-for-more-cases.patch (text/plain), 4.27 KB, created by Tomás Cohen Arazi (tcohen) on 2020-07-17 12:50:27 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2020-07-17 12:50:27 UTC

Size: 4.27 KB

patch

obsolete

>From 566e18cd9895ec9eced4184bdc513845c65f9625 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Wed, 24 Jun 2020 13:08:25 -0300
>Subject: [PATCH] Bug 15851: Display analytics links for more cases
>
>The current XSLT displays the link to linked analytics only for serials.
>This patch makes it show for all the relevant position 7 on the leader
>values. I left out a and b as recommended by expert librarians on my
>team, but I can revert that if required.
>
>The current implementation adds a new CSS class for each case, so
>libraries willing to keep the current behaviour or just have more
>granular control on the cases they want the link to display, just can.
>
>This patch makes sense with the follow-up one, which will display the
>link only if there are really related records.
>
>To test:
>1. Open a non-serial record, notice there's no link to analytics
>2. Apply this patch and reload
>=> SUCCESS: There's an 'Analytics' link
>3. Inspect the produced HTML
>=> SUCCESS: A special class with analytic_* value has been added, and
>thus we now can control its display through CSS
>4. Sign off :-D
>
>Sponsored-by: Orex Digital
>
>Signed-off-by: Hugo Agud <hagud@orex.es>
>---
> .../en/xslt/MARC21slim2OPACDetail.xsl | 35 +++++++++++++------
> 1 file changed, 24 insertions(+), 11 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl b/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl
>index afaa37bcd56..884b02a334a 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl
>@@ -339,22 +339,35 @@
> 
> </xsl:if>
> 
>- 
>- <xsl:if test="$leader7='s'">
>- Analytics: 
>+ 
>+ <xsl:variable name="leader7_class">
>+ <xsl:choose>
>+ 
>+ 
>+ <xsl:when test="$leader7='c'">analytic_collection</xsl:when>
>+ <xsl:when test="$leader7='d'">analytic_subunit</xsl:when>
>+ <xsl:when test="$leader7='i'">analytic_ires</xsl:when>
>+ <xsl:when test="$leader7='m'">analytic_monograph</xsl:when>
>+ <xsl:when test="$leader7='s'">analytic_serial</xsl:when>
>+ <xsl:otherwise>analytic_undefined</xsl:otherwise>
>+ </xsl:choose>
>+ </xsl:variable>
>+
>+ <xsl:element name="span">
>+ <xsl:attribute name="class">results_summary analytics <xsl:value-of select="$leader7_class"/></xsl:attribute>
>+ Analytics: 
> <a>
> <xsl:choose>
>- <xsl:when test="$UseControlNumber = '1' and marc:controlfield[@tag=001]">
>- <xsl:attribute name="href">/cgi-bin/koha/opac-search.pl?q=rcn:<xsl:value-of select="str:encode-uri(marc:controlfield[@tag=001], true())"/>+and+(bib-level:a+or+bib-level:b)</xsl:attribute>
>- </xsl:when>
>- <xsl:otherwise>
>- <xsl:attribute name="href">/cgi-bin/koha/opac-search.pl?q=Host-item:<xsl:value-of select="str:encode-uri(translate(marc:datafield[@tag=245]/marc:subfield[@code='a'], '/', ''), true())"/></xsl:attribute>
>- </xsl:otherwise>
>+ <xsl:when test="$UseControlNumber = '1' and marc:controlfield[@tag=001]">
>+ <xsl:attribute name="href">/cgi-bin/koha/opac-search.pl?q=rcn:<xsl:value-of select="str:encode-uri(marc:controlfield[@tag=001], true())"/>+and+(bib-level:a+or+bib-level:b)</xsl:attribute>
>+ </xsl:when>
>+ <xsl:otherwise>
>+ <xsl:attribute name="href">/cgi-bin/koha/opac-search.pl?q=Host-item:<xsl:value-of select="str:encode-uri(translate(marc:datafield[@tag=245]/marc:subfield[@code='a'], '/', ''), true())"/></xsl:attribute>
>+ </xsl:otherwise>
> </xsl:choose>
> <xsl:text>Show analytics</xsl:text>
> </a>
>- 
>- </xsl:if>
>+ </xsl:element>
> 
> 
> <xsl:if test="$materialTypeCode='ST' or substring($controlField008,22,1)='m'">
>-- 
>2.27.0

Actions: View | Diff | Splinter Review

Attachments on bug 15851: 66593 | 106260 | 106261 | 106821 | 106822 | 106988 | 106989 | 106990 | 108049 | 108050 | 108051 | 109267 | 109268 | 109269 | 109459 | 109460 | 109461 | 109462