itracker
Download
Documentation
Getting Involved
itracker Reports
itracker Modules
itracker Web
PMD Results
The following document contains the results of PMD 6.13.0.
Violations By Priority
Priority 3
org/itracker/web/ApplicationInitialization.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'servletConfig'. | 49 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'reportService'. | 51 |
org/itracker/web/actions/PortalHomeAction.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 63 |
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 239–240 |
org/itracker/web/actions/admin/configuration/EditConfigurationAction.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 187–234 |
org/itracker/web/actions/admin/configuration/ImportDataProcessAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'importer'. | 132 |
| UnusedFormalParameter | Avoid unused method parameters such as 'importer'. | 178 |
org/itracker/web/actions/admin/configuration/ImportDataVerifyAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'ic'. | 126 |
| UnusedFormalParameter | Avoid unused method parameters such as 'ic'. | 240 |
| UnusedFormalParameter | Avoid unused method parameters such as 'ic'. | 284 |
| UnusedFormalParameter | Avoid unused method parameters such as 'ic'. | 322 |
org/itracker/web/actions/admin/language/EditLanguageAction.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 138–140 |
org/itracker/web/actions/admin/language/EditLanguageFormAction.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 95–103 |
org/itracker/web/actions/admin/workflow/EditWorkflowScriptFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedLocalVariable | Avoid unused local variables such as 'isUpdate'. | 51 |
org/itracker/web/actions/issuesearch/SearchIssuesFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedNullCheckInEquals | Invoke equals() on the object you've already ensured is not null | 128 |
org/itracker/web/actions/project/CreateIssueAction.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 181–189 |
| UnusedFormalParameter | Avoid unused method parameters such as 'issue'. | 336 |
| UnusedFormalParameter | Avoid unused method parameters such as 'issueForm'. | 337 |
org/itracker/web/actions/project/MoveIssueAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'mapping'. | 116 |
org/itracker/web/actions/project/MoveIssueFormAction.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 143–148 |
org/itracker/web/actions/project/ViewIssueActivityAction.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'setupJspEnv': private methods cannot be overridden | 107–112 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'prepareActivitiesMap': private methods cannot be overridden | 119–133 |
org/itracker/web/forms/IssueForm.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 167 |
| EmptyCatchBlock | Avoid empty catch blocks | 173 |
| UnusedFormalParameter | Avoid unused method parameters such as 'user'. | 429 |
| UnusedFormalParameter | Avoid unused method parameters such as 'issueService'. | 430 |
| CollapsibleIfStatements | These nested if statements could be combined | 614–620 |
| EmptyStatementNotInLoop | An empty statement (semicolon) not part of a loop | 779 |
org/itracker/web/forms/ProjectForm.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 108–109 |
| EmptyIfStmt | Avoid empty if statements | 235–236 |
org/itracker/web/forms/ProjectScriptForm.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'action'. | 38 |
| UnusedPrivateField | Avoid unused private fields such as 'delId'. | 39 |
org/itracker/web/forms/ReportForm.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'reportType'. | 44 |
org/itracker/web/forms/WorkflowScriptForm.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'log'. | 49 |
| UnusedLocalVariable | Avoid unused local variables such as 'eof'. | 108 |
| EmptyWhileStmt | Avoid empty while statements | 109–110 |
org/itracker/web/security/AuthenticationErrorHandler.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'log'. | 15 |
org/itracker/web/servlets/GenericController.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 66 |
org/itracker/web/servlets/RssFeedController.java
| Rule | Violation | Line |
|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 45–48 |
org/itracker/web/taglib/AddErrorTag.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 72–73 |
org/itracker/web/taglib/ClearErrorsTag.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 59–60 |
org/itracker/web/taglib/FormatCustomFieldTag.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 47 |
org/itracker/web/taglib/FormatHistoryEntryTag.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 163–164 |
org/itracker/web/taglib/FormatImageActionTag.java
| Rule | Violation | Line |
|---|---|---|
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 172–173 |
org/itracker/web/taglib/FormatResolutionTag.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 94–95 |
| EmptyCatchBlock | Avoid empty catch blocks | 100–102 |
org/itracker/web/util/AttachmentUtilities.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 45–48 |
org/itracker/web/util/EditCustomFieldActionUtil.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 7 |
org/itracker/web/util/EditProjectFormActionUtil.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'log'. | 12 |
org/itracker/web/util/HTMLUtilities.java
| Rule | Violation | Line |
|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 26 |
| EmptyInitializer | Empty initializer was found | 28–31 |
| UnnecessaryConversionTemporary | Avoid unnecessary temporaries when converting primitives to Strings | 134 |
org/itracker/web/util/ImportHandler.java
| Rule | Violation | Line |
|---|---|---|
| EmptyIfStmt | Avoid empty if statements | 263–266 |
| EmptyCatchBlock | Avoid empty catch blocks | 376–378 |
org/itracker/web/util/ListIssuesActionUtil.java
| Rule | Violation | Line |
|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 130–131 |
| CollapsibleIfStatements | These nested if statements could be combined | 138–140 |
| CollapsibleIfStatements | These nested if statements could be combined | 184–186 |
org/itracker/web/util/LoginUtilities.java
| Rule | Violation | Line |
|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 275–277 |
| CollapsibleIfStatements | These nested if statements could be combined | 282–284 |
| CollapsibleIfStatements | These nested if statements could be combined | 290–292 |
| UnusedLocalVariable | Avoid unused local variables such as 'user'. | 445 |
| CollapsibleIfStatements | These nested if statements could be combined | 561–568 |
| CollapsibleIfStatements | These nested if statements could be combined | 604–610 |
Priority 4
org/itracker/web/actions/PortalHomeAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 16 |
| UselessParentheses | Useless parentheses. | 275 |
org/itracker/web/actions/admin/configuration/EditCustomFieldAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 31 |
| UselessParentheses | Useless parentheses. | 80 |
| UselessParentheses | Useless parentheses. | 81 |
| UselessParentheses | Useless parentheses. | 92 |
| UselessParentheses | Useless parentheses. | 93 |
org/itracker/web/actions/admin/configuration/ImportDataFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 23 |
org/itracker/web/actions/admin/configuration/ImportDataVerifyAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 28 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 29 |
| UselessParentheses | Useless parentheses. | 265–267 |
org/itracker/web/actions/admin/configuration/ListConfigurationAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 8 |
org/itracker/web/actions/admin/configuration/OrderCustomFieldValueAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 28 |
org/itracker/web/actions/admin/configuration/RemoveConfigurationItemAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 98 |
| UselessParentheses | Useless parentheses. | 142 |
org/itracker/web/actions/admin/configuration/RemoveCustomFieldAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 29 |
org/itracker/web/actions/admin/configuration/RemoveCustomFieldValueAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 30 |
org/itracker/web/actions/admin/project/EditProjectAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 31 |
org/itracker/web/actions/admin/project/EditProjectScriptAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 28 |
| UselessParentheses | Useless parentheses. | 81 |
org/itracker/web/actions/admin/report/DownloadReportAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 54 |
org/itracker/web/actions/admin/report/EditReportAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.PermissionType' | 26 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 28 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.RequestHelper' | 34 |
| UnusedImports | Avoid unused imports such as 'java.util.Map' | 42 |
| UnusedImports | Avoid unused imports such as 'java.util.Set' | 43 |
org/itracker/web/actions/admin/report/EditReportFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.PermissionType' | 6 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 8 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.RequestHelper' | 13 |
| UnusedImports | Avoid unused imports such as 'javax.xml.transform.TransformerConfigurationException' | 25 |
| UnusedImports | Avoid unused imports such as 'java.util.Map' | 32 |
| UnusedImports | Avoid unused imports such as 'java.util.Set' | 33 |
org/itracker/web/actions/admin/report/ExportReportAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 49 |
org/itracker/web/actions/admin/report/RemoveReportAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 53 |
org/itracker/web/actions/admin/user/EditUserFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 56 |
org/itracker/web/actions/admin/user/LockUserAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.LoginUtilities' | 26 |
| UselessParentheses | Useless parentheses. | 45 |
| UselessParentheses | Useless parentheses. | 45 |
org/itracker/web/actions/admin/user/UnlockUserAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 51 |
| UselessParentheses | Useless parentheses. | 51 |
org/itracker/web/actions/admin/workflow/EditWorkflowScriptAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.commons.beanutils.PropertyUtils' | 21 |
org/itracker/web/actions/admin/workflow/EditWorkflowScriptFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.NameValuePair' | 25 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 27 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.WorkflowUtilities' | 28 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.LoginUtilities' | 33 |
| UnusedImports | Avoid unused imports such as 'java.util.Arrays' | 41 |
org/itracker/web/actions/issuesearch/SearchIssuesFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 88 |
org/itracker/web/actions/preferences/EditPreferencesFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 108–109 |
| UselessParentheses | Useless parentheses. | 111–113 |
| UselessParentheses | Useless parentheses. | 115–118 |
| UselessParentheses | Useless parentheses. | 119–120 |
| UselessParentheses | Useless parentheses. | 128–129 |
| UselessParentheses | Useless parentheses. | 130–131 |
| UselessParentheses | Useless parentheses. | 143–144 |
| UselessParentheses | Useless parentheses. | 149–150 |
| UselessParentheses | Useless parentheses. | 156–157 |
| UselessParentheses | Useless parentheses. | 196–198 |
| UselessParentheses | Useless parentheses. | 200–201 |
| UselessParentheses | Useless parentheses. | 206–207 |
| UselessParentheses | Useless parentheses. | 214–215 |
| UselessParentheses | Useless parentheses. | 221–222 |
org/itracker/web/actions/project/CreateIssueAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 38 |
| UselessParentheses | Useless parentheses. | 183–186 |
org/itracker/web/actions/project/CreateIssueFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 33 |
| UselessParentheses | Useless parentheses. | 61–62 |
| UselessParentheses | Useless parentheses. | 62 |
org/itracker/web/actions/project/EditIssueAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 30 |
org/itracker/web/actions/project/EditIssueFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 29 |
| UselessParentheses | Useless parentheses. | 65–66 |
| UselessParentheses | Useless parentheses. | 65–66 |
org/itracker/web/actions/project/MoveIssueFormAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 62–63 |
| UselessParentheses | Useless parentheses. | 63 |
org/itracker/web/actions/project/RemoveIssueAttachmentAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 54 |
org/itracker/web/actions/project/ViewIssueAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 9 |
| UnusedImports | Avoid unused imports such as 'org.springframework.security.core.GrantedAuthority' | 18 |
| UnusedImports | Avoid unused imports such as 'org.springframework.security.core.context.SecurityContextHolder' | 19 |
| UnusedImports | Avoid unused imports such as 'org.springframework.security.core.userdetails.UserDetails' | 20 |
| UselessParentheses | Useless parentheses. | 149–151 |
| UselessParentheses | Useless parentheses. | 154 |
org/itracker/web/actions/project/ViewIssueActivityAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 76–80 |
| UselessParentheses | Useless parentheses. | 77–80 |
| UselessParentheses | Useless parentheses. | 79 |
| UselessParentheses | Useless parentheses. | 79 |
org/itracker/web/actions/project/WatchIssueAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 56 |
| UselessParentheses | Useless parentheses. | 56 |
org/itracker/web/actions/report/DisplayReportAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 40 |
org/itracker/web/actions/user/ForgotPasswordAction.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 119 |
org/itracker/web/actions/user/LoginAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.commons.beanutils.PropertyUtils' | 21 |
| UnusedImports | Avoid unused imports such as 'org.itracker.PasswordException' | 24 |
| UnusedImports | Avoid unused imports such as 'org.itracker.core.AuthenticationConstants' | 25 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.User' | 26 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 27 |
| UnusedImports | Avoid unused imports such as 'org.itracker.services.UserService' | 28 |
| UnusedImports | Avoid unused imports such as 'org.itracker.services.exceptions.AuthenticatorException' | 29 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.forms.LoginForm' | 31 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.Constants' | 32 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.LoginUtilities' | 33 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.ServletContextUtils' | 34 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.SessionManager' | 35 |
| UnusedImports | Avoid unused imports such as 'java.lang.reflect.InvocationTargetException' | 41 |
org/itracker/web/actions/user/LogoffAction.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.log4j.Logger' | 21 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.User' | 25 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.Constants' | 27 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.SessionManager' | 28 |
| UnusedImports | Avoid unused imports such as 'javax.servlet.http.Cookie' | 31 |
| UnusedImports | Avoid unused imports such as 'javax.servlet.http.HttpSession' | 34 |
org/itracker/web/filters/ExecuteAlwaysFilter.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 10 |
| UselessParentheses | Useless parentheses. | 146 |
org/itracker/web/filters/SetRequestCharacterEncoding.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 65 |
org/itracker/web/forms/ForgotPasswordForm.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 37 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 65 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 69 |
org/itracker/web/forms/IssueForm.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util' | 36 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 38 |
| UselessParentheses | Useless parentheses. | 94 |
| UselessParentheses | Useless parentheses. | 114–115 |
| UselessParentheses | Useless parentheses. | 142–143 |
| UselessParentheses | Useless parentheses. | 256–257 |
| UselessParentheses | Useless parentheses. | 283–284 |
| UselessParentheses | Useless parentheses. | 364 |
| UselessParentheses | Useless parentheses. | 365 |
| UselessParentheses | Useless parentheses. | 536–538 |
| UselessParentheses | Useless parentheses. | 539–541 |
| UselessParentheses | Useless parentheses. | 705–708 |
| UselessParentheses | Useless parentheses. | 923–924 |
org/itracker/web/forms/LoginForm.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 79 |
org/itracker/web/forms/RemoveHistoryForm.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 29 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 30 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 32 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 36 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 40 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 44 |
org/itracker/web/forms/RemoveRelationForm.java
| Rule | Violation | Line |
|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 37 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 38 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 57 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 61 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 65 |
org/itracker/web/ptos/ProjectPTO.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.PermissionType' | 4 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 6 |
| UnusedImports | Avoid unused imports such as 'org.itracker.services.ProjectService' | 7 |
| UnusedImports | Avoid unused imports such as 'java.util.Map' | 10 |
| UnusedImports | Avoid unused imports such as 'java.util.Set' | 11 |
org/itracker/web/ptos/UserPTO.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 22 |
| UselessParentheses | Useless parentheses. | 26 |
org/itracker/web/servlets/AttachmentDownloadController.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 65 |
| UselessParentheses | Useless parentheses. | 65 |
org/itracker/web/servlets/GenericController.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 21 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.PermissionType' | 25 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.User' | 26 |
| UnusedImports | Avoid unused imports such as 'org.itracker.services.ITrackerServices' | 27 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 28 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.Constants' | 29 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.RequestHelper' | 31 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.ServletContextUtils' | 32 |
| UnusedImports | Avoid unused imports such as 'javax.servlet.ServletContext' | 35 |
| UnusedImports | Avoid unused imports such as 'javax.servlet.http.HttpSession' | 40 |
| UnusedImports | Avoid unused imports such as 'java.util.Map' | 43 |
| UnusedImports | Avoid unused imports such as 'java.util.Set' | 44 |
| UselessParentheses | Useless parentheses. | 73 |
org/itracker/web/servlets/ReportDownloadController.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 58 |
| UselessParentheses | Useless parentheses. | 58 |
org/itracker/web/servlets/RssFeedController.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 257 |
org/itracker/web/taglib/AddErrorTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 59 |
org/itracker/web/taglib/ClearErrorsTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 47 |
org/itracker/web/taglib/FormatCustomFieldTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 89–90 |
| UselessParentheses | Useless parentheses. | 178–180 |
| UselessParentheses | Useless parentheses. | 207–209 |
| UselessParentheses | Useless parentheses. | 219 |
org/itracker/web/taglib/FormatDescriptionTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 106 |
org/itracker/web/taglib/FormatHistoryEntryTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 80 |
| UselessParentheses | Useless parentheses. | 176 |
org/itracker/web/taglib/FormatIconActionTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 95 |
| UselessParentheses | Useless parentheses. | 135 |
| UselessParentheses | Useless parentheses. | 185–186 |
| UselessParentheses | Useless parentheses. | 220 |
| UselessParentheses | Useless parentheses. | 244 |
org/itracker/web/taglib/FormatImageActionTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 91 |
| UselessParentheses | Useless parentheses. | 115 |
| UselessParentheses | Useless parentheses. | 172–173 |
| UselessParentheses | Useless parentheses. | 203 |
| UselessParentheses | Useless parentheses. | 242 |
| UselessParentheses | Useless parentheses. | 245 |
| UselessParentheses | Useless parentheses. | 257 |
org/itracker/web/taglib/FormatLinkTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 83 |
| UselessParentheses | Useless parentheses. | 107 |
| UselessParentheses | Useless parentheses. | 200 |
| UselessParentheses | Useless parentheses. | 211 |
org/itracker/web/taglib/FormatMessageTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 126 |
org/itracker/web/taglib/FormatPaginationLinkTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 104 |
| UselessParentheses | Useless parentheses. | 114 |
| UselessParentheses | Useless parentheses. | 126 |
org/itracker/web/taglib/FormatResolutionTag.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 109 |
org/itracker/web/util/EditIssueActionUtil.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util' | 9 |
| UselessParentheses | Useless parentheses. | 26 |
| UselessParentheses | Useless parentheses. | 141 |
org/itracker/web/util/ImportExportUtilities.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 361 |
| UselessParentheses | Useless parentheses. | 543 |
| UselessParentheses | Useless parentheses. | 638 |
| UselessParentheses | Useless parentheses. | 682 |
| UselessParentheses | Useless parentheses. | 771 |
org/itracker/web/util/ImportHandler.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 269 |
| UselessParentheses | Useless parentheses. | 271 |
| UselessParentheses | Useless parentheses. | 380 |
org/itracker/web/util/ListIssuesActionUtil.java
| Rule | Violation | Line |
|---|---|---|
| UselessParentheses | Useless parentheses. | 83 |
org/itracker/web/util/LoginUtilities.java
| Rule | Violation | Line |
|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.security.Principal' | 39 |
| UselessParentheses | Useless parentheses. | 319 |
| UselessParentheses | Useless parentheses. | 347 |
| UselessParentheses | Useless parentheses. | 422–423 |
Files
org/itracker/web/ApplicationInitialization.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'servletConfig'. | 3 | 49 |
| UnusedFormalParameter | Avoid unused constructor parameters such as 'reportService'. | 3 | 51 |
org/itracker/web/actions/PortalHomeAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 4 | 16 |
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 63 |
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 239–240 |
| UselessParentheses | Useless parentheses. | 4 | 275 |
org/itracker/web/actions/admin/configuration/EditConfigurationAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 187–234 |
org/itracker/web/actions/admin/configuration/EditCustomFieldAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 4 | 31 |
| UselessParentheses | Useless parentheses. | 4 | 80 |
| UselessParentheses | Useless parentheses. | 4 | 81 |
| UselessParentheses | Useless parentheses. | 4 | 92 |
| UselessParentheses | Useless parentheses. | 4 | 93 |
org/itracker/web/actions/admin/configuration/ImportDataFormAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 4 | 23 |
org/itracker/web/actions/admin/configuration/ImportDataProcessAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'importer'. | 3 | 132 |
| UnusedFormalParameter | Avoid unused method parameters such as 'importer'. | 3 | 178 |
org/itracker/web/actions/admin/configuration/ImportDataVerifyAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 4 | 28 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 4 | 29 |
| UnusedFormalParameter | Avoid unused method parameters such as 'ic'. | 3 | 126 |
| UnusedFormalParameter | Avoid unused method parameters such as 'ic'. | 3 | 240 |
| UselessParentheses | Useless parentheses. | 4 | 265–267 |
| UnusedFormalParameter | Avoid unused method parameters such as 'ic'. | 3 | 284 |
| UnusedFormalParameter | Avoid unused method parameters such as 'ic'. | 3 | 322 |
org/itracker/web/actions/admin/configuration/ListConfigurationAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 4 | 8 |
org/itracker/web/actions/admin/configuration/OrderCustomFieldValueAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 4 | 28 |
org/itracker/web/actions/admin/configuration/RemoveConfigurationItemAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 98 |
| UselessParentheses | Useless parentheses. | 4 | 142 |
org/itracker/web/actions/admin/configuration/RemoveCustomFieldAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 4 | 29 |
org/itracker/web/actions/admin/configuration/RemoveCustomFieldValueAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.SystemConfigurationUtilities' | 4 | 30 |
org/itracker/web/actions/admin/language/EditLanguageAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 138–140 |
org/itracker/web/actions/admin/language/EditLanguageFormAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 95–103 |
org/itracker/web/actions/admin/project/EditProjectAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 4 | 31 |
org/itracker/web/actions/admin/project/EditProjectScriptAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 4 | 28 |
| UselessParentheses | Useless parentheses. | 4 | 81 |
org/itracker/web/actions/admin/report/DownloadReportAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 54 |
org/itracker/web/actions/admin/report/EditReportAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.PermissionType' | 4 | 26 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 4 | 28 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.RequestHelper' | 4 | 34 |
| UnusedImports | Avoid unused imports such as 'java.util.Map' | 4 | 42 |
| UnusedImports | Avoid unused imports such as 'java.util.Set' | 4 | 43 |
org/itracker/web/actions/admin/report/EditReportFormAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.PermissionType' | 4 | 6 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 4 | 8 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.RequestHelper' | 4 | 13 |
| UnusedImports | Avoid unused imports such as 'javax.xml.transform.TransformerConfigurationException' | 4 | 25 |
| UnusedImports | Avoid unused imports such as 'java.util.Map' | 4 | 32 |
| UnusedImports | Avoid unused imports such as 'java.util.Set' | 4 | 33 |
org/itracker/web/actions/admin/report/ExportReportAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 49 |
org/itracker/web/actions/admin/report/RemoveReportAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 53 |
org/itracker/web/actions/admin/user/EditUserFormAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 56 |
org/itracker/web/actions/admin/user/LockUserAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.LoginUtilities' | 4 | 26 |
| UselessParentheses | Useless parentheses. | 4 | 45 |
| UselessParentheses | Useless parentheses. | 4 | 45 |
org/itracker/web/actions/admin/user/UnlockUserAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 51 |
| UselessParentheses | Useless parentheses. | 4 | 51 |
org/itracker/web/actions/admin/workflow/EditWorkflowScriptAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.commons.beanutils.PropertyUtils' | 4 | 21 |
org/itracker/web/actions/admin/workflow/EditWorkflowScriptFormAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.NameValuePair' | 4 | 25 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 4 | 27 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.WorkflowUtilities' | 4 | 28 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.LoginUtilities' | 4 | 33 |
| UnusedImports | Avoid unused imports such as 'java.util.Arrays' | 4 | 41 |
| UnusedLocalVariable | Avoid unused local variables such as 'isUpdate'. | 3 | 51 |
org/itracker/web/actions/issuesearch/SearchIssuesFormAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 88 |
| UnusedNullCheckInEquals | Invoke equals() on the object you've already ensured is not null | 3 | 128 |
org/itracker/web/actions/preferences/EditPreferencesFormAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 108–109 |
| UselessParentheses | Useless parentheses. | 4 | 111–113 |
| UselessParentheses | Useless parentheses. | 4 | 115–118 |
| UselessParentheses | Useless parentheses. | 4 | 119–120 |
| UselessParentheses | Useless parentheses. | 4 | 128–129 |
| UselessParentheses | Useless parentheses. | 4 | 130–131 |
| UselessParentheses | Useless parentheses. | 4 | 143–144 |
| UselessParentheses | Useless parentheses. | 4 | 149–150 |
| UselessParentheses | Useless parentheses. | 4 | 156–157 |
| UselessParentheses | Useless parentheses. | 4 | 196–198 |
| UselessParentheses | Useless parentheses. | 4 | 200–201 |
| UselessParentheses | Useless parentheses. | 4 | 206–207 |
| UselessParentheses | Useless parentheses. | 4 | 214–215 |
| UselessParentheses | Useless parentheses. | 4 | 221–222 |
org/itracker/web/actions/project/CreateIssueAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 4 | 38 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 181–189 |
| UselessParentheses | Useless parentheses. | 4 | 183–186 |
| UnusedFormalParameter | Avoid unused method parameters such as 'issue'. | 3 | 336 |
| UnusedFormalParameter | Avoid unused method parameters such as 'issueForm'. | 3 | 337 |
org/itracker/web/actions/project/CreateIssueFormAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 4 | 33 |
| UselessParentheses | Useless parentheses. | 4 | 61–62 |
| UselessParentheses | Useless parentheses. | 4 | 62 |
org/itracker/web/actions/project/EditIssueAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 4 | 30 |
org/itracker/web/actions/project/EditIssueFormAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 4 | 29 |
| UselessParentheses | Useless parentheses. | 4 | 65–66 |
| UselessParentheses | Useless parentheses. | 4 | 65–66 |
org/itracker/web/actions/project/MoveIssueAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedFormalParameter | Avoid unused method parameters such as 'mapping'. | 3 | 116 |
org/itracker/web/actions/project/MoveIssueFormAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 62–63 |
| UselessParentheses | Useless parentheses. | 4 | 63 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 143–148 |
org/itracker/web/actions/project/RemoveIssueAttachmentAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 54 |
org/itracker/web/actions/project/ViewIssueAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 4 | 9 |
| UnusedImports | Avoid unused imports such as 'org.springframework.security.core.GrantedAuthority' | 4 | 18 |
| UnusedImports | Avoid unused imports such as 'org.springframework.security.core.context.SecurityContextHolder' | 4 | 19 |
| UnusedImports | Avoid unused imports such as 'org.springframework.security.core.userdetails.UserDetails' | 4 | 20 |
| UselessParentheses | Useless parentheses. | 4 | 149–151 |
| UselessParentheses | Useless parentheses. | 4 | 154 |
org/itracker/web/actions/project/ViewIssueActivityAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 76–80 |
| UselessParentheses | Useless parentheses. | 4 | 77–80 |
| UselessParentheses | Useless parentheses. | 4 | 79 |
| UselessParentheses | Useless parentheses. | 4 | 79 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'setupJspEnv': private methods cannot be overridden | 3 | 107–112 |
| UnnecessaryModifier | Unnecessary modifier 'final' on method 'prepareActivitiesMap': private methods cannot be overridden | 3 | 119–133 |
org/itracker/web/actions/project/WatchIssueAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 56 |
| UselessParentheses | Useless parentheses. | 4 | 56 |
org/itracker/web/actions/report/DisplayReportAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 4 | 40 |
org/itracker/web/actions/user/ForgotPasswordAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 119 |
org/itracker/web/actions/user/LoginAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.commons.beanutils.PropertyUtils' | 4 | 21 |
| UnusedImports | Avoid unused imports such as 'org.itracker.PasswordException' | 4 | 24 |
| UnusedImports | Avoid unused imports such as 'org.itracker.core.AuthenticationConstants' | 4 | 25 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.User' | 4 | 26 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 4 | 27 |
| UnusedImports | Avoid unused imports such as 'org.itracker.services.UserService' | 4 | 28 |
| UnusedImports | Avoid unused imports such as 'org.itracker.services.exceptions.AuthenticatorException' | 4 | 29 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.forms.LoginForm' | 4 | 31 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.Constants' | 4 | 32 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.LoginUtilities' | 4 | 33 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.ServletContextUtils' | 4 | 34 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.SessionManager' | 4 | 35 |
| UnusedImports | Avoid unused imports such as 'java.lang.reflect.InvocationTargetException' | 4 | 41 |
org/itracker/web/actions/user/LogoffAction.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.log4j.Logger' | 4 | 21 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.User' | 4 | 25 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.Constants' | 4 | 27 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.SessionManager' | 4 | 28 |
| UnusedImports | Avoid unused imports such as 'javax.servlet.http.Cookie' | 4 | 31 |
| UnusedImports | Avoid unused imports such as 'javax.servlet.http.HttpSession' | 4 | 34 |
org/itracker/web/filters/ExecuteAlwaysFilter.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 4 | 10 |
| UselessParentheses | Useless parentheses. | 4 | 146 |
org/itracker/web/filters/SetRequestCharacterEncoding.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 65 |
org/itracker/web/forms/ForgotPasswordForm.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 37 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 65 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 69 |
org/itracker/web/forms/IssueForm.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util' | 4 | 36 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util' | 4 | 38 |
| UselessParentheses | Useless parentheses. | 4 | 94 |
| UselessParentheses | Useless parentheses. | 4 | 114–115 |
| UselessParentheses | Useless parentheses. | 4 | 142–143 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 167 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 173 |
| UselessParentheses | Useless parentheses. | 4 | 256–257 |
| UselessParentheses | Useless parentheses. | 4 | 283–284 |
| UselessParentheses | Useless parentheses. | 4 | 364 |
| UselessParentheses | Useless parentheses. | 4 | 365 |
| UnusedFormalParameter | Avoid unused method parameters such as 'user'. | 3 | 429 |
| UnusedFormalParameter | Avoid unused method parameters such as 'issueService'. | 3 | 430 |
| UselessParentheses | Useless parentheses. | 4 | 536–538 |
| UselessParentheses | Useless parentheses. | 4 | 539–541 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 614–620 |
| UselessParentheses | Useless parentheses. | 4 | 705–708 |
| EmptyStatementNotInLoop | An empty statement (semicolon) not part of a loop | 3 | 779 |
| UselessParentheses | Useless parentheses. | 4 | 923–924 |
org/itracker/web/forms/LoginForm.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 79 |
org/itracker/web/forms/ProjectForm.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 108–109 |
| EmptyIfStmt | Avoid empty if statements | 3 | 235–236 |
org/itracker/web/forms/ProjectScriptForm.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'action'. | 3 | 38 |
| UnusedPrivateField | Avoid unused private fields such as 'delId'. | 3 | 39 |
org/itracker/web/forms/RemoveHistoryForm.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 29 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 30 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 32 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 36 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 40 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 44 |
org/itracker/web/forms/RemoveRelationForm.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 37 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 38 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 53 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.String' due to existing implicit import 'java.lang.*' | 4 | 57 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 61 |
| UnnecessaryFullyQualifiedName | Unnecessary use of fully qualified name 'java.lang.Integer' due to existing implicit import 'java.lang.*' | 4 | 65 |
org/itracker/web/forms/ReportForm.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'reportType'. | 3 | 44 |
org/itracker/web/forms/WorkflowScriptForm.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'log'. | 3 | 49 |
| UnusedLocalVariable | Avoid unused local variables such as 'eof'. | 3 | 108 |
| EmptyWhileStmt | Avoid empty while statements | 3 | 109–110 |
org/itracker/web/ptos/ProjectPTO.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.PermissionType' | 4 | 4 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 4 | 6 |
| UnusedImports | Avoid unused imports such as 'org.itracker.services.ProjectService' | 4 | 7 |
| UnusedImports | Avoid unused imports such as 'java.util.Map' | 4 | 10 |
| UnusedImports | Avoid unused imports such as 'java.util.Set' | 4 | 11 |
org/itracker/web/ptos/UserPTO.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 22 |
| UselessParentheses | Useless parentheses. | 4 | 26 |
org/itracker/web/security/AuthenticationErrorHandler.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'log'. | 3 | 15 |
org/itracker/web/servlets/AttachmentDownloadController.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 65 |
| UselessParentheses | Useless parentheses. | 4 | 65 |
org/itracker/web/servlets/GenericController.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 4 | 21 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.PermissionType' | 4 | 25 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.User' | 4 | 26 |
| UnusedImports | Avoid unused imports such as 'org.itracker.services.ITrackerServices' | 4 | 27 |
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util.UserUtilities' | 4 | 28 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.Constants' | 4 | 29 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.RequestHelper' | 4 | 31 |
| UnusedImports | Avoid unused imports such as 'org.itracker.web.util.ServletContextUtils' | 4 | 32 |
| UnusedImports | Avoid unused imports such as 'javax.servlet.ServletContext' | 4 | 35 |
| UnusedImports | Avoid unused imports such as 'javax.servlet.http.HttpSession' | 4 | 40 |
| UnusedImports | Avoid unused imports such as 'java.util.Map' | 4 | 43 |
| UnusedImports | Avoid unused imports such as 'java.util.Set' | 4 | 44 |
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 3 | 66 |
| UselessParentheses | Useless parentheses. | 4 | 73 |
org/itracker/web/servlets/ReportDownloadController.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 58 |
| UselessParentheses | Useless parentheses. | 4 | 58 |
org/itracker/web/servlets/RssFeedController.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessOverridingMethod | Overriding method merely calls super | 3 | 45–48 |
| UselessParentheses | Useless parentheses. | 4 | 257 |
org/itracker/web/taglib/AddErrorTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 59 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 72–73 |
org/itracker/web/taglib/ClearErrorsTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 47 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 59–60 |
org/itracker/web/taglib/FormatCustomFieldTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 3 | 47 |
| UselessParentheses | Useless parentheses. | 4 | 89–90 |
| UselessParentheses | Useless parentheses. | 4 | 178–180 |
| UselessParentheses | Useless parentheses. | 4 | 207–209 |
| UselessParentheses | Useless parentheses. | 4 | 219 |
org/itracker/web/taglib/FormatDescriptionTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 106 |
org/itracker/web/taglib/FormatHistoryEntryTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 80 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 163–164 |
| UselessParentheses | Useless parentheses. | 4 | 176 |
org/itracker/web/taglib/FormatIconActionTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 95 |
| UselessParentheses | Useless parentheses. | 4 | 135 |
| UselessParentheses | Useless parentheses. | 4 | 185–186 |
| UselessParentheses | Useless parentheses. | 4 | 220 |
| UselessParentheses | Useless parentheses. | 4 | 244 |
org/itracker/web/taglib/FormatImageActionTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 91 |
| UselessParentheses | Useless parentheses. | 4 | 115 |
| UselessParentheses | Useless parentheses. | 4 | 172–173 |
| SimplifiedTernary | Ternary operators that can be simplified with || or && | 3 | 172–173 |
| UselessParentheses | Useless parentheses. | 4 | 203 |
| UselessParentheses | Useless parentheses. | 4 | 242 |
| UselessParentheses | Useless parentheses. | 4 | 245 |
| UselessParentheses | Useless parentheses. | 4 | 257 |
org/itracker/web/taglib/FormatLinkTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 83 |
| UselessParentheses | Useless parentheses. | 4 | 107 |
| UselessParentheses | Useless parentheses. | 4 | 200 |
| UselessParentheses | Useless parentheses. | 4 | 211 |
org/itracker/web/taglib/FormatMessageTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 126 |
org/itracker/web/taglib/FormatPaginationLinkTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 104 |
| UselessParentheses | Useless parentheses. | 4 | 114 |
| UselessParentheses | Useless parentheses. | 4 | 126 |
org/itracker/web/taglib/FormatResolutionTag.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 94–95 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 100–102 |
| UselessParentheses | Useless parentheses. | 4 | 109 |
org/itracker/web/util/AttachmentUtilities.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 45–48 |
org/itracker/web/util/EditCustomFieldActionUtil.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 3 | 7 |
org/itracker/web/util/EditIssueActionUtil.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'org.itracker.model.util' | 4 | 9 |
| UselessParentheses | Useless parentheses. | 4 | 26 |
| UselessParentheses | Useless parentheses. | 4 | 141 |
org/itracker/web/util/EditProjectFormActionUtil.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'log'. | 3 | 12 |
org/itracker/web/util/HTMLUtilities.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedPrivateField | Avoid unused private fields such as 'logger'. | 3 | 26 |
| EmptyInitializer | Empty initializer was found | 3 | 28–31 |
| UnnecessaryConversionTemporary | Avoid unnecessary temporaries when converting primitives to Strings | 3 | 134 |
org/itracker/web/util/ImportExportUtilities.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 361 |
| UselessParentheses | Useless parentheses. | 4 | 543 |
| UselessParentheses | Useless parentheses. | 4 | 638 |
| UselessParentheses | Useless parentheses. | 4 | 682 |
| UselessParentheses | Useless parentheses. | 4 | 771 |
org/itracker/web/util/ImportHandler.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| EmptyIfStmt | Avoid empty if statements | 3 | 263–266 |
| UselessParentheses | Useless parentheses. | 4 | 269 |
| UselessParentheses | Useless parentheses. | 4 | 271 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 376–378 |
| UselessParentheses | Useless parentheses. | 4 | 380 |
org/itracker/web/util/ListIssuesActionUtil.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UselessParentheses | Useless parentheses. | 4 | 83 |
| EmptyCatchBlock | Avoid empty catch blocks | 3 | 130–131 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 138–140 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 184–186 |
org/itracker/web/util/LoginUtilities.java
| Rule | Violation | Priority | Line |
|---|---|---|---|
| UnusedImports | Avoid unused imports such as 'java.security.Principal' | 4 | 39 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 275–277 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 282–284 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 290–292 |
| UselessParentheses | Useless parentheses. | 4 | 319 |
| UselessParentheses | Useless parentheses. | 4 | 347 |
| UselessParentheses | Useless parentheses. | 4 | 422–423 |
| UnusedLocalVariable | Avoid unused local variables such as 'user'. | 3 | 445 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 561–568 |
| CollapsibleIfStatements | These nested if statements could be combined | 3 | 604–610 |
