The following document contains the results of PMD 4.2.5.
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.DataDictionaryException' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.InactivationBlockingDefinition' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.exception.ClassValidationException' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.document.authorization.DocumentAuthorizer' | 20 |
| Avoid unused imports such as 'org.kuali.rice.krad.document.authorization.DocumentPresentationController' | 21 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 274 - 276 |
| These nested if statements could be combined | 280 - 282 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 125 - 127 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 195 - 197 |
| These nested if statements could be combined | 201 - 203 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.DataDictionary' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.ReferenceDefinition' | 22 |
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.exception.AttributeValidationException' | 23 |
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.exception.ClassValidationException' | 24 |
| Avoid unused imports such as 'org.kuali.rice.krad.document.authorization.MaintenanceDocumentAuthorizer' | 30 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 412 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 99 - 101 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 270 - 278 |
| These nested if statements could be combined | 274 - 276 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 21 |
| No need to import a type that lives in the same package | 22 |
| No need to import a type that lives in the same package | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 26 |
| No need to import a type that lives in the same package | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 21 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 241 - 253 |
| Avoid empty catch blocks | 263 - 265 |
| Avoid empty catch blocks | 434 - 436 |
| Avoid empty catch blocks | 448 - 450 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 377 - 386 |
| These nested if statements could be combined | 974 - 976 |
| These nested if statements could be combined | 1022 - 1029 |
| Avoid unused local variables such as 'fieldsForLookup'. | 1098 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.util.GlobalVariables' | 37 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getBusinessObjectAuthorizationService()'. | 207 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 430 - 449 |
| These nested if statements could be combined | 503 - 505 |
| Avoid unused local variables such as 'attributeRefName'. | 560 |
| Avoid unused method parameters such as 'nestedObjectPrefix'. | 630 - 667 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 161 - 163 |
| These nested if statements could be combined | 169 - 171 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 371 - 377 |
| These nested if statements could be combined | 394 - 397 |
| Avoid empty if statements | 427 - 435 |
| Avoid unused local variables such as 'cachedValues'. | 687 |
| These nested if statements could be combined | 1261 - 1299 |
| Avoid unused local variables such as 'catchme'. | 1273 |
| Avoid unused local variables such as 'catchme'. | 1351 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 139 - 141 |
| These nested if statements could be combined | 767 - 789 |
| These nested if statements could be combined | 1127 - 1129 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 221 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 48 - 52 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'containsIntermediateNull(Object,String)'. | 716 |
| These nested if statements could be combined | 772 - 778 |
| Avoid empty if statements | 986 - 989 |
| These nested if statements could be combined | 1038 - 1040 |
| Avoid unused local variables such as 'propertyName'. | 1177 |
| Unnecessary final modifier in final class | 1427 - 1431 |
| Unnecessary final modifier in final class | 1440 - 1442 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 83 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 156 - 169 |
| Unnecessary final modifier in final class | 171 - 198 |
| Unnecessary final modifier in final class | 223 - 225 |
| Unnecessary final modifier in final class | 227 - 230 |
| Avoid unused private methods such as 'getMaintenanceDocumentService()'. | 357 |
| Avoid unused private methods such as 'getWorkflowDocumentService()'. | 364 |
| Avoid unused private methods such as 'getKualiConfigurationService()'. | 371 |
| Avoid unused private methods such as 'getKualiExceptionIncidentService()'. | 378 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'params'. | 376 |
| Avoid unused local variables such as 'files'. | 394 |
| These nested if statements could be combined | 475 - 478 |
| These nested if statements could be combined | 526 - 529 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 69 - 71 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'OUTPUT_ENCRYPTION_WARNING'. | 78 |
| Avoid unused method parameters such as 'response'. | 243 - 254 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 521 |
| These nested if statements could be combined | 607 - 609 |
| Avoid unused method parameters such as 'response'. | 762 - 795 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1166 - 1183 |
| Avoid empty catch blocks | 1306 - 1307 |
| Avoid unused local variables such as 'newNote'. | 1475 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.DocumentEntry' | 32 |
| Avoid unused local variables such as 'label'. | 317 |
| Avoid unused local variables such as 'objectDescription'. | 318 |
| Avoid unused local variables such as 'helpForm'. | 461 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'kualiInquirable'. | 299 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 121 - 123 |
| These nested if statements could be combined | 133 - 135 |
| Avoid unused method parameters such as 'request'. | 164 - 168 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 240 - 257 |
| Avoid empty catch blocks | 255 |
| Avoid empty catch blocks | 796 - 798 |
| Avoid empty catch blocks | 799 - 801 |
| Avoid empty catch blocks | 802 - 804 |
| These nested if statements could be combined | 837 - 840 |
| Avoid unused local variables such as 'documentAuthorizer'. | 960 |
| Avoid unused local variables such as 'newBo'. | 966 |
| Avoid unused local variables such as 'maintenanceDocumentAuthorizer'. | 1002 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 30 |
| Avoid unused private fields such as 'inactiveRecordDisplay'. | 81 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 352 - 354 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 429 - 431 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 119 - 153 |
| These nested if statements could be combined | 191 - 194 |
| These nested if statements could be combined | 197 - 200 |
| Avoid unused local variables such as 'docTypeName'. | 458 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 25 |
| Avoid unused local variables such as 'ddService'. | 148 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 227 |
| These nested if statements could be combined | 327 - 330 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 37 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 185 - 187 |
| Avoid unused method parameters such as 'requestParams'. | 241 - 259 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 291 - 292 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 258 - 260 |
| These nested if statements could be combined | 280 - 282 |
| These nested if statements could be combined | 504 - 506 |
