PMD Results
The following document contains the results of PMD 7.14.0.
Violations By Priority
Priority 3
org/forgerock/api/CrestApiProducer.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 17–158 |
org/forgerock/api/commons/CommonsApi.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Errors(int, String)': enum constructors are implicitly private | 176 |
org/forgerock/api/jackson/CrestPropertyDetailsSchemaFactoryWrapper.java
| Rule | Violation | Line |
|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 137 |
org/forgerock/api/jackson/CrestStringSchema.java
| Rule | Violation | Line |
|---|---|---|
| UselessOperationOnImmutable | The result of an operation on an immutable object is ignored | 139 |
| UselessOperationOnImmutable | The result of an operation on an immutable object is ignored | 149 |
| UnnecessaryReturn | Unnecessary return statement | 187 |
org/forgerock/api/markup/ApiDocGenerator.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 18–1130 |
org/forgerock/api/markup/asciidoc/AsciiDoc.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 17–623 |
| UnusedLocalVariable | Avoid unused local variables such as 'symbol'. | 283 |
org/forgerock/api/models/Action.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'action': the method is already in a final class | 117 |
org/forgerock/api/models/ApiDescription.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 16–340 |
org/forgerock/api/models/Create.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'create': the method is already in a final class | 77 |
org/forgerock/api/models/Delete.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'delete': the method is already in a final class | 41 |
org/forgerock/api/models/Patch.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'patch': the method is already in a final class | 85 |
org/forgerock/api/models/Read.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'read': the method is already in a final class | 41 |
org/forgerock/api/models/Update.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'update': the method is already in a final class | 41 |
org/forgerock/api/transform/OpenApiTransformer.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 17–1625 |
org/forgerock/api/util/ValidationUtil.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 91 |
Priority 4
org/forgerock/api/jackson/CrestObjectSchema.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryImport | Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.AdditionalProperties' | 26 |
| UnnecessaryImport | Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.NoAdditionalProperties' | 27 |
| UnnecessaryImport | Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.SchemaAdditionalProperties' | 28 |
org/forgerock/api/markup/asciidoc/AsciiDoc.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'DOC_TITLE' is already in scope because it is imported in this file | 262 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'BLOCK_TITLE' is already in scope because it is imported in this file | 272 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_1' is already in scope because it is imported in this file | 287 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_2' is already in scope because it is imported in this file | 289 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_3' is already in scope because it is imported in this file | 291 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_4' is already in scope because it is imported in this file | 293 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_5' is already in scope because it is imported in this file | 295 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_1' is already in scope because it is imported in this file | 309 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_2' is already in scope because it is imported in this file | 319 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_3' is already in scope because it is imported in this file | 329 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_4' is already in scope because it is imported in this file | 339 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_5' is already in scope because it is imported in this file | 349 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'ANCHOR_START' is already in scope because it is imported in this file | 433 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'ANCHOR_END' is already in scope because it is imported in this file | 435 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'ANCHOR_START' is already in scope because it is imported in this file | 454 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'ANCHOR_END' is already in scope because it is imported in this file | 457 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'CROSS_REF_START' is already in scope because it is imported in this file | 474 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'CROSS_REF_END' is already in scope because it is imported in this file | 476 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'CROSS_REF_START' is already in scope because it is imported in this file | 495 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'CROSS_REF_END' is already in scope because it is imported in this file | 498 |
org/forgerock/api/models/Schema.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'JacksonUtils': 'OBJECT_MAPPER' is already in scope because it is imported in this file | 138 |
org/forgerock/api/transform/OpenApiTransformer.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'PathUtil': 'buildPath' is already in scope | 199 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'PathUtil': 'buildPath' is already in scope | 203 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'PathUtil': 'buildPath' is already in scope | 329 |
Files
org/forgerock/api/CrestApiProducer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 17–158 |
org/forgerock/api/commons/CommonsApi.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'private' on constructor 'Errors(int, String)': enum constructors are implicitly private | 3 | 176 |
org/forgerock/api/jackson/CrestObjectSchema.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryImport | Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.AdditionalProperties' | 4 | 26 |
| UnnecessaryImport | Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.NoAdditionalProperties' | 4 | 27 |
| UnnecessaryImport | Unused import 'com.fasterxml.jackson.module.jsonSchema.jakarta.types.ObjectSchema.SchemaAdditionalProperties' | 4 | 28 |
org/forgerock/api/jackson/CrestPropertyDetailsSchemaFactoryWrapper.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 3 | 137 |
org/forgerock/api/jackson/CrestStringSchema.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessOperationOnImmutable | The result of an operation on an immutable object is ignored | 3 | 139 |
| UselessOperationOnImmutable | The result of an operation on an immutable object is ignored | 3 | 149 |
| UnnecessaryReturn | Unnecessary return statement | 3 | 187 |
org/forgerock/api/markup/ApiDocGenerator.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 18–1130 |
org/forgerock/api/markup/asciidoc/AsciiDoc.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 17–623 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'DOC_TITLE' is already in scope because it is imported in this file | 4 | 262 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'BLOCK_TITLE' is already in scope because it is imported in this file | 4 | 272 |
| UnusedLocalVariable | Avoid unused local variables such as 'symbol'. | 3 | 283 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_1' is already in scope because it is imported in this file | 4 | 287 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_2' is already in scope because it is imported in this file | 4 | 289 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_3' is already in scope because it is imported in this file | 4 | 291 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_4' is already in scope because it is imported in this file | 4 | 293 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_5' is already in scope because it is imported in this file | 4 | 295 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_1' is already in scope because it is imported in this file | 4 | 309 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_2' is already in scope because it is imported in this file | 4 | 319 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_3' is already in scope because it is imported in this file | 4 | 329 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_4' is already in scope because it is imported in this file | 4 | 339 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'SECTION_TITLE_5' is already in scope because it is imported in this file | 4 | 349 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'ANCHOR_START' is already in scope because it is imported in this file | 4 | 433 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'ANCHOR_END' is already in scope because it is imported in this file | 4 | 435 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'ANCHOR_START' is already in scope because it is imported in this file | 4 | 454 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'ANCHOR_END' is already in scope because it is imported in this file | 4 | 457 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'CROSS_REF_START' is already in scope because it is imported in this file | 4 | 474 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'CROSS_REF_END' is already in scope because it is imported in this file | 4 | 476 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'CROSS_REF_START' is already in scope because it is imported in this file | 4 | 495 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'AsciiDocSymbols': 'CROSS_REF_END' is already in scope because it is imported in this file | 4 | 498 |
org/forgerock/api/models/Action.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'action': the method is already in a final class | 3 | 117 |
org/forgerock/api/models/ApiDescription.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 16–340 |
org/forgerock/api/models/Create.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'create': the method is already in a final class | 3 | 77 |
org/forgerock/api/models/Delete.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'delete': the method is already in a final class | 3 | 41 |
org/forgerock/api/models/Patch.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'patch': the method is already in a final class | 3 | 85 |
org/forgerock/api/models/Read.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'read': the method is already in a final class | 3 | 41 |
org/forgerock/api/models/Schema.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'JacksonUtils': 'OBJECT_MAPPER' is already in scope because it is imported in this file | 4 | 138 |
org/forgerock/api/models/Update.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'update': the method is already in a final class | 3 | 41 |
org/forgerock/api/transform/OpenApiTransformer.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 17–1625 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'PathUtil': 'buildPath' is already in scope | 4 | 199 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'PathUtil': 'buildPath' is already in scope | 4 | 203 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'PathUtil': 'buildPath' is already in scope | 4 | 329 |
org/forgerock/api/util/ValidationUtil.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 91 |