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/workflow/modules/alertforms/business/retrieval/RetrievalTypeFactory.java

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

fr/paris/lutece/plugins/workflow/modules/alertforms/util/constants/AlertConstants.java

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

Priority 3

fr/paris/lutece/plugins/workflow/modules/alertforms/business/AlertDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 63–77

fr/paris/lutece/plugins/workflow/modules/alertforms/business/TaskAlertConfigDAO.java

Rule Violation Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 59–74

fr/paris/lutece/plugins/workflow/modules/alertforms/service/AlertService.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–464
ImmutableField Private field '_listAcceptedEntryTypesDate' could be made final; it is only initialized in the declaration or constructor. 91
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 207–211
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 331
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 348
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 421
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 422
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 423
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 424
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 425

fr/paris/lutece/plugins/workflow/modules/alertforms/service/TaskAlert.java

Rule Violation Line
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 99–112

fr/paris/lutece/plugins/workflow/modules/alertforms/service/daemon/AlertCleanerDaemon.java

Rule Violation Line
MissingOverride The method 'run()' is missing an @Override annotation. 61–97
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 86 characters appended. 63
ConfusingTernary Avoid if (x != y) ..; else ..; 74–88

fr/paris/lutece/plugins/workflow/modules/alertforms/service/daemon/AlertDaemon.java

Rule Violation Line
MissingOverride The method 'run()' is missing an @Override annotation. 63–109
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 82 characters appended. 65
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 93–98

fr/paris/lutece/plugins/workflow/modules/alertforms/util/constants/AlertConstants.java

Rule Violation Line
ShortVariable Avoid variables with short names like ID 45

fr/paris/lutece/plugins/workflow/modules/alertforms/web/AlertTaskComponent.java

Rule Violation Line
UnusedPrivateField Avoid unused private fields such as '_stateService'. 76
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 86
UnusedLocalVariable Avoid unused local variables such as 'resourceHistory'. 111
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 111
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 116

Priority 4

fr/paris/lutece/plugins/workflow/modules/alertforms/service/AlertService.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'java.text.ParseException' 67
UnusedImports Avoid unused imports such as 'java.text.SimpleDateFormat' 68

fr/paris/lutece/plugins/workflow/modules/alertforms/web/AlertTaskComponent.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.workflowcore.business.state.State' 43

Files

fr/paris/lutece/plugins/workflow/modules/alertforms/business/AlertDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 63–77

fr/paris/lutece/plugins/workflow/modules/alertforms/business/TaskAlertConfigDAO.java

Rule Violation Priority Line
AvoidSynchronizedAtMethodLevel Use block level rather than method level synchronization 3 59–74

fr/paris/lutece/plugins/workflow/modules/alertforms/business/retrieval/RetrievalTypeFactory.java

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

fr/paris/lutece/plugins/workflow/modules/alertforms/service/AlertService.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–464
UnusedImports Avoid unused imports such as 'java.text.ParseException' 4 67
UnusedImports Avoid unused imports such as 'java.text.SimpleDateFormat' 4 68
ImmutableField Private field '_listAcceptedEntryTypesDate' could be made final; it is only initialized in the declaration or constructor. 3 91
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 207–211
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 331
AvoidCatchingGenericException Avoid catching generic exceptions such as NullPointerException, RuntimeException, Exception in try-catch block 3 348
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 421
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 422
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 423
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 424
ConsecutiveAppendsShouldReuse StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable. 3 425

fr/paris/lutece/plugins/workflow/modules/alertforms/service/TaskAlert.java

Rule Violation Priority Line
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 99–112

fr/paris/lutece/plugins/workflow/modules/alertforms/service/daemon/AlertCleanerDaemon.java

Rule Violation Priority Line
MissingOverride The method 'run()' is missing an @Override annotation. 3 61–97
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 86 characters appended. 3 63
ConfusingTernary Avoid if (x != y) ..; else ..; 3 74–88

fr/paris/lutece/plugins/workflow/modules/alertforms/service/daemon/AlertDaemon.java

Rule Violation Priority Line
MissingOverride The method 'run()' is missing an @Override annotation. 3 63–109
InsufficientStringBufferDeclaration StringBuffer constructor is initialized with size 16, but has at least 82 characters appended. 3 65
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 93–98

fr/paris/lutece/plugins/workflow/modules/alertforms/util/constants/AlertConstants.java

Rule Violation Priority Line
ClassNamingConventions The utility class name 'AlertConstants' doesn't match '[A-Z][a-zA-Z0-9]+(Utils?|Helper|Home|Service)' 1 41–97
ShortVariable Avoid variables with short names like ID 3 45

fr/paris/lutece/plugins/workflow/modules/alertforms/web/AlertTaskComponent.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.workflowcore.business.state.State' 4 43
UnusedPrivateField Avoid unused private fields such as '_stateService'. 3 76
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 86
UnusedLocalVariable Avoid unused local variables such as 'resourceHistory'. 3 111
PrematureDeclaration Avoid declaring a variable if it is unreferenced before a possible exit point. 3 111
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 116