javadoc-plugin: non default doclet will not exec

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: None
  • Labels:
    None
  • Environment:
    Linux

Description

antProperty doclet ist set from maven.javadoc.docletpath

plugin.jelly reads:

<!-- allow custom doclets -->
<j:if test="${context.getVariable('maven.dotuml.doclet') != null}">
<ant:setProperty name="doclet" value="${maven.dotuml.doclet}" />
</j:if>
<j:if test="${context.getVariable('maven.dotuml.docletpath') != null}">
<ant:setProperty name="doclet" value="${maven.dotuml.docletpath}" />
</j:if>

must read:
<!-- allow custom doclets -->
<j:if test="${context.getVariable('maven.dotuml.doclet') != null}">
<ant:setProperty name="doclet" value="${maven.dotuml.doclet}" />
</j:if>
<j:if test="${context.getVariable('maven.dotuml.docletpath') != null}">
<ant:setProperty name="docletpath" value="${maven.dotuml.docletpath}" />
</j:if>

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Jason Horne added a comment -

The code has changed slightly since this was originally reported, but the bug remains.

This:
<!-- allow custom doclets -->
<j:if test="${context.getVariable('maven.javadoc.doclet') != null}">
<ant:setProperty name="doclet" value="${maven.javadoc.doclet}" />
</j:if>
<j:if test="${context.getVariable('maven.javadoc.docletpath') != null}">
<ant:setProperty name="doclet" value="${maven.javadoc.docletpath}" />
</j:if>

...needs to be changed to this:

<!-- allow custom doclets -->
<j:if test="${context.getVariable('maven.javadoc.doclet') != null}">
<ant:setProperty name="doclet" value="${maven.javadoc.doclet}" />
</j:if>
<j:if test="${context.getVariable('maven.javadoc.docletpath') != null}">
<ant:setProperty name="docletpath" value="${maven.javadoc.docletpath}" />
</j:if>

Show
Jason Horne added a comment - The code has changed slightly since this was originally reported, but the bug remains. This: <!-- allow custom doclets --> <j:if test="${context.getVariable('maven.javadoc.doclet') != null}"> <ant:setProperty name="doclet" value="${maven.javadoc.doclet}" /> </j:if> <j:if test="${context.getVariable('maven.javadoc.docletpath') != null}"> <ant:setProperty name="doclet" value="${maven.javadoc.docletpath}" /> </j:if> ...needs to be changed to this: <!-- allow custom doclets --> <j:if test="${context.getVariable('maven.javadoc.doclet') != null}"> <ant:setProperty name="doclet" value="${maven.javadoc.doclet}" /> </j:if> <j:if test="${context.getVariable('maven.javadoc.docletpath') != null}"> <ant:setProperty name="docletpath" value="${maven.javadoc.docletpath}" /> </j:if>
Hide
Permalink
Jason Horne added a comment -

Attached a patch file which performs this fix

Show
Jason Horne added a comment - Attached a patch file which performs this fix
Hide
Permalink
dion gillard added a comment -

Patch applied today.Fixed in v1.4 of the plugin

Show
dion gillard added a comment - Patch applied today.Fixed in v1.4 of the plugin

People

  • Assignee:
    dion gillard
    Reporter:
    Anonymous
Vote (2)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: