itracker
Download
Documentation
Getting Involved
Project Documentation
PMD Results
The following document contains the results of PMD 4.2.2.
Files
org/itracker/core/ApplicationInitialization.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.jws.soap.InitParam' | 33 |
| Avoid unused constructor parameters such as 'reportService'. | 57 - 61 |
org/itracker/core/resources/ITrackerResources.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'bundle'. | 340 |
org/itracker/model/CustomField.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'name'. | 150 - 153 |
org/itracker/services/implementations/ConfigurationServiceImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 131 - 133 |
| Do not use if statements that are always true or always false | 600 |
| Avoid empty if statements | 748 - 751 |
| Avoid empty catch blocks | 779 - 780 |
| These nested if statements could be combined | 1091 - 1104 |
org/itracker/services/implementations/NotificationServiceImpl.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 969 - 973 |
org/itracker/services/implementations/ProjectServiceImpl.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'user'. | 421 |
| These nested if statements could be combined | 433 - 435 |
org/itracker/services/implementations/UserServiceImpl.java
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'projectDAO'. | 122 - 130 |
| Avoid unused constructor parameters such as 'reportDAO'. | 122 - 130 |
| Avoid empty catch blocks | 207 - 209 |
| Avoid empty if statements | 483 - 485 |
| Avoid unused local variables such as 'i'. | 792 |
org/itracker/services/util/HTMLUtilities.java
| Violation | Line |
|---|---|
| Avoid unnecessary temporaries when converting primitives to Strings | 153 |
org/itracker/services/util/IssueUtilities.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 128 - 130 |
| These nested if statements could be combined | 674 - 681 |
| These nested if statements could be combined | 766 - 772 |
org/itracker/web/actions/ShowHelpAction.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.io.File' | 3 |
org/itracker/web/actions/admin/AdminHomeAction.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'numberDefinedKeys'. | 92 |
org/itracker/web/actions/admin/configuration/EditConfigurationAction.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 190 - 245 |
org/itracker/web/actions/admin/configuration/EditCustomFieldValueAction.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'highestSortOrder'. | 87 |
org/itracker/web/actions/admin/configuration/ImportDataProcessAction.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'ic'. | 118 - 160 |
| Avoid unused method parameters such as 'importer'. | 118 - 160 |
| Avoid unused method parameters such as 'ic'. | 162 - 188 |
| Avoid unused method parameters such as 'importer'. | 162 - 188 |
| Avoid unused method parameters such as 'ic'. | 190 - 231 |
| Avoid unused method parameters such as 'ic'. | 233 - 302 |
org/itracker/web/actions/admin/configuration/ImportDataVerifyAction.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'ic'. | 136 - 248 |
| Avoid unused method parameters such as 'ic'. | 250 - 287 |
| Avoid unused method parameters such as 'ic'. | 289 - 330 |
| Avoid unused method parameters such as 'ic'. | 332 - 340 |
org/itracker/web/actions/admin/language/EditLanguageAction.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 177 - 179 |
org/itracker/web/actions/admin/language/EditLanguageFormAction.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 116 - 125 |
org/itracker/web/actions/admin/project/EditProjectScriptFormAction.java
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'i'. | 108 |
org/itracker/web/actions/admin/report/EditReportAction.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 103 - 105 |
| Avoid empty if statements | 106 - 108 |
org/itracker/web/actions/admin/workflow/EditWorkflowScriptAction.java
| Violation | Line |
|---|---|
| Avoid empty if statements | 71 - 82 |
org/itracker/web/actions/feed/RssFeedAction.java
| Violation | Line |
|---|---|
| Overriding method merely calls super | 16 - 19 |
org/itracker/web/actions/issuesearch/SearchIssuesFormAction.java
| Violation | Line |
|---|---|
| Invoke equals() on the object you've already ensured is not null | 139 |
org/itracker/web/actions/project/CreateIssueAction.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 209 - 217 |
| Avoid unused method parameters such as 'issueForm'. | 375 - 382 |
| Avoid unused method parameters such as 'issue'. | 375 - 382 |
org/itracker/web/actions/project/EditIssueActionUtil.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'issueService'. | 188 - 262 |
| Avoid unused method parameters such as 'user'. | 188 - 262 |
| Avoid unused method parameters such as 'issueService'. | 277 - 312 |
| Avoid unused method parameters such as 'issueService'. | 314 - 360 |
| These nested if statements could be combined | 378 - 384 |
| Avoid empty if statements | 656 - 658 |
| Avoid empty if statements | 709 - 711 |
org/itracker/web/actions/project/ListIssuesActionUtil.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.net.URL' | 3 |
| Avoid empty catch blocks | 135 - 136 |
| These nested if statements could be combined | 184 - 186 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 208 |
org/itracker/web/actions/project/MoveIssueAction.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'mapping'. | 121 - 132 |
org/itracker/web/actions/project/MoveIssueFormAction.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 151 - 156 |
org/itracker/web/actions/report/DisplayReportAction.java
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'request'. | 209 - 226 |
org/itracker/web/actions/user/SelfRegisterAction.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.itracker.services.ITrackerServices' | 39 |
| Avoid unused imports such as 'org.itracker.web.util.RequestHelper' | 47 |
org/itracker/web/filters/ExecuteAlwaysFilter.java
| Violation | Line |
|---|---|
| These nested if statements could be combined | 127 - 139 |
org/itracker/web/ptos/ProjectScriptPTO.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.itracker.model.Project' | 5 |
| Avoid unused imports such as 'org.itracker.services.util.IssueUtilities' | 8 |
org/itracker/web/servlets/AttachmentDownloadController.java
| Violation | Line |
|---|---|
| Avoid unnecessary return statements | 118 |
org/itracker/web/servlets/GenericController.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'logger'. | 66 |
org/itracker/web/servlets/ReportDownloadController.java
| Violation | Line |
|---|---|
| Avoid unnecessary return statements | 88 |
org/itracker/web/servlets/RssFeedController.java
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.sun.syndication.feed.synd.SyndCategory' | 6 |
| Avoid unused imports such as 'com.sun.syndication.feed.synd.SyndCategoryImpl' | 7 |
| Avoid unused imports such as 'com.sun.syndication.io.impl.RSS20Generator' | 10 |
| Avoid unused imports such as 'org.itracker.services.util.ProjectUtilities' | 20 |
| Avoid unused imports such as 'org.itracker.web.util.LoginUtilities' | 23 |
| Avoid unused imports such as 'org.springframework.util.StringUtils' | 27 |
| Avoid unused imports such as 'java.io.ByteArrayOutputStream' | 36 |
| Avoid unused imports such as 'java.util.regex.MatchResult' | 39 |
| Overriding method merely calls super | 57 - 60 |
| Avoid unused local variables such as 'f'. | 102 |
| Avoid unused method parameters such as 'p'. | 217 - 304 |
| Avoid unused local variables such as 'statusLocalizedString'. | 244 |
| Avoid unused local variables such as 'severityLocalizedString'. | 244 |
| Avoid unused local variables such as 'componentsSize'. | 244 |
org/itracker/web/taglib/FormatCustomFieldTag.java
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'logger'. | 56 |
| Avoid empty catch blocks | 225 - 227 |
| Avoid empty catch blocks | 227 - 229 |
| Avoid unused private methods such as 'formatDate(Date,ResourceBundle)'. | 329 |
org/itracker/web/taglib/FormatResolutionTag.java
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 95 - 96 |
| Avoid empty catch blocks | 101 - 103 |
