PMD Results
The following document contains the results of PMD 7.7.0.
Violations By Priority
Priority 3
org/apache/maven/DefaultMaven.java
| Rule | Violation | Line |
|---|---|---|
| ReturnFromFinallyBlock | Avoid returning from a finally block | 274 |
| UnusedFormalParameter | Avoid unused method parameters such as 'result'. | 447 |
org/apache/maven/ReactorReader.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateMethod | Avoid unused private methods such as 'determinePreviouslyPackagedArtifactFile(MavenProject, Artifact)'. | 161 |
org/apache/maven/bridge/MavenRepositorySystem.java
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingHardCodedIP | Do not hard code the IP address ${variableName} | 793 |
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 | 330–343 |
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/MojoExecutor.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'lock'. | 313 |
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/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 | 361–369 |
org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 118–124 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 394–398 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1589–1591 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1609–1613 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1742–1744 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 1758–1760 |
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 | 41 |
| UselessOverridingMethod | Overriding method merely calls super | 46 |
Priority 4
org/apache/maven/ReactorReader.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 225 |
org/apache/maven/artifact/metadata/AbstractArtifactMetadata.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.artifact.metadata': 'ArtifactMetadata' is already in scope because it is declared in the same package | 29 |
org/apache/maven/artifact/repository/metadata/AbstractRepositoryMetadata.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'RepositoryMetadata': 'RELEASE_OR_SNAPSHOT' is already in scope because it is inherited by an enclosing type | 179 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'RepositoryMetadata': 'RELEASE_OR_SNAPSHOT' is already in scope because it is inherited by an enclosing type | 179 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'RepositoryMetadata': 'SNAPSHOT' is already in scope because it is inherited by an enclosing type | 183 |
org/apache/maven/bridge/MavenRepositorySystem.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.model': 'Repository' is already in scope because it is imported in this file | 306 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.model': 'Repository' is already in scope because it is imported in this file | 308 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.model': 'Repository' is already in scope because it is imported in this file | 308 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.model': 'Repository' is already in scope because it is imported in this file | 350 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.model': 'Repository' is already in scope because it is imported in this file | 358 |
org/apache/maven/plugin/PluginParameterExpressionEvaluator.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 316 |
| UselessParentheses | Useless parentheses. | 345–346 |
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 | 42 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 48 |
| 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 | 55 |
org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.eclipse.aether.artifact': 'Artifact' is already in scope because it is imported in this file | 250 |
org/apache/maven/project/DefaultProjectBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 772 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 911 |
| UselessParentheses | Useless parentheses. | 1125 |
| UselessParentheses | Useless parentheses. | 1136 |
| UselessParentheses | Useless parentheses. | 1146 |
| UselessParentheses | Useless parentheses. | 1150 |
| UselessParentheses | Useless parentheses. | 1154 |
org/apache/maven/settings/DefaultMavenSettingsBuilder.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenSettingsBuilder': 'ALT_USER_SETTINGS_XML_LOCATION' is already in scope because it is inherited by an enclosing type | 47 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenSettingsBuilder': 'ALT_GLOBAL_SETTINGS_XML_LOCATION' is already in scope because it is inherited by an enclosing type | 58 |
Files
org/apache/maven/DefaultMaven.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ReturnFromFinallyBlock | Avoid returning from a finally block | 3 | 274 |
| UnusedFormalParameter | Avoid unused method parameters such as 'result'. | 3 | 447 |
org/apache/maven/ReactorReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateMethod | Avoid unused private methods such as 'determinePreviouslyPackagedArtifactFile(MavenProject, Artifact)'. | 3 | 161 |
| UselessParentheses | Useless parentheses. | 4 | 225 |
org/apache/maven/artifact/metadata/AbstractArtifactMetadata.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.artifact.metadata': 'ArtifactMetadata' is already in scope because it is declared in the same package | 4 | 29 |
org/apache/maven/artifact/repository/metadata/AbstractRepositoryMetadata.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'RepositoryMetadata': 'RELEASE_OR_SNAPSHOT' is already in scope because it is inherited by an enclosing type | 4 | 179 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'RepositoryMetadata': 'RELEASE_OR_SNAPSHOT' is already in scope because it is inherited by an enclosing type | 4 | 179 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'RepositoryMetadata': 'SNAPSHOT' is already in scope because it is inherited by an enclosing type | 4 | 183 |
org/apache/maven/bridge/MavenRepositorySystem.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.model': 'Repository' is already in scope because it is imported in this file | 4 | 306 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.model': 'Repository' is already in scope because it is imported in this file | 4 | 308 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.model': 'Repository' is already in scope because it is imported in this file | 4 | 308 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.model': 'Repository' is already in scope because it is imported in this file | 4 | 350 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.apache.maven.model': 'Repository' is already in scope because it is imported in this file | 4 | 358 |
| AvoidUsingHardCodedIP | Do not hard code the IP address ${variableName} | 3 | 793 |
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 |
org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyControlStatement | Empty if statement | 3 | 330–343 |
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/MojoExecutor.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'lock'. | 3 | 313 |
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/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 | 316 |
| UselessParentheses | Useless parentheses. | 4 | 345–346 |
| EmptyControlStatement | Empty if statement | 3 | 361–369 |
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 | 42 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'DefaultLegacySupport': 'SESSION' is already in scope because it is declared in an enclosing type | 4 | 48 |
| 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 | 55 |
org/apache/maven/plugin/internal/DefaultPluginDependenciesResolver.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 118–124 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'org.eclipse.aether.artifact': 'Artifact' is already in scope because it is imported in this file | 4 | 250 |
org/apache/maven/project/DefaultProjectBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 772 |
org/apache/maven/project/MavenProject.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 394–398 |
| UselessParentheses | Useless parentheses. | 4 | 911 |
| UselessParentheses | Useless parentheses. | 4 | 1125 |
| UselessParentheses | Useless parentheses. | 4 | 1136 |
| UselessParentheses | Useless parentheses. | 4 | 1146 |
| UselessParentheses | Useless parentheses. | 4 | 1150 |
| UselessParentheses | Useless parentheses. | 4 | 1154 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1589–1591 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1609–1613 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1742–1744 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 1758–1760 |
org/apache/maven/settings/DefaultMavenSettingsBuilder.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenSettingsBuilder': 'ALT_USER_SETTINGS_XML_LOCATION' is already in scope because it is inherited by an enclosing type | 4 | 47 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'MavenSettingsBuilder': 'ALT_GLOBAL_SETTINGS_XML_LOCATION' is already in scope because it is inherited by an enclosing type | 4 | 58 |
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 | 41 |
| UselessOverridingMethod | Overriding method merely calls super | 3 | 46 |