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/blog/modules/extendcomment/utils/BlogCommentConstants.java

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

Priority 2

fr/paris/lutece/plugins/blog/modules/extendcomment/web/BlogExtendCommentJspBean.java

Rule Violation Line
BooleanInstantiation Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. 199
BooleanInstantiation Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. 203

Priority 3

fr/paris/lutece/plugins/blog/modules/extendcomment/web/AbstractManageBlogExtendcommentJspBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 49–102
SingularField Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. 65
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 92

fr/paris/lutece/plugins/blog/modules/extendcomment/web/BlogExtendCommentJspBean.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–281
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 86–281
ImmutableField Private field '_extenderService' could be made final; it is only initialized in the declaration or constructor. 100
ImmutableField Private field '_commentService' could be made final; it is only initialized in the declaration or constructor. 102
ImmutableField Private field '_configService' could be made final; it is only initialized in the declaration or constructor. 103
ImmutableField Private field '_resourceExtenderService' could be made final; it is only initialized in the declaration or constructor. 104
NPathComplexity The method 'getCommentBlog(HttpServletRequest)' has an NPath complexity of 46080 114–279
ExcessiveMethodLength Avoid really long methods. 114–279
NcssCount The method 'getCommentBlog(HttpServletRequest)' has a NCSS line count of 93. 114–279
CyclomaticComplexity The method 'getCommentBlog(HttpServletRequest)' has a cyclomatic complexity of 20. 114–279
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 120
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 121
ControlStatementBraces This statement should have braces 136
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 220–221
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 223
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 223
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strContent' 277

Files

fr/paris/lutece/plugins/blog/modules/extendcomment/utils/BlogCommentConstants.java

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

fr/paris/lutece/plugins/blog/modules/extendcomment/web/AbstractManageBlogExtendcommentJspBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 49–102
SingularField Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. 3 65
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 92

fr/paris/lutece/plugins/blog/modules/extendcomment/web/BlogExtendCommentJspBean.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–281
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 86–281
ImmutableField Private field '_extenderService' could be made final; it is only initialized in the declaration or constructor. 3 100
ImmutableField Private field '_commentService' could be made final; it is only initialized in the declaration or constructor. 3 102
ImmutableField Private field '_configService' could be made final; it is only initialized in the declaration or constructor. 3 103
ImmutableField Private field '_resourceExtenderService' could be made final; it is only initialized in the declaration or constructor. 3 104
NPathComplexity The method 'getCommentBlog(HttpServletRequest)' has an NPath complexity of 46080 3 114–279
ExcessiveMethodLength Avoid really long methods. 3 114–279
NcssCount The method 'getCommentBlog(HttpServletRequest)' has a NCSS line count of 93. 3 114–279
CyclomaticComplexity The method 'getCommentBlog(HttpServletRequest)' has a cyclomatic complexity of 20. 3 114–279
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 120
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 121
ControlStatementBraces This statement should have braces 3 136
BooleanInstantiation Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. 2 199
BooleanInstantiation Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. 2 203
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 220–221
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 223
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 223
UnnecessaryLocalBeforeReturn Consider simply returning the value vs storing it in local variable 'strContent' 3 277