From: Norman Walsh <ndw@nwalsh.com>
Date: Tue, 11 Jun 2002 15:25:36 +0000 (+0000)
Subject: When unwrapping; don't produce empty elements if there's no need
X-Git-Tag: release/1.79.1~6^2~5570
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=35e7a0364c2accb82ce62cd5c354ab5fa452ef55;p=docbook-dsssl

When unwrapping; don't produce empty elements if there's no need
---

diff --git a/xsl/html/html-rtf.xsl b/xsl/html/html-rtf.xsl
index 6890b5f9a..02d25ed5a 100644
--- a/xsl/html/html-rtf.xsl
+++ b/xsl/html/html-rtf.xsl
@@ -161,14 +161,16 @@
         <xsl:variable name="leading" select="set:leading($nodes,$block)"/>
         <xsl:variable name="trailing" select="set:trailing($nodes,$block)"/>
 
-        <xsl:element name="{local-name($wrap)}" namespace="{namespace-uri($wrap)}">
-          <xsl:for-each select="$wrap/@*">
-            <xsl:if test="$first != 0 or local-name(.) != 'id'">
-              <xsl:copy/>
-            </xsl:if>
-          </xsl:for-each>
-          <xsl:apply-templates select="$leading" mode="unwrap.p"/>
-        </xsl:element>
+        <xsl:if test="($wrap/@id and $first = 1) or $leading">
+          <xsl:element name="{local-name($wrap)}" namespace="{namespace-uri($wrap)}">
+            <xsl:for-each select="$wrap/@*">
+              <xsl:if test="$first != 0 or local-name(.) != 'id'">
+                <xsl:copy/>
+              </xsl:if>
+            </xsl:for-each>
+            <xsl:apply-templates select="$leading" mode="unwrap.p"/>
+          </xsl:element>
+        </xsl:if>
 
         <xsl:apply-templates select="$block" mode="unwrap.p"/>
 
@@ -182,14 +184,16 @@
       </xsl:when>
 
       <xsl:otherwise>
-        <xsl:element name="{local-name($wrap)}" namespace="{namespace-uri($wrap)}">
-          <xsl:for-each select="$wrap/@*">
-            <xsl:if test="$first != 0 or local-name(.) != 'id'">
-              <xsl:copy/>
-            </xsl:if>
-          </xsl:for-each>
-          <xsl:apply-templates select="$nodes" mode="unwrap.p"/>
-        </xsl:element>
+        <xsl:if test="($wrap/@id and $first = 1) or $nodes">
+          <xsl:element name="{local-name($wrap)}" namespace="{namespace-uri($wrap)}">
+            <xsl:for-each select="$wrap/@*">
+              <xsl:if test="$first != 0 or local-name(.) != 'id'">
+                <xsl:copy/>
+              </xsl:if>
+            </xsl:for-each>
+            <xsl:apply-templates select="$nodes" mode="unwrap.p"/>
+          </xsl:element>
+        </xsl:if>
       </xsl:otherwise>
     </xsl:choose>
   </xsl:if>