The following document contains the results of PMD 4.2.5.
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'fetchLimit'. | 64 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.ActionRequestDTO' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 24 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_futureElements'. | 56 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 4 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 6 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 82 - 86 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.logging.Log' | 24 |
| Avoid unused imports such as 'org.apache.commons.logging.LogFactory' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 4 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 6 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LAST_ACCCESS_TIMEOUT_FIELD'. | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 20 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 129 - 131 |
| These nested if statements could be combined | 229 - 231 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.ModelObjectComplete' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Timestamp' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlEnum' | 5 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.ModelObjectComplete' | 5 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'b'. | 36 - 38 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlElement' | 9 |
| Avoid unused constructor parameters such as 'b'. | 36 - 37 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'b'. | 36 - 38 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 18 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 122 - 128 |
| Violation | Line |
|---|---|
| Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! | 89 |
| Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! | 106 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 75 - 77 |
| Avoid unused local variables such as 'copySize'. | 89 |
| Avoid empty catch blocks | 110 - 112 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.SerializationException' | 19 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 78 - 80 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 131 - 132 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 60 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 83 - 85 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 114 - 116 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 42 - 44 |
| Overriding method merely calls super | 50 - 52 |
| Overriding method merely calls super | 54 - 56 |
| Overriding method merely calls super | 58 - 60 |
| Overriding method merely calls super | 62 - 64 |
| Overriding method merely calls super | 66 - 68 |
| Violation | Line |
|---|---|
| Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. | 54 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 66 - 68 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'parentCriteria'. | 348 - 373 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'allowWildcards'. | 335 - 346 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'otherClass'. | 33 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 139 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'JPA_PARENT_INITIAL_ALIAS'. | 96 |
| Avoid unused private fields such as 'ALIAS_PREFIX_LEN'. | 99 |
| Avoid unused local variables such as 'valueLen'. | 393 |
| Avoid empty if statements | 1225 - 1227 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 246 |
| Avoid empty catch blocks | 311 |
| Violation | Line |
|---|---|
| Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal | 37 |
| Violation | Line |
|---|---|
| Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal | 37 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'converted'. | 45 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 44 - 60 |
| These nested if statements could be combined | 75 - 91 |
| These nested if statements could be combined | 104 - 120 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 124 - 126 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'S_TRUE'. | 26 |
| Avoid unused private fields such as 'S_FALSE'. | 27 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 33 - 36 |
| These nested if statements could be combined | 47 - 53 |
| Violation | Line |
|---|---|
| Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal | 29 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 77 - 78 |
| Avoid empty catch blocks | 83 - 84 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 90 - 91 |
| Avoid empty catch blocks | 96 - 97 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashSet' | 18 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 43 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 113 - 115 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.lang.reflect.Field' | 9 |
| Avoid unused imports such as 'java.util.Collection' | 10 |
| Avoid unused imports such as 'java.util.Collections' | 11 |
| Avoid unused imports such as 'java.util.Map' | 13 |
| Avoid unused imports such as 'java.util.Set' | 14 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 97 - 99 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 144 - 152 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 79 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'keys'. | 84 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 307 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 129 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'criteria'. | 121 - 127 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 219 - 221 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 96 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'successful'. | 184 - 203 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.resourceloader.ResourceLoader' | 19 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.BaseLifecycle' | 24 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 75 - 77 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'parameterService'. | 41 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'parameterService'. | 55 |
| These nested if statements could be combined | 103 - 105 |
| Avoid unused method parameters such as 'parameters'. | 180 - 188 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'toReturn'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'fieldName'. | 417 |
| Avoid unused private fields such as 'fieldType'. | 418 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'sourceFilePath'. | 105 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'length'. | 73 |
| Avoid empty while statements | 76 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 171 - 173 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 108 - 111 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 189 |
| Avoid empty if statements | 193 - 195 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'currentPage'. | 57 - 65 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'edlContext'. | 54 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 41 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 82 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'requestParser'. | 94 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.config.module.RunMode' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 22 |
| Avoid unused imports such as 'org.kuali.rice.core.framework.persistence.jpa.OrmUtils' | 24 |
| Avoid unused imports such as 'org.kuali.rice.kew.lifecycle.EmbeddedLifeCycle' | 28 |
| Avoid unused imports such as 'java.util.Collections' | 37 |
| Avoid unused imports such as 'java.util.LinkedList' | 38 |
| Avoid unused private methods such as 'getClientProtocol()'. | 132 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 37 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 27 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 25 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 106 - 108 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 32 |
| Avoid empty catch blocks | 96 - 98 |
| Avoid empty if statements | 113 - 115 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'KEW_CHANNEL'. | 48 |
| Avoid unused local variables such as 'error'. | 141 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 259 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notification'. | 50 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notification'. | 48 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 26 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.LifecycleBean' | 22 |
| Avoid unused imports such as 'org.springframework.beans.factory.BeanInitializationException' | 26 |
| Avoid unused private fields such as 'LOG'. | 35 |
| Avoid unused private fields such as 'dataSource'. | 40 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'NOTIFICATION_MESSAGE_DELIVERY_DISPATCH_SERVICE'. | 53 |
| Avoid unused private fields such as 'NOTIFICATION_MESSAGE_DELIVERER_REGISTRY_SERVICE'. | 58 |
| Avoid unused private fields such as 'NOTIFICATION_EMAIL_SERVICE'. | 61 |
| Avoid unused private fields such as 'NOTIFICATION_CONFIG'. | 62 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 38 |
| Avoid unused local variables such as 'lockedDeliveries'. | 101 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 40 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 111 - 114 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.xml.sax.SAXException' | 21 |
| Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 18 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'prefsService'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 38 |
| Avoid unused private fields such as 'businessObjectDao'. | 40 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 61 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 133 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'content'. | 364 - 380 |
| Avoid unused method parameters such as 'notification'. | 364 - 380 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'firstMessageDelivery'. | 71 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'userPreferenceService'. | 58 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notificationWorkflowDocumentService'. | 55 |
| These nested if statements could be combined | 146 - 150 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notificationChannelService'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 301 - 307 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 268 - 346 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 295 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'response'. | 302 - 335 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.web.servlet.view.RedirectView' | 42 |
| Avoid unused method parameters such as 'response'. | 293 - 350 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 603 - 605 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ken.deliverer.NotificationMessageDeliverer' | 31 |
| Avoid unused imports such as 'org.kuali.rice.ken.exception.ErrorList' | 32 |
| An empty statement (semicolon) not part of a loop | 149 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 183 - 186 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'toDocumentRouteHeaderValueActionListExtension(DocumentRouteHeaderValue)'. | 135 - 136 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 578 |
| Avoid empty catch blocks | 583 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 376 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 385 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 388 |
| Avoid unused local variables such as 'index'. | 651 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 218 - 220 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1094 - 1096 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 388 - 391 |
| Avoid unused local variables such as 'allDeactivated'. | 463 |
| Avoid unused local variables such as 'matchedArs'. | 524 |
| Avoid unused local variables such as 'arGroups'. | 541 |
| These nested if statements could be combined | 705 - 707 |
| These nested if statements could be combined | 819 - 823 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 25 |
| These nested if statements could be combined | 141 - 143 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 25 |
| These nested if statements could be combined | 144 - 146 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 24 |
| These nested if statements could be combined | 130 - 132 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 24 |
| These nested if statements could be combined | 139 - 141 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 201 - 206 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'actionTaken'. | 79 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 148 - 156 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.messaging.MessageServiceNames' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.impex.xml.XmlIngesterService' | 37 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 150 |
| Avoid unused local variables such as 'pkNames'. | 186 |
| Avoid empty catch blocks | 256 - 258 |
| Avoid empty if statements | 369 - 371 |
| An empty statement (semicolon) not part of a loop | 746 |
| Avoid empty catch blocks | 950 - 953 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.ResourceLoader' | 27 |
| Avoid duplicate imports such as 'org.kuali.rice.core.impl.resourceloader.RiceResourceLoaderFactory' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.config.KIMThinClientConfigurer' | 18 |
| Avoid unused imports such as 'org.kuali.rice.ksb.messaging.config.KSBThinClientConfigurer' | 19 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.BaseLifecycle' | 20 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 22 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 233 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 20 |
| Avoid unused constructor parameters such as 'ruleAttribute'. | 36 - 39 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.framework.document.lookup.SearchableAttribute' | 28 |
| Avoid empty if statements | 490 - 492 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceIllegalStateException' | 7 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiServiceLocator' | 11 |
| Avoid unused imports such as 'org.kuali.rice.kew.doctype.DocumentTypeAttribute' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kew.doctype.bo.DocumentType' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 25 |
| Avoid unused imports such as 'javax.jws.WebParam' | 27 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceIllegalArgumentException' | 3 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.document.lookup.DocumentLookupCriteria' | 6 |
| Avoid unused imports such as 'javax.jws.WebParam' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.DocumentSearchCriteriaDTO' | 11 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.document.attribute.DocumentAttribute' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 21 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'searchCriteriaDisplayed'. | 36 - 38 |
| Avoid unused constructor parameters such as 'headerBarDisplayed'. | 36 - 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'parseAndConvertDateToRange(String)'. | 246 |
| Avoid unused local variables such as 'queryTableColumnName'. | 599 |
| These nested if statements could be combined | 1698 - 1700 |
| Avoid unused private methods such as 'tokenizeCriteria(String)'. | 1799 |
| Avoid unused private methods such as 'validateNumber(List)'. | 1806 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 51 |
| Avoid unnecessary return statements | 262 |
| These nested if statements could be combined | 284 - 301 |
| These nested if statements could be combined | 285 - 300 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 333 |
| Violation | Line |
|---|---|
| Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. | 92 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'documentSecurityService'. | 67 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.collections.ListUtils' | 20 |
| These nested if statements could be combined | 293 - 299 |
| These nested if statements could be combined | 321 - 327 |
| These nested if statements could be combined | 349 - 355 |
| These nested if statements could be combined | 377 - 383 |
| Avoid unused private methods such as 'validateNetworkId(List)'. | 388 |
| Avoid unused private methods such as 'validatePersonByPrincipalName(String)'. | 408 |
| Avoid unused method parameters such as 'principalName'. | 408 - 415 |
| Avoid unused private methods such as 'getValidDocumentType(String)'. | 651 |
| Avoid unused private methods such as 'getValidDocumentTypeOld(String)'. | 678 |
| Avoid unused private methods such as 'tokenizeCriteria(String)'. | 765 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'hasXPathExpression'. | 267 |
| Avoid empty if statements | 292 - 294 |
| These nested if statements could be combined | 573 - 575 |
| Avoid unused local variables such as 'rangeDefinitionAttributes'. | 693 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 159 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.xml.sax.SAXException' | 35 |
| Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' | 38 |
| Avoid unused imports such as 'java.io.IOException' | 42 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 87 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 90 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 99 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 102 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 112 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 115 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1298 - 1300 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'docType'. | 62 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 159 - 161 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 94 - 96 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 164 - 167 |
| These nested if statements could be combined | 174 - 177 |
| These nested if statements could be combined | 218 - 222 |
| Avoid unused local variables such as 'extensions'. | 240 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'ruleTemplate'. | 630 |
| Violation | Line |
|---|---|
| This usage of the Collection.toArray() method will throw a ClassCastException. | 1361 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'nextNodeInstance'. | 348 - 356 |
| Avoid unused method parameters such as 'nodeInstance'. | 572 - 594 |
| Avoid unused method parameters such as 'nodeInstanceId'. | 608 - 629 |
| Avoid unused method parameters such as 'nodeInstanceId'. | 671 - 691 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'documentId'. | 52 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 23 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 122 - 124 |
| Overriding method merely calls super | 154 - 156 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 76 - 79 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.actionrequest.ActionRequestValue' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 28 |
| These nested if statements could be combined | 129 - 132 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 36 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 304 - 306 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 34 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 35 |
| Avoid unused imports such as 'org.kuali.rice.kew.engine.node.Branch' | 40 |
| Avoid unused local variables such as 'i'. | 119 |
| Avoid unused local variables such as 'documentType'. | 160 |
| Avoid empty if statements | 166 - 191 |
| Avoid unused method parameters such as 'helper'. | 234 - 283 |
| Avoid unused method parameters such as 'helper'. | 407 - 427 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 247 - 249 |
| Avoid empty catch blocks | 386 - 388 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 107 - 110 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceIllegalArgumentException' | 18 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'logger'. | 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.util.KEWConstants' | 38 |
| Avoid unused imports such as 'java.util.Date' | 46 |
| An empty statement (semicolon) not part of a loop | 304 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 22 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.BaseLifecycle' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.ActionRequestDTO' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.ActionRequestDTO' | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 70 - 72 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 37 - 43 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.document.attribute.DocumentAttributeIndexingQueue' | 26 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 233 - 235 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.logging.Logger' | 22 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 212 |
| Avoid unused method parameters such as 'request'. | 216 - 268 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 241 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 260 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 27 |
| Overriding method merely calls super | 81 - 84 |
| Avoid empty catch blocks | 142 - 144 |
| Avoid empty catch blocks | 156 - 158 |
| Avoid empty catch blocks | 169 - 171 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.ResourceLoader' | 22 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_COLOR'. | 72 |
| Avoid unused private fields such as 'DEFAULT_ACTION_LIST_SIZE'. | 73 |
| Avoid unused private fields such as 'DEFAULT_REFRESH_RATE'. | 74 |
| Avoid unused local variables such as 'stagingDirectory'. | 96 |
| Avoid empty if statements | 178 - 180 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 85 - 86 |
| Avoid empty catch blocks | 92 - 93 |
| Avoid empty catch blocks | 99 - 100 |
| Avoid empty catch blocks | 106 - 107 |
| Avoid empty catch blocks | 175 - 176 |
| Avoid empty catch blocks | 181 - 182 |
| Avoid empty catch blocks | 224 - 225 |
| Avoid empty catch blocks | 230 - 231 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 105 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 52 |
| Avoid unused private fields such as 'searchableAttributeValues'. | 212 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.bouncycastle.ocsp.OCSPReqGenerator' | 18 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'routeContext'. | 75 - 92 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 150 - 152 |
| Avoid empty if statements | 171 - 173 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 133 - 134 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'principal'. | 452 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.RouteNodeInstanceDTO' | 36 |
| Avoid unused local variables such as 'applicationId'. | 216 |
| These nested if statements could be combined | 296 - 300 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 54 |
| Avoid empty if statements | 121 - 123 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.ActionRequestDTO' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' | 20 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 21 |
| Avoid unused method parameters such as 'ruleTemplateName'. | 81 - 83 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' | 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'makeActionRequests(ActionRequestFactory,RouteContext,RuleBaseValues,DocumentRouteHeaderValue,ActionRequestValue,RuleDelegation)'. | 243 |
| Avoid empty catch blocks | 360 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'parseParameters(String)'. | 56 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 99 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'documentContent'. | 92 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 20 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 47 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 126 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 124 |
| These nested if statements could be combined | 225 - 227 |
| Avoid unused private methods such as 'getWorkgroupOrCriteria(Collection)'. | 392 |
| Avoid unused private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. | 415 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 38 |
| Avoid unused private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. | 148 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 131 - 133 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 869 - 872 |
| Avoid empty if statements | 899 - 901 |
| Avoid empty if statements | 949 - 951 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 77 - 80 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'isDelegate'. | 375 - 436 |
| Avoid unused method parameters such as 'request'. | 458 - 460 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 92 - 98 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.role.dto.KimPermissionInfo' | 42 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 47 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'o'. | 148 |
| An empty statement (semicolon) not part of a loop | 319 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'quickfinderService'. | 140 |
| Avoid empty if statements | 152 - 155 |
| These nested if statements could be combined | 459 - 461 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiServiceLocator' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.document.Document' | 19 |
| These nested if statements could be combined | 83 - 85 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 59 - 63 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.action.ActionRequest' | 31 |
| Avoid unused imports such as 'org.kuali.rice.kew.dto.ActionRequestDTO' | 39 |
| Avoid unused imports such as 'org.kuali.rice.kew.rule.bo.RuleAttribute' | 72 |
| Avoid unused private methods such as 'actionRequestMatches(ActionRequestValue,String,String)'. | 224 |
| Avoid unused local variables such as 'extensionDefinition'. | 279 |
| These nested if statements could be combined | 775 - 786 |
| Avoid unused local variables such as 'principalId'. | 1020 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 24 |
| Violation | Line |
|---|---|
| Avoid unnecessary temporaries when converting primitives to Strings | 59 |
| Avoid unnecessary temporaries when converting primitives to Strings | 75 |
| Avoid unnecessary temporaries when converting primitives to Strings | 95 |
| Avoid unnecessary temporaries when converting primitives to Strings | 128 |
| Avoid unnecessary temporaries when converting primitives to Strings | 153 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 104 - 106 |
| Avoid empty catch blocks | 130 - 132 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'request'. | 301 - 308 |
| These nested if statements could be combined | 506 - 509 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getNewOptionIdForActionList()'. | 43 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.sql.DataSource' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.ResultSet' | 20 |
| Avoid unused local variables such as 'result'. | 66 |
| Avoid empty catch blocks | 68 - 70 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 71 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'filterConfig'. | 68 |
| These nested if statements could be combined | 170 - 176 |
| These nested if statements could be combined | 173 - 175 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'filterConfig'. | 51 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'nodeTypeName'. | 1159 - 1312 |
| These nested if statements could be combined | 1216 - 1219 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_GROUP_DESCRIPTION'. | 57 |
| Avoid unused private fields such as 'groupAttributes'. | 61 |
| Avoid empty if statements | 186 - 188 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 321 - 323 |
| Avoid empty if statements | 391 - 393 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'checkForDuplicateRuleDelegations(List)'. | 159 |
| Avoid unused local variables such as 'delegations'. | 381 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'attributes'. | 144 - 154 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 344 - 347 |
| Avoid unused method parameters such as 'nodeType'. | 429 - 431 |
| Avoid unused method parameters such as 'nodeType'. | 436 - 438 |
| Violation | Line |
|---|---|
| Avoid unnecessary temporaries when converting primitives to Strings | 87 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 87 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'exportRuleDelegations(Element,RuleBaseValues)'. | 191 |
| Violation | Line |
|---|---|
| The null check here is misplaced; if the variable is null therell be a NullPointerException | 186 |
| These nested if statements could be combined | 206 - 212 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Date' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Date' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Date' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.TypeContract' | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 85 - 87 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.TypeContract' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.permission.Permission' | 26 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'spacesPattern'. | 103 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 93 - 100 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 307 - 316 |
| These nested if statements could be combined | 464 - 475 |
| Avoid unused local variables such as 'i'. | 718 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 160 - 164 |
| These nested if statements could be combined | 375 - 388 |
| These nested if statements could be combined | 376 - 387 |
| These nested if statements could be combined | 533 - 546 |
| These nested if statements could be combined | 534 - 545 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 608 - 610 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 90 - 92 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 86 - 88 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 47 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'namespaceCode'. | 174 - 182 |
| Avoid unused method parameters such as 'namespaceCode'. | 198 - 207 |
| Avoid unused method parameters such as 'qualification'. | 402 - 407 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 329 - 330 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.AttributeDefinition' | 36 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'roleBo'. | 214 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 64 |
| These nested if statements could be combined | 109 - 111 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.role.dto.KimPermissionInfo' | 19 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'i'. | 69 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'document'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'i'. | 70 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getPersistenceService()'. | 763 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 85 - 88 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.service.BusinessObjectService' | 45 |
| Avoid unused private fields such as 'LOG'. | 67 |
| These nested if statements could be combined | 270 - 272 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'externalIdentifierCriteria'. | 408 |
| These nested if statements could be combined | 436 - 452 |
| These nested if statements could be combined | 613 - 615 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getPrincipalImpl(String)'. | 605 |
| Avoid unused local variables such as 'empInfo'. | 614 |
| These nested if statements could be combined | 1316 - 1319 |
| Avoid empty if statements | 1732 - 1734 |
| Avoid empty if statements | 1740 - 1742 |
| Avoid empty if statements | 1747 - 1749 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 563 - 566 |
| These nested if statements could be combined | 593 - 595 |
| These nested if statements could be combined | 607 - 609 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 27 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocatorWeb' | 28 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 23 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'dataDictionaryService'. | 73 |
| Avoid unused private fields such as 'documentHelperService'. | 76 |
| These nested if statements could be combined | 213 - 237 |
| These nested if statements could be combined | 276 - 286 |
| Avoid empty catch blocks | 533 - 536 |
| Avoid unused local variables such as 'namespaceCode'. | 590 |
| Avoid unused local variables such as 'componentName'. | 591 |
| Avoid unused local variables such as 'propertyName'. | 592 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'attributeRefName'. | 347 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.MaintenanceDocumentEntry' | 19 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 272 - 274 |
| Avoid empty catch blocks | 290 - 292 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 555 - 558 |
| Avoid unused method parameters such as 'businessObject'. | 631 - 648 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' | 22 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'form'. | 74 - 81 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 531 - 542 |
| These nested if statements could be combined | 559 - 570 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'targetName'. | 37 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 178 - 180 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 205 - 207 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'ojbCollectionHelper'. | 58 |
| Avoid empty catch blocks | 83 |
| Avoid empty catch blocks | 101 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 30 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 240 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'fk'. | 89 |
| Avoid unused local variables such as 'foreignPK'. | 90 |
| Avoid empty catch blocks | 111 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Avoid unused private fields such as 'businessObjectDao'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 44 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'START_INDEX'. | 28 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'START_INDEX'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'START_INDEX'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'START_INDEX'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'entityManager'. | 56 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.document.Document' | 26 |
| These nested if statements could be combined | 248 - 250 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 207 - 209 |
| These nested if statements could be combined | 224 - 226 |
| These nested if statements could be combined | 240 - 242 |
| These nested if statements could be combined | 257 - 259 |
| These nested if statements could be combined | 274 - 277 |
| These nested if statements could be combined | 342 - 344 |
| Avoid empty if statements | 500 - 507 |
| These nested if statements could be combined | 517 - 531 |
| These nested if statements could be combined | 521 - 530 |
| These nested if statements could be combined | 554 - 572 |
| These nested if statements could be combined | 556 - 571 |
| These nested if statements could be combined | 564 - 568 |
| Avoid empty catch blocks | 582 - 586 |
| These nested if statements could be combined | 655 - 657 |
| These nested if statements could be combined | 725 - 727 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'maintenanceDocumentEntry'. | 60 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 244 - 247 |
| Avoid unused method parameters such as 'refFieldName'. | 379 - 394 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 66 |
| Avoid unused private fields such as 'workflowInfoService'. | 72 |
| These nested if statements could be combined | 253 - 255 |
| These nested if statements could be combined | 709 - 712 |
| These nested if statements could be combined | 723 - 727 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 162 - 167 |
| These nested if statements could be combined | 206 - 209 |
| These nested if statements could be combined | 227 - 230 |
| These nested if statements could be combined | 288 - 290 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 607 - 609 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| This usage of the Collection.toArray() method will throw a ClassCastException. | 175 |
| Avoid unused method parameters such as 'documentHeaderId'. | 697 - 704 |
| These nested if statements could be combined | 793 - 796 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collections' | 24 |
| Violation | Line |
|---|---|
| Empty initializer was found | 34 - 37 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.service.BusinessObjectDictionaryService' | 22 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'persistenceService'. | 47 |
| Avoid unused local variables such as 'formatter'. | 120 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 37 - 39 |
| These nested if statements could be combined | 46 - 48 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 26 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'user'. | 299 |
| Violation | Line |
|---|---|
| Empty initializer was found | 34 - 36 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 70 - 72 |
| These nested if statements could be combined | 80 - 82 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 131 - 133 |
| Avoid unused local variables such as 'refPkNames'. | 310 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 407 - 409 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 552 - 554 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 582 - 584 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'strValue'. | 181 - 186 |
| Avoid unused method parameters such as 'clazz'. | 181 - 186 |
| Avoid unused method parameters such as 'key'. | 181 - 186 |
| Avoid unused local variables such as 'caseInsensitive'. | 183 |
| Avoid unused method parameters such as 'clazz'. | 188 - 192 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 198 - 200 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' | 31 |
| Avoid unused private fields such as 'cachedObjects'. | 61 |
| Avoid unused local variables such as 'key'. | 214 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.expression.EvaluationException' | 27 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 52 - 56 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 66 - 68 |
| Avoid empty catch blocks | 108 - 110 |
| Avoid unused method parameters such as 'origSource'. | 121 - 131 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 61 - 69 |
| Unnecessary final modifier in final class | 71 - 79 |
| Unnecessary final modifier in final class | 91 - 101 |
| Unnecessary final modifier in final class | 110 - 112 |
| Unnecessary final modifier in final class | 139 - 141 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.util.jaxb.MultiValuedStringMapAdapter' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kew.docsearch.SearchableAttributeValue' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kew.rule.WorkflowAttributeValidationError' | 35 |
| Avoid unused imports such as 'org.kuali.rice.kns.web.ui.Row' | 36 |
| Avoid unused imports such as 'javax.jws.WebParam' | 39 |
| Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' | 40 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.util.jaxb.MultiValuedStringMapAdapter' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kew.rule.WorkflowAttributeValidationError' | 33 |
| Avoid unused imports such as 'javax.jws.WebParam' | 66 |
| Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' | 67 |
| Avoid unused imports such as 'java.util.Collection' | 69 |
| Avoid empty catch blocks | 199 - 201 |
| Avoid unused local variables such as 'boEntry'. | 433 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.rule.bo.RuleAttribute' | 19 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'docContent'. | 95 |
| Avoid unused local variables such as 'name'. | 117 |
| These nested if statements could be combined | 162 - 199 |
| Avoid unused private methods such as 'getLastXPath(Element)'. | 449 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 50 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.docsearch.SearchableAttributeValue' | 19 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 362 - 365 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.InvalidActionTakenException' | 28 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 145 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 65 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 114 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 138 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ksb.api.registry.ServiceRegistry' | 20 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 32 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 92 - 94 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ksb.util.KSBConstants.Config' | 23 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 77 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 25 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 25 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 271 - 273 |
| These nested if statements could be combined | 276 - 278 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 143 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'transactionManager'. | 41 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 68 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 62 - 64 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 139 |
| Violation | Line |
|---|---|
| Avoid importing anything from the package 'java.lang' | 20 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 48 - 50 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 391 - 394 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 121 - 122 |
| Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) | 214 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 118 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 57 - 59 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ksb.api.messaging.MessageHelper' | 31 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 96 - 98 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 103 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.impl.resourceloader.SpringResourceLoader' | 19 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 146 |
| Avoid empty catch blocks | 149 - 151 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 20 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 16 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 16 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 58 - 61 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 110 - 112 |
| Avoid empty catch blocks | 120 - 122 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.action.InvalidActionTakenException' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.doctype.IllegalDocumentTypeException' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 18 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DISAPPROVE'. | 134 |
| Avoid unused private fields such as 'USE_KEW_SUPERUSER_DOCHANDLER'. | 135 |
| Avoid unused private fields such as 'ALLOW_UNREQUESTED_ACTION'. | 136 |
| Avoid unused private fields such as 'DEFAULT_APPROVE'. | 137 |
| Avoid unused private fields such as 'INITIATOR_MUST_ROUTE'. | 138 |
| Avoid unused private fields such as 'INITIATOR_MUST_SAVE'. | 139 |
| Avoid unused private fields such as 'INITIATOR_MUST_CANCEL'. | 140 |
| Avoid unused private fields such as 'INITIATOR_MUST_BLANKET_APPROVE'. | 141 |
| Avoid unused private fields such as 'LOOK_FUTURE'. | 142 |
| Avoid unused private fields such as 'SEND_NOTIFICATION_ON_SU_APPROVE'. | 143 |
| Avoid unused private fields such as 'SUPPORTS_QUICK_INITIATE'. | 144 |
| Avoid unused private fields such as 'NOTIFY_ON_SAVE'. | 145 |
| Avoid unused private fields such as 'DOCUMENT_STATUS_POLICY'. | 146 |
| Avoid unused private fields such as 'ALLOW_SU_POSTPROCESSOR_OVERRIDE'. | 147 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceIllegalStateException' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.action.ActionType' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.jws.WebResult' | 23 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlElement' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.uif.RemotableAttributeField' | 3 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.util.KeyValue' | 3 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.document.DocumentContract' | 19 |
| Avoid unused imports such as 'java.util.ArrayList' | 21 |
| Avoid unused imports such as 'java.util.Map' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.util.KeyValue' | 3 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 6 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 6 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 6 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 6 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 20 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.joda.time.DateTime' | 3 |
| Avoid unused imports such as 'java.util.Date' | 7 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 20 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.uif.AttributeField' | 16 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 6 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 8 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.collections.CollectionUtils' | 19 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.GenericQueryResults' | 22 |
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.LookupCustomizer' | 23 |
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.QueryByCriteria' | 24 |
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceIllegalArgumentException' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.group.Group' | 26 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.group.GroupMember' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.group.GroupMemberQueryResults' | 28 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.group.GroupQueryResults' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.common.attribute.AttributeTransform' | 30 |
| Avoid unused imports such as 'org.kuali.rice.kim.util.KIMPropertyConstants' | 31 |
| Avoid unused imports such as 'org.kuali.rice.kim.util.KimConstants' | 32 |
| Avoid unused imports such as 'java.sql.Timestamp' | 35 |
| Avoid unused imports such as 'java.util.ArrayList' | 36 |
| Avoid unused imports such as 'java.util.Collection' | 37 |
| Avoid unused imports such as 'java.util.Collections' | 38 |
| Avoid unused imports such as 'java.util.HashMap' | 39 |
| Avoid unused imports such as 'java.util.HashSet' | 40 |
| Avoid unused imports such as 'java.util.List' | 41 |
| Avoid unused imports such as 'java.util.Map' | 42 |
| Avoid unused imports such as 'java.util.Set' | 43 |
| Avoid unused private fields such as 'criteriaLookupService'. | 49 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'CACHE_MAX_AGE_SECONDS'. | 57 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.util.KIMWebServiceConstants' | 35 |
| Avoid unused imports such as 'javax.jws.WebService' | 45 |
| These nested if statements could be combined | 219 - 221 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 183 - 186 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 370 - 372 |
| Overriding method merely calls super | 375 - 377 |
| Overriding method merely calls super | 423 - 425 |
| Overriding method merely calls super | 458 - 460 |
| Overriding method merely calls super | 463 - 465 |
| Overriding method merely calls super | 468 - 470 |
| Overriding method merely calls super | 473 - 475 |
| These nested if statements could be combined | 845 - 847 |
| These nested if statements could be combined | 857 - 859 |
| Avoid unused local variables such as 'delegateMember'. | 1237 |
| 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 | 98 - 100 |
| 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 |
|---|---|
| These nested if statements could be combined | 217 - 229 |
| Avoid empty catch blocks | 239 - 241 |
| Avoid empty catch blocks | 407 - 409 |
| Avoid empty catch blocks | 421 - 423 |
| 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 | 410 - 429 |
| These nested if statements could be combined | 483 - 485 |
| Avoid unused local variables such as 'attributeRefName'. | 540 |
| Avoid unused method parameters such as 'nestedObjectPrefix'. | 610 - 647 |
| 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)'. | 696 |
| These nested if statements could be combined | 752 - 758 |
| Avoid empty if statements | 966 - 969 |
| These nested if statements could be combined | 1018 - 1020 |
| Avoid unused local variables such as 'propertyName'. | 1157 |
| Unnecessary final modifier in final class | 1407 - 1411 |
| Unnecessary final modifier in final class | 1420 - 1422 |
| 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. | 533 |
| These nested if statements could be combined | 619 - 621 |
| Avoid unused method parameters such as 'response'. | 773 - 806 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.DocumentEntry' | 42 |
| These nested if statements could be combined | 1165 - 1181 |
| Avoid empty catch blocks | 1304 - 1305 |
| Avoid unused local variables such as 'newNote'. | 1473 |
| 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'. | 463 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'kualiInquirable'. | 299 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 120 - 122 |
| These nested if statements could be combined | 132 - 134 |
| Avoid unused method parameters such as 'request'. | 163 - 167 |
| 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 |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.MaintenanceDocumentEntry' | 34 |
| These nested if statements could be combined | 353 - 355 |
| 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 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 193 - 195 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getBusinessObjectAuthorizationService()'. | 65 |
| Avoid unused local variables such as 'row'. | 103 |
| These nested if statements could be combined | 356 - 682 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 81 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 83 |
| Avoid empty catch blocks | 86 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' | 27 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'persistenceStructureService'. | 43 |
| 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'. | 128 |
| These nested if statements could be combined | 576 - 604 |
| Avoid empty catch blocks | 653 - 655 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 144 - 146 |
| Avoid empty catch blocks | 162 - 163 |
| 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 local variables such as 'componentBeans'. | 167 |
| 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 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.core.api.config.property.ConfigurationService' | 22 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 23 |
| Avoid unused local variables such as 'key'. | 43 |
| Overriding method merely calls super | 59 - 61 |
| 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.core.api.config.property.ConfigurationService' | 22 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 23 |
| Avoid unused local variables such as 'key'. | 61 |
| 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 | 120 - 125 |
| 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 imports such as 'org.kuali.rice.krad.uif.container.Group' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.container.LookupView' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.container.View' | 20 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'pkNames'. | 149 |
| Avoid unused local variables such as 'user'. | 150 |
| Avoid unused local variables such as 'searchUsingOnlyPrimaryKeyValues'. | 167 |
| Avoid unused local variables such as 'searchPropertyValue'. | 399 |
| Avoid unused local variables such as 'dataObjectExample'. | 492 |
| These nested if statements could be combined | 516 - 526 |
| Avoid unused local variables such as 'lookupView'. | 667 |
| 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 | 105 - 108 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 240 - 242 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 116 - 122 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 202 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 276 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.field.AttributeField' | 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'multipleValues'. | 70 |
| Avoid unused private fields such as 'lookupCriteriaEnabled'. | 71 |
| Avoid unused private fields such as 'supplementalActionsEnabled'. | 72 |
| Avoid unused private fields such as 'disableSearchButtons'. | 73 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.service.DocumentDictionaryService' | 15 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.service.DataObjectMetaDataService' | 14 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocatorWeb' | 15 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.core.BindingInfo' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.util.ViewModelUtils' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.group.Group' | 22 |
| These nested if statements could be combined | 70 - 72 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 42 - 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.util.ComponentUtils' | 23 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'customValidatorClass'. | 81 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 305 - 310 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 67 - 69 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 97 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 103 - 105 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 108 - 111 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 183 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 45 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 87 - 90 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 236 - 238 |
| 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 unused imports such as 'org.codehaus.groovy.tools.javac.JavaAwareCompilationUnit' | 13 |
| Avoid unused imports such as 'org.hibernate.dialect.Oracle10gDialect' | 14 |
| Avoid unused imports such as 'java.util.Arrays' | 20 |
| Avoid empty if statements | 78 - 80 |
| Avoid empty if statements | 81 - 83 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 98 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 61 - 63 |
| 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 | 277 - 314 |
| These nested if statements could be combined | 310 - 312 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.util.ConcreteKeyValue' | 21 |
| Avoid unused imports such as 'org.kuali.rice.core.api.util.KeyValue' | 22 |
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.AttributeDefinition' | 24 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.control.CheckboxControl' | 26 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.control.Control' | 27 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.control.RadioGroupControl' | 28 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.control.TextAreaControl' | 29 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.field.AttributeField' | 30 |
| Avoid unused imports such as 'java.util.ArrayList' | 36 |
| Avoid unused imports such as 'java.util.List' | 37 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.beans.factory.support.ManagedArray' | 33 |
| Avoid empty if statements | 137 - 139 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 159 - 161 |
| Avoid empty catch blocks | 161 - 163 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 18 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 13 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 15 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 59 - 75 |
| Unnecessary final modifier in final class | 80 - 92 |
| Unnecessary final modifier in final class | 104 - 110 |
| Unnecessary final modifier in final class | 464 - 466 |
| 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 |
|---|---|
| These nested if statements could be combined | 76 - 78 |
| Avoid empty catch blocks | 153 - 155 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'attachmentTypeCode'. | 288 |
| Avoid unused local variables such as 'attachmentType'. | 322 |
| Avoid unused local variables such as 'tmpNote'. | 339 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 42 - 45 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Enumeration' | 52 |
| Avoid unused private fields such as 'sessionDocumentService'. | 83 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 44 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifParameters' | 22 |
| These nested if statements could be combined | 96 - 99 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 163 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.ActionRequestDTO' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.lang.reflect.InvocationTargetException' | 18 |
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' | 26 |
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.ModelObjectComplete' | 28 |
| Avoid unused private fields such as 'COULDNT_INVOKE_BUILDER_CREATE'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 289 - 291 |
| These nested if statements could be combined | 335 - 337 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'rule'. | 203 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 5 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.RepositoryConstants' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.RepositoryConstants.Namespaces' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.RepositoryConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.ResultEvent' | 19 |
| Avoid unused constructor parameters such as 'termResolutionEngine'. | 31 - 51 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 6 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'termPropositionMap'. | 16 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'parent'. | 176 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' | 20 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 33 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 39 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 45 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.type.KrmsTypeDefinition' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.type.KrmsTypeDefinition' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.Term' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.term.TermDefinition' | 25 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'typeRepositoryService'. | 66 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionEngine' | 27 |
| Avoid unused local variables such as 'parameterDefinition'. | 108 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreApiServiceLocator' | 11 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'firstItem'. | 133 |
| An empty statement (semicolon) not part of a loop | 311 |
| Avoid unused private methods such as 'treeToInOrderList(AgendaItemBo,List)'. | 898 |
| Avoid empty if statements | 947 |
| An empty statement (semicolon) not part of a loop | 947 |
| Avoid unused local variables such as 'firstItem'. | 973 |
| Avoid empty if statements | 998 - 1000 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.bo.BusinessObject' | 20 |
| Avoid unused imports such as 'org.kuali.rice.krad.bo.PersistableBusinessObject' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krad.util.KRADConstants' | 27 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.AgendaBo' | 28 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.ContextBo' | 29 |
| Overriding method merely calls super | 86 - 89 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collections' | 18 |
| Avoid unused imports such as 'java.util.List' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.BusinessObjectService' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 22 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.proposition.PropositionParameterType' | 23 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.proposition.PropositionType' | 24 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.AgendaBo' | 25 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.PropositionParameterBo' | 27 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.TermBo' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.util.KimConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 56 - 58 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'firstItem'. | 135 |
| An empty statement (semicolon) not part of a loop | 294 |
| Avoid unused method parameters such as 'request'. | 811 - 816 |
| Avoid unused private methods such as 'treeToInOrderList(AgendaItemBo,List)'. | 818 |
| Avoid empty if statements | 867 |
| An empty statement (semicolon) not part of a loop | 867 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collections' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.AgendaBo' | 25 |
| Avoid empty if statements | 53 - 55 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.KrmsAttributeDefinitionBo' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.config.ConfigurationException' | 23 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 57 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 72 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 74 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Avoid unused imports such as 'org.kuali.rice.ksb.api.KsbApiConstants' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 5 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Avoid unused imports such as 'org.kuali.rice.ksb.api.KsbApiConstants' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 6 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 36 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 36 |
| Avoid unused private fields such as 'vendorNumber'. | 40 |
| Avoid unused private fields such as 'vendorSoldToNumber'. | 63 |
| Avoid empty catch blocks | 176 - 178 |
| Avoid empty catch blocks | 251 - 253 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 109 - 111 |
| Avoid unused local variables such as 'message'. | 129 |
| Avoid empty if statements | 181 - 183 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 101 - 103 |
| Avoid empty catch blocks | 127 - 129 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 212 - 214 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 92 - 94 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'rows'. | 33 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' | 22 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' | 22 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 24 |