From 1545af592c7af03eb23c204a9085ddd49342c8fc Mon Sep 17 00:00:00 2001
From: Norman Walsh <ndw@nwalsh.com>
Date: Thu, 2 Jan 2003 13:45:34 +0000
Subject: [PATCH] Rework indexing to avoid XSLT limitation/Saxon bug with
 context dependencies; finish porting Bob's changes from FO to HTML

---
 xsl/fo/autoidx.xsl   |  76 ++++++++++++++++++++--
 xsl/fo/index.xsl     |  24 ++-----
 xsl/html/autoidx.xsl | 149 +++++++++++++++++++++++++++++++++----------
 xsl/html/index.xsl   |   8 +--
 4 files changed, 194 insertions(+), 63 deletions(-)

diff --git a/xsl/fo/autoidx.xsl b/xsl/fo/autoidx.xsl
index ec0a028ad..101d59e74 100644
--- a/xsl/fo/autoidx.xsl
+++ b/xsl/fo/autoidx.xsl
@@ -55,20 +55,58 @@
          match="indexterm[see]"
          use="concat(&primary;, &sep;, &secondary;, &sep;, &tertiary;, &sep;, see)"/>
 
+<!-- The obvious way to implement generate-index, generate-setindex, and
+     generate-index-from-terms would be to pass the root node for term
+     selection as a parameter. But if you do that, Saxon says:
+
+     java.lang.UnsupportedOperationException: Cannot create intensional
+       node-set with context dependencies: class com.icl.saxon.expr.PathExpression:128
+       at com.icl.saxon.expr.NodeSetIntent.<init>(NodeSetIntent.java:26)
+       ...
+
+     I can't decide if that's a Saxon bug or an XSLT limitation. But I want
+     to avoid it in either event. -->
+
 <xsl:template name="generate-index">
   <xsl:param name="root" select="/"/>
 
-  <xsl:variable name="scope" select=".." />
+  <xsl:variable name="scope" select="(ancestor::book|/)[last()]"/>
   <xsl:variable name="terms"
-                select="$root//indexterm[count(.|key('letter',
+                select="(ancestor::book|/)[last()]//indexterm[count(.|key('letter',
                                                 translate(substring(&primary;, 1, 1),
                                                           &lowercase;,
                                                           &uppercase;))[ancestor::* = $scope][1]) = 1
                                     and not(@class = 'endofrange')]"/>
 
+  <xsl:call-template name="generate-index-from-terms">
+    <xsl:with-param name="terms" select="$terms"/>
+    <xsl:with-param name="scope" select="$scope"/>
+  </xsl:call-template>
+</xsl:template>
+
+<xsl:template name="generate-setindex">
+  <xsl:variable name="scope" select="/"/>
+  <xsl:variable name="terms"
+                select="//indexterm[count(.|key('letter',
+                                                translate(substring(&primary;, 1, 1),
+                                                          &lowercase;,
+                                                          &uppercase;))[ancestor::* = $scope][1]) = 1
+                                    and not(@class = 'endofrange')]"/>
+
+  <xsl:call-template name="generate-index-from-terms">
+    <xsl:with-param name="terms" select="$terms"/>
+    <xsl:with-param name="scope" select="$scope"/>
+  </xsl:call-template>
+</xsl:template>
+
+<xsl:template name="generate-index-from-terms">
+  <xsl:param name="terms"/>
+  <xsl:param name="scope"/>
+
   <xsl:variable name="alphabetical"
                 select="$terms[contains(concat(&lowercase;, &uppercase;),
                                         substring(&primary;, 1, 1))]"/>
+
   <xsl:variable name="others" select="$terms[not(contains(concat(&lowercase;,
                                                  &uppercase;),
                                              substring(&primary;, 1, 1)))]"/>
@@ -102,6 +140,7 @@
 
 <xsl:template match="indexterm" mode="index-div">
   <xsl:param name="scope" select="."/>
