xsl must be simple 1

[jrenae]

Hello,

I'm very new to using xsl stylesheets so please bear with me on this, probably, very simple problem.

on the Models line I have 3 models a 328, 335, and 525.
All I'm trying to do is put a comma between the models.

What am I doing wrong?

Thanks in advance.


here is my output:
Make: BMW
Models: 328335525
Year:


here is my xml node:

    <AutoFilter>
      <Make>BMW</Make>
      <Trim />
      <Year />
      <Models>
        <Model>328</Model>
        <Model>335</Model>
        <Model>525</Model>
      </Models>
    </AutoFilter>

here is my xsl stylesheet snippet

      <xsl:for-each select="CampaignFilter/AutoFilters/AutoFilter">
        <tr>
          <td>
            Make:
            <xsl:value-of select="Make" />
          </td>
          <td>
            <img src="../App_Themes/LMP/Images/icon_check.gif"/>
          </td>
        </tr>
        <tr>
          <td>
            Models:
            <xsl:value-of select="Models"/> 
            <xsl:for-each select="Model">
            ,
            </xsl:for-each>
          </td>
          <td>
            <img src="../App_Themes/LMP/Images/icon_check.gif"/>
          </td>
        </tr>
        <tr>
          <td>
            Year:
            <xsl:value-of select="Year" />
          </td>
          <td>
            <img src="../App_Themes/LMP/Images/icon_check.gif"/>
          </td>
        </tr>
      </xsl:for-each>

 

[k5tm]

            Models:
            <xsl:for-each select="Model">
            <xsl:value-of select="."/>
            <xsl:if test="position() != last()">,</xsl:if>
            </xsl:for-each>

[small]Warning: typed, not tested[/small]

Tom Morrison

http://www.liant.com

 

[jrenae]

Thanks, that helped but what actually worked was this:

Models:
<xsl:for-each select="Models/Model">
<xsl:value-of select="."/>
<xsl:if test="position() != last()">, </xsl:if>
</xsl:for-each>

notice that I had to include the parent node "Models" in the foreach loop.

Thanks!