Fork me on GitHub
Built by Maven

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 1

fr/paris/lutece/plugins/workflow/modules/crmforms/util/constants/NotifyCRMConstants.java

Rule Violation Line
ClassNamingConventions The utility class name 'NotifyCRMConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 41–73

Priority 2

fr/paris/lutece/plugins/workflow/modules/crmforms/service/task/NotifyCRMTask.java

Rule Violation Line
AvoidReassigningParameters Avoid reassigning parameters such as 'strContent' 162

Priority 3

fr/paris/lutece/plugins/workflow/modules/crmforms/business/TaskNotifyCRMConfigDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 56–71

fr/paris/lutece/plugins/workflow/modules/crmforms/service/FormsNotifyCRMService.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–197
DefaultPackage Use explicit scoping instead of the default package private level 89–91
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the _actionService field if you want a default access modifier 91
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 99
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 137

fr/paris/lutece/plugins/workflow/modules/crmforms/service/INotifyCRMService.java

Rule Violation Line
CommentRequired Header comments are required 43–78
UnnecessaryModifier Unnecessary modifier 'public' on method 'fillModelInfoResource': the method is declared in an interface type 52
UnnecessaryModifier Unnecessary modifier 'public' on method 'getListMarkers': the method is declared in an interface type 59
UnnecessaryModifier Unnecessary modifier 'public' on method 'getIdDemand': the method is declared in an interface type 68
UnnecessaryModifier Unnecessary modifier 'public' on method 'getUserGuid': the method is declared in an interface type 77

fr/paris/lutece/plugins/workflow/modules/crmforms/service/taskinfo/NotifyCRMTaskInfoProvider.java

Rule Violation Line
ShortVariable Avoid variables with short names like t 95

fr/paris/lutece/plugins/workflow/modules/crmforms/web/NotifyCRMTaskComponent.java

Rule Violation Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 136

Priority 4

fr/paris/lutece/plugins/workflow/modules/crmforms/business/TaskNotifyCRMConfig.java

Rule Violation Line
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 161–164

Files

fr/paris/lutece/plugins/workflow/modules/crmforms/business/TaskNotifyCRMConfig.java

Rule Violation Priority Line
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 161–164

fr/paris/lutece/plugins/workflow/modules/crmforms/business/TaskNotifyCRMConfigDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 56–71

fr/paris/lutece/plugins/workflow/modules/crmforms/service/FormsNotifyCRMService.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–197
DefaultPackage Use explicit scoping instead of the default package private level 3 89–91
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the _actionService field if you want a default access modifier 3 91
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 99
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 137

fr/paris/lutece/plugins/workflow/modules/crmforms/service/INotifyCRMService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 43–78
UnnecessaryModifier Unnecessary modifier 'public' on method 'fillModelInfoResource': the method is declared in an interface type 3 52
UnnecessaryModifier Unnecessary modifier 'public' on method 'getListMarkers': the method is declared in an interface type 3 59
UnnecessaryModifier Unnecessary modifier 'public' on method 'getIdDemand': the method is declared in an interface type 3 68
UnnecessaryModifier Unnecessary modifier 'public' on method 'getUserGuid': the method is declared in an interface type 3 77

fr/paris/lutece/plugins/workflow/modules/crmforms/service/task/NotifyCRMTask.java

Rule Violation Priority Line
AvoidReassigningParameters Avoid reassigning parameters such as 'strContent' 2 162

fr/paris/lutece/plugins/workflow/modules/crmforms/service/taskinfo/NotifyCRMTaskInfoProvider.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like t 3 95

fr/paris/lutece/plugins/workflow/modules/crmforms/util/constants/NotifyCRMConstants.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'NotifyCRMConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 41–73

fr/paris/lutece/plugins/workflow/modules/crmforms/web/NotifyCRMTaskComponent.java

Rule Violation Priority Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 136