Fork me on GitHub
Built by Maven

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 1

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfigValueHome.java

Rule Violation Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 42–116

fr/paris/lutece/plugins/workflow/modules/forms/business/FormResponseValueStateControllerConfigHome.java

Rule Violation Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 40–93

fr/paris/lutece/plugins/workflow/modules/forms/service/task/AbstractEditFormsTask.java

Rule Violation Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 59
FieldNamingConventions The final field name '_editFormResponseTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 60
FieldNamingConventions The final field name '_editFormResponseTaskHistoryService' doesn't match '[a-z][a-zA-Z0-9]*' 61

fr/paris/lutece/plugins/workflow/modules/forms/service/task/DuplicateFormResponseTask.java

Rule Violation Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 42

fr/paris/lutece/plugins/workflow/modules/forms/service/task/EditFormResponseAutoUpdateTask.java

Rule Violation Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 67

fr/paris/lutece/plugins/workflow/modules/forms/service/task/EditFormResponseTaskHistoryService.java

Rule Violation Line
FieldNamingConventions The final field name '_editFormResponseTaskHistoryDAO' doesn't match '[a-z][a-zA-Z0-9]*' 49

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ModifyFormResponseUpdateDateTask.java

Rule Violation Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 52

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ModifyFormResponseUpdateStatusTask.java

Rule Violation Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 53

fr/paris/lutece/plugins/workflow/modules/forms/utils/EditableResponse.java

Rule Violation Line
FieldNamingConventions The final field name '_question' doesn't match '[a-z][a-zA-Z0-9]*' 42
FieldNamingConventions The final field name '_responseSaved' doesn't match '[a-z][a-zA-Z0-9]*' 43
FieldNamingConventions The final field name '_responseFromForm' doesn't match '[a-z][a-zA-Z0-9]*' 44

fr/paris/lutece/plugins/workflow/modules/forms/web/task/AbstractFormResponseTaskComponent.java

Rule Violation Line
AbstractClassWithoutAnyMethod No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 38–43

fr/paris/lutece/plugins/workflow/modules/forms/web/task/EditFormResponseAutoUpdateTaskComponent.java

Rule Violation Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 101
FieldNamingConventions The final field name '_editFormResponseTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 102

fr/paris/lutece/plugins/workflow/modules/forms/web/task/EditFormResponseTaskComponent.java

Rule Violation Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 115
FieldNamingConventions The final field name '_editFormResponseTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 116
FieldNamingConventions The final field name '_editFormResponseTaskHistoryService' doesn't match '[a-z][a-zA-Z0-9]*' 117

Priority 3

fr/paris/lutece/plugins/workflow/modules/forms/business/AbstractCompleteFormResponse.java

Rule Violation Line
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 38–146
CommentRequired Header comments are required 38–146

fr/paris/lutece/plugins/workflow/modules/forms/business/AbstractCompleteFormResponseTaskConfig.java

Rule Violation Line
CommentRequired Header comments are required 41–89

fr/paris/lutece/plugins/workflow/modules/forms/business/AbstractCompleteFormResponseValue.java

Rule Violation Line
CommentRequired Header comments are required 36–82
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 36–82

fr/paris/lutece/plugins/workflow/modules/forms/business/AbstractFormResponseTaskHistory.java

Rule Violation Line
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 38–113
CommentRequired Header comments are required 38–113

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponse.java

Rule Violation Line
CommentRequired Header comments are required 38–62

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 66–87
ConfusingTernary Avoid if (x != y) ..; else ..; 76–83
ConfusingTernary Avoid if (x != y) ..; else ..; 101–108
AssignmentInOperand Avoid assignments in operands 204

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseTaskConfigDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 58–70

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseTaskHistory.java

Rule Violation Line
CommentRequired Header comments are required 36–38

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseTaskHistoryDAO.java

Rule Violation Line
CommentRequired Header comments are required 44–122
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 56–69

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseValue.java

Rule Violation Line
CommentRequired Header comments are required 36–38

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseValueDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 57–68

fr/paris/lutece/plugins/workflow/modules/forms/business/DuplicateFormResponseTaskConfig.java

Rule Violation Line
CommentRequired Header comments are required 5–7

fr/paris/lutece/plugins/workflow/modules/forms/business/DuplicateFormResponseTaskConfigDAO.java

Rule Violation Line
CommentRequired Header comments are required 7–90

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfig.java

Rule Violation Line
CommentRequired Header comments are required 41–102
RedundantFieldInitializer Avoid using redundant field initializer for '_bMultiform' 44
CommentRequired Public method and constructor comments are required 81–84

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfigDao.java