+
   <xsl:variable name="key" select="translate(substring(&primary;, 1, 1),&lowercase;,&uppercase;)"/>
   <xsl:variable name="terms" select="key('letter', $key)[ancestor::* = $scope][count(.|key('primary', &primary;)[1]) = 1]"/>
   <fo:block>
@@ -116,8 +155,8 @@
     <fo:block>
       <xsl:apply-templates select="$terms"
                            mode="index-primary">
-        <xsl:sort select="translate(&primary;, &lowercase;, &uppercase;)"/>
         <xsl:with-param name="scope" select="$scope"/>
+        <xsl:sort select="translate(&primary;, &lowercase;, &uppercase;)"/>
       </xsl:apply-templates>
     </fo:block>
   </fo:block>
@@ -391,14 +430,37 @@
 <!-- ====================================================================== -->
 
 <xsl:template name="generate-index-markup">
-  <xsl:param name="root" select="/"/>
+  <xsl:variable name="scope" select="(ancestor::book|/)[last()]"/>
+  <xsl:variable name="terms" select="$scope//indexterm[count(.|key('letter',
+                                     translate(substring(&primary;, 1, 1),&lowercase;,
+                                     &uppercase;))[ancestor::* = $scope][1]) = 1]"/>
+
+  <xsl:call-template name="generate-index-markup-from-terms">
+    <xsl:with-param name="terms" select="$terms"/>
+    <xsl:with-param name="scope" select="$scope"/>
+  </xsl:call-template>
+</xsl:template>
+
+<xsl:template name="generate-setindex-markup">
+  <xsl:variable name="scope" select="/"/>
+  <xsl:variable name="terms" select="$scope//indexterm[count(.|key('letter',
+                                     translate(substring(&primary;, 1, 1),&lowercase;,
+                                     &uppercase;))[ancestor::* = $scope][1]) = 1]"/>
+
+  <xsl:call-template name="generate-index-markup-from-terms">
+    <xsl:with-param name="terms" select="$terms"/>
+    <xsl:with-param name="scope" select="$scope"/>
+  </xsl:call-template>
+</xsl:template>
+
+<xsl:template name="generate-index-markup-from-terms">
+  <xsl:param name="terms"/>
+  <xsl:param name="scope"/>
 
-  <xsl:variable name="scope" select=".."/>
-  <xsl:variable name="terms" select="$root//indexterm[count(.|key('letter',
-                                     translate(substring(&primary;, 1, 1),&lowercase;,&uppercase;))[ancestor::* = $scope][1]) = 1]"/>
   <xsl:variable name="alphabetical"
                 select="$terms[contains(concat(&lowercase;, &uppercase;),
                                         substring(&primary;, 1, 1))]"/>
+
   <xsl:variable name="others" select="$terms[not(contains(concat(&lowercase;,
                                                  &uppercase;),
                                              substring(&primary;, 1, 1)))]"/>
diff --git a/xsl/fo/index.xsl b/xsl/fo/index.xsl
index acddfceda..ef492bed1 100644
--- a/xsl/fo/index.xsl
+++ b/xsl/fo/index.xsl
@@ -24,9 +24,7 @@
   <xsl:choose>
     <xsl:when test="$make.index.markup != 0">
       <fo:block>
-        <xsl:call-template name="generate-index-markup">
-          <xsl:with-param name="root" select="(ancestor::book|/)[last()]"/>
-        </xsl:call-template>
+        <xsl:call-template name="generate-index-markup"/>
       </fo:block>
     </xsl:when>
     <xsl:otherwise>
@@ -35,9 +33,7 @@
         <xsl:call-template name="index.titlepage"/>
         <xsl:apply-templates/>
         <xsl:if test="count(indexentry) = 0 and count(indexdiv) = 0">
-          <xsl:call-template name="generate-index">
-            <xsl:with-param name="root" select="(ancestor::book|/)[last()]"/>
-          </xsl:call-template>
+          <xsl:call-template name="generate-index"/>
         </xsl:if>
       </fo:block>
     </xsl:otherwise>
