The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FormalParameterNamingConventions | The method parameter name '_bNbMaximumUsersReached' doesn't match '[a-z][a-zA-Z0-9]*' | 87 |
| Rule | Violation | Line |
|---|---|---|
| BooleanInstantiation | Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 187 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 38–48 |
| Rule | Violation | Line |
|---|---|---|
| ImmutableField | Private field '_listSessionsActive' could be made final; it is only initialized in the declaration or constructor. | 47 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 47 |
| NonThreadSafeSingleton | Singleton is not thread safe | 56–59 |
| Rule | Violation | Line |
|---|---|---|
| RedundantFieldInitializer | Avoid using redundant field initializer for '_filterConfig' | 104 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bActivate' | 106 |
| UncommentedEmptyMethodBody | Document empty method body | 113–114 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 139–167 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 150 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 205 |
| UnusedPrivateMethod | Avoid unused private methods such as 'getDateTimeString(long)'. | 222 |
| Rule | Violation | Line |
|---|---|---|
| UncommentedEmptyMethodBody | Document empty method body | 53–54 |
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.util.date.DateUtil' | 66 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' | 226 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 38–48 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ImmutableField | Private field '_listSessionsActive' could be made final; it is only initialized in the declaration or constructor. | 3 | 47 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 47 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 56–59 |
| FormalParameterNamingConventions | The method parameter name '_bNbMaximumUsersReached' doesn't match '[a-z][a-zA-Z0-9]*' | 1 | 87 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'fr.paris.lutece.util.date.DateUtil' | 4 | 66 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_filterConfig' | 3 | 104 |
| RedundantFieldInitializer | Avoid using redundant field initializer for '_bActivate' | 3 | 106 |
| UncommentedEmptyMethodBody | Document empty method body | 3 | 113–114 |
| AvoidDeeplyNestedIfStmts | Deeply nested if..then statements are hard to read | 3 | 139–167 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 150 |
| BooleanInstantiation | Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 2 | 187 |
| UseDiamondOperator | Explicit type arguments can be replaced by Diamond Operator | 3 | 205 |
| UnusedPrivateMethod | Avoid unused private methods such as 'getDateTimeString(long)'. | 3 | 222 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.util.Date' due to existing import 'java.util.Date' | 4 | 226 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UncommentedEmptyMethodBody | Document empty method body | 3 | 53–54 |