PMD Results
The following document contains the results of PMD 5.3.5.
Files
fr/paris/lutece/plugins/workflow/modules/automaticassignment/business/AutomaticAssignmentDAO.java
| Violation |
Line |
| These nested if statements could be combined |
87–92 |
| Use block level rather than method level synchronization |
135–148 |
fr/paris/lutece/plugins/workflow/modules/automaticassignment/business/TaskAutomaticAssignmentConfigDAO.java
| Violation |
Line |
| Use block level rather than method level synchronization |
68–88 |
fr/paris/lutece/plugins/workflow/modules/automaticassignment/service/AutomaticAssignmentPlugin.java
| Violation |
Line |
| All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. |
47–59 |
fr/paris/lutece/plugins/workflow/modules/automaticassignment/service/AutomaticAssignmentService.java
| Violation |
Line |
| Possible God class (WMC=74, ATFD=106, TCC=0.0) |
34–659 |
| A high number of imports can indicate a high degree of coupling within an object. |
34–659 |
| The class 'AutomaticAssignmentService' has a Cyclomatic Complexity of 4 (Highest = 18). |
95–659 |
| Private field '_listAcceptedEntryTypesFile' could be made final; it is only initialized in the declaration or constructor. |
144 |
| Deeply nested if..then statements are hard to read |
306–311 |
| The method 'notify' has a Cyclomatic Complexity of 11. |
324–396 |
| The method 'getFiles' has a Cyclomatic Complexity of 12. |
434–488 |
| Avoid variables with short names like t |
529 |
| The method 'buildModel' has a Cyclomatic Complexity of 18. |
567–658 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
571 |
| Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' due to existing import 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' |
585 |
| Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' due to existing import 'fr.paris.lutece.plugins.directory.business.EntryTypeGeolocation' |
593 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
642 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
648 |
fr/paris/lutece/plugins/workflow/modules/automaticassignment/service/TaskAutomaticAssignmentConfigService.java
| Violation |
Line |
| Deeply nested if..then statements are hard to read |
107–111 |
| Deeply nested if..then statements are hard to read |
161–165 |
fr/paris/lutece/plugins/workflow/modules/automaticassignment/web/AutomaticAssignmentJspBean.java
| Violation |
Line |
| Private field '_automaticAssignmentService' could be made final; it is only initialized in the declaration or constructor. |
92 |
| Private field '_taskAutomaticAssignmentConfigService' could be made final; it is only initialized in the declaration or constructor. |
93 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
175 |
fr/paris/lutece/plugins/workflow/modules/automaticassignment/web/AutomaticAssignmentTaskComponent.java
| Violation |
Line |
| A high number of imports can indicate a high degree of coupling within an object. |
34–416 |
| The class 'AutomaticAssignmentTaskComponent' has a Standard Cyclomatic Complexity of 8 (Highest = 20). |
84–416 |
| The class 'AutomaticAssignmentTaskComponent' has a Modified Cyclomatic Complexity of 8 (Highest = 20). |
84–416 |
| The class 'AutomaticAssignmentTaskComponent' has a Cyclomatic Complexity of 11 (Highest = 29). |
84–416 |
| The method 'doSaveConfig' has a Modified Cyclomatic Complexity of 20. |
151–301 |
| The method 'doSaveConfig' has a Cyclomatic Complexity of 29. |
151–301 |
| Avoid really long methods. |
151–301 |
| The method doSaveConfig() has an NPath complexity of 13824000 |
151–301 |
| The method 'doSaveConfig' has a Standard Cyclomatic Complexity of 20. |
151–301 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
160 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
161 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
164 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
165 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
166 |
| Avoid declaring a variable if it is unreferenced before a possible exit point. |
167 |
| The method 'getDisplayConfigForm' has a Standard Cyclomatic Complexity of 10. |
307–397 |
| The method 'getDisplayConfigForm' has a Modified Cyclomatic Complexity of 10. |
307–397 |
| The method 'getDisplayConfigForm' has a Cyclomatic Complexity of 11. |
307–397 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
311 |
| If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
326 |
