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:
- https://github.com/apache/maven-jar-plugin/blob/0ad60f4f2862bf896b6c0dc57ea5a6999c49b0d4/src/main/java/org/apache/maven/plugins/jar/AbstractJarMojo.java#L107-L115
- https://github.com/apache/maven-deploy-plugin/blob/36a2030c8851e21cd1e0bec01c3cea4dc6055a48/src/main/java/org/apache/maven/plugins/deploy/DeployFileMojo.java#L180-L188
- and few others
which is to have @Deprecated parameter, and its presence is used for breaking the build.
Attachments
Issue Links
- is related to
-
MNG-7468 Unsupported plugins parameters in configuration should be verified
- Closed
- links to