@@ -90,18 +86,14 @@
                       white-space-collapse='false'
                       xsl:use-attribute-sets="monospace.verbatim.properties"
                       linefeed-treatment="preserve">
-              <xsl:call-template name="generate-index-markup">
-                <xsl:with-param name="root" select="(ancestor::book|/)[last()]"/>
-              </xsl:call-template>
+              <xsl:call-template name="generate-index-markup"/>
             </fo:block>
           </xsl:when>
           <xsl:when test="indexentry|indexdiv/indexentry">
             <xsl:apply-templates/>
           </xsl:when>
           <xsl:otherwise>
-            <xsl:call-template name="generate-index">
-              <xsl:with-param name="root" select="(ancestor::book|/)[last()]"/>
-            </xsl:call-template>
+            <xsl:call-template name="generate-index"/>
           </xsl:otherwise>
         </xsl:choose>
       </xsl:if>
@@ -155,18 +147,14 @@
                       white-space-collapse='false'
                       xsl:use-attribute-sets="monospace.verbatim.properties"
                       linefeed-treatment="preserve">
-              <xsl:call-template name="generate-index-markup">
-                <xsl:with-param name="root" select="/"/>
-              </xsl:call-template>
+              <xsl:call-template name="generate-setindex-markup"/>
             </fo:block>
           </xsl:when>
           <xsl:when test="indexentry|indexdiv/indexentry">
             <xsl:apply-templates/>
           </xsl:when>
           <xsl:otherwise>
-            <xsl:call-template name="generate-index">
-              <xsl:with-param name="root" select="/"/>
-            </xsl:call-template>
+            <xsl:call-template name="generate-setindex"/>
           </xsl:otherwise>
         </xsl:choose>
       </xsl:if>
diff --git a/xsl/html/autoidx.xsl b/xsl/html/autoidx.xsl
index 4f9f65b21..41803d2c6 100644
--- a/xsl/html/autoidx.xsl
+++ b/xsl/html/autoidx.xsl
@@ -85,19 +85,54 @@
 
 <xsl:key name="sections" match="*[@id]" use="@id"/>
 
+<!-- The obvious way to implement generate-index, generate-setindex, and
+     generate-index-from-terms would be to pass the root node for term
+     selection as a parameter. But if you do that, Saxon says:
+
+     java.lang.UnsupportedOperationException: Cannot create intensional
+       node-set with context dependencies: class com.icl.saxon.expr.PathExpression:128
+       at com.icl.saxon.expr.NodeSetIntent.<init>(NodeSetIntent.java:26)
+       ...
+
+     I can't decide if that's a Saxon bug or an XSLT limitation. But I want
+     to avoid it in either event. -->
+
 <xsl:template name="generate-index">
-  <xsl:param name="root" select="/"/>
+  <xsl:variable name="scope" select="(ancestor::book|/)[last()]"/>
+  <xsl:variable name="terms"
+                select="(ancestor::book|/)[last()]//indexterm[count(.|key('letter',
+                                                translate(substring(&primary;, 1, 1),
+                                                          &lowercase;,
+                                                          &uppercase;))[1]) = 1
+                                    and not(@class = 'endofrange')]"/>
+  <xsl:call-template name="generate-index-from-terms">
+    <xsl:with-param name="terms" select="$terms"/>
+    <xsl:with-param name="scope" select="$scope"/>
+  </xsl:call-template>
+</xsl:template>
 
+<xsl:template name="generate-setindex">
+  <xsl:variable name="scope" select="/"/>
   <xsl:variable name="terms"
-                select="$root//indexterm[count(.|key('letter',
+                select="//indexterm[count(.|key('letter',
                                                 translate(substring(&primary;, 1, 1),
                                                           &lowercase;,
                                                           &uppercase;))[1]) = 1
                                     and not(@class = 'endofrange')]"/>
+  <xsl:call-template name="generate-index-from-terms">
+    <xsl:with-param name="terms" select="$terms"/>
+    <xsl:with-param name="scope" select="$scope"/>
+  </xsl:call-template>
+</xsl:template>
+
+<xsl:template name="generate-index-from-terms">
+  <xsl:param name="terms"/>
+  <xsl:param name="scope"/>
 
   <xsl:variable name="alphabetical"
                 select="$terms[contains(concat(&lowercase;, &uppercase;),
                                         substring(&primary;, 1, 1))]"/>
+
   <xsl:variable name="others" select="$terms[not(contains(concat(&lowercase;,
                                                  &uppercase;),
                                              substring(&primary;, 1, 1)))]"/>
@@ -111,30 +146,37 @@
         </h3>
         <dl>
           <xsl:apply-templates select="$others[count(.|key('primary',
-                                       &primary;)[1]) = 1]"
+                                       &primary;)[ancestor::* = $scope][1]) = 1]"
                                mode="index-symbol-div">
+            <xsl:with-param name="scope" select="$scope"/>
             <xsl:sort select="translate(&primary;, &lowercase;, &uppercase;)"/>
           </xsl:apply-templates>
         </dl>
       </div>
     </xsl:if>
     <xsl:apply-templates select="$alphabetical[count(.|key('letter',
-                                 translate(substring(&primary;, 1, 1),&lowercase;,&uppercase;))[1]) = 1]"
+                                 translate(substring(&primary;, 1, 1),&lowercase;,&uppercase;))[ancestor::* = $scope][1]) = 1]"
                          mode="index-div">
+      <xsl:with-param name="scope" select="$scope"/>
       <xsl:sort select="translate(&primary;, &lowercase;, &uppercase;)"/>
     </xsl:apply-templates>
   </div>
 </xsl:template>
 
 <xsl:template match="indexterm" mode="index-div">
+  <xsl:param name="scope" select="."/>
+
   <xsl:variable name="key" select="translate(substring(&primary;, 1, 1),&lowercase;,&uppercase;)"/>
+  <xsl:variable name="terms" select="key('letter', $key)[ancestor::* = $scope][count(.|key('primary', &primary;)[1]) = 1]"/>
+
   <div class="indexdiv">
     <h3>
       <xsl:value-of select="translate($key, &lowercase;, &uppercase;)"/>
     </h3>
     <dl>
-      <xsl:apply-templates select="key('letter', $key)[count(.|key('primary', &primary;)[1]) = 1]"
+      <xsl:apply-templates select="$terms"
                            mode="index-primary">
+        <xsl:with-param name="scope" select="$scope"/>
         <xsl:sort select="translate(&primary;, &lowercase;, &uppercase;)"/>
       </xsl:apply-templates>
     </dl>
@@ -142,38 +184,51 @@
 </xsl:template>
 
 <xsl:template match="indexterm" mode="index-symbol-div">
+  <xsl:param name="scope" select="."/>
+
   <xsl:variable name="key" select="translate(substring(&primary;, 1, 1),&lowercase;,&uppercase;)"/>
-  <xsl:apply-templates select="key('letter', $key)[count(.|key('primary', &primary;)[1]) = 1]"
+  <xsl:apply-templates select="key('letter', $key)[count(.|key('primary', &primary;)[ancestor::* = $scope][1]) = 1]"
                        mode="index-primary">
+    <xsl:with-param name="scope" select="$scope"/>
     <xsl:sort select="translate(&primary;, &lowercase;, &uppercase;)"/>
   </xsl:apply-templates>
 </xsl:template>
 
 <xsl:template match="indexterm" mode="index-primary">
+  <xsl:param name="scope" select="."/>
+
   <xsl:variable name="key" select="&primary;"/>
-  <xsl:variable name="refs" select="key('primary', $key)"/>
+  <xsl:variable name="refs" select="key('primary', $key)[ancestor::* = $scope]"/>
+
   <dt>
     <xsl:value-of select="primary"/>
-    <xsl:for-each select="$refs[generate-id() = generate-id(key('primary-section', concat($key, &sep;, &section.id;))[1])]">
-      <xsl:apply-templates select="." mode="reference"/>
+
+    <xsl:for-each select="$refs[generate-id() = generate-id(key('primary-section', concat($key, &sep;, &section.id;))[ancestor::* = $scope][1])]">
+      <xsl:apply-templates select="." mode="reference">
+        <xsl:with-param name="scope" select="$scope"/>
+      </xsl:apply-templates>
     </xsl:for-each>
 
     <xsl:if test="$refs[not(secondary)]/*[self::see]">
-      <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see', concat(&primary;, &sep;, &sep;, &sep;, see))[1])]"
+      <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see', concat(&primary;, &sep;, &sep;, &sep;, see))[ancestor::* = $scope][1])]"
                            mode="index-see">
+        <xsl:with-param name="scope" select="$scope"/>
         <xsl:sort select="translate(see, &lowercase;, &uppercase;)"/>
       </xsl:apply-templates>
     </xsl:if>
   </dt>
+
   <xsl:if test="$refs/secondary or $refs[not(secondary)]/*[self::seealso]">
     <dd>
       <dl>
-        <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see-also', concat(&primary;, &sep;, &sep;, &sep;, seealso))[1])]"
+        <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see-also', concat(&primary;, &sep;, &sep;, &sep;, seealso))[ancestor::* = $scope][1])]"
                              mode="index-seealso">
+          <xsl:with-param name="scope" select="$scope"/>
           <xsl:sort select="translate(seealso, &lowercase;, &uppercase;)"/>
         </xsl:apply-templates>
-        <xsl:apply-templates select="$refs[secondary and count(.|key('secondary', concat($key, &sep;, &secondary;))[1]) = 1]" 
+        <xsl:apply-templates select="$refs[secondary and count(.|key('secondary', concat($key, &sep;, &secondary;))[ancestor::* = $scope][1]) = 1]" 
                              mode="index-secondary">
+         <xsl:with-param name="scope" select="$scope"/>
           <xsl:sort select="translate(&secondary;, &lowercase;, &uppercase;)"/>
         </xsl:apply-templates>
       </dl>
@@ -182,30 +237,39 @@
 </xsl:template>
 
 <xsl:template match="indexterm" mode="index-secondary">
+  <xsl:param name="scope" select="."/>
+
   <xsl:variable name="key" select="concat(&primary;, &sep;, &secondary;)"/>
   <xsl:variable name="refs" select="key('secondary', $key)"/>
+
   <dt>
     <xsl:value-of select="secondary"/>
-    <xsl:for-each select="$refs[generate-id() = generate-id(key('secondary-section', concat($key, &sep;, &section.id;))[1])]">
-      <xsl:apply-templates select="." mode="reference"/>
+    <xsl:for-each select="$refs[generate-id() = generate-id(key('secondary-section', concat($key, &sep;, &section.id;))[ancestor::* = $scope][1])]">
+      <xsl:apply-templates select="." mode="reference">
+        <xsl:with-param name="scope" select="$scope"/>
+      </xsl:apply-templates>
     </xsl:for-each>
 
     <xsl:if test="$refs[not(tertiary)]/*[self::see]">
-      <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see', concat(&primary;, &sep;, &secondary;, &sep;, &sep;, see))[1])]"
+      <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see', concat(&primary;, &sep;, &secondary;, &sep;, &sep;, see))[ancestor::* = $scope][1])]"
                            mode="index-see">
+        <xsl:with-param name="scope" select="$scope"/>
         <xsl:sort select="translate(see, &lowercase;, &uppercase;)"/>
       </xsl:apply-templates>
     </xsl:if>
   </dt>
+
   <xsl:if test="$refs/tertiary or $refs[not(tertiary)]/*[self::seealso]">
     <dd>
       <dl>
-        <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see-also', concat(&primary;, &sep;, &secondary;, &sep;, &sep;, seealso))[1])]"
+        <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see-also', concat(&primary;, &sep;, &secondary;, &sep;, &sep;, seealso))[ancestor::* = $scope][1])]"
                              mode="index-seealso">
+          <xsl:with-param name="scope" select="$scope"/>
           <xsl:sort select="translate(seealso, &lowercase;, &uppercase;)"/>
         </xsl:apply-templates>
