[ http://jira.codehaus.org/browse/MJXR-5?page=all ]
     
Brett Porter closed MJXR-5:
---------------------------

      Assign To: Brett Porter
     Resolution: Won't Fix
    Fix Version:     (was: 2.0)

> Relative link to Xref not determined reliably
> ---------------------------------------------
>
>          Key: MJXR-5
>          URL: http://jira.codehaus.org/browse/MJXR-5
>      Project: Maven 2.x JXR Plugin
>         Type: Bug

>     Reporter: Nick Giles
>     Assignee: Brett Porter
>  Attachments: MOJO-jxr.patch
>
>
> Using the HEAD revision of the JXR plugin, I find that with certain of my 
> projects determining the relative link between the xref and the jxr.html page 
> can both crash and provide incorrect results. The problem is basically this 
> line:
>                 String relativPath = des.getAbsolutePath().substring( 
> out.getAbsolutePath().length() + 1 );
> It does no content checking, and thus when for instance the output directory 
> contains some '..' elements, it doesn't consider how that affects the length. 
> A slight improvement is:
>                 String relativPath = des.getCanonicalPath().substring( 
> out.getCanonicalPath().length() + 1 );
> which will at least be better so long as the locations point to similar 
> filesystem locations (see attached patch). 
> What is really needed is to do a filesystem based compare of the two 
> locations and determine what the link should be - or just add a parameter to 
> let the link be defined as a parameter, and reduce the cases that are 
> guaranteed to work automatically.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to