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 90–112
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 100–111

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

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

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. 34–740
Possible God class (WMC=55, ATFD=144, TCC=0.23333333333333334) 34–740
The class 'DocumentToCalendarJspBean' has a Cyclomatic Complexity of 4 (Highest = 11). 84–740
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. 540–639
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