The following document contains the results of PMD 4.2.5.
| Violation | Line |
|---|---|
| Avoid unused private fields such as '_futureElements'. | 56 |
| 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' | 18 |
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 20 |
| 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 private fields such as 'LOG'. | 31 |
| 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 'org.jacorb.idl.Member' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants.KimUIConstants' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.RoleMemberContract' | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collections' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants.KimUIConstants' | 23 |
| 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 private fields such as 'LAST_ACCCESS_TIMEOUT_FIELD'. | 28 |
| 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 |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 19 |
| 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' | 18 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| 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 |
|---|---|
| Avoid empty catch blocks | 132 - 134 |
| Avoid empty catch blocks | 134 - 136 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Timestamp' | 20 |
| 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 |
|---|---|
| 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 |
|---|---|
| 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 |
|---|---|
| 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 |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 21 |
| 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 |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlEnum' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.ModelObjectComplete' | 20 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'b'. | 51 - 53 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlElement' | 24 |
| Avoid unused constructor parameters such as 'b'. | 51 - 52 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'b'. | 51 - 53 |
| 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 |
|---|---|
| Avoid empty catch blocks | 54 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 121 - 127 |
| Violation | Line |
|---|---|
| Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! | 88 |
| Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! | 105 |
| 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' | 18 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| 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 |
|---|---|
| Avoid empty catch blocks | 131 - 132 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 59 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 84 - 86 |
| 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. | 51 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 78 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| 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'. | 339 - 350 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'otherClass'. | 32 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 138 |
| 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 | 123 - 125 |
| 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'. | 42 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 112 - 114 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 18 |
| Avoid unused imports such as 'org.apache.log4j.Logger' | 23 |
| Avoid unused imports such as 'org.kuali.rice.core.api.config.property.ConfigContext' | 24 |
| Avoid unused imports such as 'org.kuali.rice.core.api.config.property.Config' | 25 |
| Avoid unused imports such as 'java.util.Collections' | 27 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 28 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'components'. | 78 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 92 - 94 |
| Avoid empty if statements | 95 - 98 |
| Avoid empty if statements | 99 - 101 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 143 - 151 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 128 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 218 - 220 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 16 - 95 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'successful'. | 183 - 202 |
| 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 imports such as 'org.kuali.rice.core.framework.parameter.ParameterService' | 20 |
| Avoid unused local variables such as 'detailTypeRegex'. | 62 |
| Avoid unused local variables such as 'namespaceRegex'. | 63 |
| Avoid unused local variables such as 'nameRegex'. | 64 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.framework.parameter.ParameterService' | 21 |
| Avoid unused imports such as 'org.kuali.rice.core.impl.component.ComponentBo' | 22 |
| Avoid unused imports such as 'java.util.Collection' | 32 |
| Avoid unused private fields such as 'LOG'. | 48 |
| Avoid unused private fields such as 'NAMESPACE_CODE'. | 51 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.impl.component.ComponentBo' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.DataDictionaryException' | 27 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocatorWeb' | 28 |
| Avoid unused imports such as 'org.kuali.rice.krad.util.ObjectUtils' | 31 |
| Avoid unused imports such as 'java.util.List' | 35 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'toReturn'. | 42 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 177 - 179 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 204 - 206 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'com.sun.codemodel.JInvocation' | 30 |
| Avoid unused imports such as 'org.apache.commons.lang.builder.EqualsBuilder' | 36 |
| Avoid unused imports such as 'org.apache.commons.lang.builder.HashCodeBuilder' | 37 |
| Avoid unused imports such as 'org.apache.commons.lang.builder.ToStringBuilder' | 38 |
| Avoid unused private fields such as 'fieldName'. | 373 |
| Avoid unused private fields such as 'fieldType'. | 374 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'sourceFilePath'. | 106 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'length'. | 73 |
| Avoid empty while statements | 76 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 177 - 179 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 107 - 110 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 106 - 124 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 188 |
| Avoid empty if statements | 192 - 194 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'currentPage'. | 56 - 64 |
| 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'. | 40 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 81 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'requestParser'. | 93 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.config.module.RunMode' | 19 |
| Avoid unused imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 21 |
| Avoid unused imports such as 'org.kuali.rice.core.framework.persistence.jpa.OrmUtils' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kew.lifecycle.EmbeddedLifeCycle' | 27 |
| Avoid unused imports such as 'java.util.Collections' | 36 |
| Avoid unused imports such as 'java.util.LinkedList' | 37 |
| Avoid unused private methods such as 'getClientProtocol()'. | 131 |
| 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' | 24 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 105 - 107 |
| 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'. | 47 |
| Avoid unused local variables such as 'error'. | 140 |
| 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'. | 52 |
| Avoid unused private fields such as 'NOTIFICATION_MESSAGE_DELIVERER_REGISTRY_SERVICE'. | 57 |
| Avoid unused private fields such as 'NOTIFICATION_EMAIL_SERVICE'. | 60 |
| Avoid unused private fields such as 'NOTIFICATION_CONFIG'. | 61 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 37 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 37 |
| Avoid unused local variables such as 'lockedDeliveries'. | 100 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 111 - 114 |
| 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'. | 36 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 37 |
| Avoid unused private fields such as 'businessObjectDao'. | 39 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 60 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 132 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'content'. | 365 - 381 |
| Avoid unused method parameters such as 'notification'. | 365 - 381 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'firstMessageDelivery'. | 70 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'userPreferenceService'. | 57 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notificationWorkflowDocumentService'. | 54 |
| These nested if statements could be combined | 145 - 149 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notificationChannelService'. | 34 |
| 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 | 267 - 345 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 294 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'response'. | 301 - 334 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'response'. | 297 - 354 |
| 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 | 185 - 188 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'toDocumentRouteHeaderValueActionListExtension(DocumentRouteHeaderValue)'. | 134 - 135 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 579 |
| Avoid empty catch blocks | 584 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.framework.document.security.DocumentSecurityDirective' | 66 |
| Avoid unused imports such as 'org.kuali.rice.kew.framework.document.security.DocumentSecurityHandlerService' | 67 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 389 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 398 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 401 |
| Avoid unused local variables such as 'index'. | 664 |
| Avoid unused local variables such as 'partitions'. | 863 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 191 - 193 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1097 - 1099 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 395 - 398 |
| These nested if statements could be combined | 409 - 412 |
| Avoid unused local variables such as 'allDeactivated'. | 484 |
| Avoid unused local variables such as 'matchedArs'. | 545 |
| Avoid unused local variables such as 'arGroups'. | 562 |
| These nested if statements could be combined | 728 - 730 |
| These nested if statements could be combined | 842 - 846 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.exception.ResourceUnavailableException' | 24 |
| These nested if statements could be combined | 140 - 142 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 142 - 144 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.exception.ResourceUnavailableException' | 23 |
| These nested if statements could be combined | 129 - 131 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 137 - 139 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 201 - 206 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'actionTaken'. | 78 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 147 - 155 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocatorWeb' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.impex.xml.XmlIngesterService' | 36 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.ResourceLoader' | 26 |
| Avoid duplicate imports such as 'org.kuali.rice.core.impl.resourceloader.RiceResourceLoaderFactory' | 28 |
| 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' | 19 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 21 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 232 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 132 - 134 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'orderedResultSetFieldMap'. | 61 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 100 - 102 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.document.attribute.DocumentAttribute' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 20 |
| Violation | Line |
|---|---|
| Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. | 80 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getOptionCriteriaField(UserOptions,String)'. | 135 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 46 |
| Avoid unused local variables such as 'hasXPathExpression'. | 270 |
| Avoid empty if statements | 295 - 297 |
| These nested if statements could be combined | 576 - 578 |
| Avoid unused local variables such as 'rangeDefinitionAttributes'. | 700 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 158 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.xml.sax.SAXException' | 33 |
| Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' | 36 |
| Avoid unused imports such as 'java.io.IOException' | 40 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 86 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 89 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 98 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 101 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 111 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 114 |
| Avoid unused local variables such as 'attributeOrClassName'. | 247 |
| Avoid unused local variables such as 'applicationId'. | 248 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1239 - 1241 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'docType'. | 61 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 255 - 258 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 91 - 93 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 157 - 160 |
| These nested if statements could be combined | 167 - 170 |
| These nested if statements could be combined | 204 - 208 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'ruleTemplate'. | 633 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.framework.postprocessor.ActionTakenEvent' | 41 |
| Avoid unused imports such as 'org.kuali.rice.kew.framework.postprocessor.AfterProcessEvent' | 42 |
| Avoid unused imports such as 'org.kuali.rice.kew.framework.postprocessor.BeforeProcessEvent' | 43 |
| Avoid unused imports such as 'org.kuali.rice.kew.framework.postprocessor.DeleteEvent' | 44 |
| Avoid unused imports such as 'org.kuali.rice.kew.framework.postprocessor.DocumentLockingEvent' | 45 |
| Avoid unused imports such as 'org.kuali.rice.kew.framework.postprocessor.DocumentRouteLevelChange' | 46 |
| Avoid unused imports such as 'org.kuali.rice.kew.framework.postprocessor.DocumentRouteStatusChange' | 47 |
| Avoid unused private fields such as 'LOG'. | 80 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 25 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'nextNodeInstance'. | 347 - 355 |
| Avoid unused method parameters such as 'nodeInstance'. | 571 - 593 |
| Avoid unused method parameters such as 'nodeInstanceId'. | 607 - 628 |
| Avoid unused method parameters such as 'nodeInstanceId'. | 670 - 690 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'documentId'. | 51 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 123 - 125 |
| Overriding method merely calls super | 155 - 157 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 75 - 78 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'isParallel'. | 132 |
| These nested if statements could be combined | 147 - 149 |
| 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 |
|---|---|
| These nested if statements could be combined | 168 - 171 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 303 - 305 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'i'. | 114 |
| Avoid unused local variables such as 'documentType'. | 155 |
| Avoid empty if statements | 161 - 186 |
| Avoid unused method parameters such as 'helper'. | 229 - 278 |
| Avoid unused method parameters such as 'helper'. | 402 - 422 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 247 - 249 |
| Avoid empty catch blocks | 386 - 388 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 106 - 109 |
| 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 'java.util.HashSet' | 18 |
| Avoid unused imports such as 'java.util.Set' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kew.actions.MoveDocumentAction' | 24 |
| Avoid unused imports such as 'javax.jws.WebParam' | 36 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1019 - 1030 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.doctype.RouteNode' | 27 |
| Avoid unused imports such as 'java.util.HashMap' | 48 |
| Avoid unused imports such as 'java.util.LinkedHashSet' | 49 |
| Avoid unused imports such as 'java.util.Set' | 52 |
| An empty statement (semicolon) not part of a loop | 306 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 47 |
| Avoid unused private fields such as 'DOCUMENT_ATTRIBUTE_PROPERTY_NAME_PREFIX'. | 83 |
| These nested if statements could be combined | 177 - 179 |
| These nested if statements could be combined | 299 - 301 |
| Avoid empty catch blocks | 403 - 406 |
| Avoid unused local variables such as 'superUserSearch'. | 728 |
| Avoid unused method parameters such as 'column'. | 868 - 871 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DOCUMENT_STATUSES'. | 50 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DATABASE_WILDCARD_CHARACTER'. | 79 |
| Avoid unused private fields such as 'messageMap'. | 82 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'bo'. | 188 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.util.jaxb.DateTimeAdapter' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kew.doctype.bo.DocumentType' | 40 |
| Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' | 50 |
| Avoid unused imports such as 'java.sql.Timestamp' | 51 |
| Avoid unused imports such as 'java.util.Calendar' | 53 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.criteria.PredicateFactory.and' | 61 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 42 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 21 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.BaseLifecycle' | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 67 - 69 |
| Avoid unused local variables such as 'principalId'. | 89 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getQName(String,DocumentRouteHeaderValue)'. | 48 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 232 - 234 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.logging.Logger' | 21 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 211 |
| Avoid unused method parameters such as 'request'. | 215 - 267 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 240 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 259 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 78 - 81 |
| Avoid empty catch blocks | 139 - 141 |
| Avoid empty catch blocks | 153 - 155 |
| Avoid empty catch blocks | 166 - 168 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.ResourceLoader' | 21 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 124 - 126 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 84 - 85 |
| Avoid empty catch blocks | 91 - 92 |
| Avoid empty catch blocks | 98 - 99 |
| Avoid empty catch blocks | 105 - 106 |
| Avoid empty catch blocks | 174 - 175 |
| Avoid empty catch blocks | 180 - 181 |
| Avoid empty catch blocks | 223 - 224 |
| Avoid empty catch blocks | 229 - 230 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 104 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.responsibility.ResponsibilityAction' | 19 |
| Avoid unused imports such as 'java.util.List' | 22 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'extensionDefinition'. | 126 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.exception.ResourceUnavailableException' | 52 |
| Avoid unused private fields such as 'searchableAttributeValues'. | 209 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.bouncycastle.ocsp.OCSPReqGenerator' | 18 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'routeContext'. | 74 - 91 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 149 - 151 |
| Avoid empty if statements | 170 - 172 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 132 - 133 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'principal'. | 451 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'applicationId'. | 213 |
| These nested if statements could be combined | 293 - 297 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 53 |
| Avoid empty if statements | 120 - 122 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'ruleTemplateName'. | 87 - 89 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'makeActionRequests(ActionRequestFactory,RouteContext,org.kuali.rice.kew.api.rule.Rule,DocumentRouteHeaderValue,ActionRequestValue,RuleDelegation)'. | 241 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.exception.WorkflowException' | 23 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'ruleAttribute'. | 192 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.kew.api.rule.RuleExpression' | 22 |
| Ensure you override both equals() and hashCode() | 101 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'documentContent'. | 95 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'curExtId'. | 242 |
| Avoid unused private methods such as 'getRuleTemplateService()'. | 335 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'errors'. | 138 |
| Avoid unused local variables such as 'curExtId'. | 230 |
| Avoid unused private methods such as 'getRuleTemplateService()'. | 326 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 46 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 125 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 125 |
| These nested if statements could be combined | 230 - 232 |
| Avoid unused private methods such as 'getWorkgroupOrCriteria(Collection)'. | 397 |
| Avoid unused private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. | 420 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. | 146 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 118 - 120 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 604 - 607 |
| Avoid empty if statements | 635 - 637 |
| Avoid empty if statements | 685 - 687 |
| These nested if statements could be combined | 1144 - 1150 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 76 - 79 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 45 |
| Avoid unused method parameters such as 'isDelegate'. | 369 - 428 |
| Avoid unused method parameters such as 'request'. | 450 - 452 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 91 - 97 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 55 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'o'. | 147 |
| An empty statement (semicolon) not part of a loop | 318 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'quickfinderService'. | 140 |
| Avoid empty if statements | 152 - 155 |
| These nested if statements could be combined | 460 - 462 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 83 - 86 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 59 - 63 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 21 |
| Violation | Line |
|---|---|
| Avoid unnecessary temporaries when converting primitives to Strings | 58 |
| Avoid unnecessary temporaries when converting primitives to Strings | 74 |
| Avoid unnecessary temporaries when converting primitives to Strings | 94 |
| Avoid unnecessary temporaries when converting primitives to Strings | 127 |
| Avoid unnecessary temporaries when converting primitives to Strings | 152 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 103 - 105 |
| Avoid empty catch blocks | 129 - 131 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 50 |
| Avoid unused method parameters such as 'request'. | 304 - 311 |
| These nested if statements could be combined | 509 - 512 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getNewOptionIdForActionList()'. | 42 |
| 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 unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 25 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocatorWeb' | 26 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 40 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 37 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'filterConfig'. | 67 |
| These nested if statements could be combined | 169 - 175 |
| These nested if statements could be combined | 172 - 174 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 34 |
| Avoid unused private fields such as 'filterConfig'. | 52 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'nodeTypeName'. | 1162 - 1335 |
| These nested if statements could be combined | 1219 - 1222 |
| 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 | 324 - 326 |
| Avoid empty if statements | 394 - 396 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 48 |
| Avoid unused private methods such as 'checkForDuplicateRuleDelegations(List)'. | 158 |
| Avoid unused local variables such as 'delegations'. | 378 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'attributes'. | 144 - 154 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 343 - 346 |
| Avoid unused method parameters such as 'nodeType'. | 428 - 430 |
| Avoid unused method parameters such as 'nodeType'. | 435 - 437 |
| 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)'. | 190 |
| 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.CodedAttributeContract' | 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.CodedAttributeContract' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.permission.Permission' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.responsibility.ReviewResponsibilityBo' | 24 |
| Avoid unused imports such as 'java.util.HashMap' | 29 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.impl.permission.PermissionBo' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.permission.PermissionTemplateBo' | 26 |
| Avoid unused local variables such as 'spacesPattern'. | 105 |
| 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 | 376 - 389 |
| These nested if statements could be combined | 377 - 388 |
| These nested if statements could be combined | 534 - 547 |
| These nested if statements could be combined | 535 - 546 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 86 - 88 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 45 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 91 - 93 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.principal.Principal' | 35 |
| Avoid empty if statements | 322 - 323 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 64 |
| These nested if statements could be combined | 109 - 111 |
| 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 imports such as 'org.kuali.rice.kim.framework.identity.email.EntityEmailTypeContractEbo' | 43 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.group.GroupBo' | 52 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.role.RoleBo' | 53 |
| These nested if statements could be combined | 163 - 167 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getPrincipalImpl(String)'. | 602 |
| Avoid unused local variables such as 'empInfo'. | 611 |
| These nested if statements could be combined | 1314 - 1317 |
| Avoid empty if statements | 1730 - 1732 |
| Avoid empty if statements | 1738 - 1740 |
| Avoid empty if statements | 1745 - 1747 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 30 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 566 - 569 |
| These nested if statements could be combined | 596 - 598 |
| These nested if statements could be combined | 610 - 612 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 21 |
| 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'. | 74 |
| Avoid unused private fields such as 'documentHelperService'. | 77 |
| These nested if statements could be combined | 214 - 238 |
| These nested if statements could be combined | 277 - 287 |
| Avoid empty catch blocks | 534 - 537 |
| Avoid unused local variables such as 'namespaceCode'. | 591 |
| Avoid unused local variables such as 'componentName'. | 592 |
| Avoid unused local variables such as 'propertyName'. | 593 |
| 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 method parameters such as 'form'. | 74 - 81 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 534 - 545 |
| These nested if statements could be combined | 562 - 573 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'targetName'. | 36 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ksb.api.KsbApiServiceLocator' | 26 |
| 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'. | 43 |
| 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' | 25 |
| These nested if statements could be combined | 245 - 247 |
| 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 | 353 - 355 |
| Avoid empty if statements | 511 - 518 |
| These nested if statements could be combined | 528 - 542 |
| These nested if statements could be combined | 532 - 541 |
| These nested if statements could be combined | 565 - 583 |
| These nested if statements could be combined | 567 - 582 |
| These nested if statements could be combined | 575 - 579 |
| Avoid empty catch blocks | 593 - 597 |
| These nested if statements could be combined | 666 - 668 |
| These nested if statements could be combined | 736 - 738 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'maintenanceDocumentEntry'. | 60 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 243 - 246 |
| Avoid unused method parameters such as 'refFieldName'. | 378 - 393 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceIllegalArgumentException' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 32 |
| Avoid unused imports such as 'org.kuali.rice.kns.kim.type.DataDictionaryTypeServiceHelper' | 33 |
| Avoid unused imports such as 'org.kuali.rice.kns.lookup.LookupUtils' | 34 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.FieldUtils' | 35 |
| Avoid unused imports such as 'org.kuali.rice.kns.web.ui.Field' | 36 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants' | 46 |
| Avoid unused private fields such as 'LOG'. | 67 |
| 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 | 170 - 175 |
| These nested if statements could be combined | 214 - 217 |
| These nested if statements could be combined | 235 - 238 |
| These nested if statements could be combined | 296 - 298 |
| 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. | 174 |
| Avoid unused method parameters such as 'documentHeaderId'. | 696 - 703 |
| These nested if statements could be combined | 792 - 795 |
| 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'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 26 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'user'. | 299 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'classModifiers'. | 330 |
| 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 | 551 - 553 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 581 - 583 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'strValue'. | 196 - 201 |
| Avoid unused method parameters such as 'clazz'. | 196 - 201 |
| Avoid unused method parameters such as 'key'. | 196 - 201 |
| Avoid unused local variables such as 'caseInsensitive'. | 198 |
| Avoid unused method parameters such as 'clazz'. | 203 - 207 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'key'. | 170 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.DataDictionaryException' | 25 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.view.InquiryView' | 31 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.view.MaintenanceView' | 33 |
| 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 |
|---|---|
| Avoid unused imports such as 'java.util.Properties' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krad.util.KRADConstants' | 35 |
| Avoid unused imports such as 'org.kuali.rice.krad.util.KRADPropertyConstants' | 36 |
| Avoid unused imports such as 'org.kuali.rice.krad.util.UrlFactory' | 37 |
| 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 | 121 - 123 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 199 - 201 |
| Avoid unused local variables such as 'boEntry'. | 434 |
| 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 | 363 - 366 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.exception.InvalidActionTakenException' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.codec.binary.Base64' | 18 |
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' | 20 |
| Avoid unused imports such as 'java.io.ByteArrayOutputStream' | 32 |
| Avoid unused imports such as 'java.io.IOException' | 33 |
| Avoid unused imports such as 'java.io.ObjectOutput' | 34 |
| Avoid unused imports such as 'java.io.ObjectOutputStream' | 35 |
| Avoid unused imports such as 'java.io.UnsupportedEncodingException' | 36 |
| Avoid unused imports such as 'java.security.GeneralSecurityException' | 37 |
| Avoid unused imports such as 'java.security.MessageDigest' | 38 |
| 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. | 76 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ksb.api.KsbApiServiceLocator' | 19 |
| 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 | 270 - 272 |
| These nested if statements could be combined | 275 - 277 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 142 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'transactionManager'. | 40 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 67 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 61 - 63 |
| 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 unused imports such as 'org.springframework.web.context.WebApplicationContext' | 46 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 47 - 49 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.print.attribute.AttributeSet' | 22 |
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreApiServiceLocator' | 33 |
| Avoid unused imports such as 'org.kuali.rice.core.impl.services.CoreImplServiceLocator' | 37 |
| Avoid unused imports such as 'org.kuali.rice.kew.util.Utilities' | 38 |
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 41 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 42 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 391 - 394 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 120 - 121 |
| Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) | 213 |
| 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 |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.group.Group' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.Person' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kim.framework.group.GroupEbo' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kim.framework.role.RoleEbo' | 24 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.role.RoleBo' | 25 |
| These nested if statements could be combined | 102 - 108 |
| Avoid unused private methods such as 'toQuery(Map)'. | 263 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 96 - 98 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.dao.DataAccessException' | 25 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 97 |
| 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' | 19 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 19 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 57 - 60 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 109 - 111 |
| Avoid empty catch blocks | 119 - 121 |
| 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 'org.apache.commons.lang.StringUtils' | 27 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.rule.Rule' | 42 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.rule.RuleReportCriteria' | 43 |
| 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.apache.commons.lang.StringUtils' | 28 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.repository.type.KewAttributeDefinitionContract' | 31 |
| 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 private fields such as 'DISAPPROVE'. | 137 |
| Avoid unused private fields such as 'USE_KEW_SUPERUSER_DOCHANDLER'. | 138 |
| Avoid unused private fields such as 'ALLOW_UNREQUESTED_ACTION'. | 139 |
| Avoid unused private fields such as 'DEFAULT_APPROVE'. | 140 |
| Avoid unused private fields such as 'INITIATOR_MUST_ROUTE'. | 141 |
| Avoid unused private fields such as 'INITIATOR_MUST_SAVE'. | 142 |
| Avoid unused private fields such as 'INITIATOR_MUST_CANCEL'. | 143 |
| Avoid unused private fields such as 'INITIATOR_MUST_BLANKET_APPROVE'. | 144 |
| Avoid unused private fields such as 'LOOK_FUTURE'. | 145 |
| Avoid unused private fields such as 'SEND_NOTIFICATION_ON_SU_APPROVE'. | 146 |
| Avoid unused private fields such as 'SUPPORTS_QUICK_INITIATE'. | 147 |
| Avoid unused private fields such as 'NOTIFY_ON_SAVE'. | 148 |
| Avoid unused private fields such as 'DOCUMENT_STATUS_POLICY'. | 149 |
| Avoid unused private fields such as 'ALLOW_SU_POSTPROCESSOR_OVERRIDE'. | 150 |
| Avoid unused private fields such as 'FAIL_ON_INACTIVE_GROUP'. | 151 |
| Avoid unused private fields such as 'REGENERATE_ACTION_REQUESTS_ON_CHANGE'. | 152 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.rule.Rule' | 27 |
| 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.Set' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collections' | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceIllegalStateException' | 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 'java.util.Collections' | 42 |
| 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.springframework.cache.annotation.Cacheable' | 26 |
| Avoid duplicate imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 28 |
| No need to import a type that lives in the same package | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 22 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 161 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.util.KeyValue' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.lang.reflect.Type' | 19 |
| Avoid unused imports such as 'org.apache.commons.lang.math.NumberUtils' | 34 |
| Ensure you override both equals() and hashCode() | 418 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.common.GloballyUnique' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.collections.CollectionUtils' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.rule.RuleContract' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.rule.RuleDelegationContract' | 24 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.rule.RuleResponsibility' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kew.api.rule.RuleResponsibilityContract' | 26 |
| Avoid unused imports such as 'java.util.ArrayList' | 37 |
| Avoid unused imports such as 'java.util.List' | 41 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 18 |
| 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.Map' | 21 |
| 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 |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 19 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.web.form.MaintenanceForm' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.keyvalues.KeyValuesBase' | 20 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.view.View' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| 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.identity.name.StringToKimEntityNameInfoMapAdapter' | 35 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.principal.EntityNamePrincipalName' | 39 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.principal.StringToKimEntityNamePrincipalInfoMapAdapter' | 41 |
| Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' | 54 |
| Avoid unused imports such as 'java.util.List' | 55 |
| Avoid unused imports such as 'java.util.Map' | 56 |
| 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 'org.kuali.rice.kim.api.KimApiConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 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 'java.text.SimpleDateFormat' | 19 |
| Avoid unused imports such as 'java.util.Date' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlSchemaType' | 27 |
| Avoid duplicate imports such as 'javax.xml.bind.annotation.XmlAccessType' | 41 |
| Avoid duplicate imports such as 'javax.xml.bind.annotation.XmlAccessorType' | 42 |
| Avoid duplicate imports such as 'javax.xml.bind.annotation.XmlAnyElement' | 43 |
| Avoid duplicate imports such as 'javax.xml.bind.annotation.XmlElement' | 44 |
| Avoid duplicate imports such as 'javax.xml.bind.annotation.XmlRootElement' | 45 |
| Avoid duplicate imports such as 'javax.xml.bind.annotation.XmlType' | 46 |
| Avoid duplicate imports such as 'java.io.Serializable' | 47 |
| Avoid duplicate imports such as 'java.util.Collection' | 48 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Date' | 26 |
| 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' | 18 |
| Avoid unused imports such as 'java.util.Date' | 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' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 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' | 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.kuali.rice.kim.api.responsibility.Responsibility' | 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.core.api.uif.AttributeField' | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimApiConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.impl.role.RoleBo' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.role.RoleDao' | 28 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.services.KimImplServiceLocator' | 29 |
| Avoid unused imports such as 'org.kuali.rice.krad.util.KRADConstants' | 31 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.criteria.PredicateFactory.and' | 42 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.criteria.PredicateFactory.lessThan' | 43 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'criteriaLookupService'. | 23 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 551 - 553 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.principal.EntityNamePrincipalName' | 37 |
| Avoid unused imports such as 'java.util.List' | 44 |
| Avoid unused imports such as 'java.util.Map' | 45 |
| Violation | Line |
|---|---|
| The null check here is misplaced; if the variable is null therell be a NullPointerException | 178 |
| These nested if statements could be combined | 198 - 204 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.Predicate' | 19 |
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.PredicateUtils' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.QueryByCriteria' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.group.Group' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.group.GroupQueryResults' | 24 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.group.GroupService' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.group.GroupBo' | 29 |
| Avoid unused imports such as 'java.sql.Timestamp' | 33 |
| Avoid unused imports such as 'java.util.ArrayList' | 34 |
| Avoid unused imports such as 'java.util.Calendar' | 35 |
| Avoid unused imports such as 'java.util.HashMap' | 36 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'externalIdentifierCriteria'. | 360 |
| These nested if statements could be combined | 388 - 404 |
| These nested if statements could be combined | 565 - 567 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 36 |
| These nested if statements could be combined | 202 - 204 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 37 |
| Avoid unused method parameters such as 'namespaceCode'. | 203 - 213 |
| Avoid unused method parameters such as 'namespaceCode'. | 243 - 252 |
| Avoid unused method parameters such as 'qualification'. | 438 - 443 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.responsibility.Responsibility' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.common.attribute.KimAttributeDataBo' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kim.impl.permission.PermissionTemplateBo' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.RoleMember' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.RoleMembership' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 38 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.RoleMember' | 39 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.RoleMembership' | 40 |
| Avoid unused private methods such as 'getPrincipalIdsForPrincipalName(String)'. | 405 |
| Avoid unused private methods such as 'hasCoreRoleMemberCriteria(Map)'. | 689 |
| Avoid unused private methods such as 'hasExtraRoleMemberCriteria(Map)'. | 698 |
| Avoid unused private methods such as 'getRoleMembersRoles(String,String)'. | 704 |
| Avoid unused private methods such as 'getRoleMembersGroupIds(String,String)'. | 712 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 106 - 109 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1055 - 1057 |
| These nested if statements could be combined | 1067 - 1069 |
| Avoid unused local variables such as 'delegateMember'. | 1344 |
| Avoid unused private methods such as 'deleteNullMemberAttributeData(List)'. | 1748 |
| Avoid unused private methods such as 'deleteNullDelegationMemberAttributeData(List)'. | 1758 |
| 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 | 273 - 275 |
| These nested if statements could be combined | 279 - 281 |
| 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() | 411 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 99 - 101 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 270 - 278 |
| These nested if statements could be combined | 274 - 276 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 21 |
| No need to import a type that lives in the same package | 22 |
| No need to import a type that lives in the same package | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 26 |
| No need to import a type that lives in the same package | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' | 21 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 241 - 253 |
| Avoid empty catch blocks | 263 - 265 |
| Avoid empty catch blocks | 434 - 436 |
| Avoid empty catch blocks | 448 - 450 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 376 - 385 |
| These nested if statements could be combined | 973 - 975 |
| These nested if statements could be combined | 1021 - 1028 |
| Avoid unused local variables such as 'fieldsForLookup'. | 1097 |
| 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 | 429 - 448 |
| These nested if statements could be combined | 502 - 504 |
| Avoid unused local variables such as 'attributeRefName'. | 559 |
| Avoid unused method parameters such as 'nestedObjectPrefix'. | 629 - 666 |
| 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 | 138 - 140 |
| These nested if statements could be combined | 766 - 788 |
| These nested if statements could be combined | 1126 - 1128 |
| 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)'. | 717 |
| These nested if statements could be combined | 773 - 779 |
| Avoid empty if statements | 987 - 990 |
| These nested if statements could be combined | 1039 - 1041 |
| Avoid unused local variables such as 'propertyName'. | 1178 |
| Unnecessary final modifier in final class | 1428 - 1432 |
| Unnecessary final modifier in final class | 1441 - 1443 |
| 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 | 155 - 168 |
| Unnecessary final modifier in final class | 170 - 197 |
| Unnecessary final modifier in final class | 222 - 224 |
| Unnecessary final modifier in final class | 226 - 229 |
| Avoid unused private methods such as 'getMaintenanceDocumentService()'. | 356 |
| Avoid unused private methods such as 'getWorkflowDocumentService()'. | 363 |
| Avoid unused private methods such as 'getKualiConfigurationService()'. | 370 |
| Avoid unused private methods such as 'getKualiExceptionIncidentService()'. | 377 |
| 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'. | 80 |
| Avoid unused method parameters such as 'response'. | 250 - 261 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 528 |
| These nested if statements could be combined | 614 - 616 |
| Avoid unused method parameters such as 'response'. | 769 - 802 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1165 - 1182 |
| Avoid empty catch blocks | 1305 - 1306 |
| Avoid unused local variables such as 'newNote'. | 1474 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.DocumentEntry' | 31 |
| Avoid unused local variables such as 'label'. | 316 |
| Avoid unused local variables such as 'objectDescription'. | 317 |
| Avoid unused local variables such as 'helpForm'. | 460 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'kualiInquirable'. | 299 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 122 - 124 |
| These nested if statements could be combined | 134 - 136 |
| Avoid unused method parameters such as 'request'. | 165 - 169 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 239 - 256 |
| Avoid empty catch blocks | 254 |
| Avoid empty catch blocks | 795 - 797 |
| Avoid empty catch blocks | 798 - 800 |
| Avoid empty catch blocks | 801 - 803 |
| These nested if statements could be combined | 836 - 839 |
| Avoid unused local variables such as 'documentAuthorizer'. | 959 |
| Avoid unused local variables such as 'newBo'. | 965 |
| Avoid unused local variables such as 'maintenanceDocumentAuthorizer'. | 1001 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 30 |
| Avoid unused private fields such as 'inactiveRecordDisplay'. | 81 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 352 - 354 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 429 - 431 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 118 - 152 |
| These nested if statements could be combined | 190 - 193 |
| These nested if statements could be combined | 196 - 199 |
| Avoid unused local variables such as 'docTypeName'. | 457 |
| 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 | 184 - 186 |
| Avoid unused method parameters such as 'requestParams'. | 240 - 258 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 290 - 291 |
| 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 unused imports such as 'org.kuali.rice.krad.util.KRADConstants' | 23 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 80 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 82 |
| Avoid empty catch blocks | 85 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' | 26 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'persistenceStructureService'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.CriteriaLookupService' | 23 |
| Avoid unused method parameters such as 'queryClass'. | 101 - 129 |
| Avoid unused method parameters such as 'criteria'. | 140 - 147 |
| Avoid unused method parameters such as 'queryClass'. | 140 - 147 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.CriteriaDateTimeValue' | 25 |
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.CriteriaLookupService' | 26 |
| Avoid unused method parameters such as 'criteria'. | 139 - 145 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreApiServiceLocator' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.exception.CompletionException' | 25 |
| Avoid unused local variables such as 'indexName'. | 129 |
| These nested if statements could be combined | 643 - 671 |
| Avoid empty catch blocks | 720 - 722 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 146 - 148 |
| Avoid empty catch blocks | 164 - 165 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 49 - 53 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.component.Component' | 23 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.util.ComponentUtils' | 24 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 18 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 38 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'regexString'. | 82 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.ComplexAttributeDefinition' | 23 |
| Avoid empty catch blocks | 130 - 132 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 61 - 63 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'key'. | 39 |
| Overriding method merely calls super | 58 - 60 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'key'. | 60 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 44 |
| 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' | 18 |
| Avoid unused imports such as 'java.util.List' | 19 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 36 - 38 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'java.util.List' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants' | 22 |
| Overriding method merely calls super | 39 - 41 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.DataObjectEntry' | 19 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'entryName'. | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.exception.ValidationException' | 28 |
| Avoid unused imports such as 'java.util.Map' | 36 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 242 - 246 |
| These nested if statements could be combined | 505 - 529 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 714 - 726 |
| Avoid empty if statements | 744 - 751 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 121 - 126 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.StringTokenizer' | 44 |
| Avoid empty catch blocks | 87 - 89 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'pkNames'. | 152 |
| Avoid unused local variables such as 'user'. | 153 |
| Avoid unused local variables such as 'searchUsingOnlyPrimaryKeyValues'. | 170 |
| Avoid unused local variables such as 'lookupView'. | 225 |
| Avoid unused local variables such as 'lookupView'. | 446 |
| Avoid unused local variables such as 'dataObjectExample'. | 511 |
| These nested if statements could be combined | 535 - 545 |
| Avoid unused local variables such as 'fieldName'. | 601 |
| Avoid unused local variables such as 'lookupView'. | 707 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'docOrBoClass'. | 385 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 710 - 732 |
| These nested if statements could be combined | 1065 - 1067 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.namespace.QName' | 28 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 106 - 109 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 240 - 242 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 43 - 45 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.field.InputField' | 29 |
| These nested if statements could be combined | 131 - 137 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 248 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 332 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 70 - 72 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 305 - 310 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 96 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.web.format.Formatter' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krad.bo.DataObjectRelationship' | 22 |
| Avoid unused imports such as 'org.kuali.rice.krad.bo.KualiCode' | 23 |
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.AttributeSecurity' | 25 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocatorWeb' | 32 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.control.MultiValueControl' | 34 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.util.ComponentFactory' | 36 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.component.BindingInfo' | 41 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.component.DataBinding' | 43 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.util.ViewModelUtils' | 47 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.widget.Inquiry' | 50 |
| Avoid unused imports such as 'org.kuali.rice.krad.util.KRADPropertyConstants' | 53 |
| Avoid unused imports such as 'org.kuali.rice.krad.valuefinder.ValueFinder' | 55 |
| Avoid unused private fields such as 'customValidatorClass'. | 82 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 107 - 109 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 96 - 99 |
| These nested if statements could be combined | 103 - 106 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.field.InputField' | 26 |
| These nested if statements could be combined | 150 - 152 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 229 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 54 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 88 - 91 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.web.form.UifFormBase' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.view.View' | 19 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 215 - 221 |
| Avoid unused local variables such as 'layoutManager'. | 314 |
| Avoid empty if statements | 324 - 326 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 732 - 734 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 86 - 88 |
| Avoid empty if statements | 89 - 91 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 106 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'defaultPathSet'. | 56 |
| These nested if statements could be combined | 141 - 143 |
| These nested if statements could be combined | 281 - 318 |
| These nested if statements could be combined | 314 - 316 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.DataDictionary' | 21 |
| Avoid empty if statements | 147 - 149 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' | 20 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants.ViewType' | 35 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 166 - 168 |
| Avoid empty catch blocks | 168 - 170 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'lookupCriteriaEnabled'. | 80 |
| Avoid unused private fields such as 'supplementalActionsEnabled'. | 81 |
| Avoid unused private fields such as 'disableSearchButtons'. | 82 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.component.BindingInfo' | 29 |
| 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 'org.kuali.rice.krad.uif.control.TextControl' | 33 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.lang.annotation.Annotation' | 37 |
| Unnecessary final modifier in final class | 62 - 78 |
| Unnecessary final modifier in final class | 83 - 95 |
| Unnecessary final modifier in final class | 107 - 113 |
| Unnecessary final modifier in final class | 467 - 469 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 243 - 245 |
| Avoid empty catch blocks | 245 - 247 |
| Avoid empty catch blocks | 298 - 300 |
| Avoid empty catch blocks | 303 - 305 |
| An empty statement (semicolon) not part of a loop | 597 |
| These nested if statements could be combined | 828 - 830 |
| These nested if statements could be combined | 918 - 920 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 112 - 114 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 35 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants' | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 84 - 86 |
| Avoid empty catch blocks | 161 - 163 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 358 - 364 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.framework.parameter.ParameterService' | 26 |
| Avoid unused imports such as 'org.kuali.rice.core.framework.services.CoreFrameworkServiceLocator' | 27 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 38 - 40 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' | 20 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 43 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.uif.UifConstants' | 19 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 28 |
| Avoid unused imports such as 'java.util.List' | 31 |
| These nested if statements could be combined | 98 - 101 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 161 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 45 - 47 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.api.exception.ResourceUnavailableException' | 22 |
| 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 'org.kuali.rice.krms.api.KrmsConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.KrmsConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.KrmsConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 304 - 306 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.KrmsConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.KrmsConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.KrmsConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'rule'. | 218 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.KrmsConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.KrmsConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.KrmsConstants' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.KrmsConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.agenda.AgendaDefinition' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.ResultEvent' | 34 |
| Avoid unused constructor parameters such as 'termResolutionEngine'. | 46 - 66 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.framework.engine.result.EngineResultListener' | 23 |
| Avoid unused imports such as 'org.kuali.rice.krms.framework.engine.result.Log4jResultListener' | 24 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'parent'. | 190 |
| 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. | 48 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 54 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 60 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceIllegalArgumentException' | 18 |
| Avoid unused imports such as 'org.kuali.rice.core.api.uif.RemotableAttributeError' | 19 |
| Avoid unused imports such as 'org.kuali.rice.core.api.uif.RemotableAttributeField' | 20 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.agenda.AgendaDefinition' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krms.framework.engine.Agenda' | 22 |
| Avoid unused imports such as 'java.util.List' | 24 |
| Avoid unused imports such as 'java.util.Map' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.AgendaBo' | 23 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 235 - 238 |
| Avoid empty catch blocks | 238 - 241 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'typeRepositoryService'. | 63 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'parameterDefinition'. | 106 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.hibernate.mapping.TableOwner' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.util.KrmsImplConstants' | 23 |
| Avoid unused imports such as 'java.util.HashMap' | 27 |
| Avoid unused imports such as 'java.util.Map' | 29 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'context'. | 347 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.cxf.common.util.StringUtils' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krad.web.form.MaintenanceForm' | 26 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.ui.AgendaEditor' | 27 |
| Avoid unused imports such as 'java.util.HashSet' | 33 |
| Avoid unused imports such as 'java.util.Set' | 36 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krad.keyvalues.KeyValuesBase' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krad.web.form.UifFormBase' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.cxf.common.util.StringUtils' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krad.keyvalues.KeyValuesBase' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krad.web.form.MaintenanceForm' | 26 |
| Avoid unused imports such as 'org.kuali.rice.krad.web.form.UifFormBase' | 27 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.ui.AgendaEditor' | 28 |
| Avoid unused imports such as 'java.util.Collections' | 32 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.GenericQueryResults' | 28 |
| Avoid unused imports such as 'org.kuali.rice.core.api.criteria.Predicate' | 29 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.util.KrmsImplConstants' | 34 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.type.KrmsTypeAttribute' | 25 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'pList'. | 125 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 90 - 112 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.framework.engine.Agenda' | 24 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'typeAttributeFields'. | 87 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.impl.util.KRMSServiceLocatorInternal' | 25 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 364 |
| Avoid unused private methods such as 'treeToInOrderList(AgendaItemBo,List)'. | 938 |
| Avoid empty if statements | 987 |
| An empty statement (semicolon) not part of a loop | 987 |
| Avoid unused local variables such as 'firstItem'. | 1013 |
| Avoid empty if statements | 1038 - 1040 |
| Avoid unused private methods such as 'addOpCodeNode(Node,PropositionBo,int)'. | 1373 |
| Avoid unused local variables such as 'opCodeLabel'. | 1374 |
| Avoid unused private methods such as 'findPropositionTreeNode(Node,String)'. | 1396 |
| These nested if statements could be combined | 1497 - 1511 |
| Avoid empty if statements | 1549 - 1553 |
| Avoid unused local variables such as 'child'. | 1582 |
| Avoid empty if statements | 1622 - 1624 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 65 |
| Avoid unused private methods such as 'getContextBoService()'. | 77 |
| 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.krad.service.BusinessObjectService' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' | 19 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.proposition.PropositionParameterType' | 20 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.proposition.PropositionType' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.PropositionParameterBo' | 23 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.TermBo' | 24 |
| Avoid unused imports such as 'java.util.List' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.LogicalOperator' | 22 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 55 - 57 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.io.Serializable' | 26 |
| Avoid empty if statements | 54 - 56 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.ActionBo' | 30 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.CategoryBo' | 31 |
| Avoid unused private fields such as 'LOG'. | 53 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.uif.DataType' | 19 |
| Avoid unused imports such as 'org.kuali.rice.core.api.uif.RemotableAttributeField' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.api.uif.RemotableTextInput' | 21 |
| Avoid unused imports such as 'org.kuali.rice.krad.uif.container.Container' | 27 |
| Avoid unused imports such as 'org.kuali.rice.krad.web.form.MaintenanceForm' | 29 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.TermBo' | 32 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.TermResolverBo' | 33 |
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.TermResolverParameterSpecificationBo' | 34 |
| Avoid unused imports such as 'java.util.ArrayList' | 37 |
| Avoid unused imports such as 'java.util.Comparator' | 40 |
| Avoid unused imports such as 'java.util.List' | 41 |
| Avoid unused private fields such as 'LOG'. | 54 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collections' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.KrmsAttributeDefinitionBo' | 18 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'sortCodeMap'. | 51 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'validationService'. | 36 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.config.ConfigurationException' | 22 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 280 - 283 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Avoid unused imports such as 'org.kuali.rice.ksb.api.KsbApiConstants' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Avoid unused imports such as 'org.kuali.rice.ksb.api.KsbApiConstants' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.name.EntityName' | 23 |
| Avoid unused imports such as 'org.springframework.ldap.core.DirContextOperations' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.ldap.core.DirContextOperations' | 28 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 131 |
| Avoid empty catch blocks | 145 - 147 |
| Avoid unused local variables such as 'hasTaxId'. | 277 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.ldap.core.ContextMapper' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.name.EntityName' | 30 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'names'. | 74 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.ldap.core.ContextMapper' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.address.EntityAddress' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.email.EntityEmail' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.type.EntityTypeContactInfo' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.ldap.core.ContextMapper' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.principal.Principal' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| 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 'java.util.Map' | 20 |
| Avoid unused imports such as 'org.apache.commons.collections.CollectionUtils' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.identity.principal.EntityNamePrincipalName' | 27 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'kimEntity'. | 129 |
| Avoid unused local variables such as 'creatingNew'. | 130 |
| Avoid unused local variables such as 'initiatorPrincipalId'. | 132 |
| Avoid empty if statements | 354 - 356 |
| Avoid empty if statements | 362 - 364 |
| Avoid empty if statements | 369 - 371 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Avoid unused imports such as 'org.kuali.rice.location.api.LocationConstants' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.location.api.LocationConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.location.api.LocationConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.location.api.LocationConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.location.api.LocationConstants' | 21 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Avoid unused private fields such as 'vendorNumber'. | 38 |
| Avoid unused private fields such as 'vendorSoldToNumber'. | 61 |
| Avoid empty catch blocks | 174 - 176 |
| Avoid empty catch blocks | 249 - 251 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 112 - 114 |
| Avoid unused local variables such as 'message'. | 132 |
| Avoid empty if statements | 184 - 186 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 100 - 102 |
| Avoid empty catch blocks | 126 - 128 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 238 - 240 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 92 - 94 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'rows'. | 33 |