PMD Results
The following document contains the results of PMD 7.14.0.
Violations By Priority
Priority 1
org/forgerock/audit/handlers/csv/CsvAuditEventHandler.java
| Rule | Violation | Line |
|---|---|---|
| DoubleCheckedLocking | Double checked locking is not thread safe in Java. | 303 |
Priority 3
org/forgerock/audit/handlers/csv/CsvAuditEventHandler.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 16–609 |
org/forgerock/audit/handlers/csv/CsvSecureMapReader.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 34 |
org/forgerock/audit/handlers/csv/SecureCsvWriter.java
| Rule | Violation | Line |
|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 16–435 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 302 |
org/forgerock/audit/handlers/csv/StandardCsvWriter.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 45 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 116 |
Priority 4
org/forgerock/audit/handlers/csv/CsvAuditEventHandler.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'ResourceResponse': 'FIELD_CONTENT_ID' is already in scope because it is imported in this file | 248 |
| UselessParentheses | Useless parentheses. | 537 |
org/forgerock/audit/handlers/csv/SecureCsvWriter.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'CsvSecureConstants': 'HEADER_SIGNATURE' is already in scope because it is imported in this file | 329 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'CsvSecureConstants': 'HEADER_HMAC' is already in scope because it is imported in this file | 359 |
Files
org/forgerock/audit/handlers/csv/CsvAuditEventHandler.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 16–609 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'ResourceResponse': 'FIELD_CONTENT_ID' is already in scope because it is imported in this file | 4 | 248 |
| DoubleCheckedLocking | Double checked locking is not thread safe in Java. | 1 | 303 |
| UselessParentheses | Useless parentheses. | 4 | 537 |
org/forgerock/audit/handlers/csv/CsvSecureMapReader.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 3 | 34 |
org/forgerock/audit/handlers/csv/SecureCsvWriter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyStaticImports | Too many static imports may lead to messy code | 3 | 16–435 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 302 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'CsvSecureConstants': 'HEADER_SIGNATURE' is already in scope because it is imported in this file | 4 | 329 |
| UnnecessaryFullyQualifiedName | Unnecessary qualifier 'CsvSecureConstants': 'HEADER_HMAC' is already in scope because it is imported in this file | 4 | 359 |
org/forgerock/audit/handlers/csv/StandardCsvWriter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 3 | 45 |
| SimplifiedTernary | This conditional expression can be simplified with || or && | 3 | 116 |