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/mydashboard/modules/grusupply/service/IdentityStoreService.java

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

fr/paris/lutece/plugins/mydashboard/modules/grusupply/service/NotificationGruService.java

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

fr/paris/lutece/plugins/mydashboard/modules/grusupply/util/MydashboardGrusupplyUtil.java

Rule Violation Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 46–72

Priority 3

fr/paris/lutece/plugins/mydashboard/modules/grusupply/business/DemandDashboard.java

Rule Violation Line
UncommentedEmptyConstructor Document empty constructor 59–61

fr/paris/lutece/plugins/mydashboard/modules/grusupply/business/DemandDashboardDAO.java

Rule Violation Line
AvoidDuplicateLiterals The String literal "demand_id" appears 4 times in this file; the first occurrence is on line 92 92
ControlStatementBraces This statement should have braces 159

fr/paris/lutece/plugins/mydashboard/modules/grusupply/service/NotificationGruService.java

Rule Violation Line
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 81
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 106
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 128

fr/paris/lutece/plugins/mydashboard/modules/grusupply/web/MyDashboardComponentCompletedNotificationGRU.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–231
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 118
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 123
ConfusingTernary Avoid if (x != y) ..; else ..; 127
InefficientStringBuffering Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 225

fr/paris/lutece/plugins/mydashboard/modules/grusupply/web/MyDashboardComponentInProgressNotificationGRU.java

Rule Violation Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 34–232
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 119
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 124
ConfusingTernary Avoid if (x != y) ..; else ..; 128
InefficientStringBuffering Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 226

fr/paris/lutece/plugins/mydashboard/modules/grusupply/web/MyDashboardComponentLastNotificationGRU.java

Rule Violation Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 98
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 114–117

Priority 4

fr/paris/lutece/plugins/mydashboard/modules/grusupply/business/DemandDashboard.java

Rule Violation Line
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 101–104

fr/paris/lutece/plugins/mydashboard/modules/grusupply/service/NotificationGruService.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.grubusiness.business.demand.TemporaryStatus' 41

fr/paris/lutece/plugins/mydashboard/modules/grusupply/web/MyDashboardComponentInProgressNotificationGRU.java

Rule Violation Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.grubusiness.business.demand.TemporaryStatus' 50

Files

fr/paris/lutece/plugins/mydashboard/modules/grusupply/business/DemandDashboard.java

Rule Violation Priority Line
UncommentedEmptyConstructor Document empty constructor 3 59–61
BooleanGetMethodName A getX() method which returns a boolean should be named isX() 4 101–104

fr/paris/lutece/plugins/mydashboard/modules/grusupply/business/DemandDashboardDAO.java

Rule Violation Priority Line
AvoidDuplicateLiterals The String literal "demand_id" appears 4 times in this file; the first occurrence is on line 92 3 92
ControlStatementBraces This statement should have braces 3 159

fr/paris/lutece/plugins/mydashboard/modules/grusupply/service/IdentityStoreService.java

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

fr/paris/lutece/plugins/mydashboard/modules/grusupply/service/NotificationGruService.java

Rule Violation Priority Line
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.grubusiness.business.demand.TemporaryStatus' 4 41
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 55–157
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 81
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 106
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 128

fr/paris/lutece/plugins/mydashboard/modules/grusupply/util/MydashboardGrusupplyUtil.java

Rule Violation Priority Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 46–72

fr/paris/lutece/plugins/mydashboard/modules/grusupply/web/MyDashboardComponentCompletedNotificationGRU.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–231
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 118
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 123
ConfusingTernary Avoid if (x != y) ..; else ..; 3 127
InefficientStringBuffering Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 3 225

fr/paris/lutece/plugins/mydashboard/modules/grusupply/web/MyDashboardComponentInProgressNotificationGRU.java

Rule Violation Priority Line
ExcessiveImports A high number of imports can indicate a high degree of coupling within an object. 3 34–232
UnusedImports Avoid unused imports such as 'fr.paris.lutece.plugins.grubusiness.business.demand.TemporaryStatus' 4 50
UseStringBufferForStringAppends Prefer StringBuilder (non-synchronized) or StringBuffer (synchronized) over += for concatenating strings 3 119
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 124
ConfusingTernary Avoid if (x != y) ..; else ..; 3 128
InefficientStringBuffering Avoid concatenating nonliterals in a StringBuffer/StringBuilder constructor or append(). 3 226

fr/paris/lutece/plugins/mydashboard/modules/grusupply/web/MyDashboardComponentLastNotificationGRU.java

Rule Violation Priority Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 98
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 114–117