Fork me on GitHub

Résultats PMD

Le document suivant contient les résultats de PMD 6.13.0.

Violations By Priority

Priority 1

fr/paris/lutece/plugins/identitystore/v1/web/rs/service/IdentityTransportApiManagerRest.java

Rule Violation Ligne
FieldNamingConventions The constant name '_objectMapper' doesn't match '[A-Z][A-Z_0-9]*' 62

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/AbstractIdentityTransportRest.java

Rule Violation Ligne
FieldNamingConventions The constant name '_mapper' doesn't match '[A-Z][A-Z_0-9]*' 64

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/IdentityTransportApiManagerRest.java

Rule Violation Ligne
FieldNamingConventions The constant name '_objectMapper' doesn't match '[A-Z][A-Z_0-9]*' 61

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/MockIdentityTransportRest.java

Rule Violation Ligne
FieldNamingConventions The final field name 'CONNECTION_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 57
FieldNamingConventions The final field name 'CUSTOMER_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 58

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/HttpApiManagerAccessTransport.java

Rule Violation Ligne
FieldNamingConventions The constant name '_objectMapper' doesn't match '[A-Z][A-Z_0-9]*' 61

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/IdentityTransportRest.java

Rule Violation Ligne
FieldNamingConventions The final field name '_strIdentityStoreEndPoint' doesn't match '[a-z][a-zA-Z0-9]*' 67

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/MockIdentityTransportRest.java

Rule Violation Ligne
FieldNamingConventions The final field name '_listIdentities' doesn't match '[a-z][a-zA-Z0-9]*' 74
FieldNamingConventions The final field name 'CONNECTION_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 75
FieldNamingConventions The final field name 'CUSTOMER_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 76

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/ServiceContractTransportRest.java

Rule Violation Ligne
FieldNamingConventions The final field name '_strIdentityStoreEndPoint' doesn't match '[a-z][a-zA-Z0-9]*' 54

fr/paris/lutece/plugins/identitystore/v3/web/service/CustomResponseStatusValidator.java

Rule Violation Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 3860
FieldNamingConventions The static field name 'instance' doesn't match '_[a-z][a-zA-Z0-9]*' 40

Priority 2

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/IdentityTransportApiManagerRest.java

Rule Violation Ligne
GuardLogStatement Logger calls should be surrounded by log level guards. 132

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/MockServiceContractTransportDataStore.java

Rule Violation Ligne
GuardLogStatement Logger calls should be surrounded by log level guards. 100
GuardLogStatement Logger calls should be surrounded by log level guards. 117
GuardLogStatement Logger calls should be surrounded by log level guards. 126
GuardLogStatement Logger calls should be surrounded by log level guards. 135
GuardLogStatement Logger calls should be surrounded by log level guards. 144
GuardLogStatement Logger calls should be surrounded by log level guards. 153

Priority 3

fr/paris/lutece/plugins/identitystore/v1/web/rs/service/AbstractIdentityTransportRest.java

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 65445
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 127
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 132
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 132
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 136
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 136
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'identityDto' 143
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 154
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 154
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 157
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 157
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 183
ShortVariable Avoid variables with short names like in 217
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'in' 219
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 230
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 230
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 233
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 233
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 261
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 266
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 266
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 270
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 270
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'responseDto' 276
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 284
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 284
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 287
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 287
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 322
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 336
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 350
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 365
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 384
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 385
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 406
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 422
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 438

fr/paris/lutece/plugins/identitystore/v1/web/rs/service/IdentityTransportApiManagerRest.java

Rule Violation Ligne
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 109
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 109
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 110
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 110
EmptyCatchBlock Avoid empty catch blocks 131133

fr/paris/lutece/plugins/identitystore/v1/web/service/HttpAccessTransport.java

Rule Violation Ligne
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 66
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 66
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 74
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 92
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 92
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 104
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 135
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 152
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 152
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 159
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 200
ShortVariable Avoid variables with short names like e 218

fr/paris/lutece/plugins/identitystore/v1/web/service/IIdentityTransportProvider.java

Rule Violation Ligne
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 135

fr/paris/lutece/plugins/identitystore/v1/web/service/IdentityService.java

Rule Violation Ligne
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 102
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 120
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 213
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 230

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/AbstractIdentityTransportRest.java

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 63432
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 125
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 129
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 129
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 133
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 133
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'identityDto' 140
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 153
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 153
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 156
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 156
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 183
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 199
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 199
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 202
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 202
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 233
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 233
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 237
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 237
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'responseDto' 243
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 252
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 256
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 256
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 260
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 260
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 282
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 282
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 309
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 323
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 337
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 352
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 371
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 372
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 393
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 409
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 425

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/IdentityTransportApiManagerRest.java

Rule Violation Ligne
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 108
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 108
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 109
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 109

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/MockIdentityTransportDataStore.java

Rule Violation Ligne
CallSuperInConstructor It is a good practice to call super() in a constructor 6265
CommentRequired Public method and constructor comments are required 6265
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 71
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 87
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 98
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 109
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 120
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 165
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 187

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/MockIdentityTransportRest.java

Rule Violation Ligne
ImmutableField Private field '_listIdentities' could be made final; it is only initialized in the declaration or constructor. 56
FinalFieldCouldBeStatic This final field could be made static 57
FinalFieldCouldBeStatic This final field could be made static 58
CommentRequired Public method and constructor comments are required 6064
CallSuperInConstructor It is a good practice to call super() in a constructor 6064
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 63
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 70
CyclomaticComplexity The method 'getIdentity(String, String, String)' has a cyclomatic complexity of 10. 7088
CollapsibleIfStatements These nested if statements could be combined 8083
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 94
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 106
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 117
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 138

fr/paris/lutece/plugins/identitystore/v2/web/service/HttpAccessTransport.java

Rule Violation Ligne
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 68
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 68
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 76
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 94
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 94
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 106
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 124
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 124
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 137
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 168
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 185
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 185
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 192
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 202
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 233
ShortVariable Avoid variables with short names like e 251

fr/paris/lutece/plugins/identitystore/v2/web/service/IIdentityTransportProvider.java

Rule Violation Ligne
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 139

fr/paris/lutece/plugins/identitystore/v2/web/service/IdentityService.java

Rule Violation Ligne
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 103
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 121
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 141
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 158
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 176
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 211
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 226

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/AbstractTransportRest.java

Rule Violation Ligne
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 4387

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/HttpAccessTransport.java

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 62437
CommentRequired Public method and constructor comments are required 6669
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 79
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 89
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 106
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 120
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 137
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 151
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 168
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 182
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 214
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 246
UnusedLocalVariable Avoid unused local variables such as 'mapHeadersResponse'. 258
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 258
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 258
ConfusingTernary Avoid if (x != y) ..; else ..; 277
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 283
ShortVariable Avoid variables with short names like e 301
MissingOverride The method 'setApiEndPointUrl(String)' is missing an @Override annotation. 335338
MissingOverride The method 'getApiEndPointUrl()' is missing an @Override annotation. 345348
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 369
ShortVariable Avoid variables with short names like er 372
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 403
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 421
ShortVariable Avoid variables with short names like er 424

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/HttpApiManagerAccessTransport.java

Rule Violation Ligne
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 99
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 99
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 100
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 100
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 108
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 115

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/IdentityTransportRest.java

Rule Violation Ligne
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 88
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 93
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 115
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 136
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 155
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 176
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 197
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 213
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 232
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 251
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 267
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 283
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 299
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 315
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 335

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/MockIdentityTransportDataStore.java

Rule Violation Ligne
CommentRequired Public method and constructor comments are required 8084
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 90
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 105
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 116
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 208
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 230

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/MockIdentityTransportRest.java

Rule Violation Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34298
FinalFieldCouldBeStatic This final field could be made static 75
FinalFieldCouldBeStatic This final field could be made static 76
CommentRequired Public method and constructor comments are required 8690
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 89
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 96
ConfusingTernary Avoid if (x != y) ..; else ..; 118

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/MockServiceContractTransportDataStore.java

Rule Violation Ligne
CommentRequired Header comments are required 52204
UnusedFormalParameter Avoid unused constructor parameters such as 'httpTransport'. 80
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 173
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 196

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/ServiceContractTransportRest.java

Rule Violation Ligne
CommentRequired Header comments are required 50181
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 74
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 88
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 103
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 118
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 134
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 151
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 156
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 170
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 175

fr/paris/lutece/plugins/identitystore/v3/web/service/CustomResponseStatusValidator.java

Rule Violation Ligne
CommentRequired Header comments are required 3860
CommentRequired Public method and constructor comments are required 4249
NonThreadSafeSingleton Singleton is not thread safe 4447
ShortVariable Avoid variables with short names like i 56

fr/paris/lutece/plugins/identitystore/v3/web/service/IHttpTransportProvider.java

Rule Violation Ligne
UnnecessaryModifier Unnecessary modifier 'public' on method 'setApiEndPointUrl': the method is declared in an interface type 195
UnnecessaryModifier Unnecessary modifier 'public' on method 'getApiEndPointUrl': the method is declared in an interface type 202

fr/paris/lutece/plugins/identitystore/v3/web/service/IIdentityTransportProvider.java

Rule Violation Ligne
TooManyMethods This class has too many methods, consider refactoring it. 60285
CommentRequired Public method and constructor comments are required 280284

fr/paris/lutece/plugins/identitystore/v3/web/service/IServiceContractTransportProvider.java

Rule Violation Ligne
CommentRequired Header comments are required 44154
CommentRequired Public method and constructor comments are required 149153

fr/paris/lutece/plugins/identitystore/v3/web/service/IdentityService.java

Rule Violation Ligne
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 121
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 143
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 163
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 187
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 209
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 345
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 347

Priority 4

fr/paris/lutece/plugins/identitystore/v1/web/rs/service/AbstractIdentityTransportRest.java

Rule Violation Ligne
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.identitystore.web.exception.IdentityNotFoundException' 47
UnusedImports Avoid unused imports such as 'org.apache.log4j.Logger' 54

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/MockServiceContractTransportDataStore.java

Rule Violation Ligne
UnusedImports Avoid unused imports such as 'org.apache.log4j.Logger' 47

Fichiers

fr/paris/lutece/plugins/identitystore/v1/web/rs/service/AbstractIdentityTransportRest.java

Rule Violation Priority Ligne
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.identitystore.web.exception.IdentityNotFoundException' 4 47
UnusedImports Avoid unused imports such as 'org.apache.log4j.Logger' 4 54
TooManyMethods This class has too many methods, consider refactoring it. 3 65445
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 127
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 132
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 132
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 136
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 136
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'identityDto' 3 143
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 154
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 154
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 157
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 157
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 183
ShortVariable Avoid variables with short names like in 3 217
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'in' 3 219
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 230
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 230
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 233
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 233
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 261
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 266
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 266
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 270
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 270
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'responseDto' 3 276
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 284
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 284
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 287
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 287
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 322
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 336
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 350
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 365
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 384
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 385
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 406
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 422
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 438

fr/paris/lutece/plugins/identitystore/v1/web/rs/service/IdentityTransportApiManagerRest.java

Rule Violation Priority Ligne
FieldNamingConventions The constant name '_objectMapper' doesn't match '[A-Z][A-Z_0-9]*' 1 62
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 109
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 109
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 110
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 110
EmptyCatchBlock Avoid empty catch blocks 3 131133

fr/paris/lutece/plugins/identitystore/v1/web/service/HttpAccessTransport.java

Rule Violation Priority Ligne
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 66
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 66
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 74
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 92
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 92
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 104
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 135
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 152
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 152
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 159
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 200
ShortVariable Avoid variables with short names like e 3 218

fr/paris/lutece/plugins/identitystore/v1/web/service/IIdentityTransportProvider.java

Rule Violation Priority Ligne
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 135

fr/paris/lutece/plugins/identitystore/v1/web/service/IdentityService.java

Rule Violation Priority Ligne
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 102
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 120
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 213
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 230

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/AbstractIdentityTransportRest.java

Rule Violation Priority Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 63432
FieldNamingConventions The constant name '_mapper' doesn't match '[A-Z][A-Z_0-9]*' 1 64
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 125
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 129
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 129
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 133
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 133
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'identityDto' 3 140
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 153
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 153
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 156
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 156
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 183
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 199
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 199
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 202
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 202
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 233
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 233
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 237
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 237
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'responseDto' 3 243
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 252
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 256
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 256
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 260
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 260
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 282
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 282
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 309
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 323
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 337
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 352
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 371
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 372
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 393
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 409
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 425

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/IdentityTransportApiManagerRest.java

Rule Violation Priority Ligne
FieldNamingConventions The constant name '_objectMapper' doesn't match '[A-Z][A-Z_0-9]*' 1 61
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 108
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 108
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 109
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 109
GuardLogStatement Logger calls should be surrounded by log level guards. 2 132

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/MockIdentityTransportDataStore.java

Rule Violation Priority Ligne
CallSuperInConstructor It is a good practice to call super() in a constructor 3 6265
CommentRequired Public method and constructor comments are required 3 6265
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 71
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 87
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 98
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 109
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 120
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 165
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 187

fr/paris/lutece/plugins/identitystore/v2/web/rs/service/MockIdentityTransportRest.java

Rule Violation Priority Ligne
ImmutableField Private field '_listIdentities' could be made final; it is only initialized in the declaration or constructor. 3 56
FieldNamingConventions The final field name 'CONNECTION_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 1 57
FinalFieldCouldBeStatic This final field could be made static 3 57
FieldNamingConventions The final field name 'CUSTOMER_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 1 58
FinalFieldCouldBeStatic This final field could be made static 3 58
CommentRequired Public method and constructor comments are required 3 6064
CallSuperInConstructor It is a good practice to call super() in a constructor 3 6064
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 63
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 70
CyclomaticComplexity The method 'getIdentity(String, String, String)' has a cyclomatic complexity of 10. 3 7088
CollapsibleIfStatements These nested if statements could be combined 3 8083
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 94
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 106
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 117
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 138

fr/paris/lutece/plugins/identitystore/v2/web/service/HttpAccessTransport.java

Rule Violation Priority Ligne
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 68
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 68
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 76
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 94
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 94
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 106
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 124
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 124
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 137
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 168
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 185
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 185
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 192
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 202
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 233
ShortVariable Avoid variables with short names like e 3 251

fr/paris/lutece/plugins/identitystore/v2/web/service/IIdentityTransportProvider.java

Rule Violation Priority Ligne
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 139

fr/paris/lutece/plugins/identitystore/v2/web/service/IdentityService.java

Rule Violation Priority Ligne
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 103
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 121
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 141
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 158
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 176
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 211
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 226

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/AbstractTransportRest.java

Rule Violation Priority Ligne
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 3 4387

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/HttpAccessTransport.java

Rule Violation Priority Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 62437
CommentRequired Public method and constructor comments are required 3 6669
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 79
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 89
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 106
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 120
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 137
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 151
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 168
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 182
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 214
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 246
UnusedLocalVariable Avoid unused local variables such as 'mapHeadersResponse'. 3 258
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 258
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 258
ConfusingTernary Avoid if (x != y) ..; else ..; 3 277
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 283
ShortVariable Avoid variables with short names like e 3 301
MissingOverride The method 'setApiEndPointUrl(String)' is missing an @Override annotation. 3 335338
MissingOverride The method 'getApiEndPointUrl()' is missing an @Override annotation. 3 345348
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 369
ShortVariable Avoid variables with short names like er 3 372
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 403
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 421
ShortVariable Avoid variables with short names like er 3 424

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/HttpApiManagerAccessTransport.java

Rule Violation Priority Ligne
FieldNamingConventions The constant name '_objectMapper' doesn't match '[A-Z][A-Z_0-9]*' 1 61
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 99
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 99
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 100
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 100
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 108
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 115

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/IdentityTransportRest.java

Rule Violation Priority Ligne
FieldNamingConventions The final field name '_strIdentityStoreEndPoint' doesn't match '[a-z][a-zA-Z0-9]*' 1 67
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 88
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 93
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 115
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 136
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 155
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 176
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 197
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 213
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 232
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 251
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 267
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 283
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 299
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 315
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 335

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/MockIdentityTransportDataStore.java

Rule Violation Priority Ligne
CommentRequired Public method and constructor comments are required 3 8084
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 90
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 105
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 116
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 208
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 230

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/MockIdentityTransportRest.java

Rule Violation Priority Ligne
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34298
FieldNamingConventions The final field name '_listIdentities' doesn't match '[a-z][a-zA-Z0-9]*' 1 74
FinalFieldCouldBeStatic This final field could be made static 3 75
FieldNamingConventions The final field name 'CONNECTION_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 1 75
FinalFieldCouldBeStatic This final field could be made static 3 76
FieldNamingConventions The final field name 'CUSTOMER_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 1 76
CommentRequired Public method and constructor comments are required 3 8690
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 89
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 96
ConfusingTernary Avoid if (x != y) ..; else ..; 3 118

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/MockServiceContractTransportDataStore.java

Rule Violation Priority Ligne
UnusedImports Avoid unused imports such as 'org.apache.log4j.Logger' 4 47
CommentRequired Header comments are required 3 52204
UnusedFormalParameter Avoid unused constructor parameters such as 'httpTransport'. 3 80
GuardLogStatement Logger calls should be surrounded by log level guards. 2 100
GuardLogStatement Logger calls should be surrounded by log level guards. 2 117
GuardLogStatement Logger calls should be surrounded by log level guards. 2 126
GuardLogStatement Logger calls should be surrounded by log level guards. 2 135
GuardLogStatement Logger calls should be surrounded by log level guards. 2 144
GuardLogStatement Logger calls should be surrounded by log level guards. 2 153
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 173
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 196

fr/paris/lutece/plugins/identitystore/v3/web/rs/service/ServiceContractTransportRest.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 50181
FieldNamingConventions The final field name '_strIdentityStoreEndPoint' doesn't match '[a-z][a-zA-Z0-9]*' 1 54
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 74
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 88
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 103
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 118
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 134
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 151
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 156
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 170
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 175

fr/paris/lutece/plugins/identitystore/v3/web/service/CustomResponseStatusValidator.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 3860
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 3860
FieldNamingConventions The static field name 'instance' doesn't match '_[a-z][a-zA-Z0-9]*' 1 40
CommentRequired Public method and constructor comments are required 3 4249
NonThreadSafeSingleton Singleton is not thread safe 3 4447
ShortVariable Avoid variables with short names like i 3 56

fr/paris/lutece/plugins/identitystore/v3/web/service/IHttpTransportProvider.java

Rule Violation Priority Ligne
UnnecessaryModifier Unnecessary modifier 'public' on method 'setApiEndPointUrl': the method is declared in an interface type 3 195
UnnecessaryModifier Unnecessary modifier 'public' on method 'getApiEndPointUrl': the method is declared in an interface type 3 202

fr/paris/lutece/plugins/identitystore/v3/web/service/IIdentityTransportProvider.java

Rule Violation Priority Ligne
TooManyMethods This class has too many methods, consider refactoring it. 3 60285
CommentRequired Public method and constructor comments are required 3 280284

fr/paris/lutece/plugins/identitystore/v3/web/service/IServiceContractTransportProvider.java

Rule Violation Priority Ligne
CommentRequired Header comments are required 3 44154
CommentRequired Public method and constructor comments are required 3 149153

fr/paris/lutece/plugins/identitystore/v3/web/service/IdentityService.java

Rule Violation Priority Ligne
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 121
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 143
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 163
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 187
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 209
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 345
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 347