Résultats PMD
Le document suivant contient les résultats de PMD 6.13.0.
Violations By Priority
Priority 1
fr/paris/lutece/plugins/workflowcore/service/action/ActionStateService.java
fr/paris/lutece/plugins/workflowcore/service/action/IActionStateService.java
fr/paris/lutece/plugins/workflowcore/service/provider/AbstractProviderManager.java
fr/paris/lutece/plugins/workflowcore/service/provider/ProviderDescription.java
fr/paris/lutece/plugins/workflowcore/service/task/SimpleTask.java
fr/paris/lutece/plugins/workflowcore/web/task/SimpleTaskComponent.java
Priority 2
fr/paris/lutece/plugins/workflowcore/service/config/TaskConfigService.java
Rule |
Violation |
Ligne |
LoggerIsNotStaticFinal |
The Logger variable declaration does not contain the static and final modifiers |
51 |
fr/paris/lutece/plugins/workflowcore/web/task/TaskComponent.java
Rule |
Violation |
Ligne |
LoggerIsNotStaticFinal |
The Logger variable declaration does not contain the static and final modifiers |
64 |
Priority 3
fr/paris/lutece/plugins/workflowcore/business/action/Action.java
fr/paris/lutece/plugins/workflowcore/business/action/ActionFilter.java
Rule |
Violation |
Ligne |
LinguisticNaming |
Linguistics Antipattern - The method 'isAutomaticReflexiveAction' indicates linguistically it returns a boolean, but it returns 'int' |
244–247 |
fr/paris/lutece/plugins/workflowcore/business/action/IActionDAO.java
Rule |
Violation |
Ligne |
TooManyMethods |
This class has too many methods, consider refactoring it. |
43–221 |
fr/paris/lutece/plugins/workflowcore/business/action/IActionStateDAO.java
fr/paris/lutece/plugins/workflowcore/business/icon/Icon.java
fr/paris/lutece/plugins/workflowcore/business/resource/IResourceHistoryDAO.java
Rule |
Violation |
Ligne |
AvoidDuplicateLiterals |
The String literal "Not supported yet." appears 4 times in this file; the first occurrence is on line 127 |
127 |
fr/paris/lutece/plugins/workflowcore/business/resource/IResourceWorkflowDAO.java
Rule |
Violation |
Ligne |
TooManyMethods |
This class has too many methods, consider refactoring it. |
47–226 |
fr/paris/lutece/plugins/workflowcore/business/resource/ResourceUserHistory.java
fr/paris/lutece/plugins/workflowcore/business/state/IStateDAO.java
Rule |
Violation |
Ligne |
TooManyMethods |
This class has too many methods, consider refactoring it. |
44–151 |
fr/paris/lutece/plugins/workflowcore/business/state/State.java
fr/paris/lutece/plugins/workflowcore/business/workflow/Workflow.java
fr/paris/lutece/plugins/workflowcore/business/workflow/WorkflowFilter.java
fr/paris/lutece/plugins/workflowcore/exception/WorkflowRuntimeException.java
fr/paris/lutece/plugins/workflowcore/service/action/ActionService.java
fr/paris/lutece/plugins/workflowcore/service/action/ActionStateService.java
fr/paris/lutece/plugins/workflowcore/service/action/IActionService.java
Rule |
Violation |
Ligne |
TooManyMethods |
This class has too many methods, consider refactoring it. |
48–226 |
fr/paris/lutece/plugins/workflowcore/service/action/IActionStateService.java
Rule |
Violation |
Ligne |
CommentRequired |
Header comments are required |
5–54 |
UnnecessaryModifier |
Unnecessary modifiers 'public static final' on field 'BEAN_SERVICE': the field is declared in an interface type |
7 |
fr/paris/lutece/plugins/workflowcore/service/config/ITaskConfigService.java
Rule |
Violation |
Ligne |
CommentRequired |
Public method and constructor comments are required |
69 |
fr/paris/lutece/plugins/workflowcore/service/config/TaskConfigService.java
Rule |
Violation |
Ligne |
AvoidCatchingGenericException |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
124 |
fr/paris/lutece/plugins/workflowcore/service/prerequisite/IPrerequisiteManagementService.java
fr/paris/lutece/plugins/workflowcore/service/provider/InfoMarker.java
Rule |
Violation |
Ligne |
ImmutableField |
Private field '_strMarker' could be made final; it is only initialized in the declaration or constructor. |
52 |
fr/paris/lutece/plugins/workflowcore/service/resource/IResourceHistoryService.java
Rule |
Violation |
Ligne |
AvoidDuplicateLiterals |
The String literal "Not supported yet." appears 4 times in this file; the first occurrence is on line 144 |
144 |
fr/paris/lutece/plugins/workflowcore/service/task/AsynchronousSimpleTask.java
Rule |
Violation |
Ligne |
DoNotUseThreads |
To be compliant to J2EE, a webapp should not use any thread. |
61 |
fr/paris/lutece/plugins/workflowcore/service/task/ITask.java
Rule |
Violation |
Ligne |
UnnecessaryModifier |
Unnecessary modifier 'public' on method 'getActionUid': the method is declared in an interface type |
88 |
UnnecessaryModifier |
Unnecessary modifier 'public' on method 'setActionUid': the method is declared in an interface type |
94 |
fr/paris/lutece/plugins/workflowcore/service/task/SimpleTask.java
fr/paris/lutece/plugins/workflowcore/service/task/Task.java
fr/paris/lutece/plugins/workflowcore/service/workflow/IWorkflowService.java
Rule |
Violation |
Ligne |
TooManyMethods |
This class has too many methods, consider refactoring it. |
54–406 |
fr/paris/lutece/plugins/workflowcore/service/workflow/WorkflowService.java
Rule |
Violation |
Ligne |
ExcessiveImports |
A high number of imports can indicate a high degree of coupling within an object. |
34–784 |
TooManyMethods |
This class has too many methods, consider refactoring it. |
80–784 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
186–203 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
223 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
249 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
365 |
CyclomaticComplexity |
The method 'doProcessAction(int, String, int, Integer, HttpServletRequest, Locale, boolean, String, User)' has a cyclomatic complexity of 17. |
457–556 |
NPathComplexity |
The method 'doProcessAction(int, String, int, Integer, HttpServletRequest, Locale, boolean, String, User)' has an NPath complexity of 1152 |
457–556 |
AvoidCatchingGenericException |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
502 |
fr/paris/lutece/plugins/workflowcore/web/task/TaskComponent.java
Priority 4
fr/paris/lutece/plugins/workflowcore/business/icon/Icon.java
fr/paris/lutece/plugins/workflowcore/business/resource/IResourceWorkflowDAO.java
Rule |
Violation |
Ligne |
UnusedImports |
Avoid unused imports such as 'fr.paris.lutece.plugins.workflowcore.business.state.State' |
39 |
fr/paris/lutece/plugins/workflowcore/service/task/Task.java
Fichiers
fr/paris/lutece/plugins/workflowcore/business/action/Action.java
fr/paris/lutece/plugins/workflowcore/business/action/ActionFilter.java
Rule |
Violation |
Priority |
Ligne |
LinguisticNaming |
Linguistics Antipattern - The method 'isAutomaticReflexiveAction' indicates linguistically it returns a boolean, but it returns 'int' |
3 |
244–247 |
fr/paris/lutece/plugins/workflowcore/business/action/IActionDAO.java
Rule |
Violation |
Priority |
Ligne |
TooManyMethods |
This class has too many methods, consider refactoring it. |
3 |
43–221 |
fr/paris/lutece/plugins/workflowcore/business/action/IActionStateDAO.java
fr/paris/lutece/plugins/workflowcore/business/icon/Icon.java
fr/paris/lutece/plugins/workflowcore/business/resource/IResourceHistoryDAO.java
Rule |
Violation |
Priority |
Ligne |
AvoidDuplicateLiterals |
The String literal "Not supported yet." appears 4 times in this file; the first occurrence is on line 127 |
3 |
127 |
fr/paris/lutece/plugins/workflowcore/business/resource/IResourceWorkflowDAO.java
Rule |
Violation |
Priority |
Ligne |
UnusedImports |
Avoid unused imports such as 'fr.paris.lutece.plugins.workflowcore.business.state.State' |
4 |
39 |
TooManyMethods |
This class has too many methods, consider refactoring it. |
3 |
47–226 |
fr/paris/lutece/plugins/workflowcore/business/resource/ResourceUserHistory.java
fr/paris/lutece/plugins/workflowcore/business/state/IStateDAO.java
Rule |
Violation |
Priority |
Ligne |
TooManyMethods |
This class has too many methods, consider refactoring it. |
3 |
44–151 |
fr/paris/lutece/plugins/workflowcore/business/state/State.java
fr/paris/lutece/plugins/workflowcore/business/workflow/Workflow.java
fr/paris/lutece/plugins/workflowcore/business/workflow/WorkflowFilter.java
fr/paris/lutece/plugins/workflowcore/exception/WorkflowRuntimeException.java
fr/paris/lutece/plugins/workflowcore/service/action/ActionService.java
fr/paris/lutece/plugins/workflowcore/service/action/ActionStateService.java
fr/paris/lutece/plugins/workflowcore/service/action/IActionService.java
Rule |
Violation |
Priority |
Ligne |
TooManyMethods |
This class has too many methods, consider refactoring it. |
3 |
48–226 |
fr/paris/lutece/plugins/workflowcore/service/action/IActionStateService.java
fr/paris/lutece/plugins/workflowcore/service/config/ITaskConfigService.java
Rule |
Violation |
Priority |
Ligne |
CommentRequired |
Public method and constructor comments are required |
3 |
69 |
fr/paris/lutece/plugins/workflowcore/service/config/TaskConfigService.java
Rule |
Violation |
Priority |
Ligne |
LoggerIsNotStaticFinal |
The Logger variable declaration does not contain the static and final modifiers |
2 |
51 |
AvoidCatchingGenericException |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
3 |
124 |
fr/paris/lutece/plugins/workflowcore/service/prerequisite/IPrerequisiteManagementService.java
fr/paris/lutece/plugins/workflowcore/service/provider/AbstractProviderManager.java
Rule |
Violation |
Priority |
Ligne |
FieldNamingConventions |
The final field name '_strId' doesn't match '[a-z][a-zA-Z0-9]*' |
1 |
49 |
fr/paris/lutece/plugins/workflowcore/service/provider/InfoMarker.java
Rule |
Violation |
Priority |
Ligne |
ImmutableField |
Private field '_strMarker' could be made final; it is only initialized in the declaration or constructor. |
3 |
52 |
fr/paris/lutece/plugins/workflowcore/service/provider/ProviderDescription.java
fr/paris/lutece/plugins/workflowcore/service/resource/IResourceHistoryService.java
Rule |
Violation |
Priority |
Ligne |
AvoidDuplicateLiterals |
The String literal "Not supported yet." appears 4 times in this file; the first occurrence is on line 144 |
3 |
144 |
fr/paris/lutece/plugins/workflowcore/service/task/AsynchronousSimpleTask.java
Rule |
Violation |
Priority |
Ligne |
DoNotUseThreads |
To be compliant to J2EE, a webapp should not use any thread. |
3 |
61 |
fr/paris/lutece/plugins/workflowcore/service/task/ITask.java
Rule |
Violation |
Priority |
Ligne |
UnnecessaryModifier |
Unnecessary modifier 'public' on method 'getActionUid': the method is declared in an interface type |
3 |
88 |
UnnecessaryModifier |
Unnecessary modifier 'public' on method 'setActionUid': the method is declared in an interface type |
3 |
94 |
fr/paris/lutece/plugins/workflowcore/service/task/SimpleTask.java
fr/paris/lutece/plugins/workflowcore/service/task/Task.java
fr/paris/lutece/plugins/workflowcore/service/workflow/IWorkflowService.java
Rule |
Violation |
Priority |
Ligne |
TooManyMethods |
This class has too many methods, consider refactoring it. |
3 |
54–406 |
fr/paris/lutece/plugins/workflowcore/service/workflow/WorkflowService.java
Rule |
Violation |
Priority |
Ligne |
ExcessiveImports |
A high number of imports can indicate a high degree of coupling within an object. |
3 |
34–784 |
TooManyMethods |
This class has too many methods, consider refactoring it. |
3 |
80–784 |
ConfusingTernary |
Avoid if (x != y) ..; else ..; |
3 |
186–203 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
3 |
223 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
3 |
249 |
UseConcurrentHashMap |
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation |
3 |
365 |
CyclomaticComplexity |
The method 'doProcessAction(int, String, int, Integer, HttpServletRequest, Locale, boolean, String, User)' has a cyclomatic complexity of 17. |
3 |
457–556 |
NPathComplexity |
The method 'doProcessAction(int, String, int, Integer, HttpServletRequest, Locale, boolean, String, User)' has an NPath complexity of 1152 |
3 |
457–556 |
AvoidCatchingGenericException |
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block |
3 |
502 |
fr/paris/lutece/plugins/workflowcore/web/task/SimpleTaskComponent.java
fr/paris/lutece/plugins/workflowcore/web/task/TaskComponent.java