From: Michael Smith <xmldoc@users.sourceforge.net>
Date: Mon, 6 Aug 2007 10:18:36 +0000 (+0000)
Subject: Problem:
X-Git-Tag: release/1.79.1~6^2~1831
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=23572c8e209cae3459c08b24c874ab7b4fec877c;p=docbook-dsssl

Problem:
  Rows in tables with more than 10 rows get sorted into incorrect
  order.
Cause:
  Code in manpages table handler does sorting, using xsl:sort, to
  deal with instances of cells that span multiple rows. But that
  sorting was being done alphabetically rather than numerically --
  the reason being that the default sort method for xsl:sort is
  "text" (alphabetical).
Fix:
  Added @data-type=number to relevant xsl:sort instances in table
  code.

Closes #1763332. Thanks to kesuke.
---

diff --git a/xsl/manpages/table.xsl b/xsl/manpages/table.xsl
index 6d2d159ee..5a1bb9d1d 100644
--- a/xsl/manpages/table.xsl
+++ b/xsl/manpages/table.xsl
@@ -254,7 +254,6 @@
     </xsl:for-each>
   </xsl:template>
 
-
   <!-- * ============================================================== -->
   <!-- *    Output the tbl(1)-formatted contents of each cell.            -->
   <!-- * ============================================================== -->
@@ -322,10 +321,12 @@
     </xsl:param>
     <xsl:param  name="cell-data-sorted">
       <!-- * Sort the cells so that the dummy cells get put where we -->
-      <!-- * need them in the structure. -->
+      <!-- * need them in the structure. Note that we need to specify -->
+      <!-- * data-type="number" here because the default sorting method -->
+      <!-- * for xsl:sort is "text" (alphabetical). -->
       <xsl:for-each select="exsl:node-set($cell-data-unsorted)/cell">
-        <xsl:sort select="@row"/>
-        <xsl:sort select="@slot"/>
+        <xsl:sort select="@row" data-type="number"/>
+        <xsl:sort select="@slot" data-type="number"/>
         <xsl:copy-of select="."/>
       </xsl:for-each>
     </xsl:param>