Fork me on GitHub

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 1

fr/paris/lutece/plugins/identityimport/web/rs/service/HttpApiManagerAccessTransport.java

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

fr/paris/lutece/plugins/identityimport/web/service/CustomResponseStatusValidator.java

Rule Violation Line
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 3

fr/paris/lutece/plugins/identityimport/web/rs/service/AbstractTransportRest.java

Rule Violation Line
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 4388

fr/paris/lutece/plugins/identityimport/web/rs/service/BatchImportTransportRest.java

Rule Violation Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 85
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 89
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 108

fr/paris/lutece/plugins/identityimport/web/rs/service/HttpAccessTransport.java

Rule Violation Line
TooManyMethods This class has too many methods, consider refactoring it. 64435
CommentRequired Public method and constructor comments are required 6871
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 81
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 91
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 122
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 139
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 153
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 170
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 184
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 244
UnusedLocalVariable Avoid unused local variables such as 'mapHeadersResponse'. 256
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 256
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 256
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 279
ShortVariable Avoid variables with short names like e 297
MissingOverride The method 'setApiEndPointUrl(String)' is missing an @Override annotation. 331335
MissingOverride The method 'getApiEndPointUrl()' is missing an @Override annotation. 342346
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 367
ShortVariable Avoid variables with short names like er 370
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 401
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 419
ShortVariable Avoid variables with short names like er 422

fr/paris/lutece/plugins/identityimport/web/rs/service/HttpApiManagerAccessTransport.java

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

fr/paris/lutece/plugins/identityimport/web/service/CustomResponseStatusValidator.java

Rule Violation Line
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/identityimport/web/service/IBatchImportTransportProvider.java

Rule Violation Line
CommentRequired Public method and constructor comments are required 7882

fr/paris/lutece/plugins/identityimport/web/service/IHttpTransportProvider.java

Rule Violation Line
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

Files

fr/paris/lutece/plugins/identityimport/web/rs/service/AbstractTransportRest.java

Rule Violation Priority Line
AbstractClassWithoutAbstractMethod This abstract class does not have any abstract methods 3 4388

fr/paris/lutece/plugins/identityimport/web/rs/service/BatchImportTransportRest.java

Rule Violation Priority Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 85
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 89
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 108

fr/paris/lutece/plugins/identityimport/web/rs/service/HttpAccessTransport.java

Rule Violation Priority Line
TooManyMethods This class has too many methods, consider refactoring it. 3 64435
CommentRequired Public method and constructor comments are required 3 6871
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 81
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 91
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 122
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 139
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 153
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 170
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 184
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 244
UnusedLocalVariable Avoid unused local variables such as 'mapHeadersResponse'. 3 256
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 256
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 256
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 279
ShortVariable Avoid variables with short names like e 3 297
MissingOverride The method 'setApiEndPointUrl(String)' is missing an @Override annotation. 3 331335
MissingOverride The method 'getApiEndPointUrl()' is missing an @Override annotation. 3 342346
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 367
ShortVariable Avoid variables with short names like er 3 370
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 401
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 419
ShortVariable Avoid variables with short names like er 3 422

fr/paris/lutece/plugins/identityimport/web/rs/service/HttpApiManagerAccessTransport.java

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

fr/paris/lutece/plugins/identityimport/web/service/CustomResponseStatusValidator.java

Rule Violation Priority Line
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/identityimport/web/service/IBatchImportTransportProvider.java

Rule Violation Priority Line
CommentRequired Public method and constructor comments are required 3 7882

fr/paris/lutece/plugins/identityimport/web/service/IHttpTransportProvider.java

Rule Violation Priority Line
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