Fork me on GitHub

PMD Results

The following document contains the results of PMD 5.3.5.

Files

fr/paris/lutece/plugins/document/modules/calendar/service/DocumentActionsService.java

Violation Line
Avoid unused private fields such as 'PLUGIN_NAME'. 63
These nested if statements could be combined 90112
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 100
Deeply nested if..then statements are hard to read 100111

fr/paris/lutece/plugins/document/modules/calendar/service/MappingService.java

Violation Line
The class 'MappingService' has a Modified Cyclomatic Complexity of 3 (Highest = 10). 60269
The class 'MappingService' has a Cyclomatic Complexity of 4 (Highest = 15). 60269
The class 'MappingService' has a Standard Cyclomatic Complexity of 3 (Highest = 10). 60269
Avoid unused private fields such as 'PLUGIN_NAME'. 69
The method 'getCalendarAttributeListByIdMappingAttribute' has a Cyclomatic Complexity of 15. 205268
The method 'getCalendarAttributeListByIdMappingAttribute' has a Standard Cyclomatic Complexity of 10. 205268
The method 'getCalendarAttributeListByIdMappingAttribute' has a Modified Cyclomatic Complexity of 10. 205268

fr/paris/lutece/plugins/document/modules/calendar/web/DocumentToCalendarJspBean.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 34740
Possible God class (WMC=55, ATFD=144, TCC=0.23333333333333334) 34740
The class 'DocumentToCalendarJspBean' has a Cyclomatic Complexity of 4 (Highest = 11). 84740
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 176
Avoid declaring a variable if it is unreferenced before a possible exit point. 213
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 237
Avoid declaring a variable if it is unreferenced before a possible exit point. 256
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 328
Avoid declaring a variable if it is unreferenced before a possible exit point. 354
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 387
Avoid declaring a variable if it is unreferenced before a possible exit point. 412
Avoid using Literals in Conditional Statements 461
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 463
Avoid declaring a variable if it is unreferenced before a possible exit point. 488
Avoid declaring a variable if it is unreferenced before a possible exit point. 491
Avoid using Literals in Conditional Statements 502
Position literals first in String comparisons 512
The method 'getCreateEvent' has a Cyclomatic Complexity of 11. 540639
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 564
Avoid using Literals in Conditional Statements 656
Avoid variables with short names like st 678
Avoid variables with short names like st 701
Avoid variables with short names like i 703