Rule Violation Line
CommentRequired Header comments are required 41–123

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfigValue.java

Rule Violation Line
CommentRequired Header comments are required 40–161

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfigValueDao.java

Rule Violation Line
ConfusingTernary Avoid if (x != y) ..; else ..; 69–76
ConfusingTernary Avoid if (x != y) ..; else ..; 77–84
ConfusingTernary Avoid if (x != y) ..; else ..; 85–92
ConfusingTernary Avoid if (x != y) ..; else ..; 93–100
ConfusingTernary Avoid if (x != y) ..; else ..; 120–127
ConfusingTernary Avoid if (x != y) ..; else ..; 128–135
ConfusingTernary Avoid if (x != y) ..; else ..; 136–143
ConfusingTernary Avoid if (x != y) ..; else ..; 144–151

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfigValueHome.java

Rule Violation Line
CommentRequired Header comments are required 42–116

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseTaskHistory.java

Rule Violation Line
CommentRequired Header comments are required 36–57

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseTaskHistoryDAO.java

Rule Violation Line
CommentRequired Header comments are required 44–124
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 56–70

fr/paris/lutece/plugins/workflow/modules/forms/business/FormResponseValueStateControllerConfig.java

Rule Violation Line
CommentRequired Header comments are required 40–186
DataClass The class 'FormResponseValueStateControllerConfig' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=16, WMC=16) 40–186
ShortVariable Avoid variables with short names like id 63

fr/paris/lutece/plugins/workflow/modules/forms/business/FormResponseValueStateControllerConfigDao.java

Rule Violation Line
CommentRequired Header comments are required 44–173
ConfusingTernary Avoid if (x != y) ..; else ..; 59–66
ConfusingTernary Avoid if (x != y) ..; else ..; 67–74
ConfusingTernary Avoid if (x != y) ..; else ..; 75–82
ConfusingTernary Avoid if (x != y) ..; else ..; 98–105
ConfusingTernary Avoid if (x != y) ..; else ..; 106–113
ConfusingTernary Avoid if (x != y) ..; else ..; 114–121

fr/paris/lutece/plugins/workflow/modules/forms/business/FormResponseValueStateControllerConfigHome.java

Rule Violation Line
CommentRequired Header comments are required 40–93

fr/paris/lutece/plugins/workflow/modules/forms/business/ICompleteFormResponseTaskHistoryDAO.java

Rule Violation Line
CommentRequired Header comments are required 38–69

fr/paris/lutece/plugins/workflow/modules/forms/business/IEditFormResponseTaskHistoryDAO.java

Rule Violation Line
CommentRequired Header comments are required 38–70

fr/paris/lutece/plugins/workflow/modules/forms/business/IFormResponseValueStateControllerConfigDao.java

Rule Violation Line
CommentRequired Header comments are required 38–80

fr/paris/lutece/plugins/workflow/modules/forms/business/IModifyFormResponseUpdateStatusTaskConfigDAO.java

Rule Violation Line
CommentRequired Header comments are required 38–72

fr/paris/lutece/plugins/workflow/modules/forms/business/IResubmitFormResponseTaskHistoryDAO.java

Rule Violation Line
CommentRequired Header comments are required 38–69

fr/paris/lutece/plugins/workflow/modules/forms/business/ModifyFormResponseUpdateStatusTaskConfig.java

Rule Violation Line
CommentRequired Header comments are required 38–51
RedundantFieldInitializer Avoid using redundant field initializer for '_bPublished' 40
CommentRequired Public method and constructor comments are required 42–45
CommentRequired Public method and constructor comments are required 47–50

fr/paris/lutece/plugins/workflow/modules/forms/business/ModifyFormResponseUpdateStatusTaskConfigDAO.java

Rule Violation Line
CommentRequired Header comments are required 42–108

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponse.java

Rule Violation Line
CommentRequired Header comments are required 38–63

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 66–87
ConfusingTernary Avoid if (x != y) ..; else ..; 76–83
ConfusingTernary Avoid if (x != y) ..; else ..; 101–108
AssignmentInOperand Avoid assignments in operands 206

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseTaskConfigDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 59–71

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseTaskHistory.java

Rule Violation Line
CommentRequired Header comments are required 36–57

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseTaskHistoryDAO.java

Rule Violation Line
CommentRequired Header comments are required 44–121
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 53–67

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseValue.java

Rule Violation Line
CommentRequired Header comments are required 36–38

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseValueDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 57–68