-        <xsl:apply-templates select="$refs[tertiary and count(.|key('tertiary', concat($key, &sep;, &tertiary;))[1]) = 1]" 
+        <xsl:apply-templates select="$refs[tertiary and count(.|key('tertiary', concat($key, &sep;, &tertiary;))[ancestor::* = $scope][1]) = 1]"
                              mode="index-tertiary">
+          <xsl:with-param name="scope" select="$scope"/>
           <xsl:sort select="translate(&tertiary;, &lowercase;, &uppercase;)"/>
         </xsl:apply-templates>
       </dl>
@@ -214,30 +278,39 @@
 </xsl:template>
 
 <xsl:template match="indexterm" mode="index-tertiary">
+  <xsl:param name="scope" select="."/>
+
   <xsl:variable name="key" select="concat(&primary;, &sep;, &secondary;, &sep;, &tertiary;)"/>
   <xsl:variable name="refs" select="key('tertiary', $key)"/>
+
   <dt>
     <xsl:value-of select="tertiary"/>
-    <xsl:for-each select="$refs[generate-id() = generate-id(key('tertiary-section', concat($key, &sep;, &section.id;))[1])]">
-      <xsl:apply-templates select="." mode="reference"/>
+    <xsl:for-each select="$refs[generate-id() = generate-id(key('tertiary-section', concat($key, &sep;, &section.id;))[ancestor::* = $scope][1])]">
+      <xsl:apply-templates select="." mode="reference">
+        <xsl:with-param name="scope" select="$scope"/>
+      </xsl:apply-templates>
     </xsl:for-each>
 
     <xsl:if test="$refs/see">
-      <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see', concat(&primary;, &sep;, &secondary;, &sep;, &tertiary;, &sep;, see))[1])]"
+      <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see', concat(&primary;, &sep;, &secondary;, &sep;, &tertiary;, &sep;, see))[ancestor::* = $scope][1])]"
                            mode="index-see">
+        <xsl:with-param name="scope" select="$scope"/>
         <xsl:sort select="translate(see, &lowercase;, &uppercase;)"/>
       </xsl:apply-templates>
     </xsl:if>
   </dt>
+
   <xsl:if test="$refs/seealso">
     <dd>
       <dl>
-        <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see', concat(&primary;, &sep;, &secondary;, &sep;, &tertiary;, &sep;, see))[1])]"
+        <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see', concat(&primary;, &sep;, &secondary;, &sep;, &tertiary;, &sep;, see))[ancestor::* = $scope][1])]"
                              mode="index-see">
+          <xsl:with-param name="scope" select="$scope"/>
           <xsl:sort select="translate(see, &lowercase;, &uppercase;)"/>
         </xsl:apply-templates>
-        <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see-also', concat(&primary;, &sep;, &secondary;, &sep;, &tertiary;, &sep;, seealso))[1])]"
+        <xsl:apply-templates select="$refs[generate-id() = generate-id(key('see-also', concat(&primary;, &sep;, &secondary;, &sep;, &tertiary;, &sep;, seealso))[ancestor::* = $scope][1])]"
                              mode="index-seealso">
+          <xsl:with-param name="scope" select="$scope"/>
 	  <xsl:sort select="translate(seealso, &lowercase;, &uppercase;)"/>
         </xsl:apply-templates>
       </dl>
@@ -246,6 +319,7 @@
 </xsl:template>
 
 <xsl:template match="indexterm" mode="reference">
+  <xsl:param name="scope" select="."/>
   <xsl:param name="separator" select="', '"/>
 
   <xsl:value-of select="$separator"/>
@@ -253,6 +327,7 @@
     <xsl:when test="@zone and string(@zone)">
       <xsl:call-template name="reference">
         <xsl:with-param name="zones" select="normalize-space(@zone)"/>
+        <xsl:with-param name="scope" select="$scope"/>
       </xsl:call-template>
     </xsl:when>
     <xsl:otherwise>
