PMD Results
The following document contains the results of PMD 7.0.0.
Violations By Priority
Priority 3
org/apache/maven/artifact/resolver/filter/ExclusionArtifactFilter.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'exclusions'. | 37 |
org/apache/maven/bridge/MavenRepositorySystem.java
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address ${variableName} | 765 |
org/apache/maven/configuration/BasedirBeanConfigurationPathTranslator.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 45–53 |
org/apache/maven/execution/MavenSession.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'eventDispatcher'. | 376 |
org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 317–342 |
org/apache/maven/internal/impl/DefaultLifecycleRegistry.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–484 |
org/apache/maven/internal/impl/DefaultPluginXmlFactory.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'os'. | 96 |
org/apache/maven/internal/transformation/impl/DefaultConsumerPomArtifactTransformer.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateMethod | Avoid unused private methods such as 'doDeleteFiles()'. | 123 |
org/apache/maven/internal/transformation/impl/DefaultConsumerPomBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'project'. | 92 |
org/apache/maven/lifecycle/DefaultLifecycles.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'logger'. | 70 |
org/apache/maven/lifecycle/LifecycleExecutionException.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 49 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 54 |
org/apache/maven/lifecycle/internal/MojoExecutor.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'lock'. | 309 |
org/apache/maven/lifecycle/internal/builder/BuilderCommon.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 188–198 |
org/apache/maven/lifecycle/internal/concurrent/BuildPlanExecutor.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–942 |
| UnusedLocalVariable | Avoid unused local variables such as 'phase'. | 304 |
| EmptyControlStatement | Empty if statement | 586–596 |
org/apache/maven/plugin/CycleDetectedInPluginGraphException.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'e'. | 31 |
org/apache/maven/plugin/PluginConfigurationException.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'pluginDescriptor'. | 29 |
| UnusedPrivateField | Avoid unused private fields such as 'originalMessage'. | 31 |
org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
| Rule | Violation | Line |
|---|---|---|
| EmptyControlStatement | Empty if statement | 352–360 |
org/apache/maven/plugin/PluginParameterExpressionEvaluatorV4.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 182–189 |
| EmptyControlStatement | Empty if statement | 232–243 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'scope'. | 338 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1357–1359 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1377–1379 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1504–1509 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1523–1525 |
org/apache/maven/toolchain/java/DefaultJavaToolChain.java
| Rule | Violation | Line |
|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 41 |
| UselessOverridingMethod | Overriding method merely calls super | 46 |
Priority 4
org/apache/maven/ReactorReader.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 309 |
org/apache/maven/bridge/MavenRepositorySystem.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenRepositorySystem': 'DEFAULT_REMOTE_REPO_URL' is already in scope because it is declared in an enclosing type | 534 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenRepositorySystem': 'DEFAULT_REMOTE_REPO_ID' is already in scope because it is declared in an enclosing type | 535 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenRepositorySystem': 'DEFAULT_LOCAL_REPO_ID' is already in scope because it is declared in an enclosing type | 658 |
| UselessParentheses | Useless parentheses. | 804 |
org/apache/maven/exception/DefaultExceptionHandler.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 210 |
| UselessParentheses | Useless parentheses. | 245 |
org/apache/maven/graph/DefaultGraphBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 271 |
org/apache/maven/internal/impl/DefaultLifecycleRegistry.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.api': 'Lifecycle' is already in scope because it is imported in this file | 143 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'CLEAN' is already in scope because it is inherited by an enclosing type | 356 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'DEFAULT' is already in scope because it is inherited by an enclosing type | 377 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'SITE' is already in scope because it is inherited by an enclosing type | 466 |
org/apache/maven/internal/impl/DefaultToolchainManager.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.toolchain': 'ToolchainPrivate' is already in scope because it is imported in this file | 80 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.toolchain': 'ToolchainPrivate' is already in scope because it is imported in this file | 81 |
org/apache/maven/lifecycle/internal/DefaultLifecycleExecutionPlanCalculator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 317 |
org/apache/maven/lifecycle/internal/DefaultLifecycleTaskSegmentCalculator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 83–84 |
org/apache/maven/lifecycle/internal/concurrent/BuildPlanExecutor.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 861 |
org/apache/maven/lifecycle/internal/concurrent/ConcurrentLifecycleStarter.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 127–128 |
org/apache/maven/logging/LoggingExecutionListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 64 |
org/apache/maven/plugin/PluginParameterException.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 125 |
org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 307 |
| UselessParentheses | Useless parentheses. | 336–337 |
org/apache/maven/plugin/PluginParameterExpressionEvaluatorV4.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 216–217 |
org/apache/maven/plugin/internal/DefaultLegacySupport.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 52 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 57 |
org/apache/maven/plugin/prefix/internal/DefaultPluginPrefixResolver.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 100 |
org/apache/maven/project/DefaultProjectBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.eclipse.aether': 'RepositorySystem' is already in scope because it is imported in this file | 106 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 884 |
| UselessParentheses | Useless parentheses. | 1095 |
| UselessParentheses | Useless parentheses. | 1106 |
| UselessParentheses | Useless parentheses. | 1116 |
| UselessParentheses | Useless parentheses. | 1120 |
| UselessParentheses | Useless parentheses. | 1124 |
org/apache/maven/project/ProjectModelResolver.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.api.model': 'Parent' is already in scope because it is imported in this file | 347 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.api.model': 'Dependency' is already in scope because it is imported in this file | 357 |
org/apache/maven/project/ProjectSorter.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 244 |
org/apache/maven/project/collector/DefaultProjectsSelector.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 78 |
Files
org/apache/maven/ReactorReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 309 |
org/apache/maven/artifact/resolver/filter/ExclusionArtifactFilter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'exclusions'. | 3 | 37 |
org/apache/maven/bridge/MavenRepositorySystem.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenRepositorySystem': 'DEFAULT_REMOTE_REPO_URL' is already in scope because it is declared in an enclosing type | 4 | 534 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenRepositorySystem': 'DEFAULT_REMOTE_REPO_ID' is already in scope because it is declared in an enclosing type | 4 | 535 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenRepositorySystem': 'DEFAULT_LOCAL_REPO_ID' is already in scope because it is declared in an enclosing type | 4 | 658 |
| AvoidUsingHardCodedIP | Do not hard code the IP address ${variableName} | 3 | 765 |
| UselessParentheses | Useless parentheses. | 4 | 804 |
org/apache/maven/configuration/BasedirBeanConfigurationPathTranslator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 45–53 |
org/apache/maven/exception/DefaultExceptionHandler.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 210 |
| UselessParentheses | Useless parentheses. | 4 | 245 |
org/apache/maven/execution/MavenSession.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'eventDispatcher'. | 3 | 376 |
org/apache/maven/graph/DefaultGraphBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 271 |
org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 317–342 |
org/apache/maven/internal/impl/DefaultLifecycleRegistry.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–484 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.api': 'Lifecycle' is already in scope because it is imported in this file | 4 | 143 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'CLEAN' is already in scope because it is inherited by an enclosing type | 4 | 356 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'DEFAULT' is already in scope because it is inherited by an enclosing type | 4 | 377 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'SITE' is already in scope because it is inherited by an enclosing type | 4 | 466 |
org/apache/maven/internal/impl/DefaultPluginXmlFactory.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'os'. | 3 | 96 |
org/apache/maven/internal/impl/DefaultToolchainManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.toolchain': 'ToolchainPrivate' is already in scope because it is imported in this file | 4 | 80 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.toolchain': 'ToolchainPrivate' is already in scope because it is imported in this file | 4 | 81 |
org/apache/maven/internal/transformation/impl/DefaultConsumerPomArtifactTransformer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateMethod | Avoid unused private methods such as 'doDeleteFiles()'. | 3 | 123 |
org/apache/maven/internal/transformation/impl/DefaultConsumerPomBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'project'. | 3 | 92 |
org/apache/maven/lifecycle/DefaultLifecycles.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'logger'. | 3 | 70 |
org/apache/maven/lifecycle/LifecycleExecutionException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 3 | 49 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'execution'. | 3 | 54 |
org/apache/maven/lifecycle/internal/DefaultLifecycleExecutionPlanCalculator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 317 |
org/apache/maven/lifecycle/internal/DefaultLifecycleTaskSegmentCalculator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 83–84 |
org/apache/maven/lifecycle/internal/MojoExecutor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'lock'. | 3 | 309 |
org/apache/maven/lifecycle/internal/builder/BuilderCommon.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 188–198 |
org/apache/maven/lifecycle/internal/concurrent/BuildPlanExecutor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–942 |
| UnusedLocalVariable | Avoid unused local variables such as 'phase'. | 3 | 304 |
| EmptyControlStatement | Empty if statement | 3 | 586–596 |
| UselessParentheses | Useless parentheses. | 4 | 861 |
org/apache/maven/lifecycle/internal/concurrent/ConcurrentLifecycleStarter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 127–128 |
org/apache/maven/logging/LoggingExecutionListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 64 |
org/apache/maven/plugin/CycleDetectedInPluginGraphException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'e'. | 3 | 31 |
org/apache/maven/plugin/PluginConfigurationException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'pluginDescriptor'. | 3 | 29 |
| UnusedPrivateField | Avoid unused private fields such as 'originalMessage'. | 3 | 31 |
org/apache/maven/plugin/PluginParameterException.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 125 |
org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 307 |
| UselessParentheses | Useless parentheses. | 4 | 336–337 |
| EmptyControlStatement | Empty if statement | 3 | 352–360 |
org/apache/maven/plugin/PluginParameterExpressionEvaluatorV4.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 182–189 |
| UselessParentheses | Useless parentheses. | 4 | 216–217 |
| EmptyControlStatement | Empty if statement | 3 | 232–243 |
org/apache/maven/plugin/internal/DefaultLegacySupport.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 4 | 44 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 4 | 50 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 4 | 52 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 4 | 57 |
org/apache/maven/plugin/prefix/internal/DefaultPluginPrefixResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 100 |
org/apache/maven/project/DefaultProjectBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.eclipse.aether': 'RepositorySystem' is already in scope because it is imported in this file | 4 | 106 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'scope'. | 3 | 338 |
| UselessParentheses | Useless parentheses. | 4 | 884 |
| UselessParentheses | Useless parentheses. | 4 | 1095 |
| UselessParentheses | Useless parentheses. | 4 | 1106 |
| UselessParentheses | Useless parentheses. | 4 | 1116 |
| UselessParentheses | Useless parentheses. | 4 | 1120 |
| UselessParentheses | Useless parentheses. | 4 | 1124 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1357–1359 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1377–1379 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1504–1509 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1523–1525 |
org/apache/maven/project/ProjectModelResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.api.model': 'Parent' is already in scope because it is imported in this file | 4 | 347 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.api.model': 'Dependency' is already in scope because it is imported in this file | 4 | 357 |
org/apache/maven/project/ProjectSorter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 244 |
org/apache/maven/project/collector/DefaultProjectsSelector.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 78 |
org/apache/maven/toolchain/java/DefaultJavaToolChain.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 3 | 41 |
| UselessOverridingMethod | Overriding method merely calls super | 3 | 46 |