Attachment 72467 Details for Bug 20339 – Bug 20339: Unify MARC21 ISBN/ISSN handling in XSL

[patch] Bug 20339: Unify MARC21 ISBN/ISSN handling in XSL

0001-Bug-20339-Unify-MARC21-ISBN-ISSN-handling-in-XSL.patch (text/plain), 6.50 KB, created by paxed on 2018-03-06 10:23:52 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: paxed

Created: 2018-03-06 10:23:52 UTC

Size: 6.50 KB

patch

obsolete

>From 166275cb6d955bac186881ec5d047d2467a63139 Mon Sep 17 00:00:00 2001
>From: Pasi Kallinen <pasi.kallinen@joensuu.fi>
>Date: Tue, 6 Mar 2018 11:59:28 +0200
>Subject: [PATCH] Bug 20339: Unify MARC21 ISBN/ISSN handling in XSL
>
>The code to show the ISBN and ISSN is repeated between the search results
>and the detail view. Create a function to show the ISBN/ISSN, and call
>that instead.
>
>Test plan:
>1) Apply patch
>2) Search for any records, check that the ISBN and ISSN are shown
> correctly
>3) Go to biblio detail pages, check that the ISBN and ISSN are shown
> correctly
>
>Signed-off-by: Pasi Kallinen <pasi.kallinen@joensuu.fi>
>---
> .../prog/en/xslt/MARC21slim2intranetDetail.xsl | 38 +-------------------
> .../prog/en/xslt/MARC21slim2intranetResults.xsl | 19 +---------
> .../intranet-tmpl/prog/en/xslt/MARC21slimUtils.xsl | 40 ++++++++++++++++++++++
> 3 files changed, 42 insertions(+), 55 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl b/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl
>index b699341bc4..f4e2b04fa3 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl
>+++ b/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl
>@@ -447,43 +447,7 @@
> </xsl:if>
> 
> 
>- 
>- <xsl:if test="marc:datafield[@tag=020]/marc:subfield[@code='a']">
>- ISBN: 
>- <xsl:for-each select="marc:datafield[@tag=020]/marc:subfield[@code='a']">
>- 
>- <xsl:value-of select="."/>
>- <xsl:choose>
>- <xsl:when test="position()=last()">
>- <xsl:text>.</xsl:text>
>- </xsl:when>
>- <xsl:otherwise>
>- <xsl:text>; </xsl:text>
>- </xsl:otherwise>
>- </xsl:choose>
>- 
>- </xsl:for-each>
>- 
>- </xsl:if>
>-
>- 
>- <xsl:if test="marc:datafield[@tag=022]/marc:subfield[@code='a']">
>- ISSN: 
>- <xsl:for-each select="marc:datafield[@tag=022]/marc:subfield[@code='a']">
>- 
>- <xsl:value-of select="."/>
>- <xsl:choose>
>- <xsl:when test="position()=last()">
>- <xsl:text>.</xsl:text>
>- </xsl:when>
>- <xsl:otherwise>
>- <xsl:text>; </xsl:text>
>- </xsl:otherwise>
>- </xsl:choose>
>- 
>- </xsl:for-each>
>- 
>- </xsl:if>
>+ <xsl:call-template name="showISBNISSN"/>
> 
> <xsl:if test="marc:datafield[@tag=013]">
> 
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetResults.xsl b/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetResults.xsl
>index 0e8ebd6d31..d6c9a952c3 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetResults.xsl
>+++ b/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetResults.xsl
>@@ -895,24 +895,7 @@
> 
> </xsl:if>
> 
>- <xsl:if test="marc:datafield[@tag=020]">
>- ISBN: 
>- <xsl:for-each select="marc:datafield[@tag=020]">
>- <xsl:variable name="isbn" select="marc:subfield[@code='a']"/>
>- <xsl:value-of select="marc:subfield[@code='a']"/>
>- <xsl:choose><xsl:when test="position()=last()"><xsl:text>.</xsl:text></xsl:when><xsl:otherwise><xsl:text>; </xsl:text></xsl:otherwise></xsl:choose>
>- </xsl:for-each>
>- 
>- </xsl:if>
>-
>- <xsl:if test="marc:datafield[@tag=022]">
>- ISSN: 
>- <xsl:for-each select="marc:datafield[@tag=022]">
>- <xsl:value-of select="marc:subfield[@code='a']"/>
>- <xsl:choose><xsl:when test="position()=last()"><xsl:text>.</xsl:text></xsl:when><xsl:otherwise><xsl:text>; </xsl:text></xsl:otherwise></xsl:choose>
>- </xsl:for-each>
>- 
>- </xsl:if>
>+ <xsl:call-template name="showISBNISSN"/>
> 
> <xsl:if test="marc:datafield[@tag=250]">
> 
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slimUtils.xsl b/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slimUtils.xsl
>index 192107b280..a3db596e87 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slimUtils.xsl
>+++ b/koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slimUtils.xsl
>@@ -298,6 +298,46 @@
> 
> </xsl:template>
> 
>+ <xsl:template name="showISBNISSN">
>+ <xsl:call-template name="showSingleSubfield">
>+ <xsl:with-param name="tag">020</xsl:with-param>
>+ <xsl:with-param name="code">a</xsl:with-param>
>+ <xsl:with-param name="class">isbn</xsl:with-param>
>+ <xsl:with-param name="label">ISBN: </xsl:with-param>
>+ </xsl:call-template>
>+ <xsl:call-template name="showSingleSubfield">
>+ <xsl:with-param name="tag">022</xsl:with-param>
>+ <xsl:with-param name="code">a</xsl:with-param>
>+ <xsl:with-param name="class">issn</xsl:with-param>
>+ <xsl:with-param name="label">ISSN: </xsl:with-param>
>+ </xsl:call-template>
>+ </xsl:template>
>+
>+ <xsl:template name="showSingleSubfield">
>+ <xsl:param name="tag"/>
>+ <xsl:param name="code"/>
>+ <xsl:param name="class"/>
>+ <xsl:param name="label"/>
>+ <xsl:if test="marc:datafield[@tag=$tag]/marc:subfield[@code=$code]">
>+ <xsl:attribute name="class"><xsl:value-of select="concat('results_summary ', $class)"/></xsl:attribute>
>+ <xsl:value-of select="$label"/>
>+ <xsl:for-each select="marc:datafield[@tag=$tag]/marc:subfield[@code=$code]">
>+ <xsl:attribute name="property"><xsl:value-of select="$class"/></xsl:attribute>
>+ <xsl:value-of select="."/>
>+ <xsl:choose>
>+ <xsl:when test="position()=last()">
>+ <xsl:text>.</xsl:text>
>+ </xsl:when>
>+ <xsl:otherwise>
>+ <xsl:text>; </xsl:text>
>+ </xsl:otherwise>
>+ </xsl:choose>
>+ 
>+ </xsl:for-each>
>+ 
>+ </xsl:if>
>+ </xsl:template>
>+
> </xsl:stylesheet>
> 
> <!-- Stylus Studio meta-information - (c)1998-2002 eXcelon Corp.
>-- 
>2.11.0
>

Actions: View | Diff | Splinter Review

Attachments on bug 20339: 72467 | 72683 | 75847