PMD Results
The following document contains the results of PMD 6.53.0.
Violations By Priority
Priority 2
avc-commons3-api-tests - net/avcompris/commons3/api/tests/ControllerContextUtils.java
| Rule | Violation | Line |
|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 167 |
avc-commons3-core-tests - net/avcompris/commons3/core/tests/AbstractServiceApiTest.java
| Rule | Violation | Line |
|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 220 |
avc-commons3-web - net/avcompris/commons3/web/AbstractController.java
| Rule | Violation | Line |
|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 600 |
Priority 3
avc-commons3-api-tests - net/avcompris/commons3/api/tests/AbstractApiTest.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–358 |
avc-commons3-api-tests - net/avcompris/commons3/api/tests/DataUtils.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–154 |
avc-commons3-api-tests - net/avcompris/commons3/api/tests/ExpressionUtils.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 20–23 |
avc-commons3-api - net/avcompris/commons3/api/EntitiesQueryRaw.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 22 |
avc-commons3-client - net/avcompris/commons3/client/impl/AbstractClient.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–244 |
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Action': member interfaces are implicitly static | 154–157 |
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ActionVoid': member interfaces are implicitly static | 160–163 |
avc-commons3-core-RDS-it-utils - net/avcompris/commons3/core/it/utils/RDSTestUtils.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'runtimeDbTableName'. | 137 |
| UnusedFormalParameter | Avoid unused method parameters such as 'metadata'. | 137 |
| UnusedFormalParameter | Avoid unused method parameters such as 'dbTable'. | 138 |
avc-commons3-core-tests - net/avcompris/commons3/core/tests/AbstractServiceApiTest.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–457 |
| EmptyCatchBlock | Avoid empty catch blocks | 177–180 |
| CollapsibleIfStatements | These nested if statements could be combined | 294–297 |
avc-commons3-core-tests - net/avcompris/commons3/core/tests/CoreTestUtils.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–554 |
| UnnecessaryReturn | Avoid unnecessary return statements | 58 |
avc-commons3-core - net/avcompris/commons3/core/impl/SortBysParser.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–127 |
avc-commons3-dao-RDS - net/avcompris/commons3/dao/DbTablesUtils.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–566 |
avc-commons3-notifier-utils - net/avcompris/commons3/notifier/utils/NotificationUtils.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–355 |
avc-commons3-testutil - net/avcompris/commons3/testutil/TestUtils.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–121 |
avc-commons3-web-it-utils - net/avcompris/commons3/web/it/utils/AbstractWebApiTest.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–223 |
avc-commons3-web-it-utils - net/avcompris/commons3/web/it/utils/AbstractWebTest.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 1–71 |
avc-commons3-web - net/avcompris/commons3/web/AbstractController.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getAuthorization': private methods cannot be overridden | 77–100 |
Files
avc-commons3-api-tests - net/avcompris/commons3/api/tests/AbstractApiTest.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–358 |
avc-commons3-api-tests - net/avcompris/commons3/api/tests/ControllerContextUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 167 |
avc-commons3-api-tests - net/avcompris/commons3/api/tests/DataUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–154 |
avc-commons3-api-tests - net/avcompris/commons3/api/tests/ExpressionUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 20–23 |
avc-commons3-api - net/avcompris/commons3/api/EntitiesQueryRaw.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessarySemicolon | Unnecessary semicolon | 3 | 22 |
avc-commons3-client - net/avcompris/commons3/client/impl/AbstractClient.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–244 |
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'Action': member interfaces are implicitly static | 3 | 154–157 |
| UnnecessaryModifier | Unnecessary modifier 'static' on interface 'ActionVoid': member interfaces are implicitly static | 3 | 160–163 |
avc-commons3-core-RDS-it-utils - net/avcompris/commons3/core/it/utils/RDSTestUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'runtimeDbTableName'. | 3 | 137 |
| UnusedFormalParameter | Avoid unused method parameters such as 'metadata'. | 3 | 137 |
| UnusedFormalParameter | Avoid unused method parameters such as 'dbTable'. | 3 | 138 |
avc-commons3-core-tests - net/avcompris/commons3/core/tests/AbstractServiceApiTest.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–457 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 177–180 |
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 220 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 294–297 |
avc-commons3-core-tests - net/avcompris/commons3/core/tests/CoreTestUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–554 |
| UnnecessaryReturn | Avoid unnecessary return statements | 3 | 58 |
avc-commons3-core - net/avcompris/commons3/core/impl/SortBysParser.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–127 |
avc-commons3-dao-RDS - net/avcompris/commons3/dao/DbTablesUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–566 |
avc-commons3-notifier-utils - net/avcompris/commons3/notifier/utils/NotificationUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–355 |
avc-commons3-testutil - net/avcompris/commons3/testutil/TestUtils.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–121 |
avc-commons3-web-it-utils - net/avcompris/commons3/web/it/utils/AbstractWebApiTest.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–223 |
avc-commons3-web-it-utils - net/avcompris/commons3/web/it/utils/AbstractWebTest.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 1–71 |
avc-commons3-web - net/avcompris/commons3/web/AbstractController.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'getAuthorization': private methods cannot be overridden | 3 | 77–100 |
| AvoidBranchingStatementAsLastInLoop | Avoid using a branching statement as the last in a loop. | 2 | 600 |
Suppressed Violations
| Filename | Rule message | Suppression type | Reason |
|---|---|---|---|
| net/avcompris/commons3/web/it/utils/JSONUtils.java | Avoid unused local variables such as 'dateTime'. | annotation |