From: Norman Walsh <ndw@nwalsh.com>
Date: Mon, 12 Nov 2001 20:41:45 +0000 (+0000)
Subject: Support well-formed documents, use key() instead of id()
X-Git-Tag: release/1.79.1~6^2~6140
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=8dea11ce25423ac0f1596a692381f08fa9e26e52;p=docbook-dsssl

Support well-formed documents, use key() instead of id()
---

diff --git a/xsl/common/common.xsl b/xsl/common/common.xsl
index 6874cc27f..4241d5d7f 100644
--- a/xsl/common/common.xsl
+++ b/xsl/common/common.xsl
@@ -971,7 +971,7 @@ the ID is not unique.</para>
 <xsl:template name="check.id.unique">
   <xsl:param name="linkend"></xsl:param>
   <xsl:if test="$linkend != ''">
-    <xsl:variable name="targets" select="id($linkend)"/>
+    <xsl:variable name="targets" select="key('id',$linkend)"/>
     <xsl:variable name="target" select="$targets[1]"/>
 
     <xsl:if test="count($targets)=0">
@@ -1012,7 +1012,7 @@ pointed to by the link is one of the elements listed in
   <xsl:param name="linkend"></xsl:param>
   <xsl:param name="element-list"></xsl:param>
   <xsl:if test="$linkend != ''">
-    <xsl:variable name="targets" select="id($linkend)"/>
+    <xsl:variable name="targets" select="key('id',$linkend)"/>
     <xsl:variable name="target" select="$targets[1]"/>
 
     <xsl:if test="count($target) &gt; 0">
diff --git a/xsl/fo/docbook.xsl b/xsl/fo/docbook.xsl
index 2029640bb..1cdf05b4b 100644
--- a/xsl/fo/docbook.xsl
+++ b/xsl/fo/docbook.xsl
@@ -69,6 +69,10 @@
 
 <!-- ==================================================================== -->
 
+<xsl:key name="id" match="*" use="@id"/>
+
+<!-- ==================================================================== -->
+
 <xsl:template match="*">
   <xsl:message>
     <xsl:value-of select="name(.)"/>
@@ -122,7 +126,7 @@
     <xsl:choose>
       <xsl:when test="$rootid != ''">
         <xsl:choose>
-          <xsl:when test="count(id($rootid)) = 0">
+          <xsl:when test="count(key('id',$rootid)) = 0">
             <xsl:message terminate="yes">
               <xsl:text>ID '</xsl:text>
               <xsl:value-of select="$rootid"/>
@@ -131,14 +135,14 @@
           </xsl:when>
           <xsl:otherwise>
             <xsl:if test="$fop.extensions != 0">
-              <xsl:apply-templates select="id($rootid)" mode="outline"/>
+              <xsl:apply-templates select="key('id',$rootid)" mode="outline"/>
             </xsl:if>
             <xsl:if test="$xep.extensions != 0">
               <rx:outline xmlns:rx="http://www.renderx.com/XSL/Extensions">
-                <xsl:apply-templates select="id($rootid)" mode="xep.outline"/>
+                <xsl:apply-templates select="key('id',$rootid)" mode="xep.outline"/>
               </rx:outline>
             </xsl:if>
-            <xsl:apply-templates select="id($rootid)"/>
+            <xsl:apply-templates select="key('id',$rootid)"/>
           </xsl:otherwise>
         </xsl:choose>
       </xsl:when>
diff --git a/xsl/fo/ebnf.xsl b/xsl/fo/ebnf.xsl
index 168060c68..43e9d499a 100644
--- a/xsl/fo/ebnf.xsl
+++ b/xsl/fo/ebnf.xsl
@@ -141,7 +141,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
 </xsl:template>
 
 <xsl:template match="productionrecap">
-  <xsl:variable name="targets" select="id(@linkend)"/>
+  <xsl:variable name="targets" select="key('id',@linkend)"/>
   <xsl:variable name="target" select="$targets[1]"/>
 
   <xsl:if test="count($targets)=0">
@@ -209,7 +209,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
   <xsl:variable name="href">
     <xsl:choose>
       <xsl:when test="$linkend != ''">
-	<xsl:variable name="targets" select="id($linkend)"/>
+	<xsl:variable name="targets" select="key('id',$linkend)"/>
 	<xsl:variable name="target" select="$targets[1]"/>
         <xsl:call-template name="object.id">
           <xsl:with-param name="object" select="$target"/>
@@ -230,7 +230,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
       <xsl:otherwise>
         <xsl:choose>
           <xsl:when test="$linkend != ''">
