[ 
http://jira.codehaus.org/browse/MOJO-498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_94720
 ] 

Nicholas Daley commented on MOJO-498:
-------------------------------------

This issue should be re-opened.  It still happens with version 1.0.0 of the 
plugin, which uses FindBugs 1.1.1.

It is not an issue with the version of findbugs being used.  As said in the 
original description, it is a problem in the assembleJXRHyperlink method.

> findbugs maven plugin: Link to JXR(=xref) incorrect for inner classes
> ---------------------------------------------------------------------
>
>                 Key: MOJO-498
>                 URL: http://jira.codehaus.org/browse/MOJO-498
>             Project: Mojo
>          Issue Type: Bug
>          Components: findbugs
>         Environment: Bug found on WinXP with maven 2.0.4 + maven find bugs 
> plugin 1.0-beta-1
>            Reporter: Olivier Vierlinck
>            Priority: Minor
>
> If a java file has an inner class which contain a 
> FindBugs error, the link generated from the FindFugs 
> report to the XRef (=JXR) source doc is invalid: the 
> url built by the findbugs Reporter is based on the 
> full class name of the inner class (for example 
> MyComponent$2 or MyComponent$MyInner) where the html 
> page is named with the outer class only (MyComponent 
> in this case, no '$' part)
> I think the line 
> final String path = prefix + Reporter.URL_SEPARATOR + 
> this.mCurrentClassName.replaceAll( "[.]", "/" ); 
> from assembleJXRHyperlink() method from 
> org.codehaus.mojo.findbugs.Reporter should be 
> modified with something like
> final String path = prefix + Reporter.URL_SEPARATOR + 
> this.mCurrentClassName.replaceAll
> ( "[.]", "/" ).replaceAll("\$.*",""); 
> (not tested!)

-- 
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

        

---------------------------------------------------------------------
To unsubscribe from this list please visit:

    http://xircles.codehaus.org/manage_email

Reply via email to