Fork me on GitHub

PMD Results

The following document contains the results of PMD 6.13.0.

Violations By Priority

Priority 3

fr/paris/lutece/plugins/extend/modules/opengraph/business/OpengraphSocialHubDAO.java

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

fr/paris/lutece/plugins/extend/modules/opengraph/business/config/OpengraphExtenderConfig.java

Rule Violation Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 46
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 47
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 48
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 103
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 116

fr/paris/lutece/plugins/extend/modules/opengraph/business/config/OpengraphExtenderConfigDAO.java

Rule Violation Line
UnusedFormalParameter Avoid unused method parameters such as 'plugin'. 69
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 71
UnusedFormalParameter Avoid unused method parameters such as 'plugin'. 109
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 111
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 125
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 175

fr/paris/lutece/plugins/extend/modules/opengraph/service/OpengraphResourceIdService.java

Rule Violation Line
CallSuperInConstructor It is a good practice to call super() in a constructor 7275
ShortVariable Avoid variables with short names like rt 83
ShortVariable Avoid variables with short names like p 89

fr/paris/lutece/plugins/extend/modules/opengraph/web/OpengraphJspBean.java

Rule Violation Line
ImmutableField Private field '_opengraphService' could be made final; it is only initialized in the declaration or constructor. 117
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 133134
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 141
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 141
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 146
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 146
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 184
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 184
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 223
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 288
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 288
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 328
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 336

fr/paris/lutece/plugins/extend/modules/opengraph/web/action/OpengraphInstallerPluginAction.java

Rule Violation Line
UncommentedEmptyMethodBody Document empty method body 7273

fr/paris/lutece/plugins/extend/modules/opengraph/web/component/OpengraphResourceExtenderComponent.java

Rule Violation Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 156
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 156
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 291
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 301
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 301
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 329
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 339
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 339
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 366
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 376
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 376
CollapsibleIfStatements These nested if statements could be combined 407410
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 409

Files

fr/paris/lutece/plugins/extend/modules/opengraph/business/OpengraphSocialHubDAO.java

Rule Violation Priority Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 158

fr/paris/lutece/plugins/extend/modules/opengraph/business/config/OpengraphExtenderConfig.java

Rule Violation Priority Line
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 46
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 47
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 48
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 103
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 116

fr/paris/lutece/plugins/extend/modules/opengraph/business/config/OpengraphExtenderConfigDAO.java

Rule Violation Priority Line
UnusedFormalParameter Avoid unused method parameters such as 'plugin'. 3 69
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 3 71
UnusedFormalParameter Avoid unused method parameters such as 'plugin'. 3 109
UseCollectionIsEmpty Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 3 111
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 125
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 175

fr/paris/lutece/plugins/extend/modules/opengraph/service/OpengraphResourceIdService.java

Rule Violation Priority Line
CallSuperInConstructor It is a good practice to call super() in a constructor 3 7275
ShortVariable Avoid variables with short names like rt 3 83
ShortVariable Avoid variables with short names like p 3 89

fr/paris/lutece/plugins/extend/modules/opengraph/web/OpengraphJspBean.java

Rule Violation Priority Line
ImmutableField Private field '_opengraphService' could be made final; it is only initialized in the declaration or constructor. 3 117
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 133134
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 141
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 141
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 146
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 146
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 184
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 184
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 223
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 288
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 288
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 328
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 336

fr/paris/lutece/plugins/extend/modules/opengraph/web/action/OpengraphInstallerPluginAction.java

Rule Violation Priority Line
UncommentedEmptyMethodBody Document empty method body 3 7273

fr/paris/lutece/plugins/extend/modules/opengraph/web/component/OpengraphResourceExtenderComponent.java

Rule Violation Priority Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 156
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 156
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 291
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 301
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 301
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 329
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 339
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 339
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 366
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 376
UseDiamondOperator Explicit type arguments can be replaced by Diamond Operator 3 376
CollapsibleIfStatements These nested if statements could be combined 3 407410
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 409