From 00d19b809cc04b2b13baa41c23e1cf9f8b996736 Mon Sep 17 00:00:00 2001
From: Keith Fahlgren <abdelazer@users.sourceforge.net>
Date: Thu, 20 Aug 2009 20:10:08 +0000
Subject: [PATCH] Improve the quality of the OPF spine regression by ensuring
 that the spine elements for deeply nested refentries are in order and
 adjacent to their opening wrapper XHTML chunk.

---
 xsl/epub/bin/spec/epub_regressions_spec.rb | 8 ++++++++
 xsl/epub/docbook.xsl                       | 6 +++---
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/xsl/epub/bin/spec/epub_regressions_spec.rb b/xsl/epub/bin/spec/epub_regressions_spec.rb
index af7c7f19b..512f85bcf 100755
--- a/xsl/epub/bin/spec/epub_regressions_spec.rb
+++ b/xsl/epub/bin/spec/epub_regressions_spec.rb
@@ -182,6 +182,14 @@ describe DocBook::Epub do
     opf_lns.to_s.should =~ /<itemref idref="#{re01_id}"/
   end
 
+  it "should include chunked refentries in the spine in the correct order and adjacency even when they're deeply nested" do
+    opf_lns = opf_lines('orm.book.001.xml', @filedir)
+    before_refentry_id = opf_lns.to_s.sub(/.+<item id="([^"]+)" href="apa.html".+/m, '\1')
+    re01_id = opf_lns.to_s.sub(/.+<item id="([^"]+)" href="re01.html".+/m, '\1')
+    re02_id = opf_lns.to_s.sub(/.+<item id="([^"]+)" href="re02.html".+/m, '\1')
+    opf_lns.to_s.should =~ /<itemref idref="#{before_refentry_id}"[^>]*[^<]*<itemref idref="#{re01_id}"[^>]*>[^<]*<itemref idref="#{re02_id}"/m
+  end
+
   after(:all) do
     FileUtils.rm_r(@tmpdir, :force => true)
   end  
diff --git a/xsl/epub/docbook.xsl b/xsl/epub/docbook.xsl
index 19c9c5f28..6df4a1b39 100644
--- a/xsl/epub/docbook.xsl
+++ b/xsl/epub/docbook.xsl
@@ -761,10 +761,10 @@
       <!-- TODO: be nice to have a idref="titlepage" here -->
       <xsl:choose>
         <xsl:when test="$root.is.a.chunk != '0'">
-          <xsl:apply-templates select="/*|//refentry" mode="opf.spine"/>
+          <xsl:apply-templates select="/*" mode="opf.spine"/>
         </xsl:when>
         <xsl:otherwise>
-          <xsl:apply-templates select="/*/*|//refentry" mode="opf.spine"/>
+          <xsl:apply-templates select="/*/*" mode="opf.spine"/>
         </xsl:otherwise>
       </xsl:choose>
                                    
@@ -784,7 +784,7 @@
           <xsl:value-of select="generate-id(.)"/>
         </xsl:attribute>
       </xsl:element>
-      <xsl:apply-templates select="*" mode="opf.spine"/>
+      <xsl:apply-templates select="*|.//refentry" mode="opf.spine"/>
     </xsl:if>
   </xsl:template>
 
-- 
2.40.0