The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_strAssignmentTypeCode' doesn't match '[a-z][a-zA-Z0-9]*' | 47 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_strI18nErrorMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 44 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The constant name '_plugin' doesn't match '[A-Z][A-Z_0-9]*' | 53 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'UnitAttributeManager' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 52–218 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'UnitUserAttributeManager' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 50–100 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 102 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 195 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'setIdAction(int)' is missing an @Override annotation. | 61–64 |
| MissingOverride | The method 'getIdAction()' is missing an @Override annotation. | 71–74 |
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 82–85 |
| MissingOverride | The method 'getUrl()' is missing an @Override annotation. | 92–95 |
| MissingOverride | The method 'setUrl(String)' is missing an @Override annotation. | 103–106 |
| MissingOverride | The method 'getNameKey()' is missing an @Override annotation. | 113–116 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 123–126 |
| MissingOverride | The method 'setNameKey(String)' is missing an @Override annotation. | 134–137 |
| MissingOverride | The method 'getDescriptionKey()' is missing an @Override annotation. | 144–147 |
| MissingOverride | The method 'getDescription()' is missing an @Override annotation. | 154–157 |
| MissingOverride | The method 'setDescriptionKey(String)' is missing an @Override annotation. | 165–168 |
| MissingOverride | The method 'getIcon()' is missing an @Override annotation. | 175–178 |
| MissingOverride | The method 'setIcon(String)' is missing an @Override annotation. | 186–189 |
| MissingOverride | The method 'getPermission()' is missing an @Override annotation. | 196–199 |
| MissingOverride | The method 'setPermission(String)' is missing an @Override annotation. | 207–210 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like e | 80 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 69 |
| MissingOverride | The method 'getPermission()' is missing an @Override annotation. | 150 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 45–248 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'TreeUnit' is suspected to be a Data Class (WOC=14.286%, NOPA=0, NOAM=4, WMC=7) | 44–116 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 62 |
| ImmutableField | Private field '_mapAttributes' could be made final; it is only initialized in the declaration or constructor. | 62 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 48–551 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 105–123 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 357–367 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 478 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 51–293 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 52–55 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–84 |
| CommentRequired | Public method and constructor comments are required | 81 |
| CommentRequired | Public method and constructor comments are required | 83 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–48 |
| Rule | Violation | Line |
|---|---|---|
| AvoidDuplicateLiterals | The String literal "unittree.transactionManager" appears 4 times in this file; the first occurrence is on line 290 | 290 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'populateTreeUnit': the method is declared in an interface type | 350 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–53 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 82–85 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 90–139 |
| ShortVariable | Avoid variables with short names like rt | 92 |
| ShortVariable | Avoid variables with short names like p | 98 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 148–159 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 170–185 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–746 |
| MissingOverride | The method 'getUnitWithNoChildren()' is missing an @Override annotation. | 445–448 |
| AvoidDuplicateLiterals | The String literal "unittree.transactionManager" appears 4 times in this file; the first occurrence is on line 577 | 577 |
| LinguisticNaming | Linguistics Antipattern - The getter 'getXMLUnit' should not return void linguistically | 659–684 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 701 |
| Rule | Violation | Line |
|---|---|---|
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 194 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–1144 |
| ExcessiveClassLength | Avoid really long classes. | 91–1144 |
| CyclomaticComplexity | The class 'UnitJspBean' has a total cyclomatic complexity of 138 (highest 18). | 91–1144 |
| ImmutableField | Private field '_unitService' could be made final; it is only initialized in the declaration or constructor. | 167 |
| ImmutableField | Private field '_unitUserService' could be made final; it is only initialized in the declaration or constructor. | 168 |
| ImmutableField | Private field '_bAdminAvatar' could be made final; it is only initialized in the declaration or constructor. | 172 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 226 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 229 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 301 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 346 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 424 |
| CyclomaticComplexity | The method 'getMoveUser(HttpServletRequest)' has a cyclomatic complexity of 17. | 453–526 |
| NPathComplexity | The method 'getMoveUser(HttpServletRequest)' has an NPath complexity of 600 | 453–526 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 516 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 617 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 698 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 764 |
| CyclomaticComplexity | The method 'doAddUsers(HttpServletRequest)' has a cyclomatic complexity of 11. | 787–834 |
| CyclomaticComplexity | The method 'doMoveUser(HttpServletRequest)' has a cyclomatic complexity of 18. | 843–931 |
| NPathComplexity | The method 'doMoveUser(HttpServletRequest)' has an NPath complexity of 1280 | 843–931 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 847 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 918 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 969 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1011 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1048 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 1130 |
| Rule | Violation | Line |
|---|---|---|
| NPathComplexity | The method 'fillModelForUserSearchForm(List, String, HttpServletRequest, Map, Unit)' has an NPath complexity of 208 | 114–189 |
| CyclomaticComplexity | The method 'fillModelForUserSearchForm(List, String, HttpServletRequest, Map, Unit)' has a cyclomatic complexity of 12. | 114–189 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like Unit | 48–253 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'setIdAction(int)' is missing an @Override annotation. | 3 | 61–64 |
| MissingOverride | The method 'getIdAction()' is missing an @Override annotation. | 3 | 71–74 |
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 3 | 82–85 |
| MissingOverride | The method 'getUrl()' is missing an @Override annotation. | 3 | 92–95 |
| MissingOverride | The method 'setUrl(String)' is missing an @Override annotation. | 3 | 103–106 |
| MissingOverride | The method 'getNameKey()' is missing an @Override annotation. | 3 | 113–116 |
| MissingOverride | The method 'getName()' is missing an @Override annotation. | 3 | 123–126 |
| MissingOverride | The method 'setNameKey(String)' is missing an @Override annotation. | 3 | 134–137 |
| MissingOverride | The method 'getDescriptionKey()' is missing an @Override annotation. | 3 | 144–147 |
| MissingOverride | The method 'getDescription()' is missing an @Override annotation. | 3 | 154–157 |
| MissingOverride | The method 'setDescriptionKey(String)' is missing an @Override annotation. | 3 | 165–168 |
| MissingOverride | The method 'getIcon()' is missing an @Override annotation. | 3 | 175–178 |
| MissingOverride | The method 'setIcon(String)' is missing an @Override annotation. | 3 | 186–189 |
| MissingOverride | The method 'getPermission()' is missing an @Override annotation. | 3 | 196–199 |
| MissingOverride | The method 'setPermission(String)' is missing an @Override annotation. | 3 | 207–210 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like e | 3 | 80 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 102 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'setLocale(Locale)' is missing an @Override annotation. | 3 | 69 |
| MissingOverride | The method 'getPermission()' is missing an @Override annotation. | 3 | 150 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_strAssignmentTypeCode' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 47 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 45–248 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'TreeUnit' is suspected to be a Data Class (WOC=14.286%, NOPA=0, NOAM=4, WMC=7) | 3 | 44–116 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortClassName | Avoid short class names like Unit | 4 | 48–253 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 62 |
| ImmutableField | Private field '_mapAttributes' could be made final; it is only initialized in the declaration or constructor. | 3 | 62 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 48–551 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 105–123 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 357–367 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 478 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 51–293 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The final field name '_strI18nErrorMessage' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 44 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 52–55 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| FieldNamingConventions | The constant name '_plugin' doesn't match '[A-Z][A-Z_0-9]*' | 1 | 53 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–84 |
| CommentRequired | Public method and constructor comments are required | 3 | 81 |
| CommentRequired | Public method and constructor comments are required | 3 | 83 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–48 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidDuplicateLiterals | The String literal "unittree.transactionManager" appears 4 times in this file; the first occurrence is on line 290 | 3 | 290 |
| UnnecessaryModifier | Unnecessary modifier 'public' on method 'populateTreeUnit': the method is declared in an interface type | 3 | 350 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–53 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'UnitAttributeManager' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 52–218 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 82–85 |
| MissingOverride | The method 'register()' is missing an @Override annotation. | 3 | 90–139 |
| ShortVariable | Avoid variables with short names like rt | 3 | 92 |
| ShortVariable | Avoid variables with short names like p | 3 | 98 |
| MissingOverride | The method 'getResourceIdList(Locale)' is missing an @Override annotation. | 3 | 148–159 |
| MissingOverride | The method 'getTitle(String, Locale)' is missing an @Override annotation. | 3 | 170–185 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–746 |
| MissingOverride | The method 'getUnitWithNoChildren()' is missing an @Override annotation. | 3 | 445–448 |
| AvoidDuplicateLiterals | The String literal "unittree.transactionManager" appears 4 times in this file; the first occurrence is on line 577 | 3 | 577 |
| LinguisticNaming | Linguistics Antipattern - The getter 'getXMLUnit' should not return void linguistically | 3 | 659–684 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 701 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'UnitUserAttributeManager' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 50–100 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 194 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–1144 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 91–1144 |
| CyclomaticComplexity | The class 'UnitJspBean' has a total cyclomatic complexity of 138 (highest 18). | 3 | 91–1144 |
| ImmutableField | Private field '_unitService' could be made final; it is only initialized in the declaration or constructor. | 3 | 167 |
| ImmutableField | Private field '_unitUserService' could be made final; it is only initialized in the declaration or constructor. | 3 | 168 |
| ImmutableField | Private field '_bAdminAvatar' could be made final; it is only initialized in the declaration or constructor. | 3 | 172 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 195 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 226 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 229 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 301 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 346 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 424 |
| CyclomaticComplexity | The method 'getMoveUser(HttpServletRequest)' has a cyclomatic complexity of 17. | 3 | 453–526 |
| NPathComplexity | The method 'getMoveUser(HttpServletRequest)' has an NPath complexity of 600 | 3 | 453–526 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 516 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 617 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 698 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 764 |
| CyclomaticComplexity | The method 'doAddUsers(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 787–834 |
| CyclomaticComplexity | The method 'doMoveUser(HttpServletRequest)' has a cyclomatic complexity of 18. | 3 | 843–931 |
| NPathComplexity | The method 'doMoveUser(HttpServletRequest)' has an NPath complexity of 1280 | 3 | 843–931 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 847 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 918 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 969 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1011 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1048 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 1130 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| NPathComplexity | The method 'fillModelForUserSearchForm(List, String, HttpServletRequest, Map, Unit)' has an NPath complexity of 208 | 3 | 114–189 |
| CyclomaticComplexity | The method 'fillModelForUserSearchForm(List, String, HttpServletRequest, Map, Unit)' has a cyclomatic complexity of 12. | 3 | 114–189 |
