The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 40–142 |
| FieldNamingConventions | The final field name '_stateBefore' doesn't match '[a-z][a-zA-Z0-9]*' | 42 |
| FieldNamingConventions | The final field name '_stateAfter' doesn't match '[a-z][a-zA-Z0-9]*' | 43 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 77–366 |
| LocalVariableNamingConventions | The local variable name 'StateBefore' doesn't match '[a-z][a-zA-Z0-9]*' | 268 |
| LocalVariableNamingConventions | The local variable name 'PrerequisiteTypeList' doesn't match '[a-z][a-zA-Z0-9]*' | 351 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_listMarkerProviders' doesn't match '[a-z][a-zA-Z0-9]*' | 53 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 51–235 |
| Rule | Violation | Line |
|---|---|---|
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 65–68 |
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 74–77 |
| Rule | Violation | Line |
|---|---|---|
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 53–56 |
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 62–65 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 412 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 412 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 412 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 412 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 481 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 481 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 481 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 481 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 481 |
| Rule | Violation | Line |
|---|---|---|
| AvoidReassigningParameters | Avoid reassigning parameters such as 'listActions' | 124 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'user' | 378 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'user' | 399 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 57–530 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 99–122 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 113–114 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 144–145 |
| CyclomaticComplexity | The method 'selectActionsByFilter(ActionFilter)' has a cyclomatic complexity of 16. | 212–301 |
| NPathComplexity | The method 'selectActionsByFilter(ActionFilter)' has an NPath complexity of 32896 | 212–301 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 307–317 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 44–129 |
| UnusedPrivateField | Avoid unused private fields such as 'SQL_QUERY_SELECT_ALL'. | 46 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 58–73 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 63–81 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 86–100 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 56–63 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 58–528 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 86–105 |
| ShortVariable | Avoid variables with short names like sb | 459 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 460 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 461 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 506–509 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 59–829 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 102–124 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 112–119 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 140–147 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 605 |
| CyclomaticComplexity | The method 'buildFilterQuerydFooter(DAOUtil, ResourceWorkflowFilter, List, Integer)' has a cyclomatic complexity of 12. | 654–716 |
| NPathComplexity | The method 'buildFilterQuerydFooter(DAOUtil, ResourceWorkflowFilter, List, Integer)' has an NPath complexity of 648 | 654–716 |
| NPathComplexity | The method 'buildFilterQuerydHeader(ResourceWorkflowFilter, List, String, String)' has an NPath complexity of 432 | 731–801 |
| CyclomaticComplexity | The method 'buildFilterQuerydHeader(ResourceWorkflowFilter, List, String, String)' has a cyclomatic complexity of 11. | 731–801 |
| ShortVariable | Avoid variables with short names like sb | 770 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 771 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 772 |
| ShortVariable | Avoid variables with short names like sb | 786 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 787 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 788 |
| AssignmentInOperand | Avoid assignments in operands | 818 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 818–825 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 54–369 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 78–107 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 77–93 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 71–89 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 40–142 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_action' | 45 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Enum comments are required | 36–40 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–126 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 56–68 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–152 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 100 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 128 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–167 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 58–65 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 67–74 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 109–116 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 118–125 |
| AssignmentInOperand | Avoid assignments in operands | 154 |
| AssignmentInOperand | Avoid assignments in operands | 159 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–76 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 67–142 |
| ImmutableField | Private field '_workflowService' could be made final; it is only initialized in the declaration or constructor. | 69 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 70 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 71 |
| ImmutableField | Private field '_resourceWorkflowService' could be made final; it is only initialized in the declaration or constructor. | 72 |
| ImmutableField | Private field '_archiveService' could be made final; it is only initialized in the declaration or constructor. | 73 |
| ShortVariable | Avoid variables with short names like wf | 98 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 114–118 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Protected method constructor comments are required | 58–61 |
| CommentRequired | Protected method constructor comments are required | 63–67 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–46 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 50–105 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 63 |
| CommentRequired | Public method and constructor comments are required | 79–85 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 66–236 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 111 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like id | 62 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 59–71 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 59–75 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 133–140 |
| Rule | Violation | Line |
|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 121–136 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 159 |
| ShortVariable | Avoid variables with short names like t | 162 |
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has a cyclomatic complexity of 13. | 115–205 |
| NPathComplexity | The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has an NPath complexity of 1152 | 115–205 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 119 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 121 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 172–192 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 180–188 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 213 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 214 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 279 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 295 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 329 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 57–69 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 82 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 94 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like id | 66 |
| ShortVariable | Avoid variables with short names like sb | 115 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 56–68 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 58–71 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 61–64 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 69–83 |
| ShortVariable | Avoid variables with short names like rt | 71 |
| ShortVariable | Avoid variables with short names like p | 77 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 92–95 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 100–103 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–209 |
| ImmutableField | Private field '_commentValueService' could be made final; it is only initialized in the declaration or constructor. | 96 |
| ImmutableField | Private field '_resourceHistoryService' could be made final; it is only initialized in the declaration or constructor. | 97 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 98 |
| ImmutableField | Private field '_taskComponentManager' could be made final; it is only initialized in the declaration or constructor. | 99 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 99–102 |
| CommentRequired | Public method and constructor comments are required | 104–108 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 116 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 117 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 148 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 164 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 199 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 57–69 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 80 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 92 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–73 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _nDuration field if you want a default access modifier | 46 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 46 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–111 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 55–131 |
| MissingOverride | The method 'getPrerequisiteType()' is missing an @Override annotation. | 66–69 |
| CommentRequired | Public method and constructor comments are required | 66–69 |
| MissingOverride | The method 'getTitleI18nKey()' is missing an @Override annotation. | 71–74 |
| CommentRequired | Public method and constructor comments are required | 71–74 |
| MissingOverride | The method 'hasConfiguration()' is missing an @Override annotation. | 76–79 |
| CommentRequired | Public method and constructor comments are required | 76–79 |
| CommentRequired | Public method and constructor comments are required | 81–84 |
| MissingOverride | The method 'getEmptyConfiguration()' is missing an @Override annotation. | 81–84 |
| MissingOverride | The method 'getConfigurationDaoBeanName()' is missing an @Override annotation. | 86–89 |
| CommentRequired | Public method and constructor comments are required | 86–89 |
| MissingOverride | The method 'getConfigHtml(IPrerequisiteConfig, HttpServletRequest, Locale)' is missing an @Override annotation. | 91–99 |
| CommentRequired | Public method and constructor comments are required | 91–99 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 93 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 59–73 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 96 |
| ShortVariable | Avoid variables with short names like t | 99 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 80 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–64 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–106 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–93 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–96 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–89 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–104 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 39–114 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–93 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–96 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–89 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–67 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–67 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 62–133 |
| ImmutableField | Private field '_changeStateTaskService' could be made final; it is only initialized in the declaration or constructor. | 65 |
| ImmutableField | Private field '_workflowService' could be made final; it is only initialized in the declaration or constructor. | 66 |
| ImmutableField | Private field '_resourceWorkflowService' could be made final; it is only initialized in the declaration or constructor. | 67 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 68 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 69 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 112 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 62–133 |
| ImmutableField | Private field '_chooseStateTaskService' could be made final; it is only initialized in the declaration or constructor. | 65 |
| ImmutableField | Private field '_workflowService' could be made final; it is only initialized in the declaration or constructor. | 66 |
| ImmutableField | Private field '_resourceWorkflowService' could be made final; it is only initialized in the declaration or constructor. | 67 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 68 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 69 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 112 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 58–136 |
| CommentRequired | Public method and constructor comments are required | 75–92 |
| CommentRequired | Public method and constructor comments are required | 94–103 |
| CommentRequired | Public method and constructor comments are required | 105–133 |
| CommentRequired | Protected method constructor comments are required | 135 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 47–78 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 46–75 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 48–88 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–80 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 43–119 |
| UncommentedEmptyMethodBody | Document empty method body | 108–109 |
| UncommentedEmptyMethodBody | Document empty method body | 117–118 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 52–94 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 70 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 82 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 82 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 53–122 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 77 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 110 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 110 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 73–76 |
| ShortVariable | Avoid variables with short names like rt | 84 |
| ShortVariable | Avoid variables with short names like p | 90 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 110–125 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 136–148 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 159 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 160 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 160 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 161 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 167 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 169 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 170 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 72–75 |
| ShortVariable | Avoid variables with short names like rt | 83 |
| ShortVariable | Avoid variables with short names like p | 89 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 139 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 9–37 |
| CommentRequired | Public method and constructor comments are required | 13–16 |
| CommentRequired | Public method and constructor comments are required | 18–21 |
| CommentRequired | Public method and constructor comments are required | 23–26 |
| CommentRequired | Header comments are required | 28–31 |
| CommentRequired | Public method and constructor comments are required | 33–36 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'WorkflowAppResourceIdService' is suspected to be a Data Class (WOC=0.000%, NOPA=4, NOAM=0, WMC=1) | 18–76 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 31–34 |
| CommentRequired | Public method and constructor comments are required | 31–34 |
| ShortVariable | Avoid variables with short names like rt | 39 |
| ShortVariable | Avoid variables with short names like p | 45 |
| ShortVariable | Avoid variables with short names like rl | 66 |
| Rule | Violation | Line |
|---|---|---|
| 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. | 52–151 |
| ShortVariable | Avoid variables with short names like wf | 74 |
| CyclomaticComplexity | The method 'generate(Workflow, String)' has a cyclomatic complexity of 16. | 74–129 |
| NPathComplexity | The method 'generate(Workflow, String)' has an NPath complexity of 600 | 74–129 |
| ShortVariable | Avoid variables with short names like sb | 76 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 123 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 124 |
| ShortVariable | Avoid variables with short names like sb | 139 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 139 |
| ControlStatementBraces | This statement should have braces | 142 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–645 |
| MissingOverride | The method 'getActions(int, String, Collection, User)' is missing an @Override annotation. | 124–129 |
| MissingOverride | The method 'getAuthorizedActions(Collection, User)' is missing an @Override annotation. | 135–138 |
| MissingOverride | The method 'getActions(String, Map, User)' is missing an @Override annotation. | 144–156 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 200 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 211–237 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 261–272 |
| MissingOverride | The method 'getDisplayDocumentHistory(int, String, int, HttpServletRequest, Locale, Map, String, User)' is missing an @Override annotation. | 321–334 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 356 |
| MissingOverride | The method 'canProcessAction(int, String, int, HttpServletRequest, User)' is missing an @Override annotation. | 399–414 |
| CyclomaticComplexity | The method 'isAuthorized(int, String, int, User)' has a cyclomatic complexity of 10. | 420–470 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 426–443 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 536 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 548–555 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 591 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Enum comments are required | 8–46 |
| CommentRequired | Enum comments are required | 21 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 21–134 |
| 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. | 22–134 |
| ShortVariable | Avoid variables with short names like id | 23 |
| ShortVariable | Avoid variables with short names like id | 32 |
| CommentRequired | Public method and constructor comments are required | 37–45 |
| CommentRequired | Public method and constructor comments are required | 47–57 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 49 |
| UnusedLocalVariable | Avoid unused local variables such as 'actionsByScope'. | 49 |
| CommentRequired | Public method and constructor comments are required | 59–74 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 63 |
| CommentRequired | Public method and constructor comments are required | 76–79 |
| CommentRequired | Public method and constructor comments are required | 81–84 |
| CommentRequired | Public method and constructor comments are required | 86–89 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'WorkflowResourceIdService' is suspected to be a Data Class (WOC=0.000%, NOPA=8, NOAM=0, WMC=1) | 23–113 |
| CommentRequired | Public method and constructor comments are required | 43–46 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 43–46 |
| ShortVariable | Avoid variables with short names like rt | 51 |
| ShortVariable | Avoid variables with short names like p | 57 |
| ShortVariable | Avoid variables with short names like id | 109 |
| ShortVariable | Avoid variables with short names like wf | 110 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 111 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 51–78 |
| ShortVariable | Avoid variables with short names like p | 55 |
| ShortVariable | Avoid variables with short names like oc | 57 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 44–61 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 46–58 |
| ImmutableField | Private field '_fmt' could be made final; it is only initialized in the declaration or constructor. | 48 |
| ShortVariable | Avoid variables with short names like dt | 54 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 46–59 |
| ImmutableField | Private field '_fmt' could be made final; it is only initialized in the declaration or constructor. | 48 |
| ShortVariable | Avoid variables with short names like z | 54 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'WorkflowJsonData' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=10, WMC=10) | 45–142 |
| CommentRequired | Header comments are required | 45–142 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–366 |
| CommentRequired | Header comments are required | 77–366 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 78–366 |
| ImmutableField | Private field '_objectMapper' could be made final; it is only initialized in the declaration or constructor. | 83 |
| ImmutableField | Private field '_workflowService' could be made final; it is only initialized in the declaration or constructor. | 85 |
| ImmutableField | Private field '_stateService' could be made final; it is only initialized in the declaration or constructor. | 86 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 87 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 88 |
| ImmutableField | Private field '_taskFactory' could be made final; it is only initialized in the declaration or constructor. | 89 |
| ImmutableField | Private field '_prerequisiteManagementService' could be made final; it is only initialized in the declaration or constructor. | 90 |
| CommentRequired | Public method and constructor comments are required | 106–109 |
| NPathComplexity | The method 'jsonExportWorkflow(int)' has an NPath complexity of 768 | 117–199 |
| CommentRequired | Public method and constructor comments are required | 202–217 |
| CommentRequired | Public method and constructor comments are required | 219–232 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 236 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 268 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 279 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 351 |
| Rule | Violation | Line |
|---|---|---|
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 77 characters appended. | 107 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 108 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 108 |
| ControlStatementBraces | This statement should have braces | 110 |
| Rule | Violation | Line |
|---|---|---|
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 73 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 132 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like e | 121 |
| Rule | Violation | Line |
|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 138–145 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 51–235 |
| LinguisticNaming | Linguistics Antipattern - The method 'hasCycle' indicates linguistically it returns a boolean, but it returns 'ErrorMessage' | 95–121 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 97 |
| ControlStatementBraces | This statement should have braces | 220 |
| Rule | Violation | Line |
|---|---|---|
| AssignmentInOperand | Avoid assignments in operands | 194 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 194 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. | 122 |
| ImmutableField | Private field '_iconService' could be made final; it is only initialized in the declaration or constructor. | 125 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 126 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 137 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 216 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 255–262 |
| NPathComplexity | The method 'getIconData(HttpServletRequest, Icon)' has an NPath complexity of 784 | 337–406 |
| CyclomaticComplexity | The method 'getIconData(HttpServletRequest, Icon)' has a cyclomatic complexity of 18. | 337–406 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 344 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 345 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 392–400 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 109 |
| ShortVariable | Avoid variables with short names like t | 116 |
| Rule | Violation | Line |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–3199 |
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–3199 |
| CyclomaticComplexity | The class 'WorkflowJspBean' has a total cyclomatic complexity of 344 (highest 15). | 138–3199 |
| TooManyFields | Too many fields | 138–3199 |
| ExcessiveClassLength | Avoid really long classes. | 138–3199 |
| GodClass | Possible God Class (WMC=344, ATFD=580, TCC=30.114%) | 138–3199 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 139–3199 |
| ImmutableField | Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. | 315 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsAscSort' | 325 |
| ImmutableField | Private field '_workflowService' could be made final; it is only initialized in the declaration or constructor. | 326 |
| ImmutableField | Private field '_stateService' could be made final; it is only initialized in the declaration or constructor. | 327 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 328 |
| ImmutableField | Private field '_iconService' could be made final; it is only initialized in the declaration or constructor. | 329 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 330 |
| ImmutableField | Private field '_taskFactory' could be made final; it is only initialized in the declaration or constructor. | 331 |
| ImmutableField | Private field '_taskComponentManager' could be made final; it is only initialized in the declaration or constructor. | 332 |
| ImmutableField | Private field '_prerequisiteManagementService' could be made final; it is only initialized in the declaration or constructor. | 333 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 394 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 431 |
| CyclomaticComplexity | The method 'getModifyWorkflow(HttpServletRequest)' has a cyclomatic complexity of 14. | 487–616 |
| ExcessiveMethodLength | Avoid really long methods. | 487–616 |
| NPathComplexity | The method 'getModifyWorkflow(HttpServletRequest)' has an NPath complexity of 1728 | 487–616 |
| NcssCount | The method 'getModifyWorkflow(HttpServletRequest)' has a NCSS line count of 74. | 487–616 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 489 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 491 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 492 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 577 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 592 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 645–648 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 654–657 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 661–664 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 787 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 826 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 884 |
| CyclomaticComplexity | The method 'doCreateState(HttpServletRequest)' has a cyclomatic complexity of 10. | 905–966 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1001 |
| CyclomaticComplexity | The method 'doModifyState(HttpServletRequest)' has a cyclomatic complexity of 11. | 1022–1079 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1048 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1195 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1196 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1197 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1261 |
| CyclomaticComplexity | The method 'getActionData(HttpServletRequest, Action)' has a cyclomatic complexity of 15. | 1353–1462 |
| NcssCount | The method 'getActionData(HttpServletRequest, Action)' has a NCSS line count of 66. | 1353–1462 |
| ExcessiveMethodLength | Avoid really long methods. | 1353–1462 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1365 |
| CyclomaticComplexity | The method 'getModifyAction(HttpServletRequest)' has a cyclomatic complexity of 10. | 1473–1562 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1534 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1757 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1794 |
| ShortVariable | Avoid variables with short names like it | 2033 |
| CyclomaticComplexity | The method 'doChangeOrderState(HttpServletRequest)' has a cyclomatic complexity of 10. | 2182–2255 |
| CyclomaticComplexity | The method 'doChangeOrderAction(HttpServletRequest)' has a cyclomatic complexity of 10. | 2264–2334 |
| CyclomaticComplexity | The method 'doChangeOrderTask(HttpServletRequest)' has a cyclomatic complexity of 11. | 2343–2412 |
| CommentRequired | Public method and constructor comments are required | 2720–2727 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 2915 |
| CyclomaticComplexity | The method 'doRemoveTaskFromReflexiveAction(HttpServletRequest)' has a cyclomatic complexity of 11. | 3039–3078 |
| ShortVariable | Avoid variables with short names like os | 3114 |
| CommentRequired | Public method and constructor comments are required | 3128–3140 |
| CommentRequired | Public method and constructor comments are required | 3142–3162 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3159 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_prerequisiteManagementService' could be made final; it is only initialized in the declaration or constructor. | 88 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 129 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 137–147 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 193–203 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 262 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 270–280 |
| Rule | Violation | Line |
|---|---|---|
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 91–94 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.message.AdminMessage' | 47 |
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.message.AdminMessageService' | 48 |
| Rule | Violation | Line |
|---|---|---|
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 145 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 276 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_DELETE' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 684 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_DELETE' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 714 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 880 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 930 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 993 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 1044 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_DELETE_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 1154 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 1259 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 1311 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 1489 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 1608 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_DELETE_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 1678 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 1716 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 1811 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 2202 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 2284 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 2533 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 2564 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 2627 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 2660 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 2820 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 2847 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 2903 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 2967 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 3041 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowAppResourceIdService.RESOURCE_TYPE_KEY' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowAppResourceIdService.*' | 3170 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.RESOURCE_TYPE_KEY' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 3189 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.RESOURCE_TYPE_KEY' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 3190 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 57–530 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 99–122 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 113–114 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 144–145 |
| CyclomaticComplexity | The method 'selectActionsByFilter(ActionFilter)' has a cyclomatic complexity of 16. | 3 | 212–301 |
| NPathComplexity | The method 'selectActionsByFilter(ActionFilter)' has an NPath complexity of 32896 | 3 | 212–301 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 307–317 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 44–129 |
| UnusedPrivateField | Avoid unused private fields such as 'SQL_QUERY_SELECT_ALL'. | 3 | 46 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 58–73 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 63–81 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 86–100 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 56–63 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 91–94 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 58–528 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 86–105 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 400 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 412 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 412 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 412 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 412 |
| ShortVariable | Avoid variables with short names like sb | 3 | 459 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 460 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 461 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 481 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 481 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 481 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 481 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'nPos' | 2 | 481 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 506–509 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 59–829 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 102–124 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 112–119 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 140–147 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 605 |
| CyclomaticComplexity | The method 'buildFilterQuerydFooter(DAOUtil, ResourceWorkflowFilter, List, Integer)' has a cyclomatic complexity of 12. | 3 | 654–716 |
| NPathComplexity | The method 'buildFilterQuerydFooter(DAOUtil, ResourceWorkflowFilter, List, Integer)' has an NPath complexity of 648 | 3 | 654–716 |
| NPathComplexity | The method 'buildFilterQuerydHeader(ResourceWorkflowFilter, List, String, String)' has an NPath complexity of 432 | 3 | 731–801 |
| CyclomaticComplexity | The method 'buildFilterQuerydHeader(ResourceWorkflowFilter, List, String, String)' has a cyclomatic complexity of 11. | 3 | 731–801 |
| ShortVariable | Avoid variables with short names like sb | 3 | 770 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 771 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 772 |
| ShortVariable | Avoid variables with short names like sb | 3 | 786 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 787 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 788 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 818 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 818–825 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 54–369 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 78–107 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 77–93 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 71–89 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 40–142 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 40–142 |
| FieldNamingConventions | The final field name '_stateBefore' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 42 |
| FieldNamingConventions | The final field name '_stateAfter' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 43 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_action' | 3 | 45 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Enum comments are required | 3 | 36–40 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–126 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 56–68 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–152 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 100 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 128 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–167 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 58–65 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 67–74 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 109–116 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 118–125 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 154 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 159 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–76 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 67–142 |
| ImmutableField | Private field '_workflowService' could be made final; it is only initialized in the declaration or constructor. | 3 | 69 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 3 | 70 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 3 | 71 |
| ImmutableField | Private field '_resourceWorkflowService' could be made final; it is only initialized in the declaration or constructor. | 3 | 72 |
| ImmutableField | Private field '_archiveService' could be made final; it is only initialized in the declaration or constructor. | 3 | 73 |
| ShortVariable | Avoid variables with short names like wf | 3 | 98 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 114–118 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Protected method constructor comments are required | 3 | 58–61 |
| CommentRequired | Protected method constructor comments are required | 3 | 63–67 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–46 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 50–105 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 63 |
| CommentRequired | Public method and constructor comments are required | 3 | 79–85 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 66–236 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 111 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like id | 3 | 62 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 59–71 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 59–75 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 133–140 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 121–136 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 159 |
| ShortVariable | Avoid variables with short names like t | 3 | 162 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has a cyclomatic complexity of 13. | 3 | 115–205 |
| NPathComplexity | The method 'doSaveConfig(HttpServletRequest, Locale, ITask)' has an NPath complexity of 1152 | 3 | 115–205 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 119 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 121 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 172–192 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 180–188 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 213 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 214 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 279 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 295 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 329 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 57–69 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.message.AdminMessage' | 4 | 47 |
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.portal.service.message.AdminMessageService' | 4 | 48 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 82 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 94 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like id | 3 | 66 |
| ShortVariable | Avoid variables with short names like sb | 3 | 115 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 56–68 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 58–71 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 61–64 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 3 | 69–83 |
| ShortVariable | Avoid variables with short names like rt | 3 | 71 |
| ShortVariable | Avoid variables with short names like p | 3 | 77 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 3 | 92–95 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 3 | 100–103 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–209 |
| ImmutableField | Private field '_commentValueService' could be made final; it is only initialized in the declaration or constructor. | 3 | 96 |
| ImmutableField | Private field '_resourceHistoryService' could be made final; it is only initialized in the declaration or constructor. | 3 | 97 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 3 | 98 |
| ImmutableField | Private field '_taskComponentManager' could be made final; it is only initialized in the declaration or constructor. | 3 | 99 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 99–102 |
| CommentRequired | Public method and constructor comments are required | 3 | 104–108 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 116 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 117 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 148 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 164 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 199 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 57–69 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 80 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 92 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–73 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the _nDuration field if you want a default access modifier | 3 | 46 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 46 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–111 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 55–131 |
| MissingOverride | The method 'getPrerequisiteType()' is missing an @Override annotation. | 3 | 66–69 |
| CommentRequired | Public method and constructor comments are required | 3 | 66–69 |
| MissingOverride | The method 'getTitleI18nKey()' is missing an @Override annotation. | 3 | 71–74 |
| CommentRequired | Public method and constructor comments are required | 3 | 71–74 |
| MissingOverride | The method 'hasConfiguration()' is missing an @Override annotation. | 3 | 76–79 |
| CommentRequired | Public method and constructor comments are required | 3 | 76–79 |
| CommentRequired | Public method and constructor comments are required | 3 | 81–84 |
| MissingOverride | The method 'getEmptyConfiguration()' is missing an @Override annotation. | 3 | 81–84 |
| MissingOverride | The method 'getConfigurationDaoBeanName()' is missing an @Override annotation. | 3 | 86–89 |
| CommentRequired | Public method and constructor comments are required | 3 | 86–89 |
| MissingOverride | The method 'getConfigHtml(IPrerequisiteConfig, HttpServletRequest, Locale)' is missing an @Override annotation. | 3 | 91–99 |
| CommentRequired | Public method and constructor comments are required | 3 | 91–99 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 93 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 59–73 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 96 |
| ShortVariable | Avoid variables with short names like t | 3 | 99 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–64 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 39–106 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–93 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–96 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–89 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–104 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 39–114 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–93 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–96 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–89 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–67 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–67 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 62–133 |
| ImmutableField | Private field '_changeStateTaskService' could be made final; it is only initialized in the declaration or constructor. | 3 | 65 |
| ImmutableField | Private field '_workflowService' could be made final; it is only initialized in the declaration or constructor. | 3 | 66 |
| ImmutableField | Private field '_resourceWorkflowService' could be made final; it is only initialized in the declaration or constructor. | 3 | 67 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 3 | 68 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 3 | 69 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 112 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 62–133 |
| ImmutableField | Private field '_chooseStateTaskService' could be made final; it is only initialized in the declaration or constructor. | 3 | 65 |
| ImmutableField | Private field '_workflowService' could be made final; it is only initialized in the declaration or constructor. | 3 | 66 |
| ImmutableField | Private field '_resourceWorkflowService' could be made final; it is only initialized in the declaration or constructor. | 3 | 67 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 3 | 68 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 3 | 69 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 112 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 58–136 |
| CommentRequired | Public method and constructor comments are required | 3 | 75–92 |
| CommentRequired | Public method and constructor comments are required | 3 | 94–103 |
| CommentRequired | Public method and constructor comments are required | 3 | 105–133 |
| CommentRequired | Protected method constructor comments are required | 3 | 135 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 47–78 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 46–75 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 48–88 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 43–119 |
| UncommentedEmptyMethodBody | Document empty method body | 3 | 108–109 |
| UncommentedEmptyMethodBody | Document empty method body | 3 | 117–118 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 52–94 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 70 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 82 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 82 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 53–122 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 77 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 110 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 110 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 73–76 |
| ShortVariable | Avoid variables with short names like rt | 3 | 84 |
| ShortVariable | Avoid variables with short names like p | 3 | 90 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 3 | 110–125 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 3 | 136–148 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 159 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 160 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 160 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 161 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 167 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 169 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 170 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 72–75 |
| ShortVariable | Avoid variables with short names like rt | 3 | 83 |
| ShortVariable | Avoid variables with short names like p | 3 | 89 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 139 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 9–37 |
| CommentRequired | Public method and constructor comments are required | 3 | 13–16 |
| CommentRequired | Public method and constructor comments are required | 3 | 18–21 |
| CommentRequired | Public method and constructor comments are required | 3 | 23–26 |
| CommentRequired | Header comments are required | 3 | 28–31 |
| CommentRequired | Public method and constructor comments are required | 3 | 33–36 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'WorkflowAppResourceIdService' is suspected to be a Data Class (WOC=0.000%, NOPA=4, NOAM=0, WMC=1) | 3 | 18–76 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 31–34 |
| CommentRequired | Public method and constructor comments are required | 3 | 31–34 |
| ShortVariable | Avoid variables with short names like rt | 3 | 39 |
| ShortVariable | Avoid variables with short names like p | 3 | 45 |
| ShortVariable | Avoid variables with short names like rl | 3 | 66 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| 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 | 52–151 |
| ShortVariable | Avoid variables with short names like wf | 3 | 74 |
| CyclomaticComplexity | The method 'generate(Workflow, String)' has a cyclomatic complexity of 16. | 3 | 74–129 |
| NPathComplexity | The method 'generate(Workflow, String)' has an NPath complexity of 600 | 3 | 74–129 |
| ShortVariable | Avoid variables with short names like sb | 3 | 76 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 123 |
| AppendCharacterWithChar | Avoid appending characters as strings in StringBuffer.append. | 3 | 124 |
| ShortVariable | Avoid variables with short names like sb | 3 | 139 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 139 |
| ControlStatementBraces | This statement should have braces | 3 | 142 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–645 |
| MissingOverride | The method 'getActions(int, String, Collection, User)' is missing an @Override annotation. | 3 | 124–129 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'listActions' | 2 | 124 |
| MissingOverride | The method 'getAuthorizedActions(Collection, User)' is missing an @Override annotation. | 3 | 135–138 |
| MissingOverride | The method 'getActions(String, Map, User)' is missing an @Override annotation. | 3 | 144–156 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 200 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 211–237 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 261–272 |
| MissingOverride | The method 'getDisplayDocumentHistory(int, String, int, HttpServletRequest, Locale, Map, String, User)' is missing an @Override annotation. | 3 | 321–334 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 356 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'user' | 2 | 378 |
| AvoidReassigningParameters | Avoid reassigning parameters such as 'user' | 2 | 399 |
| MissingOverride | The method 'canProcessAction(int, String, int, HttpServletRequest, User)' is missing an @Override annotation. | 3 | 399–414 |
| CyclomaticComplexity | The method 'isAuthorized(int, String, int, User)' has a cyclomatic complexity of 10. | 3 | 420–470 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 426–443 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 536 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 548–555 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 591 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Enum comments are required | 3 | 8–46 |
| CommentRequired | Enum comments are required | 3 | 21 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 21–134 |
| 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 | 22–134 |
| ShortVariable | Avoid variables with short names like id | 3 | 23 |
| ShortVariable | Avoid variables with short names like id | 3 | 32 |
| CommentRequired | Public method and constructor comments are required | 3 | 37–45 |
| CommentRequired | Public method and constructor comments are required | 3 | 47–57 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 49 |
| UnusedLocalVariable | Avoid unused local variables such as 'actionsByScope'. | 3 | 49 |
| CommentRequired | Public method and constructor comments are required | 3 | 59–74 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 63 |
| CommentRequired | Public method and constructor comments are required | 3 | 76–79 |
| CommentRequired | Public method and constructor comments are required | 3 | 81–84 |
| CommentRequired | Public method and constructor comments are required | 3 | 86–89 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'WorkflowResourceIdService' is suspected to be a Data Class (WOC=0.000%, NOPA=8, NOAM=0, WMC=1) | 3 | 23–113 |
| CommentRequired | Public method and constructor comments are required | 3 | 43–46 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 43–46 |
| ShortVariable | Avoid variables with short names like rt | 3 | 51 |
| ShortVariable | Avoid variables with short names like p | 3 | 57 |
| ShortVariable | Avoid variables with short names like id | 3 | 109 |
| ShortVariable | Avoid variables with short names like wf | 3 | 110 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 111 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 51–78 |
| ShortVariable | Avoid variables with short names like p | 3 | 55 |
| ShortVariable | Avoid variables with short names like oc | 3 | 57 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 44–61 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 46–58 |
| ImmutableField | Private field '_fmt' could be made final; it is only initialized in the declaration or constructor. | 3 | 48 |
| ShortVariable | Avoid variables with short names like dt | 3 | 54 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 46–59 |
| ImmutableField | Private field '_fmt' could be made final; it is only initialized in the declaration or constructor. | 3 | 48 |
| ShortVariable | Avoid variables with short names like z | 3 | 54 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'WorkflowJsonData' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=10, WMC=10) | 3 | 45–142 |
| CommentRequired | Header comments are required | 3 | 45–142 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–366 |
| CommentRequired | Header comments are required | 3 | 77–366 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 77–366 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 78–366 |
| ImmutableField | Private field '_objectMapper' could be made final; it is only initialized in the declaration or constructor. | 3 | 83 |
| ImmutableField | Private field '_workflowService' could be made final; it is only initialized in the declaration or constructor. | 3 | 85 |
| ImmutableField | Private field '_stateService' could be made final; it is only initialized in the declaration or constructor. | 3 | 86 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 3 | 87 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 3 | 88 |
| ImmutableField | Private field '_taskFactory' could be made final; it is only initialized in the declaration or constructor. | 3 | 89 |
| ImmutableField | Private field '_prerequisiteManagementService' could be made final; it is only initialized in the declaration or constructor. | 3 | 90 |
| CommentRequired | Public method and constructor comments are required | 3 | 106–109 |
| NPathComplexity | The method 'jsonExportWorkflow(int)' has an NPath complexity of 768 | 3 | 117–199 |
| CommentRequired | Public method and constructor comments are required | 3 | 202–217 |
| CommentRequired | Public method and constructor comments are required | 3 | 219–232 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 236 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 268 |
| LocalVariableNamingConventions | The local variable name 'StateBefore' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 268 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 279 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 351 |
| LocalVariableNamingConventions | The local variable name 'PrerequisiteTypeList' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 351 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 77 characters appended. | 3 | 107 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 108 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 108 |
| ControlStatementBraces | This statement should have braces | 3 | 110 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_listMarkerProviders' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 53 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 73 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 132 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like e | 3 | 121 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 138–145 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 51–235 |
| CommentRequired | Header comments are required | 3 | 51–235 |
| LinguisticNaming | Linguistics Antipattern - The method 'hasCycle' indicates linguistically it returns a boolean, but it returns 'ErrorMessage' | 3 | 95–121 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 97 |
| ControlStatementBraces | This statement should have braces | 3 | 220 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 145 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 194 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 194 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 276 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. | 3 | 122 |
| ImmutableField | Private field '_iconService' could be made final; it is only initialized in the declaration or constructor. | 3 | 125 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 3 | 126 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 137 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 216 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 255–262 |
| NPathComplexity | The method 'getIconData(HttpServletRequest, Icon)' has an NPath complexity of 784 | 3 | 337–406 |
| CyclomaticComplexity | The method 'getIconData(HttpServletRequest, Icon)' has a cyclomatic complexity of 18. | 3 | 337–406 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 344 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 345 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 392–400 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 109 |
| ShortVariable | Avoid variables with short names like t | 3 | 116 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–3199 |
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–3199 |
| CyclomaticComplexity | The class 'WorkflowJspBean' has a total cyclomatic complexity of 344 (highest 15). | 3 | 138–3199 |
| TooManyFields | Too many fields | 3 | 138–3199 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 138–3199 |
| GodClass | Possible God Class (WMC=344, ATFD=580, TCC=30.114%) | 3 | 138–3199 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 139–3199 |
| ImmutableField | Private field '_nDefaultItemsPerPage' could be made final; it is only initialized in the declaration or constructor. | 3 | 315 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsAscSort' | 3 | 325 |
| ImmutableField | Private field '_workflowService' could be made final; it is only initialized in the declaration or constructor. | 3 | 326 |
| ImmutableField | Private field '_stateService' could be made final; it is only initialized in the declaration or constructor. | 3 | 327 |
| ImmutableField | Private field '_actionService' could be made final; it is only initialized in the declaration or constructor. | 3 | 328 |
| ImmutableField | Private field '_iconService' could be made final; it is only initialized in the declaration or constructor. | 3 | 329 |
| ImmutableField | Private field '_taskService' could be made final; it is only initialized in the declaration or constructor. | 3 | 330 |
| ImmutableField | Private field '_taskFactory' could be made final; it is only initialized in the declaration or constructor. | 3 | 331 |
| ImmutableField | Private field '_taskComponentManager' could be made final; it is only initialized in the declaration or constructor. | 3 | 332 |
| ImmutableField | Private field '_prerequisiteManagementService' could be made final; it is only initialized in the declaration or constructor. | 3 | 333 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 394 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 431 |
| CyclomaticComplexity | The method 'getModifyWorkflow(HttpServletRequest)' has a cyclomatic complexity of 14. | 3 | 487–616 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 487–616 |
| NPathComplexity | The method 'getModifyWorkflow(HttpServletRequest)' has an NPath complexity of 1728 | 3 | 487–616 |
| NcssCount | The method 'getModifyWorkflow(HttpServletRequest)' has a NCSS line count of 74. | 3 | 487–616 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 489 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 491 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 492 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 577 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 592 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 645–648 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 654–657 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 661–664 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_DELETE' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 684 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_DELETE' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 714 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 787 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 826 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 880 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 884 |
| CyclomaticComplexity | The method 'doCreateState(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 905–966 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 930 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 993 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1001 |
| CyclomaticComplexity | The method 'doModifyState(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 1022–1079 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 1044 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1048 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_DELETE_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 1154 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1195 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1196 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1197 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 1259 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1261 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 1311 |
| CyclomaticComplexity | The method 'getActionData(HttpServletRequest, Action)' has a cyclomatic complexity of 15. | 3 | 1353–1462 |
| NcssCount | The method 'getActionData(HttpServletRequest, Action)' has a NCSS line count of 66. | 3 | 1353–1462 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 1353–1462 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1365 |
| CyclomaticComplexity | The method 'getModifyAction(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 1473–1562 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 1489 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1534 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 1608 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_DELETE_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 1678 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 1716 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1757 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1794 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 1811 |
| ShortVariable | Avoid variables with short names like it | 3 | 2033 |
| CyclomaticComplexity | The method 'doChangeOrderState(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 2182–2255 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 2202 |
| CyclomaticComplexity | The method 'doChangeOrderAction(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 2264–2334 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 2284 |
| CyclomaticComplexity | The method 'doChangeOrderTask(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 2343–2412 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 2533 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 2564 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 2627 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 2660 |
| CommentRequired | Public method and constructor comments are required | 3 | 2720–2727 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_STATES' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 2820 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 2847 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 2903 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 2915 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 2967 |
| CyclomaticComplexity | The method 'doRemoveTaskFromReflexiveAction(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 3039–3078 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.PERM_WORKFLOW_EDIT_ACTIONS' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 3041 |
| ShortVariable | Avoid variables with short names like os | 3 | 3114 |
| CommentRequired | Public method and constructor comments are required | 3 | 3128–3140 |
| CommentRequired | Public method and constructor comments are required | 3 | 3142–3162 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 3159 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowAppResourceIdService.RESOURCE_TYPE_KEY' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowAppResourceIdService.*' | 4 | 3170 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.RESOURCE_TYPE_KEY' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 3189 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'WorkflowResourceIdService.RESOURCE_TYPE_KEY' due to existing static import 'fr.paris.lutece.plugins.workflow.service.WorkflowResourceIdService.*' | 4 | 3190 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_prerequisiteManagementService' could be made final; it is only initialized in the declaration or constructor. | 3 | 88 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 129 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 137–147 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 193–203 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 262 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 270–280 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 1 | 65–68 |
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 1 | 74–77 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 1 | 53–56 |
| EmptyMethodInAbstractClassShouldBeAbstract | An empty method in an abstract class should be abstract instead | 1 | 62–65 |
