The following document contains the results of PMD 4.2.5.
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' | 27 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'persistenceStructureService'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.CriteriaLookupService' | 8 |
| Avoid unused method parameters such as 'queryClass'. | 86 - 114 |
| Avoid unused method parameters such as 'criteria'. | 125 - 132 |
| Avoid unused method parameters such as 'queryClass'. | 125 - 132 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.CriteriaDateTimeValue' | 10 |
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.CriteriaLookupService' | 11 |
| Avoid unused method parameters such as 'criteria'. | 124 - 130 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreApiServiceLocator' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.exception.CompletionException' | 26 |
| Avoid unused local variables such as 'indexName'. | 130 |
| These nested if statements could be combined | 644 - 672 |
| Avoid empty catch blocks | 721 - 723 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 146 - 148 |
| Avoid empty catch blocks | 164 - 165 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 49 - 53 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.component.Component' | 23 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.util.ComponentUtils' | 24 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 38 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'regexString'. | 82 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.ComplexAttributeDefinition' | 23 |
| Avoid empty catch blocks | 130 - 132 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 61 - 63 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'key'. | 39 |
| Overriding method merely calls super | 58 - 60 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'key'. | 60 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.config.property.ConfigurationService' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.config.property.ConfigurationService' | 26 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 27 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants' | 28 |
| Avoid empty if statements | 57 - 59 |
| Avoid empty if statements | 62 - 64 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 3 |
| Avoid unused imports such as 'java.util.List' | 4 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 36 - 38 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'java.util.List' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants' | 22 |
| Overriding method merely calls super | 39 - 41 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.DataObjectEntry' | 19 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'entryName'. | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.exception.ValidationException' | 28 |
| Avoid unused imports such as 'java.util.Map' | 36 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 242 - 246 |
| These nested if statements could be combined | 505 - 529 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 714 - 726 |
| Avoid empty if statements | 744 - 751 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 121 - 126 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.StringTokenizer' | 44 |
| Avoid empty catch blocks | 87 - 89 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'pkNames'. | 152 |
| Avoid unused local variables such as 'user'. | 153 |
| Avoid unused local variables such as 'searchUsingOnlyPrimaryKeyValues'. | 170 |
| Avoid unused local variables such as 'lookupView'. | 225 |
| Avoid unused local variables such as 'lookupView'. | 446 |
| Avoid unused local variables such as 'dataObjectExample'. | 511 |
| These nested if statements could be combined | 535 - 545 |
| Avoid unused local variables such as 'lookupView'. | 714 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'docOrBoClass'. | 386 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 711 - 733 |
| These nested if statements could be combined | 1066 - 1068 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 106 - 109 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 240 - 242 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 43 - 45 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 3 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 124 - 130 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 206 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 290 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'ariaHTML'. | 31 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 70 - 72 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'customValidatorClass'. | 85 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 306 - 311 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 96 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 107 - 109 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 96 - 99 |
| These nested if statements could be combined | 103 - 106 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 149 - 151 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 228 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 49 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 88 - 91 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.web.form.UifFormBase' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.view.View' | 19 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'layoutManager'. | 296 |
| Avoid empty if statements | 306 - 308 |
| Violation | Line |
|---|---|
| An operation on an Immutable object (String, BigDecimal or BigInteger) won't change the object itself | 188 |
| Avoid empty if statements | 728 - 730 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 81 - 83 |
| Avoid empty if statements | 84 - 86 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 101 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'defaultPathSet'. | 56 |
| These nested if statements could be combined | 141 - 143 |
| These nested if statements could be combined | 281 - 318 |
| These nested if statements could be combined | 314 - 316 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.config.property.ConfigurationService' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.api.util.ConcreteKeyValue' | 22 |
| Avoid unused imports such as 'org.kuali.rice.core.api.util.KeyValue' | 23 |
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.AttributeDefinition' | 25 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.control.CheckboxControl' | 28 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.control.Control' | 29 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.control.RadioGroupControl' | 30 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.control.TextAreaControl' | 31 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.field.AttributeField' | 32 |
| Avoid unused imports such as 'java.util.ArrayList' | 38 |
| Avoid unused imports such as 'java.util.List' | 39 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.DataDictionary' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' | 15 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants.ViewType' | 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.beans.PropertyValue' | 16 |
| Avoid empty catch blocks | 162 - 164 |
| Avoid empty catch blocks | 164 - 166 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'lookupCriteriaEnabled'. | 80 |
| Avoid unused private fields such as 'supplementalActionsEnabled'. | 81 |
| Avoid unused private fields such as 'disableSearchButtons'. | 82 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.component.BindingInfo' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 13 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 15 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.lang.annotation.Annotation' | 37 |
| Unnecessary final modifier in final class | 62 - 78 |
| Unnecessary final modifier in final class | 83 - 95 |
| Unnecessary final modifier in final class | 107 - 113 |
| Unnecessary final modifier in final class | 467 - 469 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 243 - 245 |
| Avoid empty catch blocks | 245 - 247 |
| Avoid empty catch blocks | 298 - 300 |
| Avoid empty catch blocks | 303 - 305 |
| An empty statement (semicolon) not part of a loop | 597 |
| These nested if statements could be combined | 828 - 830 |
| These nested if statements could be combined | 918 - 920 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 112 - 114 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' | 18 |
| These nested if statements could be combined | 85 - 87 |
| Avoid empty catch blocks | 162 - 164 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 42 - 45 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants' | 19 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 28 |
| Avoid unused imports such as 'java.util.List' | 31 |
| These nested if statements could be combined | 98 - 101 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 161 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 45 - 47 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 21 |