fr/paris/lutece/plugins/workflow/modules/forms/service/AbstractFormResponseService.java

Rule Violation Line
CommentRequired Header comments are required 59–140
CommentRequired Protected method constructor comments are required 76–97
CommentRequired Protected method constructor comments are required 99–112
CommentRequired Protected method constructor comments are required 114
CommentRequired Protected method constructor comments are required 116–139
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 131–135

fr/paris/lutece/plugins/workflow/modules/forms/service/AbstractFormResponseStateController.java

Rule Violation Line
CommentRequired Header comments are required 66–276
CommentRequired Protected method constructor comments are required 108–139
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 112
CommentRequired Protected method constructor comments are required 190
CommentRequired Protected method constructor comments are required 192–202
CommentRequired Protected method constructor comments are required 204–221
CommentRequired Protected method constructor comments are required 223–234
CommentRequired Protected method constructor comments are required 236
CommentRequired Protected method constructor comments are required 238–275

fr/paris/lutece/plugins/workflow/modules/forms/service/CompleteFormResponseService.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–265
CommentRequired Header comments are required 70–265

fr/paris/lutece/plugins/workflow/modules/forms/service/FormResponseValueExistsStateController.java

Rule Violation Line
CommentRequired Header comments are required 51–108

fr/paris/lutece/plugins/workflow/modules/forms/service/FormResponseValueStateController.java

Rule Violation Line
CommentRequired Header comments are required 58–134
CommentRequired Protected method constructor comments are required 117–133
MissingOverride The method 'getResponseReferenceList(int)' is missing an @Override annotation. 117–133

fr/paris/lutece/plugins/workflow/modules/forms/service/ICompleteFormResponseService.java

Rule Violation Line
CommentRequired Header comments are required 48–165

fr/paris/lutece/plugins/workflow/modules/forms/service/ResubmitFormResponseService.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–310

fr/paris/lutece/plugins/workflow/modules/forms/service/provider/AbstractCompleteFormResponseTaskInfoProvider.java

Rule Violation Line
CommentRequired Header comments are required 56–172
CommentRequired Public method and constructor comments are required 80
CommentRequired Protected method constructor comments are required 125
CommentRequired Protected method constructor comments are required 127
CommentRequired Protected method constructor comments are required 129
CommentRequired Protected method constructor comments are required 131
CommentRequired Protected method constructor comments are required 133
ConfusingTernary Avoid if (x != y) ..; else ..; 146–158

fr/paris/lutece/plugins/workflow/modules/forms/service/provider/CompleteFormResponseMarkerProvider.java

Rule Violation Line
ShortVariable Avoid variables with short names like ID 59
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 66

fr/paris/lutece/plugins/workflow/modules/forms/service/provider/CompleteFormResponseTaskInfoProvider.java

Rule Violation Line
CommentRequired Header comments are required 50–108

fr/paris/lutece/plugins/workflow/modules/forms/service/provider/ResubmitFormResponseMarkerProvider.java

Rule Violation Line
ShortVariable Avoid variables with short names like ID 59
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 66

fr/paris/lutece/plugins/workflow/modules/forms/service/provider/ResubmitFormResponseTaskInfoProvider.java

Rule Violation Line
CommentRequired Header comments are required 50–108

fr/paris/lutece/plugins/workflow/modules/forms/service/signrequest/CompleteFormResponseRequestAuthenticatorService.java

Rule Violation Line
CommentRequired Header comments are required 39–60

fr/paris/lutece/plugins/workflow/modules/forms/service/signrequest/ResubmitFormResponseRequestAuthenticatorService.java

Rule Violation Line
CommentRequired Header comments are required 39–60

fr/paris/lutece/plugins/workflow/modules/forms/service/task/AbstractEditFormsTask.java

Rule Violation Line
UnusedFormalParameter Avoid unused method parameters such as 'locale'. 106

fr/paris/lutece/plugins/workflow/modules/forms/service/task/CompleteFormResponseTask.java

Rule Violation Line
CommentRequired Header comments are required 56–167
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 95–102

fr/paris/lutece/plugins/workflow/modules/forms/service/task/CompleteFormResponseTaskHistoryService.java

Rule Violation Line
CommentRequired Header comments are required 47–76

fr/paris/lutece/plugins/workflow/modules/forms/service/task/DuplicateFormResponseTask.java

Rule Violation Line
CommentRequired Header comments are required 29–198
UnusedPrivateField Avoid unused private fields such as '_config'. 33
CallSuperInConstructor It is a good practice to call super() in a constructor 45–48
CommentRequired Public method and constructor comments are required 45–48

