PMD Results
The following document contains the results of PMD 7.0.0.
Violations By Priority
Priority 3
org/apache/maven/artifact/handler/manager/DefaultArtifactHandlerManager.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 57–59 |
org/apache/maven/artifact/handler/manager/LegacyArtifactHandlerManager.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 52–54 |
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} | 788 |
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–490 |
org/apache/maven/internal/impl/DefaultPluginXmlFactory.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'os'. | 96 |
org/apache/maven/internal/impl/DefaultTypeRegistry.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 71–73 |
org/apache/maven/internal/transformation/impl/DefaultConsumerPomArtifactTransformer.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateMethod | Avoid unused private methods such as 'doDeleteFiles()'. | 121 |
org/apache/maven/internal/transformation/impl/DefaultConsumerPomBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'dep2'. | 174 |
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 | 350–358 |
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 | 230–241 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'scope'. | 465 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1574–1576 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1701–1706 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1720–1722 |
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 | 557 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenRepositorySystem': 'DEFAULT_REMOTE_REPO_ID' is already in scope because it is declared in an enclosing type | 558 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenRepositorySystem': 'DEFAULT_LOCAL_REPO_ID' is already in scope because it is declared in an enclosing type | 681 |
| UselessParentheses | Useless parentheses. | 827 |
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. | 274 |
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 | 379 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'SITE' is already in scope because it is inherited by an enclosing type | 470 |
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. | 334–335 |
org/apache/maven/plugin/PluginParameterExpressionEvaluatorV4.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 214–215 |
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 | 117 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 1074 |
| UselessParentheses | Useless parentheses. | 1307 |
| UselessParentheses | Useless parentheses. | 1318 |
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. | 83 |
Files
org/apache/maven/ReactorReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 309 |
org/apache/maven/artifact/handler/manager/DefaultArtifactHandlerManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 57–59 |
org/apache/maven/artifact/handler/manager/LegacyArtifactHandlerManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 52–54 |
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 | 557 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenRepositorySystem': 'DEFAULT_REMOTE_REPO_ID' is already in scope because it is declared in an enclosing type | 4 | 558 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenRepositorySystem': 'DEFAULT_LOCAL_REPO_ID' is already in scope because it is declared in an enclosing type | 4 | 681 |
| AvoidUsingHardCodedIP | Do not hard code the IP address ${variableName} | 3 | 788 |
| UselessParentheses | Useless parentheses. | 4 | 827 |
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 | 274 |
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–490 |
| 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 | 379 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'SITE' is already in scope because it is inherited by an enclosing type | 4 | 470 |
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/DefaultTypeRegistry.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 71–73 |
org/apache/maven/internal/transformation/impl/DefaultConsumerPomArtifactTransformer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateMethod | Avoid unused private methods such as 'doDeleteFiles()'. | 3 | 121 |
org/apache/maven/internal/transformation/impl/DefaultConsumerPomBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'dep2'. | 3 | 174 |
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 | 334–335 |
| EmptyControlStatement | Empty if statement | 3 | 350–358 |
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 | 214–215 |
| EmptyControlStatement | Empty if statement | 3 | 230–241 |
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 | 117 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'scope'. | 3 | 465 |
| UselessParentheses | Useless parentheses. | 4 | 1074 |
| UselessParentheses | Useless parentheses. | 4 | 1307 |
| UselessParentheses | Useless parentheses. | 4 | 1318 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1574–1576 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1701–1706 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1720–1722 |
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 | 83 |