[Koha-patches] [PATCH] Bug 3661 XSLT fixes

Jane Wagner jwagner at ptfs.com
Thu Sep 24 16:57:40 CEST 2009


The XSLT results list (MARC21slim2OPACResults.xsl) did not have any separators between multiple 246 fields, so additional titles all ran together.  I added the standard semicolon or period as in other repeating fields.

The display for personal and corporate authors was also confusing.  The period after the last personal author (100/700) is very hard to see, so the display looked like the first corporate author only belonged to/was associated with the last personal author, rather than being a separate author entry.  I've added a dash at the beginning of any 110/710 entries to set off the corporate authors a bit in the display.

I added the same code to insert a dash before any 111/711 (Meeting author) as well.

This patch also adds missing text labels for text_list_bullets.png
(bibliography) and remote.png (available online).
---
 .../prog/en/xslt/MARC21slim2OPACResults.xsl        |   13 +++++++++++++
 1 files changed, 13 insertions(+), 0 deletions(-)

diff --git a/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACResults.xsl b/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACResults.xsl
index 84f4714..7591f6a 100644
--- a/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACResults.xsl
+++ b/koha-tmpl/opac-tmpl/prog/en/xslt/MARC21slim2OPACResults.xsl
@@ -167,6 +167,7 @@
                 <img src="/opac-tmpl/prog/famfamfam/silk/cd.png" alt="optical disc" title="optical disc"/>
             </xsl:if>
             <xsl:if test="marc:controlfield[@tag=007][substring(text(),1,1)='c'][substring(text(),2,1)='r']">
+		available online
                 <img src="/opac-tmpl/prog/famfamfam/silk/drive_web.png" alt="remote" title="remote"/>
             </xsl:if>
             <xsl:if test="marc:controlfield[@tag=007][substring(text(),1,1)='c'][substring(text(),2,1)='a']">
@@ -452,6 +453,11 @@
 
         <xsl:for-each select="marc:datafield[@tag=110 or @tag=710]">
             <xsl:choose>
+            <xsl:when test="position()=1">
+		<xsl:text> -- </xsl:text>
+            </xsl:when>
+            </xsl:choose>
+            <xsl:choose>
             <xsl:when test="position()=last()">
                 <xsl:call-template name="nameABCDN"/>.
             </xsl:when>
@@ -463,6 +469,11 @@
 
         <xsl:for-each select="marc:datafield[@tag=111 or @tag=711]">
             <xsl:choose>
+            <xsl:when test="position()=1">
+		<xsl:text> -- </xsl:text>
+            </xsl:when>
+            </xsl:choose>
+            <xsl:choose>
             <xsl:when test="position()=last()">
                 <xsl:call-template name="nameACDEQ"/>.
             </xsl:when>
@@ -545,6 +556,7 @@
                     abstract or summary
                 </xsl:when>
                 <xsl:when test="contains($controlField008-24,'b')">
+			bibliography
                      <img src="/opac-tmpl/prog/famfamfam/silk/text_list_bullets.png" alt="bibliography" title="bibliography"/>
                 </xsl:when>
                 <xsl:when test="contains($controlField008-24,'c')">
@@ -863,6 +875,7 @@
                     <xsl:call-template name="subfieldSelect">
                         <xsl:with-param name="codes">ab</xsl:with-param>
                     </xsl:call-template>
+                    <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>
 	</span>
     </xsl:if>
-- 
1.5.6.5




More information about the Koha-patches mailing list