fr/paris/lutece/plugins/workflow/modules/forms/service/task/EditFormResponseAutoUpdateTask.java

Rule Violation Line
CallSuperInConstructor It is a good practice to call super() in a constructor 86–89
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 104

fr/paris/lutece/plugins/workflow/modules/forms/service/task/EditFormResponseConfigService.java

Rule Violation Line
CommentRequired Header comments are required 42–91

fr/paris/lutece/plugins/workflow/modules/forms/service/task/EditFormResponseTaskHistoryService.java

Rule Violation Line
CommentRequired Header comments are required 47–87

fr/paris/lutece/plugins/workflow/modules/forms/service/task/EditFormResponseTaskService.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 152–160
MissingOverride The method 'saveResponses(FormResponse, List)' is missing an @Override annotation. 152–160

fr/paris/lutece/plugins/workflow/modules/forms/service/task/FormsTaskService.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–367
ConfusingTernary Avoid if (x != y) ..; else ..; 344–358
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 356
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 362

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ICompleteFormResponseTaskHistoryService.java

Rule Violation Line
CommentRequired Header comments are required 42–69

fr/paris/lutece/plugins/workflow/modules/forms/service/task/IEditFormResponseTaskHistoryService.java

Rule Violation Line
CommentRequired Header comments are required 42–69

fr/paris/lutece/plugins/workflow/modules/forms/service/task/IEditFormResponseTaskService.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 75

fr/paris/lutece/plugins/workflow/modules/forms/service/task/IFormsTaskService.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 99–100

fr/paris/lutece/plugins/workflow/modules/forms/service/task/IResubmitFormResponseTaskHistoryService.java

Rule Violation Line
CommentRequired Header comments are required 42–69

fr/paris/lutece/plugins/workflow/modules/forms/service/task/LinkedValuesFormResponseTaskService.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as '_taskLinkedValuesFormResponseConfigService'. 65

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ModifyFormResponseUpdateDateTask.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 55–58
CallSuperInConstructor It is a good practice to call super() in a constructor 55–58

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ModifyFormResponseUpdateStatusTask.java

Rule Violation Line
CommentRequired Header comments are required 51–95
CommentRequired Public method and constructor comments are required 63–66
CallSuperInConstructor It is a good practice to call super() in a constructor 63–66

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ResubmitFormResponseTask.java

Rule Violation Line
CommentRequired Header comments are required 56–167
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 95–102

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ResubmitFormResponseTaskHistoryService.java

Rule Violation Line
CommentRequired Header comments are required 47–76

fr/paris/lutece/plugins/workflow/modules/forms/utils/EditableResponse.java

Rule Violation Line
CommentRequired Header comments are required 39–90

fr/paris/lutece/plugins/workflow/modules/forms/utils/WorkflowFormsTaskException.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 45–48

fr/paris/lutece/plugins/workflow/modules/forms/web/AbstractFormResponseApp.java

Rule Violation Line
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 96
ConfusingTernary Avoid if (x != y) ..; else ..; 112–129
CommentRequired Protected method constructor comments are required 139–155
CommentRequired Protected method constructor comments are required 157–173
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 159
CommentRequired Protected method constructor comments are required 188
CommentRequired Protected method constructor comments are required 190
CommentRequired Protected method constructor comments are required 203

fr/paris/lutece/plugins/workflow/modules/forms/web/CompleteFormResponseApp.java

Rule Violation Line
CommentRequired Header comments are required 61–163
ImmutableField Private field '_completeFormResponseService' could be made final; it is only initialized in the declaration or constructor. 76

fr/paris/lutece/plugins/workflow/modules/forms/web/ResubmitFormResponseApp.java

Rule Violation Line
CommentRequired Header comments are required 66–220
ImmutableField Private field '_resubmitFormResponseService' could be made final; it is only initialized in the declaration or constructor. 87
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 92
ConfusingTernary Avoid if (x != y) ..; else ..; 108–125

fr/paris/lutece/plugins/workflow/modules/forms/web/task/AbstractFormResponseTaskComponent.java

Rule Violation Line
CommentRequired Header comments are required 38–43

fr/paris/lutece/plugins/workflow/modules/forms/web/task/CompleteFormResponseTaskComponent.java

Rule Violation Line
CommentRequired Header comments are required 62–152
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 92
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 114
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 128

fr/paris/lutece/plugins/workflow/modules/forms/web/task/DuplicateFormResponseTaskComponent.java

