Attachment 30041 Details for Bug 12655 – Bug 12655 - PROG/CCSR deprecation: Correct hard-coded opac-tmpl/prog path in XSLT

[patch] Bug 12655 - PROG/CCSR deprecation: Correct hard-coded opac-tmpl/prog path in XSLT

Bug-12655---PROGCCSR-deprecation-Correct-hard-code.patch (text/plain), 6.47 KB, created by Owen Leonard on 2014-07-24 16:50:23 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2014-07-24 16:50:23 UTC

Size: 6.47 KB

patch

obsolete

>From 11e603b46f92ca28a736b3e40b477b0d9a59082e Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Thu, 24 Jul 2014 12:27:08 -0400
>Subject: [PATCH] Bug 12655 - PROG/CCSR deprecation: Correct hard-coded
> opac-tmpl/prog path in XSLT
>Content-Type: text/plain; charset="utf-8"
>
>Two XSL files in the bootstrap theme contain hard-coded paths to the
>prog theme directory. This patch adds a query of the opacthemes system
>preference and builds the path based on the returned value.
>
>To test in a MARC21 system:
>
>Create or locate a record which contains a subject which is linked to an
>authority record. View the detail page for that record in the bootstrap
>OPAC and confirm that there is a magnifying class icon next to the link
>for that subject heading. Inspect or right-click the image and "View
>image" [FF. "Open image in new tab" in Chrome] to verify that the image
>being shown is from the bootstrap theme directory.
>
>Also modified: The NORMARC XSL for the detail page which has been
>changed in the same way. I did not test it in a NORMARC catalog.
>---
> .../bootstrap/en/xslt/MARC21slim2OPACDetail.xsl    |    5 +++--
> .../bootstrap/en/xslt/NORMARCslim2OPACDetail.xsl   |    9 +++++----
> 2 files changed, 8 insertions(+), 6 deletions(-)
>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl b/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl
>index fdd5421..ea306ee 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACDetail.xsl
>@@ -28,6 +28,7 @@
>     <xsl:variable name="TraceSubjectSubdivisions" select="marc:sysprefs/marc:syspref[@name='TraceSubjectSubdivisions']"/>
>     <xsl:variable name="Show856uAsImage" select="marc:sysprefs/marc:syspref[@name='OPACDisplay856uAsImage']"/>
>     <xsl:variable name="OPACTrackClicks" select="marc:sysprefs/marc:syspref[@name='TrackClicks']"/>
>+    <xsl:variable name="theme" select="marc:sysprefs/marc:syspref[@name='opacthemes']"/>
>     <xsl:variable name="biblionumber" select="marc:datafield[@tag=999]/marc:subfield[@code='c']"/>
>     <xsl:variable name="TracingQuotesLeft">
>       <xsl:choose>
>@@ -578,7 +579,7 @@
>             <xsl:if test="marc:subfield[@code=9]">
>                 <a class='authlink'>
>                     <xsl:attribute name="href">/cgi-bin/koha/opac-authoritiesdetail.pl?authid=<xsl:value-of select="marc:subfield[@code=9]"/></xsl:attribute>
>-                    <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/prog/images/filefind.png"/>
>+                    <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/$theme/images/filefind.png"/>
>                 </a>
>             </xsl:if>
>             <xsl:choose>
>@@ -1051,7 +1052,7 @@
>             <xsl:if test="marc:subfield[@code=9]">
>                 <a class='authlink'>
>                     <xsl:attribute name="href">/cgi-bin/koha/opac-authoritiesdetail.pl?authid=<xsl:value-of select="marc:subfield[@code=9]"/></xsl:attribute>
>-                    <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/prog/images/filefind.png"/>
>+                    <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/$theme/images/filefind.png"/>
>                 </a>
>             </xsl:if>
>         </xsl:for-each>
>diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/xslt/NORMARCslim2OPACDetail.xsl b/koha-tmpl/opac-tmpl/bootstrap/en/xslt/NORMARCslim2OPACDetail.xsl
>index e9fd625..50cc78b 100644
>--- a/koha-tmpl/opac-tmpl/bootstrap/en/xslt/NORMARCslim2OPACDetail.xsl
>+++ b/koha-tmpl/opac-tmpl/bootstrap/en/xslt/NORMARCslim2OPACDetail.xsl
>@@ -33,6 +33,7 @@
>           </xsl:choose>
>         </xsl:variable>
>         <xsl:variable name="DisplayOPACiconsXSLT" select="marc:sysprefs/marc:syspref[@name='DisplayOPACiconsXSLT']"/>
>+        <xsl:variable name="theme" select="marc:sysprefs/marc:syspref[@name='opacthemes']"/>
> 
>         <xsl:variable name="leader" select="marc:leader"/>
>         <xsl:variable name="leader6" select="substring($leader,7,1)"/>
>@@ -105,7 +106,7 @@
>         <xsl:if test="marc:subfield[@code=9]">
>             <a class='authlink'>
>                 <xsl:attribute name="href">/cgi-bin/koha/opac-authoritiesdetail.pl?authid=<xsl:value-of select="marc:subfield[@code=9]"/></xsl:attribute>
>-                <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/prog/images/filefind.png"/>
>+                <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/$theme/images/filefind.png"/>
>             </a>
>         </xsl:if>
>         <xsl:choose>
>@@ -126,7 +127,7 @@
>         <xsl:if test="marc:subfield[@code=9]">
>             <a class='authlink'>
>                 <xsl:attribute name="href">/cgi-bin/koha/opac-authoritiesdetail.pl?authid=<xsl:value-of select="marc:subfield[@code=9]"/></xsl:attribute>
>-                <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/prog/images/filefind.png"/>
>+                <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/$theme/images/filefind.png"/>
>             </a>
>         </xsl:if>
>         <xsl:choose><xsl:when test="position()=last()"><xsl:text>.</xsl:text></xsl:when><xsl:otherwise><xsl:text>; </xsl:text></xsl:otherwise></xsl:choose>
>@@ -146,7 +147,7 @@
>         <xsl:if test="marc:subfield[@code=9]">
>             <a class='authlink'>
>                 <xsl:attribute name="href">/cgi-bin/koha/opac-authoritiesdetail.pl?authid=<xsl:value-of select="marc:subfield[@code=9]"/></xsl:attribute>
>-                <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/prog/images/filefind.png"/>
>+                <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/$theme/images/filefind.png"/>
>             </a>
>         </xsl:if>
>         <xsl:choose><xsl:when test="position()=last()"><xsl:text>.</xsl:text></xsl:when><xsl:otherwise><xsl:text>; </xsl:text></xsl:otherwise></xsl:choose>
>@@ -424,7 +425,7 @@
>             <xsl:if test="marc:subfield[@code=9]">
>                 <a class='authlink'>
>                     <xsl:attribute name="href">/cgi-bin/koha/opac-authoritiesdetail.pl?authid=<xsl:value-of select="marc:subfield[@code=9]"/></xsl:attribute>
>-                    <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/prog/images/filefind.png"/>
>+                    <img style="vertical-align:middle" height="15" width="15" src="/opac-tmpl/$theme/images/filefind.png"/>
>                 </a>
>             </xsl:if>
>             <xsl:choose>
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 12655: 30041 | 30737 | 30868 | 30872 | 31024 | 31687