Bugs item #1587993, was opened at 2006-10-31 16:08
Message generated for change (Comment added) made by paulc1
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=474851&aid=1587993&group_id=54790
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Tasks
Group: 0.85
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Paul (paulc1)
Assigned to: Nobody/Anonymous (nobody)
Summary: bad stylesheet link in nunit2report
Initial Comment:
The link to the stylesheet in the pages for the
individual fixtures is wrong. It contains far too
many '../'. (For example on a class with 5 parts to
the fully qualified name (i.e. x.x.x.x.x) the link is
'../../../../../../../../../../../../../../../styleshe
et.css'
I think this is down to the logic in the
<xsl:template name="path"> template.
I think it should be like so
<xsl:template name="path">
<xsl:param name="path"/>
<xsl:choose>
<xsl:when test="contains($path,'.')">
<xsl:text>../</xsl:text>
<xsl:call-template name="path">
<xsl:with-param
name="path"><xsl:value-of select="substring-after
($path,'.')"/></xsl:with-param>
</xsl:call-template>
</xsl:when>
<xsl:otherwise>
<xsl:if test="not(contains($path,'.'))
and not($path = '')">
<xsl:text>../</xsl:text>
</xsl:if>
</xsl:otherwise>
</xsl:choose>
</xsl:template>
The current implementation is
<xsl:template name="path">
<xsl:param name="path"/>
<xsl:if test="contains($path,'.')">
<xsl:text>../</xsl:text>
<xsl:call-template name="path">
<xsl:with-param
name="path"><xsl:value-of select="substring-after
($path,'.')"/></xsl:with-param>
</xsl:call-template>
</xsl:if>
<xsl:if test="not(contains($path,'.')) and not
($path = '')">
<xsl:text>../</xsl:text>
</xsl:if>
</xsl:template>
for which the lower if condition will be entered
after the recursive call unecessarily, which results
in the extra '../' being added.
----------------------------------------------------------------------
>Comment By: Paul (paulc1)
Date: 2006-11-01 08:16
Message:
Logged In: YES
user_id=1634579
What do you mean by a repro? Do you want the project files and the NAnt code?
----------------------------------------------------------------------
Comment By: Gert Driesen (drieseng)
Date: 2006-10-31 16:21
Message:
Logged In: YES
user_id=707851
Please provide a small repro for this issue.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=474851&aid=1587993&group_id=54790
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
NAntContrib-Developer mailing list
NAntContrib-Developer@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nantcontrib-developer
