The following document contains the results of PMD 6.13.0.
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'commonEmail' doesn't match '_[a-z][a-zA-Z0-9]*' | 8 |
| FieldNamingConventions | The field name 'commonLastName' doesn't match '_[a-z][a-zA-Z0-9]*' | 11 |
| FieldNamingConventions | The field name 'firstName' doesn't match '_[a-z][a-zA-Z0-9]*' | 14 |
| FieldNamingConventions | The field name 'birthDate' doesn't match '_[a-z][a-zA-Z0-9]*' | 17 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'processList' doesn't match '_[a-z][a-zA-Z0-9]*' | 9 |
| FieldNamingConventions | The field name 'levelList' doesn't match '_[a-z][a-zA-Z0-9]*' | 10 |
| FieldNamingConventions | The field name 'attributeKeyList' doesn't match '_[a-z][a-zA-Z0-9]*' | 11 |
| Rule | Violation | Line |
|---|---|---|
| FieldNamingConventions | The field name 'referential' doesn't match '_[a-z][a-zA-Z0-9]*' | 7 |
| FieldNamingConventions | The field name 'contract' doesn't match '_[a-z][a-zA-Z0-9]*' | 8 |
| Rule | Violation | Line |
|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 66–276 |
| FieldNamingConventions | The static field name 'instance' doesn't match '_[a-z][a-zA-Z0-9]*' | 75 |
| Rule | Violation | Line |
|---|---|---|
| ClassNamingConventions | The utility class name 'IdentityRestConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 36–53 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 5–89 |
| DataClass | The class 'IdentitySearchCriteria' is suspected to be a Data Class (WOC=16.667%, NOPA=0, NOAM=8, WMC=13) | 5–89 |
| CommentRequired | Public method and constructor comments are required | 20–22 |
| UncommentedEmptyConstructor | Document empty constructor | 20–22 |
| CommentRequired | Public method and constructor comments are required | 25–31 |
| CommentRequired | Public method and constructor comments are required | 34–37 |
| CommentRequired | Public method and constructor comments are required | 39–42 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 7–34 |
| ImmutableField | Private field 'processList' could be made final; it is only initialized in the declaration or constructor. | 9 |
| ImmutableField | Private field 'levelList' could be made final; it is only initialized in the declaration or constructor. | 10 |
| ImmutableField | Private field 'attributeKeyList' could be made final; it is only initialized in the declaration or constructor. | 11 |
| CommentRequired | Public method and constructor comments are required | 13–18 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 5–26 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the referential field if you want a default access modifier | 7 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 7 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 8 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the contract field if you want a default access modifier | 8 |
| CommentRequired | Public method and constructor comments are required | 10–14 |
| Rule | Violation | Line |
|---|---|---|
| DataClass | The class 'IdentityPickerResourceService' is suspected to be a Data Class (WOC=0.000%, NOPA=5, NOAM=0, WMC=2) | 14–98 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 29–32 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 66–276 |
| ImmutableField | Private field '_referentialService' could be made final; it is only initialized in the declaration or constructor. | 79 |
| ImmutableField | Private field '_serviceContract' could be made final; it is only initialized in the declaration or constructor. | 80 |
| NonThreadSafeSingleton | Singleton is not thread safe | 98–102 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 36–53 |
| Rule | Violation | Line |
|---|---|---|
| CommentRequired | Header comments are required | 62–221 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 99 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 135 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 164 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 190 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 199 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 5–89 |
| DataClass | The class 'IdentitySearchCriteria' is suspected to be a Data Class (WOC=16.667%, NOPA=0, NOAM=8, WMC=13) | 3 | 5–89 |
| FieldNamingConventions | The field name 'commonEmail' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 8 |
| FieldNamingConventions | The field name 'commonLastName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 11 |
| FieldNamingConventions | The field name 'firstName' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 14 |
| FieldNamingConventions | The field name 'birthDate' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 17 |
| CommentRequired | Public method and constructor comments are required | 3 | 20–22 |
| UncommentedEmptyConstructor | Document empty constructor | 3 | 20–22 |
| CommentRequired | Public method and constructor comments are required | 3 | 25–31 |
| CommentRequired | Public method and constructor comments are required | 3 | 34–37 |
| CommentRequired | Public method and constructor comments are required | 3 | 39–42 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 7–34 |
| FieldNamingConventions | The field name 'processList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 9 |
| ImmutableField | Private field 'processList' could be made final; it is only initialized in the declaration or constructor. | 3 | 9 |
| ImmutableField | Private field 'levelList' could be made final; it is only initialized in the declaration or constructor. | 3 | 10 |
| FieldNamingConventions | The field name 'levelList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 10 |
| FieldNamingConventions | The field name 'attributeKeyList' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 11 |
| ImmutableField | Private field 'attributeKeyList' could be made final; it is only initialized in the declaration or constructor. | 3 | 11 |
| CommentRequired | Public method and constructor comments are required | 3 | 13–18 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 5–26 |
| FieldNamingConventions | The field name 'referential' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 7 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the referential field if you want a default access modifier | 3 | 7 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 7 |
| DefaultPackage | Use explicit scoping instead of the default package private level | 3 | 8 |
| CommentDefaultAccessModifier | To avoid mistakes add a comment at the beginning of the contract field if you want a default access modifier | 3 | 8 |
| FieldNamingConventions | The field name 'contract' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 8 |
| CommentRequired | Public method and constructor comments are required | 3 | 10–14 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| DataClass | The class 'IdentityPickerResourceService' is suspected to be a Data Class (WOC=0.000%, NOPA=5, NOAM=0, WMC=2) | 3 | 14–98 |
| CallSuperInConstructor | It is a good practice to call super() in a constructor | 3 | 29–32 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassWithOnlyPrivateConstructorsShouldBeFinal | A class which only has private constructors should be final | 1 | 66–276 |
| CommentRequired | Header comments are required | 3 | 66–276 |
| FieldNamingConventions | The static field name 'instance' doesn't match '_[a-z][a-zA-Z0-9]*' | 1 | 75 |
| ImmutableField | Private field '_referentialService' could be made final; it is only initialized in the declaration or constructor. | 3 | 79 |
| ImmutableField | Private field '_serviceContract' could be made final; it is only initialized in the declaration or constructor. | 3 | 80 |
| NonThreadSafeSingleton | Singleton is not thread safe | 3 | 98–102 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| ClassNamingConventions | The utility class name 'IdentityRestConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' | 1 | 36–53 |
| CommentRequired | Header comments are required | 3 | 36–53 |
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CommentRequired | Header comments are required | 3 | 62–221 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 99 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 135 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 164 |
| UseConcurrentHashMap | If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation | 3 | 190 |
| AvoidCatchingGenericException | Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block | 3 | 199 |
