Maven Javadoc Plugin
  1. Maven Javadoc Plugin
  2. MJAVADOC-230

aggregate report requires to have aggregate=true configuration parameter

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.6
    • Labels:
      None
    • Environment:
      Maven 2.0.9
      Windows XP SP2
    • Number of attachments :
      1

      Description

      Project setup: I have a multi-module project (standard nested layout). The reports are configured in the parent project. I use the "aggregate" report to create an aggregated javadoc by the maven-javadoc-plugin:
      <reportSets>
      <reportSet>
      <reports>
      <report>aggregate</report>
      <reports>
      </reportSet>
      </reportSets>

      Problem: The aggregate report is ignored, when I launch the "mvn site" command, unless the aggregated parameter is set to true in the maven-javadoc-plugin configuration. So the following setup is required in pom.xml:
      <reportSets>
      <reportSet>
      <configuration>
      <encoding>UTF-8</encoding>
      <aggregate>true</aggregate>
      </configuration>
      <reports>
      <report>aggregate</report>
      <reports>
      </reportSet>
      </reportSets>

      Desired functionality: Launch the aggregate goal, if it is declared in the report section. Do not require the aggregate parameter=true setting (it is a duplication and also the aggregate parameter is deprecated). The following setup should be enough in pom.xml:
      <reportSets>
      <reportSet>
      <configuration>
      <encoding>UTF-8</encoding>
      </configuration>
      <reports>
      <report>aggregate</report>
      <reports>
      </reportSet>
      </reportSets>

      Comment:
      The aggregate goal works fine (it does not require to have the aggregate=true parameter), if it is launched directly. That is by the "mvn javadoc:aggregate" command.
      It just does not work, when it is launched indirectly through the site plugin as a report.

        Issue Links

          Activity

          Hide
          Jeff Maxwell added a comment -

          I have created a patch
          In org.apache.maven.plugin.javadoc.JavadocReport.canGenerateReport() isAggregator() is tested against the aggregate instance variable.

          This is incorrect.

          I believe the correct logic is as follows:

          isAggregator hasSourceFiles isRootProject Generate Report
          True True True True
          True True False False
          True False True False
          True False False False
          False True True True
          False True False True
          False False True False
          False False False False
          Show
          Jeff Maxwell added a comment - I have created a patch In org.apache.maven.plugin.javadoc.JavadocReport.canGenerateReport() isAggregator() is tested against the aggregate instance variable. This is incorrect. I believe the correct logic is as follows: isAggregator hasSourceFiles isRootProject Generate Report True True True True True True False False True False True False True False False False False True True True False True False True False False True False False False False False
          Hide
          Vincent Siveton added a comment -

          Patch applied in r794114

          Show
          Vincent Siveton added a comment - Patch applied in r794114

            People

            • Assignee:
              Vincent Siveton
              Reporter:
              Gabriel Forro
            • Votes:
              2 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: