Fork me on GitHub
Built by Maven

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 1

fr/paris/lutece/plugins/rest/service/RestConstants.java

Rule Violation Line
ClassNamingConventions The utility class name 'RestConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 39–58

fr/paris/lutece/plugins/rest/service/mediatype/RestMediaTypes.java

Rule Violation Line
ClassNamingConventions The utility class name 'RestMediaTypes' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 44–63

fr/paris/lutece/plugins/rest/service/param/AbstractParam.java

Rule Violation Line
FieldNamingConventions The final field name '_strOriginalParam' doesn't match '[a-z][a-zA-Z0-9]*' 48
FieldNamingConventions The final field name '_value' doesn't match '[a-z][a-zA-Z0-9]*' 49
ConstructorCallsOverridableMethod Overridable method 'onError' called during object construction 69

fr/paris/lutece/plugins/rest/service/resourceinfo/AbstractResourceInfoProvider.java

Rule Violation Line
AbstractClassWithoutAnyMethod No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 41–43

fr/paris/lutece/plugins/rest/service/resourceinfo/ResourceInfoManager.java

Rule Violation Line
ClassNamingConventions The utility class name 'ResourceInfoManager' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 48–88

Priority 3

fr/paris/lutece/plugins/rest/business/resourceinfo/AbstractResourceInfo.java

Rule Violation Line
MissingOverride The method 'setKey(String)' is missing an @Override annotation. 62–65
MissingOverride The method 'getKey()' is missing an @Override annotation. 70–73
MissingOverride The method 'setValue(String)' is missing an @Override annotation. 78–81
MissingOverride The method 'getValue()' is missing an @Override annotation. 86–89
MissingOverride The method 'setListChildren(List)' is missing an @Override annotation. 94–97
MissingOverride The method 'getListChildren()' is missing an @Override annotation. 102–105
MissingOverride The method 'hasChildren()' is missing an @Override annotation. 110–113

fr/paris/lutece/plugins/rest/business/resourceinfo/ResourceInfo.java

Rule Violation Line
CallSuperInConstructor It is a good practice to call super() in a constructor 51–55
MissingOverride The method 'setResourceInfo(ResourceInfo)' is missing an @Override annotation. 60–65

fr/paris/lutece/plugins/rest/service/LuteceApplicationResourceConfig.java

Rule Violation Line
CommentRequired Header comments are required 58–152
CallSuperInConstructor It is a good practice to call super() in a constructor 63–151
CyclomaticComplexity The constructor 'LuteceApplicationResourceConfig()' has a cyclomatic complexity of 12. 63–151
CommentRequired Public method and constructor comments are required 63–151
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 91
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 145

fr/paris/lutece/plugins/rest/service/LuteceJerseySpringServlet.java

Rule Violation Line
DefaultPackage Use explicit scoping instead of the default package private level 76
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the LOGGER field if you want a default access modifier 76
ShortVariable Avoid variables with short names like ra 143

fr/paris/lutece/plugins/rest/service/mapper/GenericUncaughtExceptionMapper.java

Rule Violation Line
CommentRequired Protected method constructor comments are required 65
CommentRequired Protected method constructor comments are required 67
CommentRequired Protected method constructor comments are required 69

fr/paris/lutece/plugins/rest/service/mapper/GenericUncaughtJerseyExceptionMapper.java

Rule Violation Line
CommentRequired Protected method constructor comments are required 63

fr/paris/lutece/plugins/rest/service/param/AbstractParam.java

Rule Violation Line
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 59
AvoidCatchingThrowable A catch statement should never catch throwable since it includes errors. 67
ShortVariable Avoid variables with short names like e 112
ShortVariable Avoid variables with short names like e 126

fr/paris/lutece/plugins/rest/service/resourceinfo/ResourceInfoManager.java

Rule Violation Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 76

fr/paris/lutece/plugins/rest/service/writers/AbstractWriter.java

Rule Violation Line
MissingOverride The method 'getSize(List, Class, Type, Annotation, MediaType)' is missing an @Override annotation. 92–95
MissingOverride The method 'writeTo(List, Class, Type, Annotation, MediaType, MultivaluedMap, OutputStream)' is missing an @Override annotation. 100–158
CyclomaticComplexity The method 'writeTo(List, Class, Type, Annotation, MediaType, MultivaluedMap, OutputStream)' has a cyclomatic complexity of 11. 100–158
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 101
ConfusingTernary Avoid if (x != y) ..; else ..; 103–157
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 105
ConfusingTernary Avoid if (x != y) ..; else ..; 114–142
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 116
ConfusingTernary Avoid if (x != y) ..; else ..; 120–127
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 130

fr/paris/lutece/plugins/rest/util/xml/XMLUtil.java

Rule Violation Line
AddEmptyString Do not add empty strings 70

Priority 4

fr/paris/lutece/plugins/rest/service/LuteceJerseySpringServlet.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.rest.service.mediatype.MediaTypeMapping' 36
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.rest.service.mediatype.RestMediaTypes' 37
UnusedImports Avoid unused imports such as 'org.apache.commons.lang3.StringUtils' 41
UnusedImports Avoid unused imports such as 'org.apache.log4j.Level' 43
UnusedImports Avoid unused imports such as 'org.glassfish.jersey.server.ResourceConfig' 46
UnusedImports Avoid unused imports such as 'java.util.List' 55
UnusedImports Avoid unused imports such as 'java.util.Map' 56

