Uploaded image for project: 'Maven Javadoc Plugin'
  1. Maven Javadoc Plugin
  2. MJAVADOC-710

Usage of removed or replaced property does not cause the build to fail

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Won't Fix
    • 3.3.2
    • None
    • None
    • None

    Description

      Per https://issues.apache.org/jira/browse/MJAVADOC-509?focusedCommentId=16581261

      Yes, additionalParam was replaced by additionalOptions in MJAVADOC-475(https://issues.apache.org/jira/browse/MJAVADOC-475) and the build will fail with 3.0.0 if this property is used.

      However current implementation does not do that. And as far as I've checked - the original implementation didn't do in 3.0.0 neither.
      This is due to mojo.getConfiguration() in AbstractJavadocMojo.verifyRemovedParameter having only parameters plugin committed for, and not all from <configuration> element. Not at the moment of goal execution, at least.

      This causes problems when upgrading from pre-3 plugin as additionalparam is silently ignored.

      I see similar case is dealt in following way:

      which is to have @Deprecated parameter, and its presence is used for breaking the build.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              pzygielo Piotr Zygielo
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: