From 4041480b22005ab6cf3e49b3a9c23309047095c9 Mon Sep 17 00:00:00 2001
From: Andrew Morrison <andrew.morrison@bodleian.ox.ac.uk>
Date: Thu, 8 Mar 2018 15:29:38 +0000
Subject: [PATCH] For Hebrew/Genizah only, turn all refs not to URLs into
 searches

Temporary fix so cross-references work.
---
 msdesc2html.xsl | 44 ++++++++++++++++++++------------------------
 1 file changed, 20 insertions(+), 24 deletions(-)

diff --git a/msdesc2html.xsl b/msdesc2html.xsl
index 665a1c2..635e5b1 100644
--- a/msdesc2html.xsl
+++ b/msdesc2html.xsl
@@ -511,6 +511,26 @@
     <xsl:template match="ref[@target]" priority="10">
         <xsl:variable name="target" as="xs:string" select="normalize-space(@target)"/>
         <xsl:choose>
+            <xsl:when test="starts-with($target, 'http') or starts-with($target, 'mailto') or starts-with($target, 'ftp')">
+                <!-- For valid-looking URLs create links to external resources -->
+                <a href="{$target}">
+                    <xsl:apply-templates/>
+                </a>
+            </xsl:when>
+            <xsl:when test="starts-with($target, 'www')">
+                <!-- Assume if the protocol is missing that it is http (hopefully the destination server will redirect if https) -->
+                <a href="http://{$target}">
+                    <xsl:apply-templates/>
+                </a>
+            </xsl:when>
+            <xsl:when test="contains(base-uri(.), 'hebrew-mss') or contains(base-uri(.), 'genizah-mss')">
+                <!-- In Hebrew and Genizah, anything that isn't a URL appearsr to be a classmarks, sometimes with part numbers appended, 
+                     so as a TEMPORARY fix, convert them into links to search the catalogue -->
+                <a href="/?q={translate($target, '_#', '  ')}">
+                    <xsl:apply-templates/>
+                    <xsl:copy-of select="bod:logging('warn', 'Converting ref with unrecognized target into a search', ., $target)"/>
+                </a>
+            </xsl:when>
             <xsl:when test="starts-with($target, '#')">
                 <xsl:choose>
                     <xsl:when test="$target = (//@xml:id)">
@@ -526,30 +546,6 @@
                     </xsl:otherwise>
                 </xsl:choose>
             </xsl:when>
-            <xsl:when test="string-length($target) le 3">
-                <!-- Anything shorter than a few chars is very likely another old ID -->
-                <xsl:apply-templates/>
-                <xsl:copy-of select="bod:logging('warn', 'Skipping ref with invalid target', ., $target)"/>
-            </xsl:when>
-            <xsl:when test="starts-with($target, 'www')">
-                <!-- Assume if the protocol is missing that it is http (hopefully if it is https the destiantion server will redirect) -->
-                <a href="http://{$target}">
-                    <xsl:apply-templates/>
-                </a>
-            </xsl:when>
-            <xsl:when test="starts-with($target, 'http') or starts-with($target, 'mailto') or starts-with($target, 'ftp')">
-                <!-- For valid-looking URLs create links to external resources -->
-                <a href="{$target}">
-                    <xsl:apply-templates/>
-                </a>
-            </xsl:when>
-            <xsl:when test="contains(base-uri(.), 'hebrew-mss') or contains(base-uri(.), 'genizah-mss')">
-                <!-- In Hebrew and Genizah, a lot of these look like classmarks, so convert them into links to search the catalogue -->
-                <a href="/?q={translate($target, '_#', '  ')}">
-                    <xsl:apply-templates/>
-                    <xsl:copy-of select="bod:logging('warn', 'Converting ref with unrecognized target into a search', ., $target)"/>
-                </a>
-            </xsl:when>
             <xsl:otherwise>
                 <!-- Probably either an old ID or a book reference -->
                 <xsl:apply-templates/>