From: Jakub Skoczen <jakub@indexdata.dk>
Date: Wed, 22 Apr 2015 11:10:37 +0000 (+0200)
Subject: Heuristics to deal with Primo s/e/page PAZ-1001
X-Git-Tag: v1.9.0~3
X-Git-Url: http://git.indexdata.com/?p=pazpar2-moved-to-github.git;a=commitdiff_plain;h=4c117d2aa5edcdfcf8a2d4e963471aecac46d77b

Heuristics to deal with Primo s/e/page PAZ-1001
---

diff --git a/etc/xsl/primo-pz2.xsl b/etc/xsl/primo-pz2.xsl
index 031cb47..caa6d9a 100644
--- a/etc/xsl/primo-pz2.xsl
+++ b/etc/xsl/primo-pz2.xsl
@@ -78,11 +78,34 @@
       </pz:metadata>
     </xsl:for-each>
 
-    <xsl:for-each select="prim:spage">
-      <pz:metadata type="pages-number">
-        <xsl:value-of select="." />
-      </pz:metadata>
-    </xsl:for-each>
+    <xsl:variable name="spage" select="prim:spage" />
+    <xsl:variable name="epage" select="prim:epage" />
+    <xsl:variable name="pages" select="prim:pages" />
+
+    <xsl:choose>
+      <!-- in some case primo wrongly returns a range in spage, we're done -->
+      <xsl:when test="contains($spage,'-')">
+        <pz:metadata type="pages-number">
+          <xsl:value-of select="$spage" />
+        </pz:metadata>
+      </xsl:when>
+      <!-- if there is a range in pages, we're also done -->
+      <xsl:when test="contains($pages,'-')">
+        <pz:metadata type="pages-number">
+          <xsl:value-of select="$pages" />
+        </pz:metadata>
+      </xsl:when>
+      <!-- otherwise, construct -->
+      <xsl:when test="$spage">
+        <pz:metadata type="pages-number">
+          <xsl:value-of select="$spage" />
+          <xsl:if test="string-length($epage)">
+            <xsl:text>-</xsl:text>
+            <xsl:value-of select="$epage" />
+          </xsl:if>
+        </pz:metadata>
+      </xsl:when>
+    </xsl:choose>
 
     <xsl:for-each select="prim:issn">
       <pz:metadata type="issn">