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-02 14:01

Message:
Logged In: YES 
user_id=1634579

To use repro:-

1. Extract entire zip into a directory, preserving the directory structure.
2. run NAnt from that directory (no need to specify any target).
3. /Out/html will contain the NUnit2Report

Have included the Out directory from when I was testing so you can see my 
results - the Out directory is deleted 
and re-created each time the NAnt script is run.

FYI this is run on Windows Server 2003 SP1, with NUnit 2.2.8, NAnt 0.85, 
NAntContrib 0.85 and the VB.NET project is 
in VS2003 format.

----------------------------------------------------------------------

Comment By: Gert Driesen (drieseng)
Date: 2006-11-01 10:57

Message:
Logged In: YES 
user_id=707851

The minimum set of files necessary to reproduce this issue. 

In this case:
- a build file in which you compile the unit test assembly, run the unit tests, 
and format the results using 
<nunit2report>.
- the sources for the unit test assembly.


----------------------------------------------------------------------

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

Reply via email to