PMD Results
The following document contains the results of PMD 6.13.0.
Violations By Priority
Priority 1
fr/paris/lutece/plugins/workflow/modules/unittree/business/task/information/TaskInformation.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/UnitAssignmentService.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/UnitSelectionService.java
Rule |
Violation |
Line |
FieldNamingConventions |
The final field name '_listUnitSelections' doesn't match '[a-z][a-zA-Z0-9]*' |
52 |
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/AbstractEmptyConfigurationHandler.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/ParametrableUnitSelection.java
Rule |
Violation |
Line |
FieldNamingConventions |
The final field name '_configurationHandler' doesn't match '[a-z][a-zA-Z0-9]*' |
65 |
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionAssignor.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionFromAll.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionParent.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionSpecificUnit.java
fr/paris/lutece/plugins/workflow/modules/unittree/util/ChangeUnitEvent.java
Rule |
Violation |
Line |
FieldNamingConventions |
The final field name '_oldAssignmentList' doesn't match '[a-z][a-zA-Z0-9]*' |
49 |
fr/paris/lutece/plugins/workflow/modules/unittree/util/WorkflowUnittreeConstants.java
Rule |
Violation |
Line |
ClassNamingConventions |
The utility class name 'WorkflowUnittreeConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' |
40–51 |
fr/paris/lutece/plugins/workflow/modules/unittree/web/task/AbstractUnitAssignmentTaskComponent.java
fr/paris/lutece/plugins/workflow/modules/unittree/web/task/AbstractUnittreeTaskComponent.java
Priority 3
fr/paris/lutece/plugins/workflow/modules/unittree/business/assignment/task/selection/config/ParametrableUnitSelectionConfig.java
fr/paris/lutece/plugins/workflow/modules/unittree/business/assignment/task/selection/config/ParametrableUnitSelectionConfigDao.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/prerequisite/ChildUnitAssignedResourcePrerequisite.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/prerequisite/UserUnitAssignedResourcePrerequisite.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/AbstractTaskUnitAssignment.java
Rule |
Violation |
Line |
CyclomaticComplexity |
The method 'buildUnitAssignmentListToDeactivate(List, UnitAssignment)' has a cyclomatic complexity of 13. |
278–335 |
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/UnitSelectionService.java
Rule |
Violation |
Line |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
111 |
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/AbstractEmptyConfigurationHandler.java
Rule |
Violation |
Line |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
61 |
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/ParametrableUnitSelection.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionAssignor.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like ID |
64 |
AccessorMethodGeneration |
Avoid autogenerated methods to access private fields and methods of inner / outer classes |
176 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
178 |
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionFromAll.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like ID |
64 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
180 |
AccessorMethodGeneration |
Avoid autogenerated methods to access private fields and methods of inner / outer classes |
197 |
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionParent.java
Rule |
Violation |
Line |
ShortVariable |
Avoid variables with short names like ID |
62 |
AccessorMethodGeneration |
Avoid autogenerated methods to access private fields and methods of inner / outer classes |
179 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
181 |
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionSpecificUnit.java
fr/paris/lutece/plugins/workflow/modules/unittree/util/ChangeUnitEvent.java
fr/paris/lutece/plugins/workflow/modules/unittree/util/ChangeUnitEventPublisher.java
Rule |
Violation |
Line |
CommentRequired |
Public method and constructor comments are required |
51–54 |
MissingOverride |
The method 'setApplicationEventPublisher(ApplicationEventPublisher)' is missing an @Override annotation. |
51–54 |
CommentRequired |
Public method and constructor comments are required |
56–59 |
fr/paris/lutece/plugins/workflow/modules/unittree/web/task/AbstractUnitAssignmentTaskComponent.java
Rule |
Violation |
Line |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
113 |
fr/paris/lutece/plugins/workflow/modules/unittree/web/task/AbstractUnittreeTaskComponent.java
Rule |
Violation |
Line |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
70 |
fr/paris/lutece/plugins/workflow/modules/unittree/web/task/UnitAssignmentAutomaticTaskComponent.java
Rule |
Violation |
Line |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
73 |
fr/paris/lutece/plugins/workflow/modules/unittree/web/task/UnitAssignmentManualTaskComponent.java
Rule |
Violation |
Line |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
100 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
118 |
Files
fr/paris/lutece/plugins/workflow/modules/unittree/business/assignment/task/selection/config/ParametrableUnitSelectionConfig.java
fr/paris/lutece/plugins/workflow/modules/unittree/business/assignment/task/selection/config/ParametrableUnitSelectionConfigDao.java
fr/paris/lutece/plugins/workflow/modules/unittree/business/task/information/TaskInformation.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/UnitAssignmentService.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/prerequisite/ChildUnitAssignedResourcePrerequisite.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/prerequisite/UserUnitAssignedResourcePrerequisite.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/AbstractTaskUnitAssignment.java
Rule |
Violation |
Priority |
Line |
CyclomaticComplexity |
The method 'buildUnitAssignmentListToDeactivate(List, UnitAssignment)' has a cyclomatic complexity of 13. |
3 |
278–335 |
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/UnitSelectionService.java
Rule |
Violation |
Priority |
Line |
FieldNamingConventions |
The final field name '_listUnitSelections' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
52 |
AccessorClassGeneration |
Avoid instantiation through private constructors from outside of the constructors class. |
3 |
111 |
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/AbstractEmptyConfigurationHandler.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/ParametrableUnitSelection.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionAssignor.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionFromAll.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionParent.java
fr/paris/lutece/plugins/workflow/modules/unittree/service/task/selection/impl/UnitSelectionSpecificUnit.java
fr/paris/lutece/plugins/workflow/modules/unittree/util/ChangeUnitEvent.java
fr/paris/lutece/plugins/workflow/modules/unittree/util/ChangeUnitEventPublisher.java
Rule |
Violation |
Priority |
Line |
CommentRequired |
Public method and constructor comments are required |
3 |
51–54 |
MissingOverride |
The method 'setApplicationEventPublisher(ApplicationEventPublisher)' is missing an @Override annotation. |
3 |
51–54 |
CommentRequired |
Public method and constructor comments are required |
3 |
56–59 |
fr/paris/lutece/plugins/workflow/modules/unittree/util/WorkflowUnittreeConstants.java
Rule |
Violation |
Priority |
Line |
ClassNamingConventions |
The utility class name 'WorkflowUnittreeConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' |
1 |
40–51 |
fr/paris/lutece/plugins/workflow/modules/unittree/web/task/AbstractUnitAssignmentTaskComponent.java
fr/paris/lutece/plugins/workflow/modules/unittree/web/task/AbstractUnittreeTaskComponent.java
fr/paris/lutece/plugins/workflow/modules/unittree/web/task/UnitAssignmentAutomaticTaskComponent.java
Rule |
Violation |
Priority |
Line |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
3 |
73 |
fr/paris/lutece/plugins/workflow/modules/unittree/web/task/UnitAssignmentManualTaskComponent.java
Rule |
Violation |
Priority |
Line |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
3 |
100 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
3 |
118 |