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/formsextend/service/DashboardNotificationService.java

Rule Violation Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 48–104

fr/paris/lutece/plugins/workflow/modules/formsextend/util/FormsExtendConstants.java

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

Priority 3

fr/paris/lutece/plugins/workflow/modules/formsextend/business/MassNotificationTaskConfig.java

Rule Violation Line
MethodReturnsInternalArray Returning '_listExtenderTypesForEmail' may expose an internal array. 133
ArrayIsStoredDirectly The user-supplied array 'listExtenderTypesForEmail' is stored directly. 140
MethodReturnsInternalArray Returning '_listExtenderTypesForDashboard' may expose an internal array. 218
ArrayIsStoredDirectly The user-supplied array 'listExtenderTypesForDashboard' is stored directly. 225

fr/paris/lutece/plugins/workflow/modules/formsextend/service/DashboardNotificationService.java

Rule Violation Line
UnusedFormalParameter Avoid unused method parameters such as 'request'. 86

fr/paris/lutece/plugins/workflow/modules/formsextend/service/MassNotificationService.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–364
DefaultPackage Use explicit scoping instead of the default package private level 110–112
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the _resourceWorkflowService field if you want a default access modifier 112
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 160
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 254
ConfusingTernary Avoid if (x != y) ..; else ..; 272–282
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 292

fr/paris/lutece/plugins/workflow/modules/formsextend/util/FormsExtendConstants.java

Rule Violation Line
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 52
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 53
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 55

fr/paris/lutece/plugins/workflow/modules/formsextend/web/MassNotificationTaskComponent.java

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

Priority 4

fr/paris/lutece/plugins/workflow/modules/formsextend/business/MassNotificationTaskConfig.java

Rule Violation Line
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 140
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 225

fr/paris/lutece/plugins/workflow/modules/formsextend/service/MassNotificationService.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppPathService' 84

Files

fr/paris/lutece/plugins/workflow/modules/formsextend/business/MassNotificationTaskConfig.java

Rule Violation Priority Line
MethodReturnsInternalArray Returning '_listExtenderTypesForEmail' may expose an internal array. 3 133
ArrayIsStoredDirectly The user-supplied array 'listExtenderTypesForEmail' is stored directly. 3 140
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 140
MethodReturnsInternalArray Returning '_listExtenderTypesForDashboard' may expose an internal array. 3 218
ArrayIsStoredDirectly The user-supplied array 'listExtenderTypesForDashboard' is stored directly. 3 225
UseVarargs Consider using varargs for methods or constructors which take an array the last parameter. 4 225

fr/paris/lutece/plugins/workflow/modules/formsextend/service/DashboardNotificationService.java

Rule Violation Priority Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 48–104
UnusedFormalParameter Avoid unused method parameters such as 'request'. 3 86

fr/paris/lutece/plugins/workflow/modules/formsextend/service/MassNotificationService.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–364
UnusedImports Avoid unused imports such as 'fr.paris.lutece.portal.service.util.AppPathService' 4 84
DefaultPackage Use explicit scoping instead of the default package private level 3 110–112
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the _resourceWorkflowService field if you want a default access modifier 3 112
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 160
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 254
ConfusingTernary Avoid if (x != y) ..; else ..; 3 272–282
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 292

fr/paris/lutece/plugins/workflow/modules/formsextend/util/FormsExtendConstants.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'FormsExtendConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 41–57
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 52
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 53
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 54
FieldDeclarationsShouldBeAtStartOfClass Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. 3 55

fr/paris/lutece/plugins/workflow/modules/formsextend/web/MassNotificationTaskComponent.java

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