PMD Results
The following document contains the results of PMD 6.49.0.
Violations By Priority
Priority 3
org/apache/maven/DefaultMaven.java
| Rule | Violation | Line |
|---|---|---|
| ReturnFromFinallyBlock | Avoid returning from a finally block | 273 |
| UnusedFormalParameter | Avoid unused method parameters such as 'result'. | 452 |
org/apache/maven/ReactorReader.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'session'. | 64 |
| UnusedPrivateMethod | Avoid unused private methods such as 'determinePreviouslyPackagedArtifactFile(MavenProject,Artifact)'. | 159 |
org/apache/maven/bridge/MavenRepositorySystem.java
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address | 750 |
org/apache/maven/configuration/BasedirBeanConfigurationPathTranslator.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 46–54 |
org/apache/maven/execution/MavenSession.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'eventDispatcher'. | 319 |
org/apache/maven/execution/SettingsAdapter.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'runtimeInfo'. | 45 |
org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 328–341 |
org/apache/maven/lifecycle/LifecycleExecutionException.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 50 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 55 |
org/apache/maven/lifecycle/internal/DefaultLifecycleExecutionPlanCalculator.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'pluginVersionResolver'. | 71 |
org/apache/maven/lifecycle/internal/LifecycleModuleBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'projectExecutionListeners'. | 64 |
org/apache/maven/lifecycle/internal/LifecycleStarter.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'sessionScope'. | 68 |
org/apache/maven/lifecycle/internal/builder/BuilderCommon.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 152–162 |
org/apache/maven/lifecycle/internal/builder/multithreaded/MultiThreadedBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'muxer'. | 178 |
org/apache/maven/plugin/CycleDetectedInPluginGraphException.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'e'. | 32 |
org/apache/maven/plugin/DefaultBuildPluginManager.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'mojoExecutionListeners'. | 61 |
org/apache/maven/plugin/PluginConfigurationException.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'pluginDescriptor'. | 30 |
| UnusedPrivateField | Avoid unused private fields such as 'originalMessage'. | 32 |
org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'pathTranslator'. | 90 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'logger'. | 91 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'project'. | 92 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'properties'. | 93 |
| EmptyControlStatement | Empty if statement | 365–373 |
org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 109–115 |
org/apache/maven/project/DefaultProjectBuildingHelper.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'container'. | 69 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 326–330 |
| CollapsibleIfStatements | These nested if statements could be combined | 1325–1327 |
| CollapsibleIfStatements | These nested if statements could be combined | 1345–1349 |
| CollapsibleIfStatements | These nested if statements could be combined | 1478–1480 |
| CollapsibleIfStatements | These nested if statements could be combined | 1494–1496 |
org/apache/maven/toolchain/building/DefaultToolchainsBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'request'. | 144 |
org/apache/maven/toolchain/java/DefaultJavaToolChain.java
| Rule | Violation | Line |
|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 40–42 |
| UselessOverridingMethod | Overriding method merely calls super | 45–47 |
Priority 4
org/apache/maven/ReactorReader.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 223 |
| UselessParentheses | Useless parentheses. | 224 |
org/apache/maven/RepositoryUtils.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 64 |
org/apache/maven/artifact/metadata/AbstractArtifactMetadata.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.maven.artifact.metadata.ArtifactMetadata' due to existing same package import 'org.apache.maven.artifact.metadata.*' | 29 |
org/apache/maven/artifact/repository/LegacyLocalRepositoryManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 101 |
| UselessParentheses | Useless parentheses. | 198 |
org/apache/maven/execution/SettingsAdapter.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 56 |
org/apache/maven/lifecycle/internal/DependencyContext.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 88 |
org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 158 |
| UselessParentheses | Useless parentheses. | 158 |
| UselessParentheses | Useless parentheses. | 320 |
| UselessParentheses | Useless parentheses. | 320 |
| UselessParentheses | Useless parentheses. | 320 |
| UselessParentheses | Useless parentheses. | 320 |
| UselessParentheses | Useless parentheses. | 349–350 |
| UselessParentheses | Useless parentheses. | 354 |
org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 496 |
| UselessParentheses | Useless parentheses. | 496 |
org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.aether.artifact.Artifact' due to existing import 'org.eclipse.aether.artifact.Artifact' | 232 |
org/apache/maven/project/DefaultProjectBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 770 |
| UselessParentheses | Useless parentheses. | 771 |
| UselessParentheses | Useless parentheses. | 771 |
| UselessParentheses | Useless parentheses. | 772 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 402 |
| UselessParentheses | Useless parentheses. | 402 |
| UselessParentheses | Useless parentheses. | 437 |
| UselessParentheses | Useless parentheses. | 437 |
| UselessParentheses | Useless parentheses. | 843 |
| UselessParentheses | Useless parentheses. | 920 |
| UselessParentheses | Useless parentheses. | 920 |
| UselessParentheses | Useless parentheses. | 1057 |
| UselessParentheses | Useless parentheses. | 1068 |
| UselessParentheses | Useless parentheses. | 1078 |
| UselessParentheses | Useless parentheses. | 1082 |
| UselessParentheses | Useless parentheses. | 1086 |
| UselessParentheses | Useless parentheses. | 1359 |
| UselessParentheses | Useless parentheses. | 1402 |
| UselessParentheses | Useless parentheses. | 1427 |
| UselessParentheses | Useless parentheses. | 1506 |
org/apache/maven/project/ProjectSorter.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 267 |
org/apache/maven/project/artifact/MavenMetadataSource.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 438 |
| UselessParentheses | Useless parentheses. | 438 |
org/apache/maven/repository/ArtifactTransferEvent.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 279 |
| UselessParentheses | Useless parentheses. | 279 |
Files
org/apache/maven/DefaultMaven.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ReturnFromFinallyBlock | Avoid returning from a finally block | 3 | 273 |
| UnusedFormalParameter | Avoid unused method parameters such as 'result'. | 3 | 452 |
org/apache/maven/ReactorReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'session'. | 3 | 64 |
| UnusedPrivateMethod | Avoid unused private methods such as 'determinePreviouslyPackagedArtifactFile(MavenProject,Artifact)'. | 3 | 159 |
| UselessParentheses | Useless parentheses. | 4 | 223 |
| UselessParentheses | Useless parentheses. | 4 | 224 |
org/apache/maven/RepositoryUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 64 |
org/apache/maven/artifact/metadata/AbstractArtifactMetadata.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.maven.artifact.metadata.ArtifactMetadata' due to existing same package import 'org.apache.maven.artifact.metadata.*' | 4 | 29 |
org/apache/maven/artifact/repository/LegacyLocalRepositoryManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 101 |
| UselessParentheses | Useless parentheses. | 4 | 198 |
org/apache/maven/bridge/MavenRepositorySystem.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address | 3 | 750 |
org/apache/maven/configuration/BasedirBeanConfigurationPathTranslator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 46–54 |
org/apache/maven/execution/MavenSession.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'eventDispatcher'. | 3 | 319 |
org/apache/maven/execution/SettingsAdapter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'runtimeInfo'. | 3 | 45 |
| UselessParentheses | Useless parentheses. | 4 | 56 |
org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 328–341 |
org/apache/maven/lifecycle/LifecycleExecutionException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 3 | 50 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 3 | 55 |
org/apache/maven/lifecycle/internal/DefaultLifecycleExecutionPlanCalculator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'pluginVersionResolver'. | 3 | 71 |
org/apache/maven/lifecycle/internal/DependencyContext.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 88 |
org/apache/maven/lifecycle/internal/LifecycleModuleBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'projectExecutionListeners'. | 3 | 64 |
org/apache/maven/lifecycle/internal/LifecycleStarter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'sessionScope'. | 3 | 68 |
org/apache/maven/lifecycle/internal/builder/BuilderCommon.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 152–162 |
org/apache/maven/lifecycle/internal/builder/multithreaded/MultiThreadedBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'muxer'. | 3 | 178 |
org/apache/maven/plugin/CycleDetectedInPluginGraphException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'e'. | 3 | 32 |
org/apache/maven/plugin/DefaultBuildPluginManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'mojoExecutionListeners'. | 3 | 61 |
org/apache/maven/plugin/PluginConfigurationException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'pluginDescriptor'. | 3 | 30 |
| UnusedPrivateField | Avoid unused private fields such as 'originalMessage'. | 3 | 32 |
org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'pathTranslator'. | 3 | 90 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'logger'. | 3 | 91 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'project'. | 3 | 92 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'properties'. | 3 | 93 |
| UselessParentheses | Useless parentheses. | 4 | 158 |
| UselessParentheses | Useless parentheses. | 4 | 158 |
| UselessParentheses | Useless parentheses. | 4 | 320 |
| UselessParentheses | Useless parentheses. | 4 | 320 |
| UselessParentheses | Useless parentheses. | 4 | 320 |
| UselessParentheses | Useless parentheses. | 4 | 320 |
| UselessParentheses | Useless parentheses. | 4 | 349–350 |
| UselessParentheses | Useless parentheses. | 4 | 354 |
| EmptyControlStatement | Empty if statement | 3 | 365–373 |
org/apache/maven/plugin/internal/DefaultMavenPluginManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 496 |
| UselessParentheses | Useless parentheses. | 4 | 496 |
org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 109–115 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.eclipse.aether.artifact.Artifact' due to existing import 'org.eclipse.aether.artifact.Artifact' | 4 | 232 |
org/apache/maven/project/DefaultProjectBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 770 |
| UselessParentheses | Useless parentheses. | 4 | 771 |
| UselessParentheses | Useless parentheses. | 4 | 771 |
| UselessParentheses | Useless parentheses. | 4 | 772 |
org/apache/maven/project/DefaultProjectBuildingHelper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'container'. | 3 | 69 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 326–330 |
| UselessParentheses | Useless parentheses. | 4 | 402 |
| UselessParentheses | Useless parentheses. | 4 | 402 |
| UselessParentheses | Useless parentheses. | 4 | 437 |
| UselessParentheses | Useless parentheses. | 4 | 437 |
| UselessParentheses | Useless parentheses. | 4 | 843 |
| UselessParentheses | Useless parentheses. | 4 | 920 |
| UselessParentheses | Useless parentheses. | 4 | 920 |
| UselessParentheses | Useless parentheses. | 4 | 1057 |
| UselessParentheses | Useless parentheses. | 4 | 1068 |
| UselessParentheses | Useless parentheses. | 4 | 1078 |
| UselessParentheses | Useless parentheses. | 4 | 1082 |
| UselessParentheses | Useless parentheses. | 4 | 1086 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1325–1327 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1345–1349 |
| UselessParentheses | Useless parentheses. | 4 | 1359 |
| UselessParentheses | Useless parentheses. | 4 | 1402 |
| UselessParentheses | Useless parentheses. | 4 | 1427 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1478–1480 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 1494–1496 |
| UselessParentheses | Useless parentheses. | 4 | 1506 |
org/apache/maven/project/ProjectSorter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 267 |
org/apache/maven/project/artifact/MavenMetadataSource.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 438 |
| UselessParentheses | Useless parentheses. | 4 | 438 |
org/apache/maven/repository/ArtifactTransferEvent.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 279 |
| UselessParentheses | Useless parentheses. | 4 | 279 |
org/apache/maven/toolchain/building/DefaultToolchainsBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'request'. | 3 | 144 |
org/apache/maven/toolchain/java/DefaultJavaToolChain.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 3 | 40–42 |
| UselessOverridingMethod | Overriding method merely calls super | 3 | 45–47 |
Suppressed Violations
| Filename | Rule message | Suppression type | Reason |
|---|---|---|---|
| org/apache/maven/lifecycle/mapping/Lifecycle.java | Avoid unused private fields such as 'optionalMojos'. | annotation |