From: Norman Walsh <ndw@nwalsh.com>
Date: Sun, 13 Apr 2003 17:59:31 +0000 (+0000)
Subject: A few bug fixes for the colsep/rowsep code
X-Git-Tag: release/1.79.1~6^2~4698
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e67cc24b9b030f1ea808f23ccd4aa2bcab7acbd0;p=docbook-dsssl

A few bug fixes for the colsep/rowsep code
---

diff --git a/xsl/common/table.xsl b/xsl/common/table.xsl
index 4d4b58d90..416d346f7 100644
--- a/xsl/common/table.xsl
+++ b/xsl/common/table.xsl
@@ -70,28 +70,6 @@
   </xsl:if>
 </xsl:template>
 
-<!-- Modified versions of finaltd and sfinaltd, return a non-empty string -->
-<!-- if there are more table cells in this row -->
-
-<xsl:template name="has-more-td">
-  <xsl:param name="spans"/>
-  <xsl:param name="col" select="0"/>
-
-  <xsl:if test="$spans != ''">
-    <xsl:choose>
-      <xsl:when test="starts-with($spans,'0:')">
-        <xsl:text>X</xsl:text>
-      </xsl:when>
-      <xsl:otherwise></xsl:otherwise>
-    </xsl:choose>
-
-    <xsl:call-template name="has-more-td">
-      <xsl:with-param name="spans" select="substring-after($spans,':')"/>
-      <xsl:with-param name="col" select="$col+1"/>
-    </xsl:call-template>
-  </xsl:if>
-</xsl:template>
-
 <xsl:template name="entry.colnum">
   <xsl:param name="entry" select="."/>
 
diff --git a/xsl/fo/table.xsl b/xsl/fo/table.xsl
index d60b109b1..c78cc6638 100644
--- a/xsl/fo/table.xsl
+++ b/xsl/fo/table.xsl
@@ -65,7 +65,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
   <xsl:variable name="rowsep">
     <xsl:choose>
       <!-- If this is the last row, rowsep never applies. -->
-      <xsl:when test="not(ancestor-or-self::row/following-sibling::row
+      <xsl:when test="not(ancestor-or-self::row[1]/following-sibling::row
                           or ancestor-or-self::thead/following-sibling::tbody
                           or ancestor-or-self::tbody/preceding-sibling::tfoot)">
         <xsl:value-of select="0"/>
@@ -328,22 +328,10 @@ to be incomplete. Don't forget to read the source, too :-)</para>
     </xsl:call-template>
   </xsl:variable>
 
-  <xsl:variable name="more.cols.str">
-    <xsl:choose>
-      <xsl:when test="following-sibling::entry|following-sibling::entrytbl">X</xsl:when>
-      <xsl:otherwise>
-        <xsl:call-template name="has-more-td">
-          <xsl:with-param name="spans" select="$following.spans"/>
-          <xsl:with-param name="col" select="$col+$entry.colspan"/>
-        </xsl:call-template>
-      </xsl:otherwise>
-    </xsl:choose>
-  </xsl:variable>
-
   <xsl:variable name="rowsep">
     <xsl:choose>
       <!-- If this is the last row, rowsep never applies. -->
-      <xsl:when test="not(ancestor-or-self::row/following-sibling::row
+      <xsl:when test="not(ancestor-or-self::row[1]/following-sibling::row
                           or ancestor-or-self::thead/following-sibling::tbody
                           or ancestor-or-self::tbody/preceding-sibling::tfoot)">
         <xsl:value-of select="0"/>
@@ -365,7 +353,7 @@ to be incomplete. Don't forget to read the source, too :-)</para>
   <xsl:variable name="colsep">
     <xsl:choose>
       <!-- If this is the last column, colsep never applies. -->
-      <xsl:when test="$more.cols.str = ''">0</xsl:when>
+      <xsl:when test="$following.spans = ''">0</xsl:when>
       <xsl:otherwise>
         <xsl:call-template name="inherited.table.attribute">
           <xsl:with-param name="entry" select="."/>
diff --git a/xsl/html/table.xsl b/xsl/html/table.xsl
index 7895f4a37..2ec97b60c 100644
--- a/xsl/html/table.xsl
+++ b/xsl/html/table.xsl
@@ -29,7 +29,7 @@
   <xsl:variable name="rowsep">
     <xsl:choose>
       <!-- If this is the last row, rowsep never applies. -->
-      <xsl:when test="not(ancestor-or-self::row/following-sibling::row
+      <xsl:when test="not(ancestor-or-self::row[1]/following-sibling::row
                           or ancestor-or-self::thead/following-sibling::tbody
                           or ancestor-or-self::tbody/preceding-sibling::tfoot)">
         <xsl:value-of select="0"/>
@@ -635,22 +635,14 @@
     </xsl:call-template>
   </xsl:variable>
 
-  <xsl:variable name="more.cols.str">
-    <xsl:choose>
-      <xsl:when test="following-sibling::entry|following-sibling::entrytbl">X</xsl:when>
-      <xsl:otherwise>
-        <xsl:call-template name="has-more-td">
-          <xsl:with-param name="spans" select="$following.spans"/>
-          <xsl:with-param name="col" select="$col+$entry.colspan"/>
-        </xsl:call-template>
-      </xsl:otherwise>
-    </xsl:choose>
-  </xsl:variable>
-
   <xsl:variable name="rowsep">
     <xsl:choose>
       <!-- If this is the last row, rowsep never applies. -->
-      <xsl:when test="not(ancestor-or-self::row/following-sibling::row
+      <xsl:when test="ancestor::entrytbl
+                      and not (ancestor-or-self::row[1]/following-sibling::row)">
+        <xsl:value-of select="0"/>
+      </xsl:when>
+      <xsl:when test="not(ancestor-or-self::row[1]/following-sibling::row
                           or ancestor-or-self::thead/following-sibling::tbody
                           or ancestor-or-self::tbody/preceding-sibling::tfoot)">
         <xsl:value-of select="0"/>
@@ -668,7 +660,7 @@
   <xsl:variable name="colsep">
     <xsl:choose>
       <!-- If this is the last column, colsep never applies. -->
-      <xsl:when test="$more.cols.str = ''">0</xsl:when>
+      <xsl:when test="$following.spans = ''">0</xsl:when>
       <xsl:otherwise>
         <xsl:call-template name="inherited.table.attribute">
           <xsl:with-param name="entry" select="."/>