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/verifybackurl/business/AuthorizedUrl.java

Rule Violation Line
FormalParameterNamingConventions The method parameter name '_strApplicationCode' doesn't match '[a-z][a-zA-Z0-9]*' 123

fr/paris/lutece/plugins/verifybackurl/service/AuthorizedUrlService.java

Rule Violation Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 55–251

fr/paris/lutece/plugins/verifybackurl/utils/VerifyBackUrlConstants.java

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

Priority 3

fr/paris/lutece/plugins/verifybackurl/business/AuthorizedUrlDAO.java

Rule Violation Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 163
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 192
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 221

fr/paris/lutece/plugins/verifybackurl/service/AuthorizedUrlDatabaseProvider.java

Rule Violation Line
CommentRequired Header comments are required 41–60

fr/paris/lutece/plugins/verifybackurl/service/AuthorizedUrlService.java

Rule Violation Line
NonThreadSafeSingleton Singleton is not thread safe 66–70
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 86
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 113

fr/paris/lutece/plugins/verifybackurl/service/DomainNameConstraint.java

Rule Violation Line
CommentRequired Header comments are required 38–50

fr/paris/lutece/plugins/verifybackurl/service/IAuthorizedUrlProvider.java

Rule Violation Line
CommentRequired Header comments are required 39–59
UnnecessaryModifier Unnecessary modifier 'public' on method 'getAuthorizedUrlsList': the method is declared in an interface type 47
UnnecessaryModifier Unnecessary modifier 'public' on method 'getAuthorizedUrlsByApplicationCode': the method is declared in an interface type 53

fr/paris/lutece/plugins/verifybackurl/service/IBackUrlConstraint.java

Rule Violation Line
CommentRequired Header comments are required 36–44

fr/paris/lutece/plugins/verifybackurl/service/NoHTMLConstraint.java

Rule Violation Line
CommentRequired Header comments are required 38–49

fr/paris/lutece/plugins/verifybackurl/service/ProcessConstraintsService.java

Rule Violation Line
CommentRequired Header comments are required 39–59
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 40–59

fr/paris/lutece/plugins/verifybackurl/utils/VerifiyBackUrlUtils.java

Rule Violation Line
CommentRequired Header comments are required 50–203
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 51–203
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 74–82
SimplifyBooleanReturns Avoid unnecessary if..then..else statements when returning booleans 129–132
ConfusingTernary Avoid if (x != y) ..; else ..; 197

fr/paris/lutece/plugins/verifybackurl/utils/VerifyBackUrlConstants.java

Rule Violation Line
CommentRequired Header comments are required 36–64

fr/paris/lutece/plugins/verifybackurl/web/AuthorizedUrlJspBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 53–233
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 107
ConfusingTernary Avoid if (x != y) ..; else ..; 123

fr/paris/lutece/plugins/verifybackurl/web/ManageVerifybackurlJspBean.java

Rule Violation Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 49–98
SingularField Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. 65
AddEmptyString Do not add empty strings 92

Files

fr/paris/lutece/plugins/verifybackurl/business/AuthorizedUrl.java

Rule Violation Priority Line
FormalParameterNamingConventions The method parameter name '_strApplicationCode' doesn't match '[a-z][a-zA-Z0-9]*' 1 123

fr/paris/lutece/plugins/verifybackurl/business/AuthorizedUrlDAO.java

Rule Violation Priority Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 163
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 192
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 221

fr/paris/lutece/plugins/verifybackurl/service/AuthorizedUrlDatabaseProvider.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 41–60

fr/paris/lutece/plugins/verifybackurl/service/AuthorizedUrlService.java

Rule Violation Priority Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 55–251
NonThreadSafeSingleton Singleton is not thread safe 3 66–70
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 86
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 113

fr/paris/lutece/plugins/verifybackurl/service/DomainNameConstraint.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–50

fr/paris/lutece/plugins/verifybackurl/service/IAuthorizedUrlProvider.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 39–59
UnnecessaryModifier Unnecessary modifier 'public' on method 'getAuthorizedUrlsList': the method is declared in an interface type 3 47
UnnecessaryModifier Unnecessary modifier 'public' on method 'getAuthorizedUrlsByApplicationCode': the method is declared in an interface type 3 53

fr/paris/lutece/plugins/verifybackurl/service/IBackUrlConstraint.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 36–44

fr/paris/lutece/plugins/verifybackurl/service/NoHTMLConstraint.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 38–49

fr/paris/lutece/plugins/verifybackurl/service/ProcessConstraintsService.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 39–59
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 3 40–59

fr/paris/lutece/plugins/verifybackurl/utils/VerifiyBackUrlUtils.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 50–203
UseUtilityClass All methods are static. Consider using a utility class instead. Alternatively, you could add a private constructor or make the class abstract to silence this warning. 3 51–203
ForLoopCanBeForeach This for loop can be replaced by a foreach loop 3 74–82
SimplifyBooleanReturns Avoid unnecessary if..then..else statements when returning booleans 3 129–132
ConfusingTernary Avoid if (x != y) ..; else ..; 3 197

fr/paris/lutece/plugins/verifybackurl/utils/VerifyBackUrlConstants.java

Rule Violation Priority Line
CommentRequired Header comments are required 3 36–64
ClassNamingConventions The utility class name 'VerifyBackUrlConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 36–64

fr/paris/lutece/plugins/verifybackurl/web/AuthorizedUrlJspBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 53–233
NullAssignment Assigning an Object to null is a code smell. Consider refactoring. 3 107
ConfusingTernary Avoid if (x != y) ..; else ..; 3 123

fr/paris/lutece/plugins/verifybackurl/web/ManageVerifybackurlJspBean.java

Rule Violation Priority Line
MissingSerialVersionUID Classes implementing Serializable should set a serialVersionUID 3 49–98
SingularField Perhaps '_nDefaultItemsPerPage' could be replaced by a local variable. 3 65
AddEmptyString Do not add empty strings 3 92