Rule Violation Line
CommentRequired Header comments are required 10–23

fr/paris/lutece/plugins/workflow/modules/forms/web/task/EditFormResponseAutoUpdateTaskComponent.java

Rule Violation Line
UnusedFormalParameter Avoid unused constructor parameters such as 'editFormResponseTaskHistoryService'. 116
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 146
CommentRequired Protected method constructor comments are required 178–181

fr/paris/lutece/plugins/workflow/modules/forms/web/task/EditFormResponseFOTaskComponent.java

Rule Violation Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 84

fr/paris/lutece/plugins/workflow/modules/forms/web/task/EditFormResponseTaskComponent.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–413
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 177–190
CommentRequired Protected method constructor comments are required 264–274
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 270
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 282
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 304
CommentRequired Protected method constructor comments are required 325–328

fr/paris/lutece/plugins/workflow/modules/forms/web/task/LinkedValuesFormResponseTaskComponent.java

Rule Violation Line
DefaultPackage Use explicit scoping instead of the default package private level 90–92
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the _configService field if you want a default access modifier 92
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 97
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 161

fr/paris/lutece/plugins/workflow/modules/forms/web/task/ModifyFormResponseUpdateStatusTaskComponent.java

Rule Violation Line
CommentRequired Header comments are required 51–84
SingularField Perhaps '_config' could be replaced by a local variable. 59
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/forms/web/task/ResubmitFormResponseTaskComponent.java

Rule Violation Line
CommentRequired Header comments are required 62–152
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 92
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 114
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 128

Files

fr/paris/lutece/plugins/workflow/modules/forms/business/AbstractCompleteFormResponse.java

Rule Violation Priority Line
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 3 38–146
CommentRequired Header comments are required 3 38–146

fr/paris/lutece/plugins/workflow/modules/forms/business/AbstractCompleteFormResponseTaskConfig.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 41–89

fr/paris/lutece/plugins/workflow/modules/forms/business/AbstractCompleteFormResponseValue.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 36–82
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 3 36–82

fr/paris/lutece/plugins/workflow/modules/forms/business/AbstractFormResponseTaskHistory.java

Rule Violation Priority Line
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 3 38–113
CommentRequired Header comments are required 3 38–113

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponse.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–62

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 66–87
ConfusingTernary Avoid if (x != y) ..; else ..; 3 76–83
ConfusingTernary Avoid if (x != y) ..; else ..; 3 101–108
AssignmentInOperand Avoid assignments in operands 3 204

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseTaskConfigDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 58–70

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseTaskHistory.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 36–38

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseTaskHistoryDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 44–122
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 56–69

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseValue.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 36–38

fr/paris/lutece/plugins/workflow/modules/forms/business/CompleteFormResponseValueDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 57–68

fr/paris/lutece/plugins/workflow/modules/forms/business/DuplicateFormResponseTaskConfig.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 5–7

fr/paris/lutece/plugins/workflow/modules/forms/business/DuplicateFormResponseTaskConfigDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 7–90

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfig.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 41–102
RedundantFieldInitializer Avoid using redundant field initializer for '_bMultiform' 3 44
CommentRequired Public method and constructor comments are required 3 81–84

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfigDao.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 41–123

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfigValue.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 40–161

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfigValueDao.java

Rule Violation Priority Line
ConfusingTernary Avoid if (x != y) ..; else ..; 3 69–76
ConfusingTernary Avoid if (x != y) ..; else ..; 3 77–84
ConfusingTernary Avoid if (x != y) ..; else ..; 3 85–92
ConfusingTernary Avoid if (x != y) ..; else ..; 3 93–100
ConfusingTernary Avoid if (x != y) ..; else ..; 3 120–127
ConfusingTernary Avoid if (x != y) ..; else ..; 3 128–135
ConfusingTernary Avoid if (x != y) ..; else ..; 3 136–143
ConfusingTernary Avoid if (x != y) ..; else ..; 3 144–151

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseConfigValueHome.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 42–116
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 42–116

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseTaskHistory.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 36–57

fr/paris/lutece/plugins/workflow/modules/forms/business/EditFormResponseTaskHistoryDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 44–124
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 56–70

fr/paris/lutece/plugins/workflow/modules/forms/business/FormResponseValueStateControllerConfig.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 40–186
DataClass The class 'FormResponseValueStateControllerConfig' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=16, WMC=16) 3 40–186
ShortVariable Avoid variables with short names like id 3 63

