From 2076a3407dd9256fb1ac6eb4f668c538f5b997c7 Mon Sep 17 00:00:00 2001
From: Bob Stayton <bobs@sagehill.net>
Date: Mon, 19 Apr 2004 02:37:59 +0000
Subject: [PATCH] Now reports proper error if $current.docid parameter used but
 the value is not found in target database.

---
 xsl/html/xref.xsl | 32 +++++++++++++++++++++++---------
 1 file changed, 23 insertions(+), 9 deletions(-)

diff --git a/xsl/html/xref.xsl b/xsl/html/xref.xsl
index 06955260c..91b92f615 100644
--- a/xsl/html/xref.xsl
+++ b/xsl/html/xref.xsl
@@ -42,8 +42,8 @@
   <xsl:choose>
     <xsl:when test="count($target) = 0">
       <xsl:message>
-	<xsl:text>XRef to nonexistent id: </xsl:text>
-	<xsl:value-of select="@linkend"/>
+        <xsl:text>XRef to nonexistent id: </xsl:text>
+        <xsl:value-of select="@linkend"/>
       </xsl:message>
       <xsl:text>???</xsl:text>
     </xsl:when>
@@ -952,15 +952,29 @@
                   <xsl:choose>
                     <!-- Was current.docid parameter set? -->
                     <xsl:when test="$current.docid != ''">
-                      <xsl:for-each select="$target.database" >
-                        <xsl:call-template name="targetpath" >
-                          <xsl:with-param name="dirnode" select="key('targetdoc-key', $current.docid)/parent::dir"/>
-                          <xsl:with-param name="targetdoc" select="$seek.targetdoc"/>
-                        </xsl:call-template>
-                      </xsl:for-each >
+                      <!-- Was it found in the database? -->
+                      <xsl:variable name="currentdoc.key" >
+                        <xsl:for-each select="$target.database" >
+                          <xsl:value-of select="key('targetdoc-key',
+                                                $current.docid)/@targetdoc" />
+                        </xsl:for-each>
+                      </xsl:variable>
+                      <xsl:choose>
+                        <xsl:when test="$currentdoc.key != ''">
+                          <xsl:for-each select="$target.database" >
+                            <xsl:call-template name="targetpath" >
+                              <xsl:with-param name="dirnode" select="key('targetdoc-key', $current.docid)/parent::dir"/>
+                              <xsl:with-param name="targetdoc" select="$seek.targetdoc"/>
+                            </xsl:call-template>
+                          </xsl:for-each >
+                        </xsl:when>
+                        <xsl:otherwise>
+                          <xsl:message>Olink error: cannot compute relative sitemap path because $current.docid '<xsl:value-of select="$current.docid"/>' not found in target database</xsl:message>
+                        </xsl:otherwise>
+                      </xsl:choose>
                     </xsl:when>
                     <xsl:otherwise>
-                      <xsl:message>Olink warning: cannot compute relative sitemap path without $current.docid parameter</xsl:message>
+                      <xsl:message>Olink error: cannot compute relative sitemap path without $current.docid parameter</xsl:message>
                     </xsl:otherwise>
                   </xsl:choose> 
                   <!-- In either case, add baseuri from its document entry-->
-- 
2.40.0