Fork me on GitHub

PMD Results

The following document contains the results of PMD 5.3.5.

Files

fr/paris/lutece/plugins/contextinclude/business/Context.java

Violation Line
The class 'Context' has a Cyclomatic Complexity of 2 (Highest = 13). 51281
The class 'Context' has a Modified Cyclomatic Complexity of 2 (Highest = 13). 51281
The class 'Context' has a Standard Cyclomatic Complexity of 2 (Highest = 13). 51281
The method 'isInvoked' has a Standard Cyclomatic Complexity of 13. 208280
The method 'isInvoked' has a Modified Cyclomatic Complexity of 13. 208280
The method 'isInvoked' has a Cyclomatic Complexity of 13. 208280
The method isInvoked() has an NPath complexity of 330 208280
These nested if statements could be combined 258261

fr/paris/lutece/plugins/contextinclude/business/ContextDAO.java

Violation Line
Use block level rather than method level synchronization 91109
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 211
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 245
StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 246

fr/paris/lutece/plugins/contextinclude/business/ContextParamsDAO.java

Violation Line
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 92

fr/paris/lutece/plugins/contextinclude/business/parameter/ContextParameter.java

Violation Line
Class cannot be instantiated and does not provide any static methods or fields 4272

fr/paris/lutece/plugins/contextinclude/business/parameter/ContextParameterDAO.java

Violation Line
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 63
Use block level rather than method level synchronization 8494

fr/paris/lutece/plugins/contextinclude/service/ContextIncludePlugin.java

Violation Line
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. 4763

fr/paris/lutece/plugins/contextinclude/service/parameter/ContextParameterService.java

Violation Line
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 131

fr/paris/lutece/plugins/contextinclude/web/ContextJspBean.java

Violation Line
A high number of imports can indicate a high degree of coupling within an object. 34645
debug logging that involves string concatenation should be guarded with isDebugEnabled() checks 152
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 157
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 180
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 203
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 245
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 278
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 290
Substitute calls to size() == 0 (or size() != 0, size() > 0, size() < 1) with calls to isEmpty() 341
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 353
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 401
Avoid variables with short names like it 441
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 461
Avoid variables with short names like it 496
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 516
Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 557
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 604
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 619

fr/paris/lutece/plugins/contextinclude/web/action/CreateContextPluginAction.java

Violation Line
Document empty method 7475

fr/paris/lutece/plugins/contextinclude/web/action/ManageAdvancedParametersPluginAction.java

Violation Line
Document empty method 7374