Fork me on GitHub
Built by Maven

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 3

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/AutomaticAssignment.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 155–158
CommentRequired Public method and constructor comments are required 160–163

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/AutomaticAssignmentDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 122–135

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/TaskAutomaticAssignmentConfigDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 67–87

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/AutomaticAssignmentService.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–602
TooManyMethods This class has too many methods, consider refactoring it. 94–602
ConfusingTernary Avoid if (x != y) ..; else ..; 448–465
ShortVariable Avoid variables with short names like t 481
CyclomaticComplexity The method 'buildModel(TaskAutomaticAssignmentConfig, ResourceHistory, List, HttpServletRequest, Locale)' has a cyclomatic complexity of 10. 516–600
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 519
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 585
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 591

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/IAutomaticAssignmentService.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 160

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/TaskAutomaticAssignment.java

Rule Violation Line
CyclomaticComplexity The method 'processTask(int, HttpServletRequest, Locale)' has a cyclomatic complexity of 10. 91–138
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 93

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/web/AutomaticAssignmentJspBean.java

Rule Violation Line
ImmutableField Private field '_automaticAssignmentService' could be made final; it is only initialized in the declaration or constructor. 90
ImmutableField Private field '_taskAutomaticAssignmentConfigService' could be made final; it is only initialized in the declaration or constructor. 91
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 178
ConfusingTernary Avoid if (x != y) ..; else ..; 210–217

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/web/AutomaticAssignmentTaskComponent.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–426
ExcessiveMethodLength Avoid really long methods. 147–295
NcssCount The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has a NCSS line count of 78. 147–295
NPathComplexity The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has an NPath complexity of 2211840 147–295
CyclomaticComplexity The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has a cyclomatic complexity of 29. 147–295
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 156
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 157
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 160
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 161
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 162
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 163
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 220–240
ConfusingTernary Avoid if (x != y) ..; else ..; 228–236
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 273–276
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 304
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 319
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 380

Files

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/AutomaticAssignment.java

Rule Violation Priority Line
CommentRequired Public method and constructor comments are required 3 155–158
CommentRequired Public method and constructor comments are required 3 160–163

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/AutomaticAssignmentDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 122–135

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/business/TaskAutomaticAssignmentConfigDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 67–87

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/AutomaticAssignmentService.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–602
TooManyMethods This class has too many methods, consider refactoring it. 3 94–602
ConfusingTernary Avoid if (x != y) ..; else ..; 3 448–465
ShortVariable Avoid variables with short names like t 3 481
CyclomaticComplexity The method 'buildModel(TaskAutomaticAssignmentConfig, ResourceHistory, List, HttpServletRequest, Locale)' has a cyclomatic complexity of 10. 3 516–600
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 519
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 585
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 591

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/IAutomaticAssignmentService.java

Rule Violation Priority Line
CommentRequired Public method and constructor comments are required 3 160

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/service/TaskAutomaticAssignment.java

Rule Violation Priority Line
CyclomaticComplexity The method 'processTask(int, HttpServletRequest, Locale)' has a cyclomatic complexity of 10. 3 91–138
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 93

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/web/AutomaticAssignmentJspBean.java

Rule Violation Priority Line
ImmutableField Private field '_automaticAssignmentService' could be made final; it is only initialized in the declaration or constructor. 3 90
ImmutableField Private field '_taskAutomaticAssignmentConfigService' could be made final; it is only initialized in the declaration or constructor. 3 91
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 178
ConfusingTernary Avoid if (x != y) ..; else ..; 3 210–217

fr/paris/lutece/plugins/workflow/modules/formsautomaticassignment/web/AutomaticAssignmentTaskComponent.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–426
ExcessiveMethodLength Avoid really long methods. 3 147–295
NcssCount The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has a NCSS line count of 78. 3 147–295
NPathComplexity The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has an NPath complexity of 2211840 3 147–295
CyclomaticComplexity The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has a cyclomatic complexity of 29. 3 147–295
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 156
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 157
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 160
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 161
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 162
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 163
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 220–240
ConfusingTernary Avoid if (x != y) ..; else ..; 3 228–236
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 273–276
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 304
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 319
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 380