Files

fr/paris/lutece/plugins/rest/business/resourceinfo/AbstractResourceInfo.java

Rule Violation Priority Line
MissingOverride The method 'setKey(String)' is missing an @Override annotation. 3 62–65
MissingOverride The method 'getKey()' is missing an @Override annotation. 3 70–73
MissingOverride The method 'setValue(String)' is missing an @Override annotation. 3 78–81
MissingOverride The method 'getValue()' is missing an @Override annotation. 3 86–89
MissingOverride The method 'setListChildren(List)' is missing an @Override annotation. 3 94–97
MissingOverride The method 'getListChildren()' is missing an @Override annotation. 3 102–105
MissingOverride The method 'hasChildren()' is missing an @Override annotation. 3 110–113

fr/paris/lutece/plugins/rest/business/resourceinfo/ResourceInfo.java

Rule Violation Priority Line
CallSuperInConstructor It is a good practice to call super() in a constructor 3 51–55
MissingOverride The method 'setResourceInfo(ResourceInfo)' is missing an @Override annotation. 3 60–65

fr/paris/lutece/plugins/rest/service/LuteceApplicationResourceConfig.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 58–152
CallSuperInConstructor It is a good practice to call super() in a constructor 3 63–151
CyclomaticComplexity The constructor 'LuteceApplicationResourceConfig()' has a cyclomatic complexity of 12. 3 63–151
CommentRequired Public method and constructor comments are required 3 63–151
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 91
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 145

fr/paris/lutece/plugins/rest/service/LuteceJerseySpringServlet.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.rest.service.mediatype.MediaTypeMapping' 4 36
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.rest.service.mediatype.RestMediaTypes' 4 37
UnusedImports Avoid unused imports such as 'org.apache.commons.lang3.StringUtils' 4 41
UnusedImports Avoid unused imports such as 'org.apache.log4j.Level' 4 43
UnusedImports Avoid unused imports such as 'org.glassfish.jersey.server.ResourceConfig' 4 46
UnusedImports Avoid unused imports such as 'java.util.List' 4 55
UnusedImports Avoid unused imports such as 'java.util.Map' 4 56
DefaultPackage Use explicit scoping instead of the default package private level 3 76
CommentDefaultAccessModifier To avoid mistakes add a comment at the beginning of the LOGGER field if you want a default access modifier 3 76
ShortVariable Avoid variables with short names like ra 3 143

fr/paris/lutece/plugins/rest/service/RestConstants.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'RestConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 39–58

fr/paris/lutece/plugins/rest/service/mapper/GenericUncaughtExceptionMapper.java

Rule Violation Priority Line
CommentRequired Protected method constructor comments are required 3 65
CommentRequired Protected method constructor comments are required 3 67
CommentRequired Protected method constructor comments are required 3 69

fr/paris/lutece/plugins/rest/service/mapper/GenericUncaughtJerseyExceptionMapper.java

Rule Violation Priority Line
CommentRequired Protected method constructor comments are required 3 63

fr/paris/lutece/plugins/rest/service/mediatype/RestMediaTypes.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'RestMediaTypes' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 44–63

fr/paris/lutece/plugins/rest/service/param/AbstractParam.java

Rule Violation Priority Line
FieldNamingConventions The final field name '_strOriginalParam' doesn't match '[a-z][a-zA-Z0-9]*' 1 48
FieldNamingConventions The final field name '_value' doesn't match '[a-z][a-zA-Z0-9]*' 1 49
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 59
AvoidCatchingThrowable A catch statement should never catch throwable since it includes errors. 3 67
ConstructorCallsOverridableMethod Overridable method 'onError' called during object construction 1 69
ShortVariable Avoid variables with short names like e 3 112
ShortVariable Avoid variables with short names like e 3 126

fr/paris/lutece/plugins/rest/service/resourceinfo/AbstractResourceInfoProvider.java

Rule Violation Priority Line
AbstractClassWithoutAnyMethod No abstract method which means that the keyword is most likely used to prevent instantiation. Use a private or protected constructor instead. 1 41–43

fr/paris/lutece/plugins/rest/service/resourceinfo/ResourceInfoManager.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'ResourceInfoManager' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 48–88
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 76

fr/paris/lutece/plugins/rest/service/writers/AbstractWriter.java

Rule Violation Priority Line
MissingOverride The method 'getSize(List, Class, Type, Annotation, MediaType)' is missing an @Override annotation. 3 92–95
MissingOverride The method 'writeTo(List, Class, Type, Annotation, MediaType, MultivaluedMap, OutputStream)' is missing an @Override annotation. 3 100–158
CyclomaticComplexity The method 'writeTo(List, Class, Type, Annotation, MediaType, MultivaluedMap, OutputStream)' has a cyclomatic complexity of 11. 3 100–158
AvoidUncheckedExceptionsInSignatures A method or constructor should not explicitly declare unchecked exceptions in its 'throws' clause 3 101
ConfusingTernary Avoid if (x != y) ..; else ..; 3 103–157
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 105
ConfusingTernary Avoid if (x != y) ..; else ..; 3 114–142
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 116
ConfusingTernary Avoid if (x != y) ..; else ..; 3 120–127
AvoidLiteralsInIfCondition Avoid using Literals in Conditional Statements 3 130

fr/paris/lutece/plugins/rest/util/xml/XMLUtil.java

Rule Violation Priority Line
AddEmptyString Do not add empty strings 3 70