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)' |
43–65 |
fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTask.java
Priority 3
fr/paris/lutece/plugins/workflow/modules/identityimport/mapper/IdentityMapper.java
Rule |
Violation |
Line |
CommentRequired |
Header comments are required |
43–65 |
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. |
44–65 |
CommentRequired |
Public method and constructor comments are required |
46–64 |
fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTask.java
Rule |
Violation |
Line |
CommentRequired |
Header comments are required |
64–141 |
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
fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTaskConfigDAO.java
fr/paris/lutece/plugins/workflow/modules/identityimport/web/IdentityImportTaskComponent.java
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 |
43–65 |
CommentRequired |
Header comments are required |
3 |
43–65 |
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 |
44–65 |
CommentRequired |
Public method and constructor comments are required |
3 |
46–64 |
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 |
64–141 |
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
fr/paris/lutece/plugins/workflow/modules/identityimport/task/IdentityImportTaskConfigDAO.java
fr/paris/lutece/plugins/workflow/modules/identityimport/web/IdentityImportTaskComponent.java
Rule |
Violation |
Priority |
Line |
CommentRequired |
Header comments are required |
3 |
70–225 |
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 |