fr/paris/lutece/plugins/workflow/modules/forms/business/FormResponseValueStateControllerConfigDao.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 44–173
ConfusingTernary Avoid if (x != y) ..; else ..; 3 59–66
ConfusingTernary Avoid if (x != y) ..; else ..; 3 67–74
ConfusingTernary Avoid if (x != y) ..; else ..; 3 75–82
ConfusingTernary Avoid if (x != y) ..; else ..; 3 98–105
ConfusingTernary Avoid if (x != y) ..; else ..; 3 106–113
ConfusingTernary Avoid if (x != y) ..; else ..; 3 114–121

fr/paris/lutece/plugins/workflow/modules/forms/business/FormResponseValueStateControllerConfigHome.java

Rule Violation Priority Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 40–93
CommentRequired Header comments are required 3 40–93

fr/paris/lutece/plugins/workflow/modules/forms/business/ICompleteFormResponseTaskHistoryDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–69

fr/paris/lutece/plugins/workflow/modules/forms/business/IEditFormResponseTaskHistoryDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–70

fr/paris/lutece/plugins/workflow/modules/forms/business/IFormResponseValueStateControllerConfigDao.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–80

fr/paris/lutece/plugins/workflow/modules/forms/business/IModifyFormResponseUpdateStatusTaskConfigDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–72

fr/paris/lutece/plugins/workflow/modules/forms/business/IResubmitFormResponseTaskHistoryDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–69

fr/paris/lutece/plugins/workflow/modules/forms/business/ModifyFormResponseUpdateStatusTaskConfig.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–51
RedundantFieldInitializer Avoid using redundant field initializer for '_bPublished' 3 40
CommentRequired Public method and constructor comments are required 3 42–45
CommentRequired Public method and constructor comments are required 3 47–50

fr/paris/lutece/plugins/workflow/modules/forms/business/ModifyFormResponseUpdateStatusTaskConfigDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 42–108

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponse.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–63

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 66–87
ConfusingTernary Avoid if (x != y) ..; else ..; 3 76–83
ConfusingTernary Avoid if (x != y) ..; else ..; 3 101–108
AssignmentInOperand Avoid assignments in operands 3 206

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseTaskConfigDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 59–71

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseTaskHistory.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 36–57

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseTaskHistoryDAO.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 44–121
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 53–67

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseValue.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 36–38

fr/paris/lutece/plugins/workflow/modules/forms/business/ResubmitFormResponseValueDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 57–68

fr/paris/lutece/plugins/workflow/modules/forms/service/AbstractFormResponseService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 59–140
CommentRequired Protected method constructor comments are required 3 76–97
CommentRequired Protected method constructor comments are required 3 99–112
CommentRequired Protected method constructor comments are required 3 114
CommentRequired Protected method constructor comments are required 3 116–139
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 131–135

fr/paris/lutece/plugins/workflow/modules/forms/service/AbstractFormResponseStateController.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 66–276
CommentRequired Protected method constructor comments are required 3 108–139
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 112
CommentRequired Protected method constructor comments are required 3 190
CommentRequired Protected method constructor comments are required 3 192–202
CommentRequired Protected method constructor comments are required 3 204–221
CommentRequired Protected method constructor comments are required 3 223–234
CommentRequired Protected method constructor comments are required 3 236
CommentRequired Protected method constructor comments are required 3 238–275

fr/paris/lutece/plugins/workflow/modules/forms/service/CompleteFormResponseService.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–265
CommentRequired Header comments are required 3 70–265

fr/paris/lutece/plugins/workflow/modules/forms/service/FormResponseValueExistsStateController.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 51–108

fr/paris/lutece/plugins/workflow/modules/forms/service/FormResponseValueStateController.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 58–134
CommentRequired Protected method constructor comments are required 3 117–133
MissingOverride The method 'getResponseReferenceList(int)' is missing an @Override annotation. 3 117–133

fr/paris/lutece/plugins/workflow/modules/forms/service/ICompleteFormResponseService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 48–165

fr/paris/lutece/plugins/workflow/modules/forms/service/ResubmitFormResponseService.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–310

fr/paris/lutece/plugins/workflow/modules/forms/service/provider/AbstractCompleteFormResponseTaskInfoProvider.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 56–172
CommentRequired Public method and constructor comments are required 3 80
CommentRequired Protected method constructor comments are required 3 125
CommentRequired Protected method constructor comments are required 3 127
CommentRequired Protected method constructor comments are required 3 129
CommentRequired Protected method constructor comments are required 3 131
CommentRequired Protected method constructor comments are required 3 133
ConfusingTernary Avoid if (x != y) ..; else ..; 3 146–158

