[ 
https://issues.apache.org/jira/browse/MJAVADOC-783?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17865782#comment-17865782
 ] 

ASF GitHub Bot commented on MJAVADOC-783:
-----------------------------------------

michael-o opened a new pull request, #300:
URL: https://github.com/apache/maven-javadoc-plugin/pull/300

   Co-authored-by: Rob Gordon <[email protected]>
   Co-authored-by: Michael Osipov <[email protected]>
   
   This closes #300
   
   Following this checklist to help us incorporate your
   contribution quickly and easily:
   
    - [ ] Make sure there is a [JIRA 
issue](https://issues.apache.org/jira/browse/MJAVADOC) filed
          for the change (usually before you start working on it).  Trivial 
changes like typos do not
          require a JIRA issue.  Your pull request should address just this 
issue, without
          pulling in other changes.
    - [ ] Each commit in the pull request should have a meaningful subject line 
and body.
    - [ ] Format the pull request title like `[MJAVADOC-XXX] - Fixes bug in 
ApproximateQuantiles`,
          where you replace `MJAVADOC-XXX` with the appropriate JIRA issue. 
Best practice
          is to use the JIRA issue title in the pull request title and in the 
first line of the
          commit message.
    - [ ] Write a pull request description that is detailed enough to 
understand what the pull request does, how, and why.
    - [ ] Run `mvn clean verify -Prun-its` to make sure basic checks pass. A 
more thorough check will
          be performed on your pull request automatically.
   
   If your pull request is about ~20 lines of code you don't need to sign an
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.pdf) if you are unsure
   please ask on the developers list.
   
   To make clear that you license your contribution under
   the [Apache License Version 2.0, January 
2004](http://www.apache.org/licenses/LICENSE-2.0)
   you have to acknowledge this by using the following check-box.
   
    - [ ] I hereby declare this contribution to be licensed under the [Apache 
License Version 2.0, January 2004](http://www.apache.org/licenses/LICENSE-2.0)
   
    - [ ] In any other case, please file an [Apache Individual Contributor 
License Agreement](https://www.apache.org/licenses/icla.pdf).
   
   




> Invalid path when using TagletArtifact and TagletPath
> -----------------------------------------------------
>
>                 Key: MJAVADOC-783
>                 URL: https://issues.apache.org/jira/browse/MJAVADOC-783
>             Project: Maven Javadoc Plugin
>          Issue Type: Bug
>    Affects Versions: 3.6.3
>            Reporter: Rob Gordon
>            Assignee: Michael Osipov
>            Priority: Major
>             Fix For: 3.7.1
>
>
> I’m trying to use the Javadoc plugin with both a tagletartifact and a 
> tagletpath but I get an InvliadPathException from the Javadoc goal.
> I can see from the generated options file that there is no separator between 
> the end of the artifact path and the taglet path.
> I think the problem is that getTagletPath method of AbstractJavadocMojo 
> should check for an existing path before adding the tagletpath. Something 
> like:
> {code:java}
>         if (tagletpath != null && !tagletpath.isEmpty()) {
>             if (path.length() > 0) {
>                  path.append(File.pathSeparator);
>            }
>             path.append(JavadocUtil.unifyPathSeparator(tagletpath));
>         }
> {code}
> I looked at creating a pull request but I can’t work out how to create a 
> suitable test. I found a taglet-test-plugin-config.xml, but not where it’s 
> used.
> Could someone familiar with this codebase possibly take a look at this 
> problem please?
> My configuration can be seen here: 
> [https://github.com/robjg/oj-parent/blob/d3e2328ca285516f1a4f6ee2bce94b44ba89b748/pom.xml#L304]
> I attempted to report this problem via '[email protected]' however my 
> mail did not make it. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to