-            <xsl:variable name="targets" select="id($linkend)"/>
+            <xsl:variable name="targets" select="key('id',$linkend)"/>
             <xsl:variable name="target" select="$targets[1]"/>
             <xsl:apply-templates select="$target/lhs"/>
           </xsl:when>
@@ -264,7 +264,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
   </xsl:call-template>
 
   <xsl:variable name="href">
-    <xsl:variable name="targets" select="id(@linkend)"/>
+    <xsl:variable name="targets" select="key('id',@linkend)"/>
     <xsl:variable name="target" select="$targets[1]"/>
     <xsl:call-template name="object.id">
       <xsl:with-param name="object" select="$target"/>
@@ -282,7 +282,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
       <xsl:text>: </xsl:text>
     </xsl:when>
     <xsl:otherwise>
-      <xsl:variable name="targets" select="id(@linkend)"/>
+      <xsl:variable name="targets" select="key('id',@linkend)"/>
       <xsl:variable name="target" select="$targets[1]"/>
       <xsl:if test="$target/@role">
 	<xsl:value-of select="$target/@role"/>
@@ -293,7 +293,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
 
   <fo:basic-link internal-destination="{$href}"
                  xsl:use-attribute-sets="xref.properties">
-    <xsl:variable name="targets" select="id(@linkend)"/>
+    <xsl:variable name="targets" select="key('id',@linkend)"/>
     <xsl:variable name="target" select="$targets[1]"/>
     <xsl:apply-templates select="$target" mode="title.markup"/>
   </fo:basic-link>
diff --git a/xsl/fo/footnote.xsl b/xsl/fo/footnote.xsl
index 014f34efb..010272181 100644
--- a/xsl/fo/footnote.xsl
+++ b/xsl/fo/footnote.xsl
@@ -27,7 +27,7 @@
 </xsl:template>
 
 <xsl:template match="footnoteref">
-  <xsl:variable name="footnote" select="id(@linkend)"/>
+  <xsl:variable name="footnote" select="key('id',@linkend)"/>
   <fo:inline>
     <xsl:text>[</xsl:text>
     <xsl:apply-templates select="$footnote" mode="footnote.number"/>
diff --git a/xsl/fo/lists.xsl b/xsl/fo/lists.xsl
index 7116207c8..29ab93f1f 100644
--- a/xsl/fo/lists.xsl
+++ b/xsl/fo/lists.xsl
@@ -568,7 +568,7 @@
 
 <xsl:template name="callout.arearef">
   <xsl:param name="arearef"></xsl:param>
-  <xsl:variable name="targets" select="id($arearef)"/>
+  <xsl:variable name="targets" select="key('id',$arearef)"/>
   <xsl:variable name="target" select="$targets[1]"/>
 
   <xsl:choose>
diff --git a/xsl/fo/synop.xsl b/xsl/fo/synop.xsl
index 1dade1aba..120704343 100644
--- a/xsl/fo/synop.xsl
+++ b/xsl/fo/synop.xsl
@@ -109,7 +109,7 @@
 <!-- ==================================================================== -->
 
 <xsl:template match="synopfragmentref">
-  <xsl:variable name="target" select="id(@linkend)"/>
+  <xsl:variable name="target" select="key('id',@linkend)"/>
   <xsl:variable name="snum">
     <xsl:apply-templates select="$target" mode="synopfragment.number"/>
   </xsl:variable>
diff --git a/xsl/fo/xref.xsl b/xsl/fo/xref.xsl
index adfcbf050..57350cec8 100644
--- a/xsl/fo/xref.xsl
+++ b/xsl/fo/xref.xsl
@@ -22,7 +22,7 @@
 <!-- ==================================================================== -->
 
 <xsl:template match="xref" name="xref">
-  <xsl:variable name="targets" select="id(@linkend)"/>
+  <xsl:variable name="targets" select="key('id',@linkend)"/>
   <xsl:variable name="target" select="$targets[1]"/>
   <xsl:variable name="refelem" select="local-name($target)"/>
 
@@ -53,7 +53,7 @@
                      xsl:use-attribute-sets="xref.properties">
         <xsl:choose>
 	  <xsl:when test="@endterm">
-	    <xsl:variable name="etargets" select="id(@endterm)"/>
+	    <xsl:variable name="etargets" select="key('id',@endterm)"/>
 	    <xsl:variable name="etarget" select="$etargets[1]"/>
 	    <xsl:choose>
 	      <xsl:when test="count($etarget) = 0">
@@ -278,7 +278,7 @@
 <!-- ==================================================================== -->
 
 <xsl:template match="link" name="link">
-  <xsl:variable name="targets" select="id(@linkend)"/>
+  <xsl:variable name="targets" select="key('id',@linkend)"/>
   <xsl:variable name="target" select="$targets[1]"/>
 
   <xsl:call-template name="check.id.unique">
@@ -296,7 +296,7 @@
         <!-- else look for an endterm -->
         <xsl:choose>
           <xsl:when test="@endterm">
-            <xsl:variable name="etargets" select="id(@endterm)"/>
+            <xsl:variable name="etargets" select="key('id',@endterm)"/>
             <xsl:variable name="etarget" select="$etargets[1]"/>
             <xsl:choose>
               <xsl:when test="count($etarget) = 0">
diff --git a/xsl/html/chunk-experimental.xsl b/xsl/html/chunk-experimental.xsl
index 64a1f232a..f8fb9f386 100644
--- a/xsl/html/chunk-experimental.xsl
+++ b/xsl/html/chunk-experimental.xsl
@@ -348,8 +348,8 @@
   <xsl:variable name="next-id"
                 select="$chunk/following-sibling::chunk[1]/@id"/>
 
-  <xsl:variable name="prev" select="id($prev-id)"/>
-  <xsl:variable name="next" select="id($next-id)"/>
+  <xsl:variable name="prev" select="key('id',$prev-id)"/>
+  <xsl:variable name="next" select="key('id',$next-id)"/>
 
   <xsl:variable name="ischunk">
     <xsl:call-template name="chunk"/>
@@ -523,7 +523,7 @@
   <xsl:choose>
     <xsl:when test="$rootid != ''">
       <xsl:choose>
-        <xsl:when test="count(id($rootid)) = 0">
+        <xsl:when test="count(key('id',$rootid)) = 0">
           <xsl:message terminate="yes">
             <xsl:text>ID '</xsl:text>
             <xsl:value-of select="$rootid"/>
@@ -531,7 +531,7 @@
           </xsl:message>
         </xsl:when>
         <xsl:otherwise>
-          <xsl:apply-templates select="id($rootid)"/>
+          <xsl:apply-templates select="key('id',$rootid)"/>
         </xsl:otherwise>
       </xsl:choose>
     </xsl:when>
diff --git a/xsl/html/chunk.xsl b/xsl/html/chunk.xsl
index 01a438117..e88e89d9d 100644
--- a/xsl/html/chunk.xsl
+++ b/xsl/html/chunk.xsl
@@ -927,7 +927,7 @@
   <xsl:choose>
     <xsl:when test="$rootid != ''">
       <xsl:choose>
-        <xsl:when test="count(id($rootid)) = 0">
+        <xsl:when test="count(key('id',$rootid)) = 0">
           <xsl:message terminate="yes">
             <xsl:text>ID '</xsl:text>
             <xsl:value-of select="$rootid"/>
@@ -935,7 +935,7 @@
           </xsl:message>
         </xsl:when>
         <xsl:otherwise>
-          <xsl:apply-templates select="id($rootid)"/>
+          <xsl:apply-templates select="key('id',$rootid)"/>
         </xsl:otherwise>
       </xsl:choose>
     </xsl:when>
diff --git a/xsl/html/docbook.xsl b/xsl/html/docbook.xsl
index d6a4b98c9..b5b447b51 100644
--- a/xsl/html/docbook.xsl
+++ b/xsl/html/docbook.xsl
@@ -64,6 +64,10 @@
 
 <!-- ==================================================================== -->
 
+<xsl:key name="id" match="*" use="@id"/>
+
+<!-- ==================================================================== -->
+
 <xsl:template match="*">
   <xsl:message>
     <xsl:text>No template matches </xsl:text>
@@ -179,7 +183,7 @@
   <xsl:choose>
     <xsl:when test="$rootid != ''">
       <xsl:choose>
-        <xsl:when test="count(id($rootid)) = 0">
+        <xsl:when test="count(key('id',$rootid)) = 0">
           <xsl:message terminate="yes">
             <xsl:text>ID '</xsl:text>
             <xsl:value-of select="$rootid"/>
@@ -187,7 +191,7 @@
           </xsl:message>
         </xsl:when>
         <xsl:otherwise>
-          <xsl:apply-templates select="id($rootid)" mode="process.root"/>
+          <xsl:apply-templates select="key('id',$rootid)" mode="process.root"/>
         </xsl:otherwise>
       </xsl:choose>
     </xsl:when>
diff --git a/xsl/html/ebnf.xsl b/xsl/html/ebnf.xsl
index 29c70212c..897192c95 100644
--- a/xsl/html/ebnf.xsl
+++ b/xsl/html/ebnf.xsl
@@ -146,7 +146,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
 </xsl:template>
 
 <xsl:template match="productionrecap">
-  <xsl:variable name="targets" select="id(@linkend)"/>
+  <xsl:variable name="targets" select="key('id',@linkend)"/>
   <xsl:variable name="target" select="$targets[1]"/>
 
   <xsl:if test="count($targets)=0">
@@ -218,7 +218,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
   <xsl:variable name="href">
     <xsl:choose>
       <xsl:when test="$linkend != ''">
-	<xsl:variable name="targets" select="id($linkend)"/>
+	<xsl:variable name="targets" select="key('id',$linkend)"/>
 	<xsl:variable name="target" select="$targets[1]"/>
 	<xsl:call-template name="href.target">
 	  <xsl:with-param name="object" select="$target"/>
@@ -238,7 +238,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
       <xsl:otherwise>
 	<xsl:choose>
 	  <xsl:when test="$linkend != ''">
-	    <xsl:variable name="targets" select="id($linkend)"/>
+	    <xsl:variable name="targets" select="key('id',$linkend)"/>
 	    <xsl:variable name="target" select="$targets[1]"/>
 	    <xsl:apply-templates select="$target/lhs"/>
 	  </xsl:when>
@@ -273,7 +273,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
   </xsl:call-template>
 
   <xsl:variable name="href">
-    <xsl:variable name="targets" select="id(@linkend)"/>
+    <xsl:variable name="targets" select="key('id',@linkend)"/>
     <xsl:variable name="target" select="$targets[1]"/>
     <xsl:call-template name="href.target">
       <xsl:with-param name="object" select="$target"/>
@@ -288,7 +288,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
       <xsl:text>: </xsl:text>
     </xsl:when>
     <xsl:otherwise>
-      <xsl:variable name="targets" select="id(@linkend)"/>
+      <xsl:variable name="targets" select="key('id',@linkend)"/>
       <xsl:variable name="target" select="$targets[1]"/>
       <xsl:if test="$target/@role">
 	<xsl:value-of select="$target/@role"/>
@@ -298,7 +298,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
   </xsl:choose>
 
   <a href="{$href}">
-    <xsl:variable name="targets" select="id(@linkend)"/>
+    <xsl:variable name="targets" select="key('id',@linkend)"/>
     <xsl:variable name="target" select="$targets[1]"/>
     <xsl:apply-templates select="$target" mode="title.markup"/>
   </a>
diff --git a/xsl/html/footnote.xsl b/xsl/html/footnote.xsl
index ee2607b8f..64b9b3d1b 100644
--- a/xsl/html/footnote.xsl
+++ b/xsl/html/footnote.xsl
@@ -44,7 +44,7 @@
 </xsl:template>
 
 <xsl:template match="footnoteref">
-  <xsl:variable name="targets" select="id(@linkend)"/>
+  <xsl:variable name="targets" select="key('id',@linkend)"/>
   <xsl:variable name="footnote" select="$targets[1]"/>
   <xsl:variable name="href">
     <xsl:text>#ftn.</xsl:text>
diff --git a/xsl/html/inline.xsl b/xsl/html/inline.xsl
index fa8c07227..7416db1f0 100644
--- a/xsl/html/inline.xsl
+++ b/xsl/html/inline.xsl
@@ -510,7 +510,7 @@
 <xsl:template match="glossterm">
   <xsl:choose>
     <xsl:when test="@linkend">
-      <xsl:variable name="targets" select="id(@linkend)"/>
+      <xsl:variable name="targets" select="key('id',@linkend)"/>
       <xsl:variable name="target" select="$targets[1]"/>
 
       <xsl:call-template name="check.id.unique">
diff --git a/xsl/html/lists.xsl b/xsl/html/lists.xsl
index 338b23169..050f7ce55 100644
--- a/xsl/html/lists.xsl
+++ b/xsl/html/lists.xsl
@@ -615,7 +615,7 @@
 
 <xsl:template name="callout.arearef">
   <xsl:param name="arearef"></xsl:param>
-  <xsl:variable name="targets" select="id($arearef)"/>
+  <xsl:variable name="targets" select="key('id',$arearef)"/>
   <xsl:variable name="target" select="$targets[1]"/>
 
   <xsl:call-template name="check.id.unique">
diff --git a/xsl/html/param.xweb b/xsl/html/param.xweb
index 40a2dace4..234b41d47 100644
--- a/xsl/html/param.xweb
+++ b/xsl/html/param.xweb
@@ -156,7 +156,6 @@ to be incomplete. Don't forget to read the source, too :-)</para>
 </reference>
 
 <reference><title>XSLT Processing</title>
-&use.id.function;
 &rootid;
 &suppress.navigation;
 </reference>
@@ -382,7 +381,6 @@ around all these parameters.</para>
 <src:fragref linkend="ulink.target.frag"/>
 <src:fragref linkend="use.extensions.frag"/>
 <src:fragref linkend="use.id.as.filename.frag"/>
-<src:fragref linkend="use.id.function.frag"/>
 <src:fragref linkend="using.chunker.frag"/>
 <src:fragref linkend="table.borders.with.css.frag"/>
 <src:fragref linkend="table.border.thickness.frag"/>
diff --git a/xsl/html/synop.xsl b/xsl/html/synop.xsl
index ee908a2d4..883ad4820 100644
--- a/xsl/html/synop.xsl
+++ b/xsl/html/synop.xsl
@@ -112,7 +112,7 @@
 <!-- ==================================================================== -->
 
 <xsl:template match="synopfragmentref">
-  <xsl:variable name="target" select="id(@linkend)"/>
+  <xsl:variable name="target" select="key('id',@linkend)"/>
   <xsl:variable name="snum">
     <xsl:apply-templates select="$target" mode="synopfragment.number"/>
   </xsl:variable>
diff --git a/xsl/html/toc.xsl b/xsl/html/toc.xsl
index f33a1a141..f2f6b5117 100644
--- a/xsl/html/toc.xsl
+++ b/xsl/html/toc.xsl
@@ -120,7 +120,7 @@
 </xsl:template>
 
 <xsl:template name="tocentry-content">
-  <xsl:variable name="targets" select="id(@linkend)"/>
+  <xsl:variable name="targets" select="key('id',@linkend)"/>
   <xsl:variable name="target" select="$targets[1]"/>
 
   <xsl:choose>
diff --git a/xsl/html/xref.xsl b/xsl/html/xref.xsl
index 8713d73bb..4fec41a6c 100644
--- a/xsl/html/xref.xsl
+++ b/xsl/html/xref.xsl
@@ -23,7 +23,7 @@
 <!-- ==================================================================== -->
 
 <xsl:template match="xref" name="xref">
-  <xsl:variable name="targets" select="id(@linkend)"/>
+  <xsl:variable name="targets" select="key('id',@linkend)"/>
   <xsl:variable name="target" select="$targets[1]"/>
   <xsl:variable name="refelem" select="local-name($target)"/>
 
@@ -64,7 +64,7 @@
 
       <xsl:choose>
         <xsl:when test="@endterm">
-          <xsl:variable name="etargets" select="id(@endterm)"/>
+          <xsl:variable name="etargets" select="key('id',@endterm)"/>
           <xsl:variable name="etarget" select="$etargets[1]"/>
           <xsl:choose>
             <xsl:when test="count($etarget) = 0">
@@ -388,7 +388,7 @@
 <!-- ==================================================================== -->
 
 <xsl:template match="link" name="link">
-  <xsl:variable name="targets" select="id(@linkend)"/>
+  <xsl:variable name="targets" select="key('id',@linkend)"/>
   <xsl:variable name="target" select="$targets[1]"/>
 
   <xsl:call-template name="check.id.unique">
@@ -440,7 +440,7 @@
         <!-- else look for an endterm -->
         <xsl:choose>
           <xsl:when test="@endterm">
-            <xsl:variable name="etargets" select="id(@endterm)"/>
+            <xsl:variable name="etargets" select="key('id',@endterm)"/>
             <xsl:variable name="etarget" select="$etargets[1]"/>
             <xsl:choose>
               <xsl:when test="count($etarget) = 0">
@@ -503,7 +503,7 @@
     <xsl:choose>
       <xsl:when test="@linkmode">
         <!-- use the linkmode to get the base URI, use localinfo as fragid -->
-        <xsl:variable name="modespec" select="id(@linkmode)"/>
+        <xsl:variable name="modespec" select="key('id',@linkmode)"/>
         <xsl:if test="count($modespec) != 1
                       or local-name($modespec) != 'modespec'">
           <xsl:message>Warning: olink linkmode pointer is wrong.</xsl:message>