PMD Results
The following document contains the results of PMD 7.14.0.
Violations By Priority
Priority 3
org/apache/maven/cling/invoker/BaseParser.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–533 |
org/apache/maven/cling/invoker/LookupInvoker.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 330 |
org/apache/maven/cling/invoker/mvnsh/builtin/BuiltinShellCommandRegistryFactory.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 186 |
| UnusedFormalParameter | Avoid unused method parameters such as 'input'. | 190 |
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 211 |
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 249 |
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 267 |
org/apache/maven/cling/invoker/mvnup/goals/CompatibilityFixStrategy.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–571 |
org/apache/maven/cling/invoker/mvnup/goals/InferenceStrategy.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–641 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 295–299 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 309–313 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 331–334 |
org/apache/maven/cling/invoker/mvnup/goals/JDomUtils.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–544 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 413–416 |
org/apache/maven/cling/invoker/mvnup/goals/ModelUpgradeStrategy.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–253 |
| UnusedFormalParameter | Avoid unused method parameters such as 'currentVersion'. | 135 |
org/apache/maven/cling/invoker/mvnup/goals/ModelVersionUtils.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–228 |
org/apache/maven/cling/invoker/mvnup/goals/PluginUpgradeStrategy.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–900 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 279–281 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 809–812 |
org/apache/maven/cling/invoker/mvnup/goals/PomDiscovery.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 19–295 |
org/apache/maven/cling/logging/impl/UnsupportedSlf4jBindingConfiguration.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'slf4jBinding'. | 38 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'supported'. | 38 |
org/apache/maven/cling/props/MavenProperties.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 364–366 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 377–379 |
org/apache/maven/cling/transfer/ConsoleMavenTransferListener.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'event'. | 139 |
org/apache/maven/cling/utils/CLIReportingUtils.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'ONE_DAY'. | 47 |
Priority 4
org/apache/maven/cling/invoker/mvnenc/CommonsCliEncryptOptions.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Options': 'SOURCE_CLI' is already in scope because it is inherited by an enclosing type | 39 |
org/apache/maven/cling/invoker/mvnsh/CommonsCliShellOptions.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Options': 'SOURCE_CLI' is already in scope because it is inherited by an enclosing type | 33 |
org/apache/maven/cling/invoker/mvnup/CommonsCliUpgradeOptions.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Options': 'SOURCE_CLI' is already in scope because it is inherited by an enclosing type | 40 |
org/apache/maven/cling/logging/Slf4jLoggerManager.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 53–55 |
org/apache/maven/cling/props/MavenPropertiesLoader.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 124 |
| UselessParentheses | Useless parentheses. | 155 |
org/apache/maven/cling/transfer/AbstractMavenTransferListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 72 |
org/apache/maven/cling/transfer/Slf4jMavenTransferListener.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 75 |
org/apache/maven/cling/utils/CLIReportingUtils.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 99 |
| UselessParentheses | Useless parentheses. | 119 |
Files
org/apache/maven/cling/invoker/BaseParser.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–533 |
org/apache/maven/cling/invoker/LookupInvoker.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 330 |
org/apache/maven/cling/invoker/mvnenc/CommonsCliEncryptOptions.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Options': 'SOURCE_CLI' is already in scope because it is inherited by an enclosing type | 4 | 39 |
org/apache/maven/cling/invoker/mvnsh/CommonsCliShellOptions.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Options': 'SOURCE_CLI' is already in scope because it is inherited by an enclosing type | 4 | 33 |
org/apache/maven/cling/invoker/mvnsh/builtin/BuiltinShellCommandRegistryFactory.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 3 | 186 |
| UnusedFormalParameter | Avoid unused method parameters such as 'input'. | 3 | 190 |
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 3 | 211 |
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 3 | 249 |
| UnusedFormalParameter | Avoid unused method parameters such as 'name'. | 3 | 267 |
org/apache/maven/cling/invoker/mvnup/CommonsCliUpgradeOptions.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'Options': 'SOURCE_CLI' is already in scope because it is inherited by an enclosing type | 4 | 40 |
org/apache/maven/cling/invoker/mvnup/goals/CompatibilityFixStrategy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–571 |
org/apache/maven/cling/invoker/mvnup/goals/InferenceStrategy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–641 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 295–299 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 309–313 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 331–334 |
org/apache/maven/cling/invoker/mvnup/goals/JDomUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–544 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 413–416 |
org/apache/maven/cling/invoker/mvnup/goals/ModelUpgradeStrategy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–253 |
| UnusedFormalParameter | Avoid unused method parameters such as 'currentVersion'. | 3 | 135 |
org/apache/maven/cling/invoker/mvnup/goals/ModelVersionUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–228 |
org/apache/maven/cling/invoker/mvnup/goals/PluginUpgradeStrategy.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–900 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 279–281 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 809–812 |
org/apache/maven/cling/invoker/mvnup/goals/PomDiscovery.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 19–295 |
org/apache/maven/cling/logging/Slf4jLoggerManager.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 53–55 |
org/apache/maven/cling/logging/impl/UnsupportedSlf4jBindingConfiguration.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused constructor parameters such as 'slf4jBinding'. | 3 | 38 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'supported'. | 3 | 38 |
org/apache/maven/cling/props/MavenProperties.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 364–366 |
| CollapsibleIfStatements | This if statement could be combined with its parent | 3 | 377–379 |
org/apache/maven/cling/props/MavenPropertiesLoader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 124 |
| UselessParentheses | Useless parentheses. | 4 | 155 |
org/apache/maven/cling/transfer/AbstractMavenTransferListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 72 |
org/apache/maven/cling/transfer/ConsoleMavenTransferListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'event'. | 3 | 139 |
org/apache/maven/cling/transfer/Slf4jMavenTransferListener.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 75 |
org/apache/maven/cling/utils/CLIReportingUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'ONE_DAY'. | 3 | 47 |
| UselessParentheses | Useless parentheses. | 4 | 99 |
| UselessParentheses | Useless parentheses. | 4 | 119 |