Fork me on GitHub

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 1

fr/paris/lutece/plugins/workflow/modules/actionsbatch/task/ActionsBatchTask.java

Rule Violation Line
FieldNamingConventions The constant name '_resourceHistoryService' doesn't match '[A-Z][A-Z_0-9]*' 65
FieldNamingConventions The constant name '_taskConfigService' doesn't match '[A-Z][A-Z_0-9]*' 66
FieldNamingConventions The constant name '_workflowService' doesn't match '[A-Z][A-Z_0-9]*' 67
FieldNamingConventions The constant name '_progressManagerService' doesn't match '[A-Z][A-Z_0-9]*' 68

fr/paris/lutece/plugins/workflow/modules/actionsbatch/web/ActionsBatchTaskComponent.java

Rule Violation Line
LocalVariableNamingConventions The local variable name '_actionService' doesn't match '[a-z][a-zA-Z0-9]*' 114

Priority 3

fr/paris/lutece/plugins/workflow/modules/actionsbatch/service/ActionsBatchService.java

Rule Violation Line
CommentRequired Header comments are required 47107
UseUtilityClass 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. 48107
DoNotUseThreads To be compliant to J2EE, a webapp should not use any thread. 81
DoNotUseThreads To be compliant to J2EE, a webapp should not use any thread. 101
DoNotUseThreads To be compliant to J2EE, a webapp should not use any thread. 101

fr/paris/lutece/plugins/workflow/modules/actionsbatch/task/ActionsBatchTask.java

Rule Violation Line
CommentRequired Header comments are required 57131
UnusedPrivateField Avoid unused private fields such as 'PARAMETER_USER_ID'. 59
UnusedPrivateField Avoid unused private fields such as 'PARAMETER_FEED_TOKEN'. 71
UnusedLocalVariable Avoid unused local variables such as 'parentId'. 78

fr/paris/lutece/plugins/workflow/modules/actionsbatch/task/ActionsBatchTaskConfig.java

Rule Violation Line
CommentRequired Header comments are required 40138

fr/paris/lutece/plugins/workflow/modules/actionsbatch/task/ActionsBatchTaskConfigDAO.java

Rule Violation Line
CommentRequired Header comments are required 39111
ShortVariable Avoid variables with short names like i 89

fr/paris/lutece/plugins/workflow/modules/actionsbatch/task/AsynchronousSimpleTask.java

Rule Violation Line
DoNotUseThreads To be compliant to J2EE, a webapp should not use any thread. 61
DefaultPackage Use explicit scoping instead of the default package private level 74
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the processAsynchronousTask method if you want a default access modifier 74

fr/paris/lutece/plugins/workflow/modules/actionsbatch/web/ActionsBatchTaskComponent.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34268
UnusedPrivateField Avoid unused private fields such as 'MARK_WORKFLOWS'. 86
UnusedPrivateField Avoid unused private fields such as 'MARK_STATUS'. 87
UnusedPrivateField Avoid unused private fields such as 'MARK_ACTION'. 88
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 117
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 129
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 130
ShortVariable Avoid variables with short names like id 205

Priority 4

fr/paris/lutece/plugins/workflow/modules/actionsbatch/web/ActionsBatchTaskComponent.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'java.util.ArrayList' 36

Files

fr/paris/lutece/plugins/workflow/modules/actionsbatch/service/ActionsBatchService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 47107
UseUtilityClass 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. 3 48107
DoNotUseThreads To be compliant to J2EE, a webapp should not use any thread. 3 81
DoNotUseThreads To be compliant to J2EE, a webapp should not use any thread. 3 101
DoNotUseThreads To be compliant to J2EE, a webapp should not use any thread. 3 101

fr/paris/lutece/plugins/workflow/modules/actionsbatch/task/ActionsBatchTask.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 57131
UnusedPrivateField Avoid unused private fields such as 'PARAMETER_USER_ID'. 3 59
FieldNamingConventions The constant name '_resourceHistoryService' doesn't match '[A-Z][A-Z_0-9]*' 1 65
FieldNamingConventions The constant name '_taskConfigService' doesn't match '[A-Z][A-Z_0-9]*' 1 66
FieldNamingConventions The constant name '_workflowService' doesn't match '[A-Z][A-Z_0-9]*' 1 67
FieldNamingConventions The constant name '_progressManagerService' doesn't match '[A-Z][A-Z_0-9]*' 1 68
UnusedPrivateField Avoid unused private fields such as 'PARAMETER_FEED_TOKEN'. 3 71
UnusedLocalVariable Avoid unused local variables such as 'parentId'. 3 78

fr/paris/lutece/plugins/workflow/modules/actionsbatch/task/ActionsBatchTaskConfig.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 40138

fr/paris/lutece/plugins/workflow/modules/actionsbatch/task/ActionsBatchTaskConfigDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 39111
ShortVariable Avoid variables with short names like i 3 89

fr/paris/lutece/plugins/workflow/modules/actionsbatch/task/AsynchronousSimpleTask.java

Rule Violation Priority Line
DoNotUseThreads To be compliant to J2EE, a webapp should not use any thread. 3 61
DefaultPackage Use explicit scoping instead of the default package private level 3 74
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the processAsynchronousTask method if you want a default access modifier 3 74

fr/paris/lutece/plugins/workflow/modules/actionsbatch/web/ActionsBatchTaskComponent.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34268
UnusedImports Avoid unused imports such as 'java.util.ArrayList' 4 36
UnusedPrivateField Avoid unused private fields such as 'MARK_WORKFLOWS'. 3 86
UnusedPrivateField Avoid unused private fields such as 'MARK_STATUS'. 3 87
UnusedPrivateField Avoid unused private fields such as 'MARK_ACTION'. 3 88
LocalVariableNamingConventions The local variable name '_actionService' doesn't match '[a-z][a-zA-Z0-9]*' 1 114
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 117
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 129
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 130
ShortVariable Avoid variables with short names like id 3 205