From: Bob Stayton <bobs@sagehill.net>
Date: Sun, 25 Feb 2007 09:26:05 +0000 (+0000)
Subject: Fixed bug 1652360 empty link with xlink:href.
X-Git-Tag: release/1.79.1~6^2~2247
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=5350ad6239e081d4006ea29cf549950b63254aee;p=docbook-dsssl

Fixed bug 1652360 empty link with xlink:href.
---

diff --git a/xsl/fo/xref.xsl b/xsl/fo/xref.xsl
index bb62bd5da..f5a56b345 100644
--- a/xsl/fo/xref.xsl
+++ b/xsl/fo/xref.xsl
@@ -789,36 +789,35 @@
           <!-- If it has content, use it -->
           <xsl:apply-templates/>
         </xsl:when>
-        <xsl:otherwise>
-          <!-- else look for an endterm -->
+        <!-- look for an endterm -->
+        <xsl:when test="@endterm">
+          <xsl:variable name="etargets" select="key('id',@endterm)"/>
+          <xsl:variable name="etarget" select="$etargets[1]"/>
           <xsl:choose>
-            <xsl:when test="@endterm">
-              <xsl:variable name="etargets" select="key('id',@endterm)"/>
-              <xsl:variable name="etarget" select="$etargets[1]"/>
-              <xsl:choose>
-                <xsl:when test="count($etarget) = 0">
-                  <xsl:message>
-                    <xsl:value-of select="count($etargets)"/>
-                    <xsl:text>Endterm points to nonexistent ID: </xsl:text>
-                    <xsl:value-of select="@endterm"/>
-                  </xsl:message>
-                  <xsl:text>???</xsl:text>
-                </xsl:when>
-                <xsl:otherwise>
-                    <xsl:apply-templates select="$etarget" mode="endterm"/>
-                </xsl:otherwise>
-              </xsl:choose>
-            </xsl:when>
-  
-            <xsl:otherwise>
+            <xsl:when test="count($etarget) = 0">
               <xsl:message>
-                <xsl:text>Link element has no content and no Endterm. </xsl:text>
-                <xsl:text>Nothing to show in the link to </xsl:text>
-                <xsl:value-of select="$target"/>
+                <xsl:value-of select="count($etargets)"/>
+                <xsl:text>Endterm points to nonexistent ID: </xsl:text>
+                <xsl:value-of select="@endterm"/>
               </xsl:message>
               <xsl:text>???</xsl:text>
+            </xsl:when>
+            <xsl:otherwise>
+                <xsl:apply-templates select="$etarget" mode="endterm"/>
             </xsl:otherwise>
           </xsl:choose>
+        </xsl:when>
+        <!-- Use the xlink:href if no other text -->
+        <xsl:when test="@xlink:href">
+          <xsl:value-of select="@xlink:href"/>
+        </xsl:when>
+        <xsl:otherwise>
+          <xsl:message>
+            <xsl:text>Link element has no content and no Endterm. </xsl:text>
+            <xsl:text>Nothing to show in the link to </xsl:text>
+            <xsl:value-of select="$target"/>
+          </xsl:message>
+          <xsl:text>???</xsl:text>
         </xsl:otherwise>
       </xsl:choose>
     </fo:inline>
diff --git a/xsl/html/xref.xsl b/xsl/html/xref.xsl
index 1aa2d6eba..54de0f081 100644
--- a/xsl/html/xref.xsl
+++ b/xsl/html/xref.xsl
@@ -901,36 +901,35 @@
         <!-- If it has content, use it -->
         <xsl:apply-templates/>
       </xsl:when>
-      <xsl:otherwise>
-        <!-- else look for an endterm -->
+      <!-- else look for an endterm -->
+      <xsl:when test="@endterm">
+        <xsl:variable name="etargets" select="key('id',@endterm)"/>
+        <xsl:variable name="etarget" select="$etargets[1]"/>
         <xsl:choose>
-          <xsl:when test="@endterm">
-            <xsl:variable name="etargets" select="key('id',@endterm)"/>
-            <xsl:variable name="etarget" select="$etargets[1]"/>
-            <xsl:choose>
-              <xsl:when test="count($etarget) = 0">
-                <xsl:message>
-                  <xsl:value-of select="count($etargets)"/>
-                  <xsl:text>Endterm points to nonexistent ID: </xsl:text>
-                  <xsl:value-of select="@endterm"/>
-                </xsl:message>
-                <xsl:text>???</xsl:text>
-              </xsl:when>
-              <xsl:otherwise>
-                  <xsl:apply-templates select="$etarget" mode="endterm"/>
-              </xsl:otherwise>
-            </xsl:choose>
-          </xsl:when>
-
-          <xsl:otherwise>
+          <xsl:when test="count($etarget) = 0">
             <xsl:message>
-              <xsl:text>Link element has no content and no Endterm. </xsl:text>
-              <xsl:text>Nothing to show in the link to </xsl:text>
-              <xsl:value-of select="(@xlink:href|@linkend)[1]"/>
+              <xsl:value-of select="count($etargets)"/>
+              <xsl:text>Endterm points to nonexistent ID: </xsl:text>
+              <xsl:value-of select="@endterm"/>
             </xsl:message>
             <xsl:text>???</xsl:text>
+          </xsl:when>
+          <xsl:otherwise>
+              <xsl:apply-templates select="$etarget" mode="endterm"/>
           </xsl:otherwise>
         </xsl:choose>
+      </xsl:when>
+      <!-- Use the xlink:href if no other text -->
+      <xsl:when test="@xlink:href">
+        <xsl:value-of select="@xlink:href"/>
+      </xsl:when>
+      <xsl:otherwise>
+        <xsl:message>
+          <xsl:text>Link element has no content and no Endterm. </xsl:text>
+          <xsl:text>Nothing to show in the link to </xsl:text>
+          <xsl:value-of select="(@xlink:href|@linkend)[1]"/>
+        </xsl:message>
+        <xsl:text>???</xsl:text>
       </xsl:otherwise>
     </xsl:choose>
   </xsl:variable>