The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 8 |
| FieldNamingConventions | The field name 'contentLabel' doesn't match '_[a-z][a-zA-Z0-9]*' | 9 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 45–253 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The final field name '_blogServiceSession' doesn't match '[a-z][a-zA-Z0-9]*' | 318 |
| LocalVariableNamingConventions | The local variable name '_nDocContentInSession' doesn't match '[a-z][a-zA-Z0-9]*' | 1397 |
| Rule | Violation | Line |
|---|---|---|
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 712 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 724 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 735 |
| Rule | Violation | Line |
|---|---|---|
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 93 |
| Rule | Violation | Line |
|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 34–655 |
| TooManyFields | Too many fields | 54–655 |
| CommentRequired | Public method and constructor comments are required | 291–294 |
| CommentRequired | Public method and constructor comments are required | 296–299 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 53–905 |
| CyclomaticComplexity | The method 'getDaoFromFilter(String, BlogFilter)' has a cyclomatic complexity of 31. | 605–741 |
| ExcessiveMethodLength | Avoid really long methods. | 605–741 |
| NPathComplexity | The method 'getDaoFromFilter(String, BlogFilter)' has an NPath complexity of 1064448 | 605–741 |
| NcssCount | The method 'getDaoFromFilter(String, BlogFilter)' has a NCSS line count of 65. | 605–741 |
| ShortVariable | Avoid variables with short names like i | 614 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 636 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 654 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 659 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 664 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 668 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 672 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 678 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 678 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 679 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 685 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 685 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 691 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 691 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 699 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 705 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 709 |
| Rule | Violation | Line |
|---|---|---|
| MethodReturnsInternalArray | Returning '_arrayTagsId' may expose an internal array. | 60 |
| ArrayIsStoredDirectly | The user-supplied array 'arrayTagsId' is stored directly. | 67 |
| MethodReturnsInternalArray | Returning '_arrayId' may expose an internal array. | 97 |
| ArrayIsStoredDirectly | The user-supplied array 'arrayId' is stored directly. | 104 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 47–348 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 65–80 |
| Rule | Violation | Line |
|---|---|---|
| MethodReturnsInternalArray | Returning '_strTag' may expose an internal array. | 102 |
| ArrayIsStoredDirectly | The user-supplied array 'strTag' is stored directly. | 111 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 6–56 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 6–56 |
| DataClass | The class 'BlogSerializable' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=6) | 6–56 |
| ShortVariable | Avoid variables with short names like id | 8 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 8 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the id field if you want a default access modifier | 8 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 9 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the contentLabel field if you want a default access modifier | 9 |
| UncommentedEmptyConstructor | Document empty constructor | 11–13 |
| CommentRequired | Public method and constructor comments are required | 11–13 |
| ShortVariable | Avoid variables with short names like id | 15 |
| CommentRequired | Public method and constructor comments are required | 15–19 |
| ShortVariable | Avoid variables with short names like id | 34 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–84 |
| Rule | Violation | Line |
|---|---|---|
| ArrayIsStoredDirectly | The user-supplied array 'bytes' is stored directly. | 106 |
| MethodReturnsInternalArray | Returning '_bytes' may expose an internal array. | 118 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 45–239 |
| CommentRequired | Public method and constructor comments are required | 155 |
| CommentRequired | Public method and constructor comments are required | 234 |
| CommentRequired | Public method and constructor comments are required | 236 |
| CommentRequired | Public method and constructor comments are required | 238 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 40–145 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–172 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 42–172 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 61–74 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 65–113 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 121–124 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 40–145 |
| UncommentedEmptyConstructor | Document empty constructor | 56–59 |
| CommentRequired | Public method and constructor comments are required | 56–59 |
| CommentRequired | Public method and constructor comments are required | 61–66 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 48–299 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 42–45 |
| CommentRequired | Public method and constructor comments are required | 47–50 |
| CommentRequired | Public method and constructor comments are required | 52–55 |
| CommentRequired | Public method and constructor comments are required | 57–60 |
| CommentRequired | Public method and constructor comments are required | 62–65 |
| CommentRequired | Public method and constructor comments are required | 67–70 |
| CommentRequired | Public method and constructor comments are required | 72–75 |
| CommentRequired | Public method and constructor comments are required | 77–80 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_removedBlogsId' could be made final; it is only initialized in the declaration or constructor. | 76 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 81–84 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 95 |
| Rule | Violation | Line |
|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 52–396 |
| ShortVariable | Avoid variables with short names like p | 82 |
| ShortVariable | Avoid variables with short names like p | 103 |
| ShortVariable | Avoid variables with short names like p | 132 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 213 |
| ShortVariable | Avoid variables with short names like p | 263 |
| CyclomaticComplexity | The method 'selectPortletByType(int, PortletOrder, PortletFilter)' has a cyclomatic complexity of 10. | 321–375 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 326 |
| Rule | Violation | Line |
|---|---|---|
| AssignmentToNonFinalStatic | Possible unsafe assignment to a non-final static field in a constructor. | 56 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_singleton' | 56 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 61–67 |
| NonThreadSafeSingleton | Singleton is not thread safe | 76–79 |
| Rule | Violation | Line |
|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 67–70 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 72 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 74 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 76 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like p | 71 |
| ShortVariable | Avoid variables with short names like p | 109 |
| ShortVariable | Avoid variables with short names like p | 160 |
| CyclomaticComplexity | The method 'selectPortletByType(int, PortletOrder, PortletFilter)' has a cyclomatic complexity of 10. | 196–246 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 201 |
| Rule | Violation | Line |
|---|---|---|
| AssignmentToNonFinalStatic | Possible unsafe assignment to a non-final static field in a constructor. | 53 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 58–64 |
| NonThreadSafeSingleton | Singleton is not thread safe | 85–88 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–171 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 45–434 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 46–434 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 45–253 |
| Rule | Violation | Line |
|---|---|---|
| MissingOverride | The method 'load(int)' is missing an @Override annotation. | 74 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 42–204 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 43–204 |
| Rule | Violation | Line |
|---|---|---|
| ArrayIsStoredDirectly | The user-supplied array 'aPageTitle' is stored directly. | 66 |
| MethodReturnsInternalArray | Returning '_aPageName' may expose an internal array. | 78 |
| ArrayIsStoredDirectly | The user-supplied array 'aPortletTitle' is stored directly. | 87 |
| MethodReturnsInternalArray | Returning '_aPortletName' may expose an internal array. | 99 |
| CyclomaticComplexity | The method 'getSQLFilter()' has a cyclomatic complexity of 13. | 170–219 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–315 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 248 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 297 |
| Rule | Violation | Line |
|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 59–70 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 47–122 |
| ShortVariable | Avoid variables with short names like rt | 62 |
| ShortVariable | Avoid variables with short names like p | 68 |
| Rule | Violation | Line |
|---|---|---|
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 102 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 105 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 124 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 146 |
| Rule | Violation | Line |
|---|---|---|
| GodClass | Possible God Class (WMC=57, ATFD=99, TCC=0.000%) | 60–532 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 61–532 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 119 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 169 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 224 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 241–252 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 284 |
| CollapsibleIfStatements | These nested if statements could be combined | 303–307 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 345 |
| CommentRequired | Public method and constructor comments are required | 506–513 |
| CommentRequired | Public method and constructor comments are required | 515–522 |
| CommentRequired | Public method and constructor comments are required | 524–531 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like e | 216 |
| CollapsibleIfStatements | These nested if statements could be combined | 222–230 |
| ShortVariable | Avoid variables with short names like e | 258 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like se | 57 |
| ShortVariable | Avoid variables with short names like se | 66 |
| CommentRequired | Public method and constructor comments are required | 73–77 |
| CommentRequired | Public method and constructor comments are required | 79–84 |
| Rule | Violation | Line |
|---|---|---|
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 309 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 47–110 |
| ShortVariable | Avoid variables with short names like rt | 60 |
| ShortVariable | Avoid variables with short names like p | 66 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 41–68 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 40–57 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'BlogSearchItem' is suspected to be a Data Class (WOC=0.000%, NOPA=8, NOAM=0, WMC=1) | 43–91 |
| CommentRequired | Public method and constructor comments are required | 45–48 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 53 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 58 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 63 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 69 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 74 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 79 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 84 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 89 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–512 |
| ImmutableField | Private field '_indexer' could be made final; it is only initialized in the declaration or constructor. | 103 |
| ImmutableField | Private field '_nWriterMergeFactor' could be made final; it is only initialized in the declaration or constructor. | 104 |
| NonThreadSafeSingleton | Singleton is not thread safe | 145–148 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 147 characters appended. | 197 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 203 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 244 |
| CommentRequired | Public method and constructor comments are required | 270–285 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 280 |
| ShortVariable | Avoid variables with short names like si | 324 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 329 |
| NPathComplexity | The method 'prepareQueryForFilter(BlogSearchFilter)' has an NPath complexity of 1296 | 338–436 |
| NcssCount | The method 'prepareQueryForFilter(BlogSearchFilter)' has a NCSS line count of 64. | 338–436 |
| CyclomaticComplexity | The method 'prepareQueryForFilter(BlogSearchFilter)' has a cyclomatic complexity of 14. | 338–436 |
| CollapsibleIfStatements | These nested if statements could be combined | 418–425 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 434 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 434 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 435 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–425 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 140–211 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 145–208 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 198 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 229 |
| ShortVariable | Avoid variables with short names like ft | 234 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 293 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 294 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 295 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 296 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 297 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 298 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 49 |
| Rule | Violation | Line |
|---|---|---|
| PositionLiteralsFirstInCaseInsensitiveComparisons | Position literals first in String comparisons for EqualsIgnoreCase | 101 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–75 |
| CommentRequired | Public method and constructor comments are required | 44–49 |
| CommentRequired | Public method and constructor comments are required | 51–55 |
| CommentRequired | Public method and constructor comments are required | 57–61 |
| CommentRequired | Public method and constructor comments are required | 63–67 |
| CommentRequired | Public method and constructor comments are required | 69–73 |
| Rule | Violation | Line |
|---|---|---|
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 110 |
| AssignmentInOperand | Avoid assignments in operands | 110 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 171–172 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 220 |
| CommentRequired | Public method and constructor comments are required | 244–247 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 108 |
| ShortVariable | Avoid variables with short names like t | 119 |
| Rule | Violation | Line |
|---|---|---|
| ShortVariable | Avoid variables with short names like os | 104 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–2189 |
| ExcessiveClassLength | Avoid really long classes. | 124–2189 |
| TooManyFields | Too many fields | 124–2189 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 125–2189 |
| UnusedPrivateField | Avoid unused private fields such as 'MARK_NUMBER_MANDATORY_TAGS'. | 207 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 300 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsChecked' | 302 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsSorted' | 310 |
| CyclomaticComplexity | The method 'getManageBlogs(HttpServletRequest)' has a cyclomatic complexity of 28. | 328–517 |
| ExcessiveMethodLength | Avoid really long methods. | 328–517 |
| NPathComplexity | The method 'getManageBlogs(HttpServletRequest)' has an NPath complexity of 36240 | 328–517 |
| NcssCount | The method 'getManageBlogs(HttpServletRequest)' has a NCSS line count of 113. | 328–517 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 330 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 337 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 346–374 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 368 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 369 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 370 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 371 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 391 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 483 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 484–492 |
| AddEmptyString | Do not add empty strings | 502 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 526 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 529 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 556 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 557 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 592 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 717 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 766 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 807 |
| CyclomaticComplexity | The method 'doUpdatePriorityTag(HttpServletRequest)' has a cyclomatic complexity of 12. | 850–911 |
| ShortVariable | Avoid variables with short names like tg | 852 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 852 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 853 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 854 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 855 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 857 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 858 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 885 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 893 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 926 |
| CommentRequired | Public method and constructor comments are required | 996–1020 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1041 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1046–1069 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1150 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1219–1226 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1259–1266 |
| ShortVariable | Avoid variables with short names like tf | 1291 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 1312 |
| CyclomaticComplexity | The method 'addContent(HttpServletRequest)' has a cyclomatic complexity of 11. | 1338–1414 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1383–1410 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1434–1472 |
| NPathComplexity | The method 'doUpdatePriorityContent(HttpServletRequest)' has an NPath complexity of 1620 | 1484–1562 |
| CyclomaticComplexity | The method 'doUpdatePriorityContent(HttpServletRequest)' has a cyclomatic complexity of 17. | 1484–1562 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1486 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1487 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1488 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1489 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1491 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1492 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 1499–1518 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 1530 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 1538 |
| CyclomaticComplexity | The method 'updateContentType(HttpServletRequest)' has a cyclomatic complexity of 10. | 1570–1608 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1573 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1574 |
| LinguisticNaming | Linguistics Antipattern - The setter 'setContent' should not return any type except void linguistically | 1618–1644 |
| UnusedLocalVariable | Avoid unused local variables such as 'adminUser'. | 1704 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 1704 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1724 |
| CyclomaticComplexity | The method 'getConfirmArchiveBlogs(HttpServletRequest)' has a cyclomatic complexity of 10. | 1806–1853 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1839 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1886 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1945 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1956 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 1973 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 2016 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 2020 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 2078–2094 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 2102–2119 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–529 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 149 |
| CyclomaticComplexity | The method 'getManageBlogPublication(HttpServletRequest)' has a cyclomatic complexity of 12. | 150–212 |
| NPathComplexity | The method 'getManageBlogPublication(HttpServletRequest)' has an NPath complexity of 360 | 150–212 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 154 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 281 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 288 |
| LinguisticNaming | Linguistics Antipattern - The setter 'setFillFilter' should not return any type except void linguistically | 319–358 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 324–355 |
| UnusedFormalParameter | Avoid unused method parameters such as 'pFilter'. | 406 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 419 |
| Rule | Violation | Line |
|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 52–141 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsSorted' | 79 |
| AddEmptyString | Do not add empty strings | 135 |
| Rule | Violation | Line |
|---|---|---|
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 124 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 160 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 181 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 229 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 283 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 314 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 23–58 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 41 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Public method and constructor comments are required | 24–45 |
| Rule | Violation | Line |
|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 114 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 50–105 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 49–169 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 47–98 |
| Rule | Violation | Line |
|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 34–596 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsChecked' | 143 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_nIsUnpublished' | 145 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsSorted' | 148 |
| AddEmptyString | Do not add empty strings | 204 |
| CyclomaticComplexity | The method 'filterBlogIds(HttpServletRequest)' has a cyclomatic complexity of 12. | 227–271 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 281–312 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 306 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 307 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 309 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 310 |
| CommentRequired | Public method and constructor comments are required | 315–346 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 317 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 360 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 380–381 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 405 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 415 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 418 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 447 |
| CyclomaticComplexity | The method 'doModify(HttpServletRequest)' has a cyclomatic complexity of 11. | 465–526 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 480 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 483 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 518 |
| Rule | Violation | Line |
|---|---|---|
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 148 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 152 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 172 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 197 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 219 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 222 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like Blog | 54–655 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 413–416 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 433–436 |
| Rule | Violation | Line |
|---|---|---|
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 67 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 104 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 114–117 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 186–189 |
| Rule | Violation | Line |
|---|---|---|
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 111 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'fr.paris.lutece.portal.service.plugin.Plugin' due to existing import 'fr.paris.lutece.portal.service.plugin.Plugin' | 236 |
| Rule | Violation | Line |
|---|---|---|
| ShortClassName | Avoid short class names like Tag | 40–145 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.GregorianCalendar' due to existing import 'java.util.GregorianCalendar' | 89 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.GregorianCalendar' due to existing import 'java.util.GregorianCalendar' | 88 |
| Rule | Violation | Line |
|---|---|---|
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 66 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 87 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.lucene.document.IntPoint' | 48 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.blog.business.IndexerAction' due to existing import 'fr.paris.lutece.plugins.blog.business.IndexerAction' | 149 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' | 222 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' | 225 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' | 225 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.io.IOException' | 37 |
| UnusedImports | Avoid unused imports such as 'java.io.InputStream' | 38 |
| UnusedImports | Avoid unused imports such as 'java.net.URLConnection' | 40 |
| UnusedImports | Avoid unused imports such as 'java.util.Base64' | 47 |
| DuplicateImports | Avoid duplicate imports such as 'fr.paris.lutece.portal.service.i18n.I18nService' | 100 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' | 2181 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' | 2184 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' | 2184 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' | 285 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' | 286 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' | 294 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' | 299 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.sql.Date' | 36 |
| UnusedImports | Avoid unused imports such as 'java.text.SimpleDateFormat' | 38 |
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.blog.web.BlogPublicationJspBean' | 62 |
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' | 255 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' | 255 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessivePublicCount | This class has a bunch of public methods and attributes | 3 | 34–655 |
| ShortClassName | Avoid short class names like Blog | 4 | 54–655 |
| TooManyFields | Too many fields | 3 | 54–655 |
| CommentRequired | Public method and constructor comments are required | 3 | 291–294 |
| CommentRequired | Public method and constructor comments are required | 3 | 296–299 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 413–416 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 433–436 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 53–905 |
| CyclomaticComplexity | The method 'getDaoFromFilter(String, BlogFilter)' has a cyclomatic complexity of 31. | 3 | 605–741 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 605–741 |
| NPathComplexity | The method 'getDaoFromFilter(String, BlogFilter)' has an NPath complexity of 1064448 | 3 | 605–741 |
| NcssCount | The method 'getDaoFromFilter(String, BlogFilter)' has a NCSS line count of 65. | 3 | 605–741 |
| ShortVariable | Avoid variables with short names like i | 3 | 614 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 636 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 654 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 659 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 664 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 668 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 672 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 678 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 678 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 679 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 685 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 685 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 691 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 691 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 699 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 705 |
| UseStringBufferForStringAppends | Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings | 3 | 709 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 712 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 724 |
| GuardLogStatement | Logger calls should be surrounded by log level guards. | 2 | 735 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MethodReturnsInternalArray | Returning '_arrayTagsId' may expose an internal array. | 3 | 60 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 67 |
| ArrayIsStoredDirectly | The user-supplied array 'arrayTagsId' is stored directly. | 3 | 67 |
| MethodReturnsInternalArray | Returning '_arrayId' may expose an internal array. | 3 | 97 |
| ArrayIsStoredDirectly | The user-supplied array 'arrayId' is stored directly. | 3 | 104 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 104 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 114–117 |
| BooleanGetMethodName | A getX() method which returns a boolean should be named isX() | 4 | 186–189 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 47–348 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 65–80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MethodReturnsInternalArray | Returning '_strTag' may expose an internal array. | 3 | 102 |
| ArrayIsStoredDirectly | The user-supplied array 'strTag' is stored directly. | 3 | 111 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 111 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 6–56 |
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 6–56 |
| DataClass | The class 'BlogSerializable' is suspected to be a Data Class (WOC=0.000%, NOPA=0, NOAM=4, WMC=6) | 3 | 6–56 |
| ShortVariable | Avoid variables with short names like id | 3 | 8 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 8 |
| FieldNamingConventions | The field name 'id' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 8 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the id field if you want a default access modifier | 3 | 8 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 9 |
| FieldNamingConventions | The field name 'contentLabel' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 9 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the contentLabel field if you want a default access modifier | 3 | 9 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 11–13 |
| CommentRequired | Public method and constructor comments are required | 3 | 11–13 |
| ShortVariable | Avoid variables with short names like id | 3 | 15 |
| CommentRequired | Public method and constructor comments are required | 3 | 15–19 |
| ShortVariable | Avoid variables with short names like id | 3 | 34 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 36–84 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ArrayIsStoredDirectly | The user-supplied array 'bytes' is stored directly. | 3 | 106 |
| MethodReturnsInternalArray | Returning '_bytes' may expose an internal array. | 3 | 118 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 45–239 |
| CommentRequired | Public method and constructor comments are required | 3 | 155 |
| CommentRequired | Public method and constructor comments are required | 3 | 234 |
| CommentRequired | Public method and constructor comments are required | 3 | 236 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'fr.paris.lutece.portal.service.plugin.Plugin' due to existing import 'fr.paris.lutece.portal.service.plugin.Plugin' | 4 | 236 |
| CommentRequired | Public method and constructor comments are required | 3 | 238 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 40–145 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–172 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 42–172 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 61–74 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 65–113 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 121–124 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortClassName | Avoid short class names like Tag | 4 | 40–145 |
| CommentRequired | Header comments are required | 3 | 40–145 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 56–59 |
| CommentRequired | Public method and constructor comments are required | 3 | 56–59 |
| CommentRequired | Public method and constructor comments are required | 3 | 61–66 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 48–299 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 42–45 |
| CommentRequired | Public method and constructor comments are required | 3 | 47–50 |
| CommentRequired | Public method and constructor comments are required | 3 | 52–55 |
| CommentRequired | Public method and constructor comments are required | 3 | 57–60 |
| CommentRequired | Public method and constructor comments are required | 3 | 62–65 |
| CommentRequired | Public method and constructor comments are required | 3 | 67–70 |
| CommentRequired | Public method and constructor comments are required | 3 | 72–75 |
| CommentRequired | Public method and constructor comments are required | 3 | 77–80 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_removedBlogsId' could be made final; it is only initialized in the declaration or constructor. | 3 | 76 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 81–84 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.GregorianCalendar' due to existing import 'java.util.GregorianCalendar' | 4 | 89 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 95 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 52–396 |
| ShortVariable | Avoid variables with short names like p | 3 | 82 |
| ShortVariable | Avoid variables with short names like p | 3 | 103 |
| ShortVariable | Avoid variables with short names like p | 3 | 132 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 213 |
| ShortVariable | Avoid variables with short names like p | 3 | 263 |
| CyclomaticComplexity | The method 'selectPortletByType(int, PortletOrder, PortletFilter)' has a cyclomatic complexity of 10. | 3 | 321–375 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 326 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AssignmentToNonFinalStatic | Possible unsafe assignment to a non-final static field in a constructor. | 3 | 56 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_singleton' | 3 | 56 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 61–67 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 76–79 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 67–70 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 72 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 74 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 76 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.GregorianCalendar' due to existing import 'java.util.GregorianCalendar' | 4 | 88 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like p | 3 | 71 |
| ShortVariable | Avoid variables with short names like p | 3 | 109 |
| ShortVariable | Avoid variables with short names like p | 3 | 160 |
| CyclomaticComplexity | The method 'selectPortletByType(int, PortletOrder, PortletFilter)' has a cyclomatic complexity of 10. | 3 | 196–246 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 201 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AssignmentToNonFinalStatic | Possible unsafe assignment to a non-final static field in a constructor. | 3 | 53 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 58–64 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 85–88 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–171 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 45–434 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 46–434 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 45–253 |
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 45–253 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingOverride | The method 'load(int)' is missing an @Override annotation. | 3 | 74 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 42–204 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 43–204 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ArrayIsStoredDirectly | The user-supplied array 'aPageTitle' is stored directly. | 3 | 66 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 66 |
| MethodReturnsInternalArray | Returning '_aPageName' may expose an internal array. | 3 | 78 |
| ArrayIsStoredDirectly | The user-supplied array 'aPortletTitle' is stored directly. | 3 | 87 |
| UseVarargs | Consider using varargs for methods or constructors which take an array the last parameter. | 4 | 87 |
| MethodReturnsInternalArray | Returning '_aPortletName' may expose an internal array. | 3 | 99 |
| CyclomaticComplexity | The method 'getSQLFilter()' has a cyclomatic complexity of 13. | 3 | 170–219 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–315 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 248 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 297 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 59–70 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 47–122 |
| ShortVariable | Avoid variables with short names like rt | 3 | 62 |
| ShortVariable | Avoid variables with short names like p | 3 | 68 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 3 | 102 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 3 | 105 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 3 | 124 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 146 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| GodClass | Possible God Class (WMC=57, ATFD=99, TCC=0.000%) | 3 | 60–532 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 61–532 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 119 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 169 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 224 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 241–252 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 284 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 303–307 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 345 |
| CommentRequired | Public method and constructor comments are required | 3 | 506–513 |
| CommentRequired | Public method and constructor comments are required | 3 | 515–522 |
| CommentRequired | Public method and constructor comments are required | 3 | 524–531 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like e | 3 | 216 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 222–230 |
| ShortVariable | Avoid variables with short names like e | 3 | 258 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like se | 3 | 57 |
| ShortVariable | Avoid variables with short names like se | 3 | 66 |
| CommentRequired | Public method and constructor comments are required | 3 | 73–77 |
| CommentRequired | Public method and constructor comments are required | 3 | 79–84 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 309 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 47–110 |
| ShortVariable | Avoid variables with short names like rt | 3 | 60 |
| ShortVariable | Avoid variables with short names like p | 3 | 66 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 41–68 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 40–57 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'BlogSearchItem' is suspected to be a Data Class (WOC=0.000%, NOPA=8, NOAM=0, WMC=1) | 3 | 43–91 |
| CommentRequired | Public method and constructor comments are required | 3 | 45–48 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 53 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 58 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 63 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 69 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 74 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 79 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 84 |
| FieldDeclarationsShouldBeAtStartOfClass | Fields should be declared at the top of the class, before any method declarations, constructors, initializers or inner classes. | 3 | 89 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–512 |
| AvoidUsingVolatile | Use of modifier volatile is not recommended. | 2 | 93 |
| ImmutableField | Private field '_indexer' could be made final; it is only initialized in the declaration or constructor. | 3 | 103 |
| ImmutableField | Private field '_nWriterMergeFactor' could be made final; it is only initialized in the declaration or constructor. | 3 | 104 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 145–148 |
| InsufficientStringBufferDeclaration | StringBuffer constructor is initialized with size 16, but has at least 147 characters appended. | 3 | 197 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 203 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 244 |
| CommentRequired | Public method and constructor comments are required | 3 | 270–285 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 280 |
| ShortVariable | Avoid variables with short names like si | 3 | 324 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 329 |
| NPathComplexity | The method 'prepareQueryForFilter(BlogSearchFilter)' has an NPath complexity of 1296 | 3 | 338–436 |
| NcssCount | The method 'prepareQueryForFilter(BlogSearchFilter)' has a NCSS line count of 64. | 3 | 338–436 |
| CyclomaticComplexity | The method 'prepareQueryForFilter(BlogSearchFilter)' has a cyclomatic complexity of 14. | 3 | 338–436 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 418–425 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 434 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 434 |
| OptimizableToArrayCall | This call to Collection.toArray() may be optimizable | 3 | 435 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–425 |
| UnusedImports | Avoid unused imports such as 'org.apache.lucene.document.IntPoint' | 4 | 48 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 140–211 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 145–208 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'fr.paris.lutece.plugins.blog.business.IndexerAction' due to existing import 'fr.paris.lutece.plugins.blog.business.IndexerAction' | 4 | 149 |
| ConsecutiveLiteralAppends | StringBuffer (or StringBuilder).append is called 2 consecutive times with literals. Use a single append with a single combined String. | 3 | 198 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' | 4 | 222 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' | 4 | 225 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'org.apache.lucene.document.Document' due to existing import 'org.apache.lucene.document.Document' | 4 | 225 |
| UseLocaleWithCaseConversions | When doing a String.toLowerCase()/toUpperCase() call, use a Locale | 3 | 229 |
| ShortVariable | Avoid variables with short names like ft | 3 | 234 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 293 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 294 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 295 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 296 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 297 |
| ConsecutiveAppendsShouldReuse | StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. | 3 | 298 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 49 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| PositionLiteralsFirstInCaseInsensitiveComparisons | Position literals first in String comparisons for EqualsIgnoreCase | 3 | 101 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–75 |
| CommentRequired | Public method and constructor comments are required | 3 | 44–49 |
| CommentRequired | Public method and constructor comments are required | 3 | 51–55 |
| CommentRequired | Public method and constructor comments are required | 3 | 57–61 |
| CommentRequired | Public method and constructor comments are required | 3 | 63–67 |
| CommentRequired | Public method and constructor comments are required | 3 | 69–73 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 110 |
| AssignmentInOperand | Avoid assignments in operands | 3 | 110 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 171–172 |
| UseObjectForClearerAPI | Rather than using a lot of String arguments, consider using a container object for those values. | 3 | 220 |
| CommentRequired | Public method and constructor comments are required | 3 | 244–247 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 108 |
| ShortVariable | Avoid variables with short names like t | 3 | 119 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ShortVariable | Avoid variables with short names like os | 3 | 104 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–2189 |
| UnusedImports | Avoid unused imports such as 'java.io.IOException' | 4 | 37 |
| UnusedImports | Avoid unused imports such as 'java.io.InputStream' | 4 | 38 |
| UnusedImports | Avoid unused imports such as 'java.net.URLConnection' | 4 | 40 |
| UnusedImports | Avoid unused imports such as 'java.util.Base64' | 4 | 47 |
| DuplicateImports | Avoid duplicate imports such as 'fr.paris.lutece.portal.service.i18n.I18nService' | 4 | 100 |
| ExcessiveClassLength | Avoid really long classes. | 3 | 124–2189 |
| TooManyFields | Too many fields | 3 | 124–2189 |
| TooManyMethods | This class has too many methods, consider refactoring it. | 3 | 125–2189 |
| UnusedPrivateField | Avoid unused private fields such as 'MARK_NUMBER_MANDATORY_TAGS'. | 3 | 207 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 300 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsChecked' | 3 | 302 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsSorted' | 3 | 310 |
| FieldNamingConventions | The final field name '_blogServiceSession' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 318 |
| CyclomaticComplexity | The method 'getManageBlogs(HttpServletRequest)' has a cyclomatic complexity of 28. | 3 | 328–517 |
| ExcessiveMethodLength | Avoid really long methods. | 3 | 328–517 |
| NPathComplexity | The method 'getManageBlogs(HttpServletRequest)' has an NPath complexity of 36240 | 3 | 328–517 |
| NcssCount | The method 'getManageBlogs(HttpServletRequest)' has a NCSS line count of 113. | 3 | 328–517 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 330 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 3 | 337 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 346–374 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 368 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 369 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 370 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 371 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 391 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 483 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 484–492 |
| AddEmptyString | Do not add empty strings | 3 | 502 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 526 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 529 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 556 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 557 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 592 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 717 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 766 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 807 |
| CyclomaticComplexity | The method 'doUpdatePriorityTag(HttpServletRequest)' has a cyclomatic complexity of 12. | 3 | 850–911 |
| ShortVariable | Avoid variables with short names like tg | 3 | 852 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 852 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 853 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 854 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 855 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 857 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 858 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 885 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 893 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 926 |
| CommentRequired | Public method and constructor comments are required | 3 | 996–1020 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1041 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1046–1069 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1150 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1219–1226 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1259–1266 |
| ShortVariable | Avoid variables with short names like tf | 3 | 1291 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 1312 |
| CyclomaticComplexity | The method 'addContent(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 1338–1414 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1383–1410 |
| LocalVariableNamingConventions | The local variable name '_nDocContentInSession' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 1397 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1434–1472 |
| NPathComplexity | The method 'doUpdatePriorityContent(HttpServletRequest)' has an NPath complexity of 1620 | 3 | 1484–1562 |
| CyclomaticComplexity | The method 'doUpdatePriorityContent(HttpServletRequest)' has a cyclomatic complexity of 17. | 3 | 1484–1562 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1486 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1487 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1488 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1489 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1491 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1492 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 1499–1518 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 1530 |
| PositionLiteralsFirstInComparisons | Position literals first in String comparisons | 3 | 1538 |
| CyclomaticComplexity | The method 'updateContentType(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 1570–1608 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1573 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1574 |
| LinguisticNaming | Linguistics Antipattern - The setter 'setContent' should not return any type except void linguistically | 3 | 1618–1644 |
| UnusedLocalVariable | Avoid unused local variables such as 'adminUser'. | 3 | 1704 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 1704 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1724 |
| CyclomaticComplexity | The method 'getConfirmArchiveBlogs(HttpServletRequest)' has a cyclomatic complexity of 10. | 3 | 1806–1853 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1839 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1886 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1945 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1956 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 1973 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 2016 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 2020 |
| AvoidSynchronizedAtMethodLevel | Use block level rather than method level synchronization | 3 | 2078–2094 |
| ForLoopCanBeForeach | This for loop can be replaced by a foreach loop | 3 | 2102–2119 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' | 4 | 2181 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' | 4 | 2184 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Timestamp' due to existing import 'java.sql.Timestamp' | 4 | 2184 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–529 |
| UnnecessaryAnnotationValueElement | Avoid the use of value in annotations when its the only element | 3 | 149 |
| CyclomaticComplexity | The method 'getManageBlogPublication(HttpServletRequest)' has a cyclomatic complexity of 12. | 3 | 150–212 |
| NPathComplexity | The method 'getManageBlogPublication(HttpServletRequest)' has an NPath complexity of 360 | 3 | 150–212 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 154 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 281 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' | 4 | 285 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' | 4 | 286 |
| SimpleDateFormatNeedsLocale | When instantiating a SimpleDateFormat object, specify a Locale | 3 | 288 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' | 4 | 294 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.sql.Date' due to existing import 'java.sql.Date' | 4 | 299 |
| LinguisticNaming | Linguistics Antipattern - The setter 'setFillFilter' should not return any type except void linguistically | 3 | 319–358 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 324–355 |
| UnusedFormalParameter | Avoid unused method parameters such as 'pFilter'. | 3 | 406 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 419 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| MissingSerialVersionUID | Classes implementing Serializable should set a serialVersionUID | 3 | 52–141 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsSorted' | 3 | 79 |
| AddEmptyString | Do not add empty strings | 3 | 135 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 124 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 160 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 181 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 229 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 283 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 314 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 23–58 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 41 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Public method and constructor comments are required | 3 | 24–45 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 114 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 50–105 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 49–169 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 47–98 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ExcessiveImports | A high number of imports can indicate a high degree of coupling within an object. | 3 | 34–596 |
| UnusedImports | Avoid unused imports such as 'java.sql.Date' | 4 | 36 |
| UnusedImports | Avoid unused imports such as 'java.text.SimpleDateFormat' | 4 | 38 |
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.plugins.blog.web.BlogPublicationJspBean' | 4 | 62 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsChecked' | 3 | 143 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_nIsUnpublished' | 3 | 145 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bIsSorted' | 3 | 148 |
| AddEmptyString | Do not add empty strings | 3 | 204 |
| CyclomaticComplexity | The method 'filterBlogIds(HttpServletRequest)' has a cyclomatic complexity of 12. | 3 | 227–271 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 281–312 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 306 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 307 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 309 |
| NullAssignment | Assigning an Object to null is a code smell. Consider refactoring. | 3 | 310 |
| CommentRequired | Public method and constructor comments are required | 3 | 315–346 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 317 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 360 |
| ConfusingTernary | Avoid if (x != y) ..; else ..; | 3 | 380–381 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 405 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 415 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 418 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 447 |
| CyclomaticComplexity | The method 'doModify(HttpServletRequest)' has a cyclomatic complexity of 11. | 3 | 465–526 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 480 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 483 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 518 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 148 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 152 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 172 |
| AvoidLiteralsInIfCondition | Avoid using Literals in Conditional Statements | 3 | 197 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 219 |
| PrematureDeclaration | Avoid declaring a variable if it is unreferenced before a possible exit point. | 3 | 222 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' | 4 | 255 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' | 4 | 255 |
