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/identityimport/mapper/IdentityMapper.java

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

fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTask.java

Rule Violation Line
FieldNamingConventions The constant name '_resourceHistoryService' doesn't match '[A-Z][A-Z_0-9]*' 71
FieldNamingConventions The field name 'identityService' doesn't match '_[a-z][a-zA-Z0-9]*' 73

Priority 3

fr/paris/lutece/plugins/workflow/modules/identityimport/mapper/IdentityMapper.java

Rule Violation Line
CommentRequired Header comments are required 4365
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. 4465
CommentRequired Public method and constructor comments are required 4664

fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTask.java

Rule Violation Line
CommentRequired Header comments are required 64141
ImmutableField Private field 'identityService' could be made final; it is only initialized in the declaration or constructor. 73

fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTaskConfig.java

Rule Violation Line
CommentRequired Header comments are required 40135

fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTaskConfigDAO.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/identityimport/web/IdentityImportTaskComponent.java

Rule Violation Line
CommentRequired Header comments are required 70225
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 98
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 110
ShortVariable Avoid variables with short names like id 173

Priority 4

fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTask.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'org.springframework.core.SpringProperties' 62

Files

fr/paris/lutece/plugins/workflow/modules/identityimport/mapper/IdentityMapper.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'IdentityMapper' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 4365
CommentRequired Header comments are required 3 4365
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 4465
CommentRequired Public method and constructor comments are required 3 4664

fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTask.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'org.springframework.core.SpringProperties' 4 62
CommentRequired Header comments are required 3 64141
FieldNamingConventions The constant name '_resourceHistoryService' doesn't match '[A-Z][A-Z_0-9]*' 1 71
FieldNamingConventions The field name 'identityService' doesn't match '_[a-z][a-zA-Z0-9]*' 1 73
ImmutableField Private field 'identityService' could be made final; it is only initialized in the declaration or constructor. 3 73

fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTaskConfig.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 40135

fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTaskConfigDAO.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/identityimport/web/IdentityImportTaskComponent.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 70225
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 98
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 110
ShortVariable Avoid variables with short names like id 3 173