You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Erik (JIRA)" <ji...@codehaus.org> on 2011/09/13 09:49:07 UTC

[jira] Commented: (MJAVADOC-326) docletPath configuration parameter is concatenated incorrectly

    [ https://jira.codehaus.org/browse/MJAVADOC-326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=278615#comment-278615 ] 

Erik commented on MJAVADOC-326:
-------------------------------

{code}
        if ( !StringUtils.isEmpty( docletPath ) )
        {
            char tail = path.charAt(path.length()-1);
            if( tail != ':' || tail != ';')
                path.append(":");
            path.append( JavadocUtil.unifyPathSeparator( docletPath ) );
        }
{code}

> docletPath configuration parameter is concatenated incorrectly
> --------------------------------------------------------------
>
>                 Key: MJAVADOC-326
>                 URL: https://jira.codehaus.org/browse/MJAVADOC-326
>             Project: Maven 2.x Javadoc Plugin
>          Issue Type: Bug
>    Affects Versions: 2.8
>            Reporter: Erik
>
> No separator is added between path induced from docletArtifacts and docletPath. This makes the docletPath parameter useless.
> Here's a patch (svn diff):
> Index: src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java
> ===================================================================
> --- src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java	(revision 1170063)
> +++ src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java	(working copy)
> @@ -2728,6 +2728,9 @@
>  
>          if ( !StringUtils.isEmpty( docletPath ) )
>          {
> +            char tail = path.charAt(path.length()-1);
> +            if( tail != ':' || tail != ';')
> +                path.append(":");
>              path.append( JavadocUtil.unifyPathSeparator( docletPath ) );
>          }

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira