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]*' 77
FieldNamingConventions The final field name '_strIdentityPath' doesn't match '[a-z][a-zA-Z0-9]*' 78
FieldNamingConventions The final field name '_strTaskStackPath' doesn't match '[a-z][a-zA-Z0-9]*' 79

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]*' 83
FieldNamingConventions The final field name 'CONNECTION_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 84
FieldNamingConventions The final field name 'CUSTOMER_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 85

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
FieldNamingConventions The final field name '_strIdentityPath' doesn't match '[a-z][a-zA-Z0-9]*' 55

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
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 136
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 136
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'identityDto' 143
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 154
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 157
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 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
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 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
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 284
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 284
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 287
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 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
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 109
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 109
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 110
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 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
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 92
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 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
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 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
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 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
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 237
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 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
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 256
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 260
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 260
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 282
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 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
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 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
CommentRequired Public method and constructor comments are required 6265
CallSuperInConstructor It is a good practice to call super() in a constructor 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
CyclomaticComplexity The method 'getIdentity(String, String, String)' has a cyclomatic complexity of 10. 7088
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 70
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
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 68
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 68
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 76
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 94
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 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
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 258
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 258
UnusedLocalVariable Avoid unused local variables such as 'mapHeadersResponse'. 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
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 99
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 100
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 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
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34464
TooManyMethods This class has too many methods, consider refactoring it. 75464
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 117
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 122
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 144
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 165
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 184
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 205
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 226
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 242
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 261
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 280
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 296
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 312
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 328
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 344
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 364
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 378
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 393
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 407
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 421
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 436
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 455

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. 34337
TooManyMethods This class has too many methods, consider refactoring it. 82337
FinalFieldCouldBeStatic This final field could be made static 84
FinalFieldCouldBeStatic This final field could be made static 85
CommentRequired Public method and constructor comments are required 9599
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 98
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 105
ConfusingTernary Avoid if (x != y) ..; else ..; 127

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 50197
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 90
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 104
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 119
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 150
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 167
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 172
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 186
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 191

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. 69357
CommentRequired Public method and constructor comments are required 352356

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 126
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 148
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 168
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 192
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 214
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 391
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 393

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
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 136
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 136
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'identityDto' 3 143
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 154
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 157
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 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
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 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
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 284
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 284
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 287
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 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
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 109
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 109
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 110
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 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
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 92
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 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
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 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
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 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
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 237
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 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
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 256
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 260
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 260
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 282
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 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
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 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
CommentRequired Public method and constructor comments are required 3 6265
CallSuperInConstructor It is a good practice to call super() in a constructor 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
FinalFieldCouldBeStatic This final field could be made static 3 57
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 58
FieldNamingConventions The final field name 'CUSTOMER_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 1 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
CyclomaticComplexity The method 'getIdentity(String, String, String)' has a cyclomatic complexity of 10. 3 7088
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 70
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
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 68
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 68
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 76
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 94
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 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
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 258
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 258
UnusedLocalVariable Avoid unused local variables such as 'mapHeadersResponse'. 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
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 99
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 100
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 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
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34464
TooManyMethods This class has too many methods, consider refactoring it. 3 75464
FieldNamingConventions The final field name '_strIdentityStoreEndPoint' doesn't match '[a-z][a-zA-Z0-9]*' 1 77
FieldNamingConventions The final field name '_strIdentityPath' doesn't match '[a-z][a-zA-Z0-9]*' 1 78
FieldNamingConventions The final field name '_strTaskStackPath' doesn't match '[a-z][a-zA-Z0-9]*' 1 79
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 117
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 122
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 144
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 165
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 184
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 205
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 226
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 242
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 261
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 280
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 296
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 312
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 328
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 344
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 364
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 378
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 393
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 407
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 421
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 436
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 455

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 34337
TooManyMethods This class has too many methods, consider refactoring it. 3 82337
FieldNamingConventions The final field name '_listIdentities' doesn't match '[a-z][a-zA-Z0-9]*' 1 83
FinalFieldCouldBeStatic This final field could be made static 3 84
FieldNamingConventions The final field name 'CONNECTION_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 1 84
FinalFieldCouldBeStatic This final field could be made static 3 85
FieldNamingConventions The final field name 'CUSTOMER_ID_PREFIX' doesn't match '[a-z][a-zA-Z0-9]*' 1 85
CommentRequired Public method and constructor comments are required 3 9599
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 98
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 105
ConfusingTernary Avoid if (x != y) ..; else ..; 3 127

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 50197
FieldNamingConventions The final field name '_strIdentityStoreEndPoint' doesn't match '[a-z][a-zA-Z0-9]*' 1 54
FieldNamingConventions The final field name '_strIdentityPath' doesn't match '[a-z][a-zA-Z0-9]*' 1 55
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 90
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 104
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 119
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 150
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 167
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 172
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 186
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 191

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

Rule Violation Priority Ligne
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 3860
CommentRequired Header comments are required 3 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 69357
CommentRequired Public method and constructor comments are required 3 352356

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 126
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 148
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 168
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 192
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 214
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 391
PreserveStackTrace New exception is thrown in catch block, original stack trace may be lost 3 393