Fork me on GitHub

PMD Results

The following document contains the results of PMD 5.3.5.

Files

fr/paris/lutece/plugins/crm/modules/form/service/CRMParametersService.java

Violation Line
Avoid using implementation types like 'HashMap'; use the interface instead 24
Avoid using implementation types like 'HashMap'; use the interface instead 24

fr/paris/lutece/plugins/crm/modules/form/service/draft/CRMDraftBackupService.java

Violation Line
Possible God class (WMC=97, ATFD=167, TCC=0.03508771929824561) 34825
A high number of imports can indicate a high degree of coupling within an object. 34825
The class 'CRMDraftBackupService' has a Modified Cyclomatic Complexity of 4 (Highest = 12). 86825
The class 'CRMDraftBackupService' has a Standard Cyclomatic Complexity of 4 (Highest = 12). 86825
The class 'CRMDraftBackupService' has a Cyclomatic Complexity of 5 (Highest = 13). 86825
The Logger variable declaration does not contain the static and final modifiers 88
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 230
Avoid unused method parameters such as 'form'. 306
Avoid unnecessary if..then..else statements when returning booleans 309312
The method 'storeFiles' has a Modified Cyclomatic Complexity of 10. 324402
The method 'storeFiles' has a Cyclomatic Complexity of 10. 324402
The method 'storeFiles' has a Standard Cyclomatic Complexity of 10. 324402
Avoid variables with short names like is 346
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 384
The method 'create' has a Cyclomatic Complexity of 11. 409482
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 474
The method 'restore' has a Modified Cyclomatic Complexity of 12. 488563
The method 'restore' has a Standard Cyclomatic Complexity of 12. 488563
The method 'restore' has a Cyclomatic Complexity of 13. 488563
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 547
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 651
Rather than using a lot of String arguments, consider using a container object for those values. 783784

fr/paris/lutece/plugins/crm/modules/form/util/JSONUtils.java

Violation Line
Possible God class (WMC=50, ATFD=48, TCC=0.06666666666666667) 1402
The class 'JSONUtils' has a Cyclomatic Complexity of 5 (Highest = 12). 37402
The method 'buildJson' has a Cyclomatic Complexity of 12. 93145
Deeply nested if..then statements are hard to read 125132
The method 'getBlobIds' has a Cyclomatic Complexity of 12. 173216
Avoid empty catch blocks 210213
Avoid unused method parameters such as 'session'. 275
Assigning an Object to null is a code smell. Consider refactoring. 388
Assigning an Object to null is a code smell. Consider refactoring. 394