From: Keith Fahlgren <abdelazer@users.sourceforge.net>
Date: Thu, 20 Aug 2009 20:10:08 +0000 (+0000)
Subject: Improve the quality of the OPF spine regression by ensuring that the spine
X-Git-Tag: release/1.79.1~6^2~957
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=541282402e42a57c9ae88003810e4a89c7b9d365;p=docbook-dsssl

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.
---

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>