Bugzilla – Attachment 20860 Details for
Bug 10793
XSLT change 780/785 search queries to use ti,phr indexes like other linking entries
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PASSED QA] Bug 10793 - XSLT change 780/785 search queries to use ti,phr indexes like other linking entries
PASSED-QA-Bug-10793---XSLT-change-780785-search-qu.patch (text/plain), 4.60 KB, created by
Katrin Fischer
on 2013-09-07 19:35:53 UTC
(
hide
)
Description:
[PASSED QA] Bug 10793 - XSLT change 780/785 search queries to use ti,phr indexes like other linking entries
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2013-09-07 19:35:53 UTC
Size:
4.60 KB
patch
obsolete
>From 128af5e543c577e196a3e241571672048fbdb1c6 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] [PASSED QA] 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. > >Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> >Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> >XSLT change only, no regressions found. >--- > .../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 5b53629..c3a846c 100644 >--- a/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACDetail.xsl >+++ b/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACDetail.xsl >@@ -902,7 +902,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> >@@ -965,7 +965,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.9.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 10793
:
20680
|
20843
| 20860