The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'ProfilesConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 41–159 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 52–441 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getResourceId()' is missing an @Override annotation. | 99–102 |
| MissingOverride | The method 'getResourceTypeCode()' is missing an @Override annotation. | 109–112 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getPermission()' is missing an @Override annotation. | 164–167 |
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 186–189 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 55–692 |
| Rule | Violation | Line |
|---|---|---|
| LinguisticNaming | Linguistics Antipattern - The setter 'setFilter' should not return any type except void linguistically | 117–134 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 122–131 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 53–563 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'canBeRemoved(String)' is missing an @Override annotation. | 63–87 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 76–81 |
| MissingOverride | The method 'getRemovalRefusedMessage(String, Locale)' is missing an @Override annotation. | 98–102 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'canBeRemoved(String)' is missing an @Override annotation. | 63–87 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 76–81 |
| MissingOverride | The method 'getRemovalRefusedMessage(String, Locale)' is missing an @Override annotation. | 98–102 |
| Rule | Violation | Line |
|---|---|---|
| CyclomaticComplexity | The method 'doCreateUserFields(AdminUser, List, Locale)' has a cyclomatic complexity of 10. | 64–112 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'right' | 82 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'role' | 93 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'workgroup' | 104 |
| CyclomaticComplexity | The method 'doModifyUserFields(AdminUser, List, Locale, AdminUser)' has a cyclomatic complexity of 23. | 118–214 |
| NPathComplexity | The method 'doModifyUserFields(AdminUser, List, Locale, AdminUser)' has an NPath complexity of 3854 | 118–214 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'right' | 137 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'role' | 149 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'workgroup' | 160 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'right' | 183 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'role' | 194 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'workgroup' | 205 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 50–314 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getResourceId()' is missing an @Override annotation. | 99–102 |
| MissingOverride | The method 'getResourceTypeCode()' is missing an @Override annotation. | 109–112 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getPermission()' is missing an @Override annotation. | 164–167 |
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 186–189 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 55–679 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 372–382 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 413–422 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 559–560 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 578–586 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 624–631 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'getDashboardComponents(AdminUser, HttpServletRequest)' is missing an @Override annotation. | 67–91 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 82–85 |
| Rule | Violation | Line |
|---|---|---|
| LinguisticNaming | Linguistics Antipattern - The setter 'setFilter' should not return any type except void linguistically | 117–134 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 122–131 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 51–406 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 60–522 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 76–79 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 84–132 |
| ShortVariable | Avoid variables with short names like rt | 86 |
| ShortVariable | Avoid variables with short names like p | 91 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 141–146 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 157–160 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–591 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 81–591 |
| CyclomaticComplexity | The method 'doUnassignUserFromProfile(int, String, AdminUser, HttpServletRequest, Locale, Plugin)' has a cyclomatic complexity of 15. | 151–209 |
| NPathComplexity | The method 'doUnassignUserFromProfile(int, String, AdminUser, HttpServletRequest, Locale, Plugin)' has an NPath complexity of 576 | 151–209 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 58–464 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 71–74 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 79–112 |
| ShortVariable | Avoid variables with short names like rt | 81 |
| ShortVariable | Avoid variables with short names like p | 86 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 121–126 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 137–143 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 142 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–778 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 77–778 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 135 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 191 |
| CyclomaticComplexity | The method 'doMoveDashboard(IDashboardComponent, int, int, boolean, String, Plugin)' has a cyclomatic complexity of 16. | 247–331 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 302–306 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 366 |
| MissingOverride | The method 'findDashboardsByFilter(DashboardFilter, String, Plugin)' is missing an @Override annotation. | 705–708 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–1429 |
| CyclomaticComplexity | The class 'ProfilesJspBean' has a total cyclomatic complexity of 137 (highest 23). | 102–1429 |
| ExcessiveClassLength | Avoid really long classes. | 102–1429 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 103–1429 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 137 |
| ImmutableField | Private field '_profilesService' could be made final; it is only initialized in the declaration or constructor. | 138 |
| ImmutableField | Private field '_profileActionService' could be made final; it is only initialized in the declaration or constructor. | 139 |
| ImmutableField | Private field '_viewsService' could be made final; it is only initialized in the declaration or constructor. | 140 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 217 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 244 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 267 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 377 |
| CyclomaticComplexity | The method 'getAssignRightsProfile(HttpServletRequest)' has a cyclomatic complexity of 11. | 427–505 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 429 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'right' | 441 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 535–553 |
| CyclomaticComplexity | The method 'getAssignWorkgroupsProfile(HttpServletRequest)' has a cyclomatic complexity of 16. | 606–755 |
| ExcessiveMethodLength | Avoid really long methods. | 606–755 |
| NPathComplexity | The method 'getAssignWorkgroupsProfile(HttpServletRequest)' has an NPath complexity of 4704 | 606–755 |
| NcssCount | The method 'getAssignWorkgroupsProfile(HttpServletRequest)' has a NCSS line count of 81. | 606–755 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 608 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'workgroup' | 620 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 785–804 |
| ExcessiveMethodLength | Avoid really long methods. | 856–964 |
| CyclomaticComplexity | The method 'getAssignRolesProfile(HttpServletRequest)' has a cyclomatic complexity of 10. | 856–964 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 858 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'role' | 870 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 994–1012 |
| NcssCount | The method 'getAssignUsersProfile(HttpServletRequest)' has a NCSS line count of 84. | 1065–1220 |
| NPathComplexity | The method 'getAssignUsersProfile(HttpServletRequest)' has an NPath complexity of 27040 | 1065–1220 |
| CyclomaticComplexity | The method 'getAssignUsersProfile(HttpServletRequest)' has a cyclomatic complexity of 23. | 1065–1220 |
| ExcessiveMethodLength | Avoid really long methods. | 1065–1220 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1067 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'user' | 1082 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1235–1263 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 1254–1258 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1306 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1355–1386 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–840 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 114 |
| ImmutableField | Private field '_viewsService' could be made final; it is only initialized in the declaration or constructor. | 115 |
| ImmutableField | Private field '_viewActionService' could be made final; it is only initialized in the declaration or constructor. | 116 |
| ImmutableField | Private field '_profilesService' could be made final; it is only initialized in the declaration or constructor. | 117 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 194 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 221 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 243 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 345 |
| ExcessiveMethodLength | Avoid really long methods. | 395–527 |
| CyclomaticComplexity | The method 'getAssignProfilesView(HttpServletRequest)' has a cyclomatic complexity of 13. | 395–527 |
| NPathComplexity | The method 'getAssignProfilesView(HttpServletRequest)' has an NPath complexity of 896 | 395–527 |
| NcssCount | The method 'getAssignProfilesView(HttpServletRequest)' has a NCSS line count of 75. | 395–527 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 397 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'profile' | 412 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 542–582 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 560–577 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 565–576 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 627 |
| CyclomaticComplexity | The method 'doMoveDashboard(HttpServletRequest)' has a cyclomatic complexity of 10. | 710–777 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like View | 45–133 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 52–441 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'getResourceId()' is missing an @Override annotation. | 3 | 99–102 |
| MissingOverride | The method 'getResourceTypeCode()' is missing an @Override annotation. | 3 | 109–112 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'getPermission()' is missing an @Override annotation. | 3 | 164–167 |
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 3 | 186–189 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 55–692 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| LinguisticNaming | Linguistics Antipattern - The setter 'setFilter' should not return any type except void linguistically | 3 | 117–134 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 122–131 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 53–563 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'canBeRemoved(String)' is missing an @Override annotation. | 3 | 63–87 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 76–81 |
| MissingOverride | The method 'getRemovalRefusedMessage(String, Locale)' is missing an @Override annotation. | 3 | 98–102 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'canBeRemoved(String)' is missing an @Override annotation. | 3 | 63–87 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 76–81 |
| MissingOverride | The method 'getRemovalRefusedMessage(String, Locale)' is missing an @Override annotation. | 3 | 98–102 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CyclomaticComplexity | The method 'doCreateUserFields(AdminUser, List, Locale)' has a cyclomatic complexity of 10. | 3 | 64–112 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'right' | 3 | 82 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'role' | 3 | 93 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'workgroup' | 3 | 104 |
| CyclomaticComplexity | The method 'doModifyUserFields(AdminUser, List, Locale, AdminUser)' has a cyclomatic complexity of 23. | 3 | 118–214 |
| NPathComplexity | The method 'doModifyUserFields(AdminUser, List, Locale, AdminUser)' has an NPath complexity of 3854 | 3 | 118–214 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'right' | 3 | 137 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'role' | 3 | 149 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'workgroup' | 3 | 160 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'right' | 3 | 183 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'role' | 3 | 194 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'workgroup' | 3 | 205 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 50–314 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortClassName | Avoid short class names like View | 4 | 45–133 |
| MissingOverride | The method 'getResourceId()' is missing an @Override annotation. | 3 | 99–102 |
| MissingOverride | The method 'getResourceTypeCode()' is missing an @Override annotation. | 3 | 109–112 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'getPermission()' is missing an @Override annotation. | 3 | 164–167 |
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 3 | 186–189 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 55–679 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 372–382 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 413–422 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 559–560 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 578–586 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 624–631 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'getDashboardComponents(AdminUser, HttpServletRequest)' is missing an @Override annotation. | 3 | 67–91 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 82–85 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| LinguisticNaming | Linguistics Antipattern - The setter 'setFilter' should not return any type except void linguistically | 3 | 117–134 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 122–131 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 51–406 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 60–522 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 76–79 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 3 | 84–132 |
| ShortVariable | Avoid variables with short names like rt | 3 | 86 |
| ShortVariable | Avoid variables with short names like p | 3 | 91 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 3 | 141–146 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 3 | 157–160 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–591 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 81–591 |
| CyclomaticComplexity | The method 'doUnassignUserFromProfile(int, String, AdminUser, HttpServletRequest, Locale, Plugin)' has a cyclomatic complexity of 15. | 3 | 151–209 |
| NPathComplexity | The method 'doUnassignUserFromProfile(int, String, AdminUser, HttpServletRequest, Locale, Plugin)' has an NPath complexity of 576 | 3 | 151–209 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 58–464 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 71–74 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 3 | 79–112 |
| ShortVariable | Avoid variables with short names like rt | 3 | 81 |
| ShortVariable | Avoid variables with short names like p | 3 | 86 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 3 | 121–126 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 3 | 137–143 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 142 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–778 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 77–778 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 135 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 191 |
| CyclomaticComplexity | The method 'doMoveDashboard(IDashboardComponent, int, int, boolean, String, Plugin)' has a cyclomatic complexity of 16. | 3 | 247–331 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 302–306 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 366 |
| MissingOverride | The method 'findDashboardsByFilter(DashboardFilter, String, Plugin)' is missing an @Override annotation. | 3 | 705–708 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'ProfilesConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 41–159 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–1429 |
| CyclomaticComplexity | The class 'ProfilesJspBean' has a total cyclomatic complexity of 137 (highest 23). | 3 | 102–1429 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 102–1429 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 103–1429 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 137 |
| ImmutableField | Private field '_profilesService' could be made final; it is only initialized in the declaration or constructor. | 3 | 138 |
| ImmutableField | Private field '_profileActionService' could be made final; it is only initialized in the declaration or constructor. | 3 | 139 |
| ImmutableField | Private field '_viewsService' could be made final; it is only initialized in the declaration or constructor. | 3 | 140 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 217 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 244 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 267 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 377 |
| CyclomaticComplexity | The method 'getAssignRightsProfile(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 427–505 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 429 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'right' | 3 | 441 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 535–553 |
| CyclomaticComplexity | The method 'getAssignWorkgroupsProfile(HttpServletRequest)' has a cyclomatic complexity of 16. | 3 | 606–755 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 606–755 |
| NPathComplexity | The method 'getAssignWorkgroupsProfile(HttpServletRequest)' has an NPath complexity of 4704 | 3 | 606–755 |
| NcssCount | The method 'getAssignWorkgroupsProfile(HttpServletRequest)' has a NCSS line count of 81. | 3 | 606–755 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 608 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'workgroup' | 3 | 620 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 785–804 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 856–964 |
| CyclomaticComplexity | The method 'getAssignRolesProfile(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 856–964 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 858 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'role' | 3 | 870 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 994–1012 |
| NcssCount | The method 'getAssignUsersProfile(HttpServletRequest)' has a NCSS line count of 84. | 3 | 1065–1220 |
| NPathComplexity | The method 'getAssignUsersProfile(HttpServletRequest)' has an NPath complexity of 27040 | 3 | 1065–1220 |
| CyclomaticComplexity | The method 'getAssignUsersProfile(HttpServletRequest)' has a cyclomatic complexity of 23. | 3 | 1065–1220 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 1065–1220 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1067 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'user' | 3 | 1082 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1235–1263 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 1254–1258 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1306 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1355–1386 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–840 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 114 |
| ImmutableField | Private field '_viewsService' could be made final; it is only initialized in the declaration or constructor. | 3 | 115 |
| ImmutableField | Private field '_viewActionService' could be made final; it is only initialized in the declaration or constructor. | 3 | 116 |
| ImmutableField | Private field '_profilesService' could be made final; it is only initialized in the declaration or constructor. | 3 | 117 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 194 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 221 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 243 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 345 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 395–527 |
| CyclomaticComplexity | The method 'getAssignProfilesView(HttpServletRequest)' has a cyclomatic complexity of 13. | 3 | 395–527 |
| NPathComplexity | The method 'getAssignProfilesView(HttpServletRequest)' has an NPath complexity of 896 | 3 | 395–527 |
| NcssCount | The method 'getAssignProfilesView(HttpServletRequest)' has a NCSS line count of 75. | 3 | 395–527 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 397 |
| AvoidReassigningLoopVariables | Avoid reassigning the loop control variable 'profile' | 3 | 412 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 542–582 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 560–577 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 565–576 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 627 |
| CyclomaticComplexity | The method 'doMoveDashboard(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 710–777 |
