Attachment 20680 Details for Bug 10793 – Bug 10793 - XSLT change 780/785 search queries to use ti,phr indexes like other linking entries

[patch] Bug 10793 - XSLT change 780/785 search queries to use ti,phr indexes like other linking entries

Bug-10793---XSLT-change-780785-search-queries-to-u.patch (text/plain), 4.43 KB, created by David Cook on 2013-08-28 02:03:30 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Cook

Created: 2013-08-28 02:03:30 UTC

Size: 4.43 KB

patch

obsolete

>From 303f2441711940e1f17d6dfe8a82da4468fca3e0 Mon Sep 17 00:00:00 2001
>From: David Cook <dcook@prosentient.com.au>
>Date: Wed, 28 Aug 2013 11:55:02 +1000
>Subject: [PATCH] Bug 10793 - XSLT change 780/785 search queries to use ti,phr
> indexes like other linking entries
>
>This patch adds the ti,phr index to the search queries for the 780
>and 785 MARC fields in the XSLT templates.
>
>Test Plan:
>
>1) Use a record with 780 and 785 MARC fields with data in the "a" or
>"t" subfields. (You might need to add these fields to an existing
>record, or create a new record. You might also need to change your
>MARC Bibliographic Framework settings to not "hide" these fields in
>your framework(s)).
>
>N.B. The data in the "a" or "t" subfields should be a title that
>exists in your database. Otherwise, this test might seem misleading.
>
>2) Set the system preferences "XSLTDetailsDisplay" and
>"OPACXSLTDetailsDisplay" to "default".
>
>3) Go to your record (which contains the 780/785 fields), and look
>for links with linktext derived from the "a" or "t" fields in the
>record. These may or may not be prefaced with labels such as
>"Continued by" or "Continues".
>
>4) Note that the links are formatted "q=TITLE". Click this link
>and note the possibly high number of results.
>
>5) APPLY THE PATCH
>
>6) Shift+Refresh your record page, and observe that the link should
>now say "q=ti,phr:TITLE". Click this link and note that you are
>either taken directly to the referenced article or that your
>search results are considerably fewer.
>---
> .../prog/en/xslt/MARC21slim2intranetDetail.xsl     |    4 ++--
> .../prog/en/xslt/MARC21slim2OPACDetail.xsl         |    4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl b/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl
>index caa0ab5..14ff809 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl
>+++ b/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl
>@@ -754,7 +754,7 @@
>                     </a>
>                 </xsl:when>
>                 <xsl:otherwise>
>-                    <a><xsl:attribute name="href">/cgi-bin/koha/catalogue/search.pl?q=<xsl:value-of select="translate($f780, '()', '')"/></xsl:attribute>
>+                    <a><xsl:attribute name="href">/cgi-bin/koha/catalogue/search.pl?q=ti,phr:<xsl:value-of select="translate($f780, '()', '')"/></xsl:attribute>
>                         <xsl:value-of select="translate($f780, '()', '')"/>
>                     </a>
>                 </xsl:otherwise>
>@@ -816,7 +816,7 @@
>                     </a>
>                 </xsl:when>
>                 <xsl:otherwise>
>-                    <a><xsl:attribute name="href">/cgi-bin/koha/catalogue/search.pl?q=<xsl:value-of select="translate($f785, '()', '')"/></xsl:attribute>
>+                    <a><xsl:attribute name="href">/cgi-bin/koha/catalogue/search.pl?q=ti,phr:<xsl:value-of select="translate($f785, '()', '')"/></xsl:attribute>
>                         <xsl:value-of select="translate($f785, '()', '')"/>
>                     </a>
>                 </xsl:otherwise>
>diff --git a/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACDetail.xsl b/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACDetail.xsl
>index fd0eb68..9901d75 100644
>--- a/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACDetail.xsl
>+++ b/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACDetail.xsl
>@@ -860,7 +860,7 @@
>                     </a>
>                 </xsl:when>
>                 <xsl:otherwise>
>-                    <a><xsl:attribute name="href">/cgi-bin/koha/opac-search.pl?q=<xsl:value-of select="translate($f780, '()', '')"/></xsl:attribute>
>+                    <a><xsl:attribute name="href">/cgi-bin/koha/opac-search.pl?q=ti,phr:<xsl:value-of select="translate($f780, '()', '')"/></xsl:attribute>
>                         <xsl:value-of select="translate($f780, '()', '')"/>
>                     </a>
>                 </xsl:otherwise>
>@@ -923,7 +923,7 @@
>                     </a>
>                 </xsl:when>
>                 <xsl:otherwise>
>-                    <a><xsl:attribute name="href">/cgi-bin/koha/opac-search.pl?q=<xsl:value-of select="translate($f785, '()', '')"/></xsl:attribute>
>+                    <a><xsl:attribute name="href">/cgi-bin/koha/opac-search.pl?q=ti,phr:<xsl:value-of select="translate($f785, '()', '')"/></xsl:attribute>
>                         <xsl:value-of select="translate($f785, '()', '')"/>
>                     </a>
>                 </xsl:otherwise>
>-- 
>1.7.7.4

Actions: View | Diff | Splinter Review

Attachments on bug 10793: 20680 | 20843 | 20860