<xsl:variable name="tmpSite"><xsl:value-of
select="site"/></xsl:variable>
it's more efficient to go
<xsl:variable name="tmpSite" select="site"/>
as otherwise you are making a result tree fragment, which is relatively
expensive.
I think you just want
<td><xsl:value-of select="site"/><br>
<img>
<xsl:attribute name="src">
<xsl:choose>
<xsl:when test="severity = 'MINOR'">Image2</xsl:when>
<xsl:when test="severity = 'CRITICAL'">Image3</xsl:when>
<xsl:otherwise>Image1</xsl:otherwise>
</xsl:choose>
</xsl:attribute>
</img>
David
________________________________________________________________________
This e-mail has been scanned for all viruses by Star Internet. The
service is powered by MessageLabs. For more information on a proactive
anti-virus service working around the clock, around the globe, visit:
http://www.star.net.uk
________________________________________________________________________
XSL-List info and archive: http://www.mulberrytech.com/xsl/xsl-list