fr/paris/lutece/plugins/workflow/modules/forms/service/provider/CompleteFormResponseMarkerProvider.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like ID 3 59
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 66

fr/paris/lutece/plugins/workflow/modules/forms/service/provider/CompleteFormResponseTaskInfoProvider.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 50–108

fr/paris/lutece/plugins/workflow/modules/forms/service/provider/ResubmitFormResponseMarkerProvider.java

Rule Violation Priority Line
ShortVariable Avoid variables with short names like ID 3 59
UnnecessaryAnnotationValueElement Avoid the use of value in annotations when its the only element 3 66

fr/paris/lutece/plugins/workflow/modules/forms/service/provider/ResubmitFormResponseTaskInfoProvider.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 50–108

fr/paris/lutece/plugins/workflow/modules/forms/service/signrequest/CompleteFormResponseRequestAuthenticatorService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 39–60

fr/paris/lutece/plugins/workflow/modules/forms/service/signrequest/ResubmitFormResponseRequestAuthenticatorService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 39–60

fr/paris/lutece/plugins/workflow/modules/forms/service/task/AbstractEditFormsTask.java

Rule Violation Priority Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 1 59
FieldNamingConventions The final field name '_editFormResponseTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 1 60
FieldNamingConventions The final field name '_editFormResponseTaskHistoryService' doesn't match '[a-z][a-zA-Z0-9]*' 1 61
UnusedFormalParameter Avoid unused method parameters such as 'locale'. 3 106

fr/paris/lutece/plugins/workflow/modules/forms/service/task/CompleteFormResponseTask.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 56–167
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 95–102

fr/paris/lutece/plugins/workflow/modules/forms/service/task/CompleteFormResponseTaskHistoryService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 47–76

fr/paris/lutece/plugins/workflow/modules/forms/service/task/DuplicateFormResponseTask.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 29–198
UnusedPrivateField Avoid unused private fields such as '_config'. 3 33
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 1 42
CallSuperInConstructor It is a good practice to call super() in a constructor 3 45–48
CommentRequired Public method and constructor comments are required 3 45–48

fr/paris/lutece/plugins/workflow/modules/forms/service/task/EditFormResponseAutoUpdateTask.java

Rule Violation Priority Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 1 67
CallSuperInConstructor It is a good practice to call super() in a constructor 3 86–89
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 104

fr/paris/lutece/plugins/workflow/modules/forms/service/task/EditFormResponseConfigService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 42–91

fr/paris/lutece/plugins/workflow/modules/forms/service/task/EditFormResponseTaskHistoryService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 47–87
FieldNamingConventions The final field name '_editFormResponseTaskHistoryDAO' doesn't match '[a-z][a-zA-Z0-9]*' 1 49

fr/paris/lutece/plugins/workflow/modules/forms/service/task/EditFormResponseTaskService.java

Rule Violation Priority Line
CommentRequired Public method and constructor comments are required 3 152–160
MissingOverride The method 'saveResponses(FormResponse, List)' is missing an @Override annotation. 3 152–160

fr/paris/lutece/plugins/workflow/modules/forms/service/task/FormsTaskService.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–367
ConfusingTernary Avoid if (x != y) ..; else ..; 3 344–358
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 356
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 362

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ICompleteFormResponseTaskHistoryService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 42–69

fr/paris/lutece/plugins/workflow/modules/forms/service/task/IEditFormResponseTaskHistoryService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 42–69

fr/paris/lutece/plugins/workflow/modules/forms/service/task/IEditFormResponseTaskService.java

Rule Violation Priority Line
CommentRequired Public method and constructor comments are required 3 75

fr/paris/lutece/plugins/workflow/modules/forms/service/task/IFormsTaskService.java

Rule Violation Priority Line
CommentRequired Public method and constructor comments are required 3 99–100

fr/paris/lutece/plugins/workflow/modules/forms/service/task/IResubmitFormResponseTaskHistoryService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 42–69

fr/paris/lutece/plugins/workflow/modules/forms/service/task/LinkedValuesFormResponseTaskService.java

Rule Violation Priority Line
UnusedPrivateField Avoid unused private fields such as '_taskLinkedValuesFormResponseConfigService'. 3 65

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ModifyFormResponseUpdateDateTask.java