@@ -271,8 +346,9 @@
       </a>
 
       <xsl:if test="key('endofrange', @id)">
-        <xsl:apply-templates select="key('endofrange', @id)[last()]"
+        <xsl:apply-templates select="key('endofrange', @id)[ancestor::* = $scope][last()]"
                              mode="reference">
+          <xsl:with-param name="scope" select="$scope"/>
           <xsl:with-param name="separator" select="'-'"/>
         </xsl:apply-templates>
       </xsl:if>
@@ -281,11 +357,13 @@
 </xsl:template>
 
 <xsl:template name="reference">
+  <xsl:param name="scope" select="."/>
   <xsl:param name="zones"/>
+
   <xsl:choose>
     <xsl:when test="contains($zones, ' ')">
       <xsl:variable name="zone" select="substring-before($zones, ' ')"/>
-      <xsl:variable name="target" select="key('sections', $zone)"/>
+      <xsl:variable name="target" select="key('sections', $zone)[ancestor::* = $scope]"/>
 
       <a>
         <xsl:attribute name="href">
@@ -298,11 +376,12 @@
       <xsl:text>, </xsl:text>
       <xsl:call-template name="reference">
         <xsl:with-param name="zones" select="substring-after($zones, ' ')"/>
+        <xsl:with-param name="scope" select="$scope"/>
       </xsl:call-template>
     </xsl:when>
     <xsl:otherwise>
       <xsl:variable name="zone" select="$zones"/>
-      <xsl:variable name="target" select="key('sections', $zone)"/>
+      <xsl:variable name="target" select="key('sections', $zone)[ancestor::* = $scope]"/>
 
       <a>
         <xsl:attribute name="href">
@@ -317,6 +396,8 @@
 </xsl:template>
 
 <xsl:template match="indexterm" mode="index-see">
+  <xsl:param name="scope" select="."/>
+
   <xsl:text> (</xsl:text>
   <xsl:call-template name="gentext">
     <xsl:with-param name="key" select="'see'"/>
@@ -327,13 +408,17 @@
 </xsl:template>
 
 <xsl:template match="indexterm" mode="index-seealso">
-  <xsl:text>(</xsl:text>
-  <xsl:call-template name="gentext">
-    <xsl:with-param name="key" select="'seealso'"/>
-  </xsl:call-template>
-  <xsl:text> </xsl:text>
-  <xsl:value-of select="seealso"/>
-  <xsl:text>)</xsl:text>
+  <xsl:param name="scope" select="."/>
+
+  <dt>
+    <xsl:text>(</xsl:text>
+    <xsl:call-template name="gentext">
+      <xsl:with-param name="key" select="'seealso'"/>
+    </xsl:call-template>
+    <xsl:text> </xsl:text>
+    <xsl:value-of select="seealso"/>
+    <xsl:text>)</xsl:text>
+  </dt>
 </xsl:template>
 
 <xsl:template match="*" mode="index-title-content">
diff --git a/xsl/html/index.xsl b/xsl/html/index.xsl
index 3b1e0b658..cfbb4248a 100644
--- a/xsl/html/index.xsl
+++ b/xsl/html/index.xsl
@@ -33,9 +33,7 @@
       <xsl:apply-templates/>
 
       <xsl:if test="count(indexentry) = 0 and count(indexdiv) = 0">
-        <xsl:call-template name="generate-index">
-          <xsl:with-param name="root" select="(ancestor::book|/)[last()]"/>
-        </xsl:call-template>
+        <xsl:call-template name="generate-index"/>
       </xsl:if>
 
       <xsl:if test="not(parent::article)">
@@ -64,9 +62,7 @@
       <xsl:apply-templates/>
 
       <xsl:if test="count(indexentry) = 0 and count(indexdiv) = 0">
-        <xsl:call-template name="generate-index">
-          <xsl:with-param name="root" select="/"/>
-        </xsl:call-template>
+        <xsl:call-template name="generate-setindex"/>
       </xsl:if>
 
       <xsl:if test="not(parent::article)">
-- 
2.40.0