PMD Results
The following document contains the results of PMD 7.14.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 | 46–54 |
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 | 319–344 |
org/apache/maven/internal/impl/DefaultLifecycleRegistry.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–502 |
org/apache/maven/internal/impl/DefaultTypeRegistry.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 66–68 |
org/apache/maven/internal/transformation/impl/ConsumerPomArtifactTransformer.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateMethod | Avoid unused private methods such as 'doDeleteFiles()'. | 119 |
org/apache/maven/internal/transformation/impl/DefaultConsumerPomBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'dep2'. | 174 |
org/apache/maven/internal/transformation/impl/TransformedArtifact.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'SHA1_BUFFER_SIZE'. | 48 |
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'. | 307 |
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–1135 |
| UnusedLocalVariable | Avoid unused local variables such as 'phase'. | 336 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 381–399 |
| EmptyControlStatement | Empty if statement | 781–791 |
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/DefaultProjectBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'repoSystem'. | 127 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'scope'. | 512 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1627–1629 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1754–1759 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1773–1775 |
Priority 4
org/apache/maven/ReactorReader.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 317 |
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 | 152 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'CLEAN' is already in scope because it is inherited by an enclosing type | 365 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'DEFAULT' is already in scope because it is inherited by an enclosing type | 388 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'SITE' is already in scope because it is inherited by an enclosing type | 482 |
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. | 1054 |
org/apache/maven/lifecycle/internal/concurrent/ConcurrentLifecycleStarter.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 128–129 |
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 | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 51 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 59 |
org/apache/maven/plugin/prefix/internal/DefaultPluginPrefixResolver.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 101 |
org/apache/maven/project/DefaultProjectBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 375–377 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 1127 |
| UselessParentheses | Useless parentheses. | 1360 |
| UselessParentheses | Useless parentheses. | 1371 |
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 | 317 |
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 | 46–54 |
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 | 319–344 |
org/apache/maven/internal/impl/DefaultLifecycleRegistry.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–502 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.api': 'Lifecycle' is already in scope because it is imported in this file | 4 | 152 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'CLEAN' is already in scope because it is inherited by an enclosing type | 4 | 365 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'DEFAULT' is already in scope because it is inherited by an enclosing type | 4 | 388 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Lifecycle': 'SITE' is already in scope because it is inherited by an enclosing type | 4 | 482 |
org/apache/maven/internal/impl/DefaultTypeRegistry.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 66–68 |
org/apache/maven/internal/transformation/impl/ConsumerPomArtifactTransformer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateMethod | Avoid unused private methods such as 'doDeleteFiles()'. | 3 | 119 |
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/internal/transformation/impl/TransformedArtifact.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'SHA1_BUFFER_SIZE'. | 3 | 48 |
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 | 307 |
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–1135 |
| UnusedLocalVariable | Avoid unused local variables such as 'phase'. | 3 | 336 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 381–399 |
| EmptyControlStatement | Empty if statement | 3 | 781–791 |
| UselessParentheses | Useless parentheses. | 4 | 1054 |
org/apache/maven/lifecycle/internal/concurrent/ConcurrentLifecycleStarter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 128–129 |
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 | 45 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 4 | 51 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 4 | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 4 | 59 |
org/apache/maven/plugin/prefix/internal/DefaultPluginPrefixResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 101 |
org/apache/maven/project/DefaultProjectBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'repoSystem'. | 3 | 127 |
| UselessParentheses | Useless parentheses. | 4 | 375–377 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'scope'. | 3 | 512 |
| UselessParentheses | Useless parentheses. | 4 | 1127 |
| UselessParentheses | Useless parentheses. | 4 | 1360 |
| UselessParentheses | Useless parentheses. | 4 | 1371 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1627–1629 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1754–1759 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1773–1775 |
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 |