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/NotificationGruService.java

Rule Violation Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 54–130

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. 80

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

Rule Violation Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 107
ConfusingTernary Avoid if (x != y) ..; else ..; 111
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 130–147

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 96
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 110–113

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

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/NotificationGruService.java

Rule Violation Priority Line
ClassWithOnlyPrivateConstructorsShouldBeFinal A class which only has private constructors should be final 1 54–130
UseObjectForClearerAPI Rather than using a lot of String arguments, consider using a container object for those values. 3 80

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/MyDashboardComponentAllNotificationGRU.java

Rule Violation Priority Line
UseConcurrentHashMap If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation 3 107
ConfusingTernary Avoid if (x != y) ..; else ..; 3 111
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 130–147

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 96
AvoidDeeplyNestedIfStmts Deeply nested if..then statements are hard to read 3 110–113