ansaurus

Question

Answer 1

+1 A:

Just take <xsl:text>Header</xsl:text> out of for-each...

Michał Chaniewski 2009-06-16 11:27:01

Yeah, that would work for this example.This question was a simplification of a real problem, I was creating an html table. In that problem the solution you gave falls down due to the table row tags not lining up within the xsl tags.Thanks though, sorry I wqasn't more explict.

Simon Nunn 2009-06-16 11:54:59

Answer 2

+4 A:

<?xml version="1.0" encoding="ISO-8859-1"?>
<xsl:stylesheet version="1.0"
xmlns:xsl="http://www.w3.org/1999/XSL/Transform"&gt;

<xsl:template match="a">Header
   <xsl:apply-templates/>
</xsl:template>

<xsl:template match="a/b">
  <xsl:apply-templates/>
</xsl:template>

</xsl:stylesheet>

Is practically what you want, and simpler than what you have now.

Alex Brown 2009-06-16 11:28:21

You need to do some work on whitespace formatting, though.

Alex Brown 2009-06-16 11:42:48

Cheers, I think you've shown me the right direction. As you said I'll need to jiggle the formating. The real world problem is going to output an html table, so there's quite a way to go. Cheers Simon

Simon Nunn 2009-06-16 12:17:54

The use of <xsl:text> is recommendable over the use of literal strings in the XSL. Output white-space is a lot more easily controlled while code format can be kept independent of output format. Also, a template that does nothing else but call <xsl:apply-templates/> is superfluous. This is happening anyway through the pre-defined default templates.

Tomalak 2009-06-16 12:21:53

Answer 3

+1 A:

The end result was more like:

        <?xml version="1.0" encoding="ISO-8859-1"?>
        <xsl:stylesheet  version = "1.0"
        xmlns:xsl = "http://www.w3.org/1999/XSL/Transform" >

          <xsl:template match="/a">
            <html><body><table>
             <xsl:apply-templates select="*"/>
            </table></body></html>
          </xsl:template>


          <xsl:template match="a/*">
            <xsl:if test="not(.='')">
              <TR>
                <xsl:if test="position()=1">
    <!--
                  <TD rowspan="6" valign="top">Address</TD>
                 improved based on Tomalak's suggestion
    -->
              <xsl:element name="TD">
                <xsl:attribute name="rowspan" >
    <!--
                  <cough />
                  <xsl:value-of select="count(*)"/>
    -->
                  <xsl:value-of select="count(*[not(.='')]"/>
                </xsl:attribute>
                <xsl:attribute name="valign" >
                  <xsl:text>top</xsl:text>
                </xsl:attribute>
                <xsl:text>Address</xsl:text>
              </xsl:element>

                </xsl:if>
                <TD>
                  <xsl:value-of select="."/>
                </TD>
              </TR>
            </xsl:if>
          </xsl:template>
        </xsl:stylesheet>

Simon Nunn 2009-06-16 12:52:25

Actually, that's a good solution. +1 -- The only check I'd remove is the "if not(.='')" - the rowspan will break if you do that.

Tomalak 2009-06-16 12:57:21

amend, cheers for bug spotting.

Simon Nunn 2009-06-16 13:48:57

ansaurus

tags:

views:

answers:

Show the header only once in a for-each

related questions