PMD Results
The following document contains the results of PMD 6.13.0.
Violations By Priority
Priority 3
org/itracker/services/ConfigurationService.java
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifiers 'public static final' on field 'PNAME_SYSTEM_BASE_URL': the field is declared in an interface type | 34 |
org/itracker/services/IssueService.java
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getNextIssues': the method is declared in an interface type | 83 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getPreviousIssues': the method is declared in an interface type | 84 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'searchIssues': the method is declared in an interface type | 241 |
org/itracker/services/ProjectService.java
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProject': the method is declared in an interface type | 31 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAllProjects': the method is declared in an interface type | 33 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAllAvailableProjects': the method is declared in an interface type | 35 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectComponent': the method is declared in an interface type | 37 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'updateProjectComponent': the method is declared in an interface type | 39 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'addProjectComponent': the method is declared in an interface type | 41 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'removeProjectComponent': the method is declared in an interface type | 43 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectVersion': the method is declared in an interface type | 45 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'updateProjectVersion': the method is declared in an interface type | 47 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'addProjectVersion': the method is declared in an interface type | 49 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'removeProjectVersion': the method is declared in an interface type | 51 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectOwners': the method is declared in an interface type | 53 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getListOfProjectOwners': the method is declared in an interface type | 55 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'setProjectOwners': the method is declared in an interface type | 57 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectFields': the method is declared in an interface type | 59 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getListOfProjectFields': the method is declared in an interface type | 61 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectFields': the method is declared in an interface type | 63 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'setProjectFields': the method is declared in an interface type | 65 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectScript': the method is declared in an interface type | 67 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectScripts': the method is declared in an interface type | 69 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'updateProjectScript': the method is declared in an interface type | 71 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'addProjectScript': the method is declared in an interface type | 73 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'removeProjectScript': the method is declared in an interface type | 75 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'countIssuesByComponent': the method is declared in an interface type | 83 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getTotalNumberIssuesByProject': the method is declared in an interface type | 85 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getTotalNumberOpenIssuesByProject': the method is declared in an interface type | 87 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getTotalNumberResolvedIssuesByProject': the method is declared in an interface type | 89 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'countIssuesByVersion': the method is declared in an interface type | 97 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectStats': the method is declared in an interface type | 108 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getLatestIssueUpdatedDateByProjectId': the method is declared in an interface type | 110 |
org/itracker/services/ReportService.java
Rule | Violation | Line |
---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAllReports': the method is declared in an interface type | 28 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getNumberReports': the method is declared in an interface type | 30 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'createReport': the method is declared in an interface type | 32 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getReportDAO': the method is declared in an interface type | 34 |
org/itracker/services/implementations/ConfigurationServiceImpl.java
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 131–133 |
EmptyCatchBlock | Avoid empty catch blocks | 695–696 |
UnusedLocalVariable | Avoid unused local variables such as 'current'. | 1135 |
org/itracker/services/implementations/IssueServiceImpl.java
Rule | Violation | Line |
---|---|---|
UnusedPrivateMethod | Avoid unused private methods such as 'addComponentsModifiedActivity(Issue,User,String)'. | 651–652 |
UnusedPrivateMethod | Avoid unused private methods such as 'addVersionsModifiedActivity(Issue,User,String)'. | 741–742 |
org/itracker/services/implementations/NotificationServiceImpl.java
Rule | Violation | Line |
---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 638–642 |
CollapsibleIfStatements | These nested if statements could be combined | 664–668 |
org/itracker/services/implementations/ProjectServiceImpl.java
Rule | Violation | Line |
---|---|---|
UnusedPrivateMethod | Avoid unused private methods such as 'getCustomFieldDAO()'. | 329 |
UnusedPrivateMethod | Avoid unused private methods such as 'getProjectScriptDAO()'. | 333 |
UnusedLocalVariable | Avoid unused local variables such as 'user'. | 377 |
CollapsibleIfStatements | These nested if statements could be combined | 389–391 |
org/itracker/services/implementations/UserServiceImpl.java
Rule | Violation | Line |
---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'projectDAO'. | 95 |
UnusedFormalParameter | Avoid unused constructor parameters such as 'reportDAO'. | 96 |
EmptyCatchBlock | Avoid empty catch blocks | 160–162 |
EmptyIfStmt | Avoid empty if statements | 442–443 |
UnusedLocalVariable | Avoid unused local variables such as 'i'. | 772 |
Priority 4
org/itracker/services/UserService.java
Rule | Violation | Line |
---|---|---|
UnusedImports | Avoid unused imports such as 'java.util.Collection' | 27 |
org/itracker/services/authentication/TestAuthenticator.java
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 24 |
org/itracker/services/authentication/adsson/ADIntegration.java
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 68 |
UselessParentheses | Useless parentheses. | 75 |
UselessParentheses | Useless parentheses. | 82 |
UselessParentheses | Useless parentheses. | 89 |
UselessParentheses | Useless parentheses. | 96 |
org/itracker/services/authentication/adsson/GetUserModelFromADPrivilegedAction.java
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 44 |
UselessParentheses | Useless parentheses. | 69 |
UselessParentheses | Useless parentheses. | 78 |
UselessParentheses | Useless parentheses. | 88 |
UselessParentheses | Useless parentheses. | 88 |
UselessParentheses | Useless parentheses. | 90 |
org/itracker/services/authentication/adsson/WindowsSSONAuthenticator.java
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 124 |
UselessParentheses | Useless parentheses. | 188 |
org/itracker/services/implementations/ConfigurationServiceImpl.java
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 163 |
UselessParentheses | Useless parentheses. | 950 |
org/itracker/services/implementations/IssueServiceImpl.java
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 940 |
UselessParentheses | Useless parentheses. | 1167 |
UselessParentheses | Useless parentheses. | 1318 |
UselessParentheses | Useless parentheses. | 1342 |
org/itracker/services/implementations/NotificationServiceImpl.java
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 238–239 |
UselessParentheses | Useless parentheses. | 382–385 |
UselessParentheses | Useless parentheses. | 411–412 |
org/itracker/services/implementations/UserServiceImpl.java
Rule | Violation | Line |
---|---|---|
UselessParentheses | Useless parentheses. | 251 |
UselessParentheses | Useless parentheses. | 603 |
UselessParentheses | Useless parentheses. | 790 |
UselessParentheses | Useless parentheses. | 818 |
UselessParentheses | Useless parentheses. | 820 |
UselessParentheses | Useless parentheses. | 850 |
UselessParentheses | Useless parentheses. | 878 |
UselessParentheses | Useless parentheses. | 906 |
UselessParentheses | Useless parentheses. | 934 |
UselessParentheses | Useless parentheses. | 962 |
Files
org/itracker/services/ConfigurationService.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifiers 'public static final' on field 'PNAME_SYSTEM_BASE_URL': the field is declared in an interface type | 3 | 34 |
org/itracker/services/IssueService.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getNextIssues': the method is declared in an interface type | 3 | 83 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getPreviousIssues': the method is declared in an interface type | 3 | 84 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'searchIssues': the method is declared in an interface type | 3 | 241 |
org/itracker/services/ProjectService.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProject': the method is declared in an interface type | 3 | 31 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAllProjects': the method is declared in an interface type | 3 | 33 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAllAvailableProjects': the method is declared in an interface type | 3 | 35 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectComponent': the method is declared in an interface type | 3 | 37 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'updateProjectComponent': the method is declared in an interface type | 3 | 39 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'addProjectComponent': the method is declared in an interface type | 3 | 41 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'removeProjectComponent': the method is declared in an interface type | 3 | 43 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectVersion': the method is declared in an interface type | 3 | 45 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'updateProjectVersion': the method is declared in an interface type | 3 | 47 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'addProjectVersion': the method is declared in an interface type | 3 | 49 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'removeProjectVersion': the method is declared in an interface type | 3 | 51 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectOwners': the method is declared in an interface type | 3 | 53 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getListOfProjectOwners': the method is declared in an interface type | 3 | 55 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'setProjectOwners': the method is declared in an interface type | 3 | 57 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectFields': the method is declared in an interface type | 3 | 59 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getListOfProjectFields': the method is declared in an interface type | 3 | 61 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectFields': the method is declared in an interface type | 3 | 63 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'setProjectFields': the method is declared in an interface type | 3 | 65 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectScript': the method is declared in an interface type | 3 | 67 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectScripts': the method is declared in an interface type | 3 | 69 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'updateProjectScript': the method is declared in an interface type | 3 | 71 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'addProjectScript': the method is declared in an interface type | 3 | 73 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'removeProjectScript': the method is declared in an interface type | 3 | 75 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'countIssuesByComponent': the method is declared in an interface type | 3 | 83 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getTotalNumberIssuesByProject': the method is declared in an interface type | 3 | 85 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getTotalNumberOpenIssuesByProject': the method is declared in an interface type | 3 | 87 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getTotalNumberResolvedIssuesByProject': the method is declared in an interface type | 3 | 89 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'countIssuesByVersion': the method is declared in an interface type | 3 | 97 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getProjectStats': the method is declared in an interface type | 3 | 108 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getLatestIssueUpdatedDateByProjectId': the method is declared in an interface type | 3 | 110 |
org/itracker/services/ReportService.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getAllReports': the method is declared in an interface type | 3 | 28 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getNumberReports': the method is declared in an interface type | 3 | 30 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'createReport': the method is declared in an interface type | 3 | 32 |
UnnecessaryModifier | Unnecessary modifier 'public' on method 'getReportDAO': the method is declared in an interface type | 3 | 34 |
org/itracker/services/UserService.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedImports | Avoid unused imports such as 'java.util.Collection' | 4 | 27 |
org/itracker/services/authentication/TestAuthenticator.java
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 24 |
org/itracker/services/authentication/adsson/ADIntegration.java
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 68 |
UselessParentheses | Useless parentheses. | 4 | 75 |
UselessParentheses | Useless parentheses. | 4 | 82 |
UselessParentheses | Useless parentheses. | 4 | 89 |
UselessParentheses | Useless parentheses. | 4 | 96 |
org/itracker/services/authentication/adsson/GetUserModelFromADPrivilegedAction.java
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 44 |
UselessParentheses | Useless parentheses. | 4 | 69 |
UselessParentheses | Useless parentheses. | 4 | 78 |
UselessParentheses | Useless parentheses. | 4 | 88 |
UselessParentheses | Useless parentheses. | 4 | 88 |
UselessParentheses | Useless parentheses. | 4 | 90 |
org/itracker/services/authentication/adsson/WindowsSSONAuthenticator.java
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 124 |
UselessParentheses | Useless parentheses. | 4 | 188 |
org/itracker/services/implementations/ConfigurationServiceImpl.java
Rule | Violation | Priority | Line |
---|---|---|---|
CollapsibleIfStatements | These nested if statements could be combined | 3 | 131–133 |
UselessParentheses | Useless parentheses. | 4 | 163 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 695–696 |
UselessParentheses | Useless parentheses. | 4 | 950 |
UnusedLocalVariable | Avoid unused local variables such as 'current'. | 3 | 1135 |
org/itracker/services/implementations/IssueServiceImpl.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedPrivateMethod | Avoid unused private methods such as 'addComponentsModifiedActivity(Issue,User,String)'. | 3 | 651–652 |
UnusedPrivateMethod | Avoid unused private methods such as 'addVersionsModifiedActivity(Issue,User,String)'. | 3 | 741–742 |
UselessParentheses | Useless parentheses. | 4 | 940 |
UselessParentheses | Useless parentheses. | 4 | 1167 |
UselessParentheses | Useless parentheses. | 4 | 1318 |
UselessParentheses | Useless parentheses. | 4 | 1342 |
org/itracker/services/implementations/NotificationServiceImpl.java
Rule | Violation | Priority | Line |
---|---|---|---|
UselessParentheses | Useless parentheses. | 4 | 238–239 |
UselessParentheses | Useless parentheses. | 4 | 382–385 |
UselessParentheses | Useless parentheses. | 4 | 411–412 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 638–642 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 664–668 |
org/itracker/services/implementations/ProjectServiceImpl.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedPrivateMethod | Avoid unused private methods such as 'getCustomFieldDAO()'. | 3 | 329 |
UnusedPrivateMethod | Avoid unused private methods such as 'getProjectScriptDAO()'. | 3 | 333 |
UnusedLocalVariable | Avoid unused local variables such as 'user'. | 3 | 377 |
CollapsibleIfStatements | These nested if statements could be combined | 3 | 389–391 |
org/itracker/services/implementations/UserServiceImpl.java
Rule | Violation | Priority | Line |
---|---|---|---|
UnusedFormalParameter | Avoid unused constructor parameters such as 'projectDAO'. | 3 | 95 |
UnusedFormalParameter | Avoid unused constructor parameters such as 'reportDAO'. | 3 | 96 |
EmptyCatchBlock | Avoid empty catch blocks | 3 | 160–162 |
UselessParentheses | Useless parentheses. | 4 | 251 |
EmptyIfStmt | Avoid empty if statements | 3 | 442–443 |
UselessParentheses | Useless parentheses. | 4 | 603 |
UnusedLocalVariable | Avoid unused local variables such as 'i'. | 3 | 772 |
UselessParentheses | Useless parentheses. | 4 | 790 |
UselessParentheses | Useless parentheses. | 4 | 818 |
UselessParentheses | Useless parentheses. | 4 | 820 |
UselessParentheses | Useless parentheses. | 4 | 850 |
UselessParentheses | Useless parentheses. | 4 | 878 |
UselessParentheses | Useless parentheses. | 4 | 906 |
UselessParentheses | Useless parentheses. | 4 | 934 |
UselessParentheses | Useless parentheses. | 4 | 962 |