Maven 1.x Javadoc Plugin
  1. Maven 1.x Javadoc Plugin
  2. MPJAVADOC-6

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
    • Number of attachments :
      1

      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

        Hide
        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
        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
        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
          • Votes:
            2 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: