ansaurus

Question

Answer 1

+7 A:

You only have one languages node, at the root.

You should be using templates instead of for-each whenever possible:

<xsl:template match="language">
    <th>
       <xsl:value-of select="."/>
    </th>
</xsl:template>

And in the main template call it:

<tr>
    <xsl:apply-templates select="languages/language"/>
</tr>

Oded 2010-09-09 15:20:40

@Oded: +1 for apply templates suggestion.

Alejandro 2010-09-09 15:24:42

Very nice way of doing it, thanks.

wonea 2010-09-09 15:39:17

Answer 2

+2 A:

@Oded has probably posted the best answer atm, try to go with that. But in your specific case, you've written

<xsl:for-each select="languages">

of which there is only one, so rewrite that as

<xsl:for-each select="languages/language">

and

<xsl:value-of select="language">

as

<xsl:value-of select=".">

or

<xsl:apply-templates/>

and this particular stylesheet should work for you.

So again: if you can, do what Oded suggests. Sometimes though, you will have to use a for-each-loop - if so, then hopefully my answer helps you understand what you did wrong in this case.

Rob 2010-09-09 15:26:19

ansaurus

tags:

views:

answers:

xsl foreach loop not working on xml

related questions