Rule Violation Priority Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 1 52
CommentRequired Public method and constructor comments are required 3 55–58
CallSuperInConstructor It is a good practice to call super() in a constructor 3 55–58

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ModifyFormResponseUpdateStatusTask.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 51–95
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 1 53
CommentRequired Public method and constructor comments are required 3 63–66
CallSuperInConstructor It is a good practice to call super() in a constructor 3 63–66

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ResubmitFormResponseTask.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 56–167
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 95–102

fr/paris/lutece/plugins/workflow/modules/forms/service/task/ResubmitFormResponseTaskHistoryService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 47–76

fr/paris/lutece/plugins/workflow/modules/forms/utils/EditableResponse.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 39–90
FieldNamingConventions The final field name '_question' doesn't match '[a-z][a-zA-Z0-9]*' 1 42
FieldNamingConventions The final field name '_responseSaved' doesn't match '[a-z][a-zA-Z0-9]*' 1 43
FieldNamingConventions The final field name '_responseFromForm' doesn't match '[a-z][a-zA-Z0-9]*' 1 44

fr/paris/lutece/plugins/workflow/modules/forms/utils/WorkflowFormsTaskException.java

Rule Violation Priority Line
CommentRequired Public method and constructor comments are required 3 45–48

fr/paris/lutece/plugins/workflow/modules/forms/web/AbstractFormResponseApp.java

Rule Violation Priority Line
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 96
ConfusingTernary Avoid if (x != y) ..; else ..; 3 112–129
CommentRequired Protected method constructor comments are required 3 139–155
CommentRequired Protected method constructor comments are required 3 157–173
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 159
CommentRequired Protected method constructor comments are required 3 188
CommentRequired Protected method constructor comments are required 3 190
CommentRequired Protected method constructor comments are required 3 203

fr/paris/lutece/plugins/workflow/modules/forms/web/CompleteFormResponseApp.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 61–163
ImmutableField Private field '_completeFormResponseService' could be made final; it is only initialized in the declaration or constructor. 3 76

fr/paris/lutece/plugins/workflow/modules/forms/web/ResubmitFormResponseApp.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 66–220
ImmutableField Private field '_resubmitFormResponseService' could be made final; it is only initialized in the declaration or constructor. 3 87
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 92
ConfusingTernary Avoid if (x != y) ..; else ..; 3 108–125

fr/paris/lutece/plugins/workflow/modules/forms/web/task/AbstractFormResponseTaskComponent.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–43
AbstractClassWithoutAnyMethod No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 1 38–43

fr/paris/lutece/plugins/workflow/modules/forms/web/task/CompleteFormResponseTaskComponent.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 62–152
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 92
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 114
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 128

fr/paris/lutece/plugins/workflow/modules/forms/web/task/DuplicateFormResponseTaskComponent.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 10–23

fr/paris/lutece/plugins/workflow/modules/forms/web/task/EditFormResponseAutoUpdateTaskComponent.java

Rule Violation Priority Line
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 1 101
FieldNamingConventions The final field name '_editFormResponseTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 1 102
UnusedFormalParameter Avoid unused constructor parameters such as 'editFormResponseTaskHistoryService'. 3 116
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 146
CommentRequired Protected method constructor comments are required 3 178–181

fr/paris/lutece/plugins/workflow/modules/forms/web/task/EditFormResponseFOTaskComponent.java

Rule Violation Priority Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 84

fr/paris/lutece/plugins/workflow/modules/forms/web/task/EditFormResponseTaskComponent.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–413
FieldNamingConventions The final field name '_formsTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 1 115
FieldNamingConventions The final field name '_editFormResponseTaskService' doesn't match '[a-z][a-zA-Z0-9]*' 1 116
FieldNamingConventions The final field name '_editFormResponseTaskHistoryService' doesn't match '[a-z][a-zA-Z0-9]*' 1 117
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 177–190
CommentRequired Protected method constructor comments are required 3 264–274
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 270
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 282
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 304
CommentRequired Protected method constructor comments are required 3 325–328

fr/paris/lutece/plugins/workflow/modules/forms/web/task/LinkedValuesFormResponseTaskComponent.java

Rule Violation Priority Line
DefaultPackage Use explicit scoping instead of the default package private level 3 90–92
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the _configService field if you want a default access modifier 3 92
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 97
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 161

fr/paris/lutece/plugins/workflow/modules/forms/web/task/ModifyFormResponseUpdateStatusTaskComponent.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 51–84
SingularField Perhaps '_config' could be replaced by a local variable. 3 59
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 70

fr/paris/lutece/plugins/workflow/modules/forms/web/task/ResubmitFormResponseTaskComponent.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 62–152
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 92
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 114
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 128