The following document contains the results of PMD 4.2.2.
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'fetchLimit'. | 64 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.UserIdDTO' | 37 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.Date' | 18 |
| Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.util.jaxb.SqlDateAdapter' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.util.jaxb.SqlDateAdapter' | 22 |
| 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 private fields such as 'LAST_ACCCESS_TIMEOUT_FIELD'. | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.config.ConfigurationException' | 20 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 125 - 127 |
| These nested if statements could be combined | 223 - 225 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 122 - 128 |
| Violation | Line |
|---|---|
| Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! | 112 |
| Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! | 129 |
| 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' | 21 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 78 - 80 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 19 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 21 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 131 - 132 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 60 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 82 - 84 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 114 - 116 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.servlet.http.HttpServletRequest' | 28 |
| 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 |
|---|---|
| Overriding method merely calls super | 37 - 41 |
| Violation | Line |
|---|---|
| Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. | 54 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 66 - 68 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'parentCriteria'. | 349 - 374 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'allowWildcards'. | 321 - 332 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'otherClass'. | 33 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 140 |
| 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 |
| An operation on an Immutable object (BigDecimal or BigInteger) won't change the object itself | 56 |
| Violation | Line |
|---|---|
| Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal | 37 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'converted'. | 45 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 44 - 60 |
| These nested if statements could be combined | 75 - 91 |
| These nested if statements could be combined | 104 - 120 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 124 - 126 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'S_TRUE'. | 26 |
| Avoid unused private fields such as 'S_FALSE'. | 27 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 33 - 36 |
| These nested if statements could be combined | 47 - 53 |
| Violation | Line |
|---|---|
| Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal | 29 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 77 - 78 |
| Avoid empty catch blocks | 83 - 84 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 90 - 91 |
| Avoid empty catch blocks | 96 - 97 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashSet' | 18 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.DataDefinition' | 30 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 31 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.PropertyDefinition' | 32 |
| Avoid unused private fields such as 'LOG'. | 46 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 113 - 115 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 144 - 152 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 79 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'keys'. | 83 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 306 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 129 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'criteria'. | 121 - 127 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 219 - 221 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 96 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'successful'. | 184 - 203 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 23 |
| 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 |
|---|---|
| Too many static imports may lead to messy code | 17 - 86 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 75 - 77 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'parameterService'. | 41 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'parameterService'. | 56 |
| These nested if statements could be combined | 104 - 106 |
| Avoid unused method parameters such as 'parameters'. | 179 - 187 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'toReturn'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'fieldName'. | 418 |
| Avoid unused private fields such as 'fieldType'. | 419 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'sourceFilePath'. | 91 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'routeHeader'. | 170 - 185 |
| These nested if statements could be combined | 179 - 181 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 99 - 102 |
| 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'. | 57 - 65 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 21 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'edlContext'. | 54 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 41 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 82 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'requestParser'. | 87 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.config.module.RunMode' | 20 |
| Avoid unused imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 22 |
| Avoid unused imports such as 'org.kuali.rice.core.framework.persistence.jpa.OrmUtils' | 24 |
| Avoid unused imports such as 'org.kuali.rice.kew.lifecycle.EmbeddedLifeCycle' | 28 |
| Avoid unused imports such as 'java.util.Collections' | 37 |
| Avoid unused imports such as 'java.util.LinkedList' | 38 |
| Avoid unused private methods such as 'getClientProtocol()'. | 132 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 37 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 27 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 25 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 107 - 109 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 151 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 32 |
| Avoid empty catch blocks | 96 - 98 |
| Avoid empty if statements | 113 - 115 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'KEW_CHANNEL'. | 48 |
| Avoid unused local variables such as 'error'. | 141 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 259 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notification'. | 50 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notification'. | 48 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 26 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.LifecycleBean' | 22 |
| Avoid unused imports such as 'org.springframework.beans.factory.BeanInitializationException' | 26 |
| Avoid unused private fields such as 'LOG'. | 35 |
| Avoid unused private fields such as 'dataSource'. | 40 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'NOTIFICATION_MESSAGE_DELIVERY_DISPATCH_SERVICE'. | 53 |
| Avoid unused private fields such as 'NOTIFICATION_MESSAGE_DELIVERER_REGISTRY_SERVICE'. | 58 |
| Avoid unused private fields such as 'NOTIFICATION_EMAIL_SERVICE'. | 61 |
| Avoid unused private fields such as 'NOTIFICATION_CONFIG'. | 62 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 38 |
| Avoid unused local variables such as 'lockedDeliveries'. | 101 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 40 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 18 |
| Avoid unused imports such as 'java.util.LinkedHashMap' | 19 |
| Avoid unused imports such as 'org.kuali.rice.ken.exception.ErrorList' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 20 |
| Avoid unused imports such as 'java.util.LinkedHashMap' | 21 |
| Avoid unused imports such as 'org.kuali.rice.ken.exception.ErrorList' | 28 |
| Avoid empty if statements | 121 - 124 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.xml.sax.SAXException' | 21 |
| Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 18 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'prefsService'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 36 |
| Avoid unused private fields such as 'businessObjectDao'. | 38 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 61 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 133 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'content'. | 364 - 380 |
| Avoid unused method parameters such as 'notification'. | 364 - 380 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'firstMessageDelivery'. | 71 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'userPreferenceService'. | 56 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notificationWorkflowDocumentService'. | 55 |
| These nested if statements could be combined | 146 - 150 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'notificationChannelService'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 301 - 307 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 268 - 346 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 295 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'response'. | 301 - 334 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.web.servlet.view.RedirectView' | 40 |
| Avoid unused method parameters such as 'response'. | 291 - 346 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 603 - 605 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.WorkflowIdDTO' | 39 |
| 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 - 187 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'toDocumentRouteHeaderValueActionListExtension(DocumentRouteHeaderValue)'. | 135 - 136 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 577 |
| Avoid empty catch blocks | 582 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 374 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 383 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 386 |
| Avoid unused local variables such as 'index'. | 651 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 218 - 220 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'groupName'. | 166 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.entity.principal.Principal' | 18 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 389 - 392 |
| Avoid unused local variables such as 'allDeactivated'. | 464 |
| Avoid unused local variables such as 'matchedArs'. | 525 |
| Avoid unused local variables such as 'arGroups'. | 542 |
| These nested if statements could be combined | 706 - 708 |
| These nested if statements could be combined | 820 - 824 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.engine.RouteContext' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 25 |
| These nested if statements could be combined | 141 - 143 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Iterator' | 22 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.DataDefinition' | 30 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.entity.principal.Principal' | 37 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 25 |
| These nested if statements could be combined | 144 - 146 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 24 |
| These nested if statements could be combined | 130 - 132 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 24 |
| These nested if statements could be combined | 139 - 141 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 202 - 207 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'actionTaken'. | 79 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.entity.principal.Principal' | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 148 - 156 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.DataDefinition' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.impex.xml.XmlIngesterService' | 36 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'parameters'. | 118 |
| An empty statement (semicolon) not part of a loop | 160 |
| Avoid unused local variables such as 'pkNames'. | 210 |
| Avoid empty catch blocks | 305 - 307 |
| Avoid empty if statements | 418 - 420 |
| An empty statement (semicolon) not part of a loop | 915 |
| Avoid empty catch blocks | 1125 - 1128 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'returnKeys'. | 145 |
| Avoid unused local variables such as 'pkNames'. | 146 |
| Avoid unused local variables such as 'user'. | 147 |
| Avoid empty try blocks | 198 - 201 |
| Avoid empty catch blocks | 201 - 204 |
| Avoid empty if statements | 266 - 268 |
| An empty statement (semicolon) not part of a loop | 437 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.ResourceLoader' | 27 |
| Avoid duplicate imports such as 'org.kuali.rice.core.impl.resourceloader.RiceResourceLoaderFactory' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.config.KIMThinClientConfigurer' | 18 |
| Avoid unused imports such as 'org.kuali.rice.ksb.messaging.config.KSBThinClientConfigurer' | 19 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.BaseLifecycle' | 20 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 22 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 234 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 20 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 587 - 589 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 32 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 72 - 75 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 21 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'searchCriteriaDisplayed'. | 36 - 38 |
| Avoid unused constructor parameters such as 'headerBarDisplayed'. | 36 - 38 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'parseAndConvertDateToRange(String)'. | 289 |
| Avoid unused local variables such as 'queryTableColumnName'. | 642 |
| These nested if statements could be combined | 1742 - 1744 |
| Avoid unused private methods such as 'tokenizeCriteria(String)'. | 1843 |
| Avoid unused private methods such as 'validateNumber(List)'. | 1850 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 49 |
| Avoid unnecessary return statements | 268 |
| These nested if statements could be combined | 290 - 307 |
| These nested if statements could be combined | 291 - 306 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 339 |
| Violation | Line |
|---|---|
| Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. | 92 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'documentSecurityService'. | 67 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 256 - 262 |
| These nested if statements could be combined | 284 - 290 |
| These nested if statements could be combined | 312 - 318 |
| These nested if statements could be combined | 340 - 346 |
| Avoid unused private methods such as 'validateNetworkId(List)'. | 351 |
| Avoid unused private methods such as 'validatePersonByPrincipalName(String)'. | 371 |
| Avoid unused method parameters such as 'principalName'. | 371 - 378 |
| Avoid unused private methods such as 'getValidDocumentType(String)'. | 718 |
| Avoid unused private methods such as 'getValidDocumentTypeOld(String)'. | 745 |
| Avoid unused private methods such as 'tokenizeCriteria(String)'. | 832 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 293 - 295 |
| These nested if statements could be combined | 407 - 409 |
| Avoid unused method parameters such as 'dataType'. | 432 - 438 |
| Avoid unused method parameters such as 'quickfinderService'. | 456 - 485 |
| Avoid unused private methods such as 'getBooleanWithPotentialOverrides(String,NamedNodeMap,NamedNodeMap,NamedNodeMap)'. | 545 |
| These nested if statements could be combined | 622 - 624 |
| Avoid unused local variables such as 'rangeDefinitionAttributes'. | 742 |
| Avoid empty if statements | 784 - 794 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 159 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.io.IOException' | 19 |
| Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' | 26 |
| Avoid unused imports such as 'org.xml.sax.SAXException' | 47 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 88 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 91 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 100 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 103 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 113 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 116 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1229 - 1231 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'docType'. | 61 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 157 - 159 |
| Avoid unused local variables such as 'replacementValue'. | 247 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 94 - 96 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 166 - 169 |
| These nested if statements could be combined | 176 - 179 |
| These nested if statements could be combined | 220 - 224 |
| Avoid unused local variables such as 'extensions'. | 242 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.notes.Note' | 24 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 631 - 633 |
| Violation | Line |
|---|---|
| This usage of the Collection.toArray() method will throw a ClassCastException. | 1287 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 21 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 22 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'nextNodeInstance'. | 343 - 351 |
| Avoid unused method parameters such as 'nodeInstance'. | 577 - 599 |
| Avoid unused method parameters such as 'nodeInstanceId'. | 613 - 634 |
| Avoid unused method parameters such as 'nodeInstanceId'. | 676 - 697 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'documentId'. | 52 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 23 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 122 - 124 |
| Overriding method merely calls super | 154 - 156 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 76 - 79 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 27 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kew.actionrequest.ActionRequestValue' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' | 25 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 28 |
| These nested if statements could be combined | 129 - 132 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 21 |
| Avoid unused imports such as 'java.util.Map' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 34 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 302 - 304 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 34 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 35 |
| Avoid unused imports such as 'org.kuali.rice.kew.engine.node.Branch' | 39 |
| Avoid unused local variables such as 'i'. | 119 |
| Avoid unused local variables such as 'documentType'. | 160 |
| Avoid empty if statements | 166 - 191 |
| Avoid unused method parameters such as 'helper'. | 234 - 283 |
| Avoid unused method parameters such as 'helper'. | 407 - 427 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 235 - 237 |
| Avoid empty catch blocks | 374 - 376 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 107 - 110 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 20 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'errors'. | 107 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.impex.ExportDataSet' | 29 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.action.ActionForm' | 19 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'logger'. | 45 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 22 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.BaseLifecycle' | 21 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 72 - 74 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 37 - 43 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 232 - 234 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.logging.Logger' | 22 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 212 |
| Avoid unused method parameters such as 'request'. | 216 - 267 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 241 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 259 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.util.KEWConstants' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kew.util.Utilities' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 24 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 22 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.impl.resourceloader.BaseWrappingResourceLoader' | 22 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 27 |
| Overriding method merely calls super | 81 - 84 |
| Avoid empty catch blocks | 142 - 144 |
| Avoid empty catch blocks | 156 - 158 |
| Avoid empty catch blocks | 169 - 171 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.ResourceLoader' | 22 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'DEFAULT_COLOR'. | 72 |
| Avoid unused private fields such as 'DEFAULT_ACTION_LIST_SIZE'. | 73 |
| Avoid unused private fields such as 'DEFAULT_REFRESH_RATE'. | 74 |
| Avoid unused local variables such as 'stagingDirectory'. | 96 |
| Avoid empty if statements | 176 - 178 |
| 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. | 105 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 24 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 44 |
| Avoid unused private fields such as 'searchableAttributeValues'. | 187 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.entity.principal.Principal' | 19 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'routeContext'. | 70 - 87 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 151 - 153 |
| Avoid empty if statements | 172 - 174 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 133 - 134 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'principal'. | 424 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 301 - 304 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 24 |
| Avoid unused private fields such as 'LOG'. | 55 |
| Avoid empty if statements | 122 - 124 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 21 |
| Avoid unused method parameters such as 'ruleTemplateName'. | 81 - 83 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'makeActionRequests(ActionRequestFactory,RouteContext,RuleBaseValues,DocumentRouteHeaderValue,ActionRequestValue,RuleDelegation)'. | 238 |
| Avoid empty catch blocks | 355 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'parseParameters(String)'. | 56 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 99 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'documentContent'. | 92 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 20 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 23 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 24 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 49 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 128 |
| 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 | 226 - 228 |
| Avoid unused private methods such as 'getWorkgroupOrCriteria(Collection)'. | 393 |
| Avoid unused private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. | 416 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. | 149 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 131 - 133 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 870 - 873 |
| Avoid empty if statements | 900 - 902 |
| Avoid empty if statements | 950 - 952 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 79 - 82 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'isDelegate'. | 363 - 424 |
| Avoid unused method parameters such as 'request'. | 446 - 448 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 92 - 98 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| 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'. | 136 |
| Avoid empty if statements | 148 - 151 |
| These nested if statements could be combined | 455 - 457 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 81 - 83 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 57 - 61 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 139 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 150 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 882 - 893 |
| These nested if statements could be combined | 955 - 957 |
| Avoid unused local variables such as 'principalId'. | 1192 |
| Violation | Line |
|---|---|
| Avoid unnecessary temporaries when converting primitives to Strings | 59 |
| Avoid unnecessary temporaries when converting primitives to Strings | 75 |
| Avoid unnecessary temporaries when converting primitives to Strings | 95 |
| Avoid unnecessary temporaries when converting primitives to Strings | 130 |
| Avoid unnecessary temporaries when converting primitives to Strings | 155 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 104 - 106 |
| Avoid empty catch blocks | 130 - 132 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'request'. | 254 - 260 |
| These nested if statements could be combined | 402 - 404 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getNewOptionIdForActionList()'. | 43 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.sql.DataSource' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.sql.ResultSet' | 20 |
| Avoid unused local variables such as 'result'. | 66 |
| Avoid empty catch blocks | 68 - 70 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 71 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 23 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'filterConfig'. | 67 |
| These nested if statements could be combined | 169 - 173 |
| These nested if statements could be combined | 170 - 172 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'filterConfig'. | 51 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 1500 |
| Avoid unused method parameters such as 'nodeTypeName'. | 1199 - 1352 |
| These nested if statements could be combined | 1256 - 1259 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.common.attribute.KimAttribute' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.common.attribute.KimAttributeData' | 28 |
| Avoid unused private fields such as 'DEFAULT_GROUP_DESCRIPTION'. | 61 |
| Avoid unused private fields such as 'groupAttributes'. | 65 |
| Avoid empty if statements | 191 - 193 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 320 - 322 |
| Avoid empty if statements | 390 - 392 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 600 |
| Avoid unused private methods such as 'checkForDuplicateRuleDelegations(List)'. | 192 |
| Avoid unused local variables such as 'delegations'. | 414 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'attributes'. | 144 - 154 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 527 |
| These nested if statements could be combined | 386 - 389 |
| Avoid unused method parameters such as 'nodeType'. | 471 - 473 |
| Avoid unused method parameters such as 'nodeType'. | 478 - 480 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.common.attribute.KimAttributeData' | 23 |
| Avoid unnecessary temporaries when converting primitives to Strings | 88 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 88 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 76 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 98 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 107 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 167 |
| Violation | Line |
|---|---|
| Too many static imports may lead to messy code | 17 - 260 |
| Avoid unused private methods such as 'exportRuleDelegations(Element,RuleBaseValues)'. | 224 |
| Violation | Line |
|---|---|
| The null check here is misplaced; if the variable is null therell be a NullPointerException | 186 |
| These nested if statements could be combined | 206 - 212 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'affiliationTypeCode'. | 47 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'employmentTypeCode'. | 42 |
| 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.entity.TypeContract' | 20 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.hibernate.annotations.Parameter' | 19 |
| Avoid duplicate imports such as 'javax.persistence.Entity' | 27 |
| Avoid duplicate imports such as 'javax.persistence.Table' | 28 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 85 - 87 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.util.AttributeSet' | 24 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'spacesPattern'. | 100 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 91 - 98 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 300 - 309 |
| These nested if statements could be combined | 455 - 466 |
| Avoid unused local variables such as 'i'. | 709 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 157 - 161 |
| These nested if statements could be combined | 372 - 385 |
| These nested if statements could be combined | 373 - 384 |
| These nested if statements could be combined | 529 - 542 |
| These nested if statements could be combined | 530 - 541 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 23 |
| No need to import a type that lives in the same package | 24 |
| No need to import a type that lives in the same package | 25 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 23 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 25 |
| No need to import a type that lives in the same package | 26 |
| No need to import a type that lives in the same package | 27 |
| Avoid empty if statements | 92 - 94 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 26 |
| No need to import a type that lives in the same package | 27 |
| No need to import a type that lives in the same package | 28 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'namespaceCode'. | 176 - 184 |
| Avoid unused method parameters such as 'namespaceCode'. | 200 - 209 |
| Avoid unused method parameters such as 'qualification'. | 405 - 410 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'routeNodeNames'. | 62 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.PermissionImpl' | 20 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 94 - 96 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 294 - 295 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 67 |
| These nested if statements could be combined | 114 - 116 |
| Avoid unused private methods such as 'getAttrDefnId(AttributeDefinition)'. | 156 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 136 - 149 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'i'. | 67 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'document'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'i'. | 69 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'authenticationService'. | 73 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 78 - 81 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 64 |
| These nested if statements could be combined | 225 - 227 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 427 - 443 |
| These nested if statements could be combined | 607 - 609 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 414 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.entity.principal.Principal' | 23 |
| These nested if statements could be combined | 209 - 212 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 679 - 703 |
| These nested if statements could be combined | 686 - 702 |
| These nested if statements could be combined | 982 - 984 |
| These nested if statements could be combined | 994 - 996 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.common.attribute.KimAttributeData' | 25 |
| Avoid unused private methods such as 'getPrincipalImpl(String)'. | 641 |
| These nested if statements could be combined | 1273 - 1276 |
| Avoid empty if statements | 1700 - 1702 |
| Avoid empty if statements | 1708 - 1710 |
| Avoid empty if statements | 1716 - 1718 |
| These nested if statements could be combined | 2310 - 2312 |
| These nested if statements could be combined | 2319 - 2321 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 245 - 257 |
| Avoid empty catch blocks | 267 - 269 |
| These nested if statements could be combined | 404 - 407 |
| These nested if statements could be combined | 414 - 418 |
| Avoid empty catch blocks | 489 - 491 |
| Avoid empty catch blocks | 504 - 506 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 40 |
| Avoid unused private methods such as 'getAttributeInfo(List,String)'. | 204 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 570 - 573 |
| These nested if statements could be combined | 598 |
| These nested if statements could be combined | 607 - 609 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 178 - 180 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 205 - 207 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.Version' | 21 |
| Avoid unused imports such as 'javax.persistence.TemporalType' | 22 |
| Avoid unused imports such as 'javax.persistence.Temporal' | 23 |
| Avoid unused imports such as 'javax.persistence.Id' | 25 |
| Avoid unused imports such as 'javax.persistence.CascadeType' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashSet' | 19 |
| Avoid unused imports such as 'java.util.Set' | 20 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 23 |
| Avoid unused private fields such as 'KNS_SPRING_BEANS_PATH'. | 37 |
| 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 | 226 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'fk'. | 89 |
| Avoid unused local variables such as 'foreignPK'. | 90 |
| Avoid empty catch blocks | 111 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Avoid unused private fields such as 'businessObjectDao'. | 35 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 44 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'START_INDEX'. | 28 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'START_INDEX'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'START_INDEX'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'START_INDEX'. | 32 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'entityManager'. | 56 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 29 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 18 |
| Avoid unused imports such as 'java.util.Map' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAttributes' | 24 |
| Avoid unused imports such as 'org.kuali.rice.kim.util.KimConstants' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.SelectedObjectIds' | 26 |
| Avoid unused imports such as 'org.kuali.rice.kns.exception.AuthorizationException' | 27 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 161 - 163 |
| These nested if statements could be combined | 169 - 171 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 222 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.document.Document' | 26 |
| These nested if statements could be combined | 245 - 247 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 215 - 239 |
| These nested if statements could be combined | 278 - 288 |
| Avoid empty catch blocks | 535 - 538 |
| Avoid unused local variables such as 'namespaceCode'. | 609 |
| Avoid unused local variables such as 'componentName'. | 610 |
| Avoid unused local variables such as 'propertyName'. | 611 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 204 - 206 |
| These nested if statements could be combined | 221 - 223 |
| These nested if statements could be combined | 237 - 239 |
| These nested if statements could be combined | 254 - 256 |
| These nested if statements could be combined | 271 - 274 |
| These nested if statements could be combined | 339 - 341 |
| Avoid empty if statements | 502 - 509 |
| These nested if statements could be combined | 519 - 533 |
| These nested if statements could be combined | 523 - 532 |
| These nested if statements could be combined | 556 - 574 |
| These nested if statements could be combined | 558 - 573 |
| These nested if statements could be combined | 566 - 570 |
| Avoid empty catch blocks | 584 - 588 |
| These nested if statements could be combined | 657 - 659 |
| These nested if statements could be combined | 727 - 729 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'attributeRefName'. | 345 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 244 - 247 |
| Avoid unused method parameters such as 'refFieldName'. | 379 - 394 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 254 - 256 |
| These nested if statements could be combined | 704 - 707 |
| These nested if statements could be combined | 718 - 722 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 157 - 163 |
| These nested if statements could be combined | 203 - 207 |
| These nested if statements could be combined | 222 - 225 |
| These nested if statements could be combined | 269 - 271 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 554 - 556 |
| Avoid empty catch blocks | 570 - 572 |
| These nested if statements could be combined | 851 - 853 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| This usage of the Collection.toArray() method will throw a ClassCastException. | 165 |
| Avoid unused method parameters such as 'documentHeaderId'. | 646 - 653 |
| These nested if statements could be combined | 740 - 742 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'persistenceService'. | 45 |
| Avoid unused local variables such as 'formatter'. | 117 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 37 - 39 |
| These nested if statements could be combined | 46 - 48 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 22 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 26 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 557 - 560 |
| Avoid unused method parameters such as 'businessObject'. | 633 - 650 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 61 - 63 |
| These nested if statements could be combined | 71 - 73 |
| 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 | 582 - 584 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'dateTimeService'. | 54 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 205 - 207 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'key'. | 258 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 52 - 57 |
| 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 constructor parameters such as 'prop'. | 74 - 76 |
| Avoid unused constructor parameters such as 'broker'. | 74 - 76 |
| Avoid unused local variables such as 'cache'. | 83 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.exception.PropertiesException' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.exception.PropertiesException' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Arrays' | 21 |
| Avoid unused imports such as 'java.util.List' | 22 |
| Avoid unused imports such as 'org.kuali.rice.core.util.RiceKeyConstants' | 24 |
| Violation | Line |
|---|---|
| Avoid unused method parameters such as 'form'. | 75 - 82 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'label'. | 315 |
| Avoid unused local variables such as 'objectDescription'. | 316 |
| Avoid unused local variables such as 'helpForm'. | 460 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'kualiInquirable'. | 295 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 119 - 121 |
| These nested if statements could be combined | 131 - 133 |
| Avoid unused method parameters such as 'request'. | 162 - 166 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 237 - 254 |
| Avoid empty catch blocks | 252 |
| Avoid empty catch blocks | 794 - 796 |
| Avoid empty catch blocks | 797 - 799 |
| Avoid empty catch blocks | 800 - 802 |
| These nested if statements could be combined | 835 - 838 |
| Avoid unused local variables such as 'documentAuthorizer'. | 958 |
| Avoid unused local variables such as 'newBo'. | 964 |
| Avoid unused local variables such as 'maintenanceDocumentAuthorizer'. | 1000 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Properties' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 30 |
| These nested if statements could be combined | 515 - 525 |
| These nested if statements could be combined | 542 - 552 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 22 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'targetName'. | 37 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 31 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 367 - 368 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.taglib.TagUtils' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.taglib.TagUtils' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.struts.taglib.TagUtils' | 22 |
| Violation | Line |
|---|---|
| Unnecessary final modifier in final class | 61 - 69 |
| Unnecessary final modifier in final class | 71 - 79 |
| Unnecessary final modifier in final class | 91 - 101 |
| Unnecessary final modifier in final class | 110 - 112 |
| Unnecessary final modifier in final class | 139 - 141 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 28 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 181 - 183 |
| 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.exception.InvalidActionTakenException' | 31 |
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 32 |
| These nested if statements could be combined | 280 - 282 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 367 - 370 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.InvalidActionTakenException' | 28 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 150 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 65 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 115 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 138 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ksb.api.registry.ServiceRegistry' | 20 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 32 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 92 - 94 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ksb.util.KSBConstants.Config' | 23 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 77 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ksb.impl.registry.ServiceInfoBo' | 28 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 25 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' | 25 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 271 - 273 |
| These nested if statements could be combined | 276 - 278 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 143 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'transactionManager'. | 41 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 68 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 62 - 64 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 139 |
| Violation | Line |
|---|---|
| Avoid importing anything from the package 'java.lang' | 20 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 48 - 50 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 392 - 395 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 121 - 122 |
| Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) | 214 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 118 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 57 - 59 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.ksb.api.messaging.MessageHelper' | 31 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 19 |
| These nested if statements could be combined | 97 - 99 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 103 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.BaseLifecycle' | 29 |
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 30 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.impl.resourceloader.SpringResourceLoader' | 19 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 146 |
| Avoid empty catch blocks | 149 - 151 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' | 20 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 16 |
| Violation | Line |
|---|---|
| Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' | 16 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 58 - 61 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 6 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 6 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 6 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 6 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.common.Defaultable' | 18 |
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.common.active.Inactivatable' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.entity.TypeContract' | 12 |
| No need to import a type that lives in the same package | 13 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.common.active.InactivatableFromTo' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.entity.TypeContract' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.common.Defaultable' | 4 |
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.common.active.Inactivatable' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.entity.email.EntityEmail' | 9 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.ModelObjectComplete' | 7 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.collections.CollectionUtils' | 19 |
| Avoid unused imports such as 'org.kuali.rice.core.util.AttributeSet' | 28 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.common.attribute.KimAttributeData' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.common.attribute.KimAttributeDataContract' | 30 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlElementWrapper' | 37 |
| Avoid unused imports such as 'java.util.ArrayList' | 41 |
| Avoid unused imports such as 'java.util.List' | 43 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.common.attribute.KimAttributeDataContract' | 24 |
| Avoid unused imports such as 'java.util.List' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.util.jaxb.AttributeSetAdapter' | 19 |
| Avoid unused imports such as 'java.util.HashMap' | 33 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 32 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 21 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' | 6 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.entity.privacy.EntityPrivacyPreferencesContract' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.role.dto.DelegateInfo' | 21 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kim.api.group.GroupContract' | 20 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 364 - 366 |
| 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.kns.exception.ClassNotPersistableException' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.exception.ClassNotPersistableException' | 24 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 81 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 83 |
| Avoid empty catch blocks | 86 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'persistenceStructureService'. | 42 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 19 |
| Avoid unused imports such as 'java.util.Set' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashSet' | 19 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' | 25 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 270 - 278 |
| These nested if statements could be combined | 274 - 276 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.persistence.Column' | 18 |
| Avoid unused imports such as 'javax.persistence.Id' | 19 |
| Avoid unused imports such as 'javax.persistence.CascadeType' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.uif.core.BindingInfo' | 46 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.exception.CompletionException' | 38 |
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.exporter.StringMap' | 39 |
| Avoid unused imports such as 'org.kuali.rice.kns.uif.core.Component' | 46 |
| Avoid unused imports such as 'org.springframework.core.convert.ConversionService' | 53 |
| Avoid unused local variables such as 'indexName'. | 131 |
| These nested if statements could be combined | 575 - 603 |
| Avoid empty catch blocks | 652 - 654 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 145 - 147 |
| Avoid empty catch blocks | 163 - 164 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 25 |
| These nested if statements could be combined | 271 - 273 |
| These nested if statements could be combined | 277 - 279 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 49 - 53 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 24 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 123 - 125 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 193 - 195 |
| These nested if statements could be combined | 199 - 201 |
| Violation | Line |
|---|---|
| An empty statement (semicolon) not part of a loop | 38 |
| Violation | Line |
|---|---|
| Ensure you override both equals() and hashCode() | 410 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 19 |
| Avoid unused imports such as 'java.util.List' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 24 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'componentBeans'. | 168 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'regexString'. | 81 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 129 - 131 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 93 - 95 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 79 - 81 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 25 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 33 - 35 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.validation.constraint.CollectionSizeConstraint' | 23 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'entryName'. | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kns.exception.ValidationException' | 31 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 247 - 251 |
| These nested if statements could be combined | 510 - 534 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 23 |
| These nested if statements could be combined | 722 - 730 |
| Avoid empty if statements | 748 - 755 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 18 |
| Avoid unused private fields such as 'LOG'. | 28 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 121 - 126 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 22 |
| No need to import a type that lives in the same package | 24 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.document.Document' | 22 |
| Avoid unused imports such as 'org.kuali.rice.kns.workflow.service.KualiWorkflowDocument' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Map' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.List' | 18 |
| Avoid unused imports such as 'java.util.Map' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'java.util.List' | 20 |
| Avoid unused private fields such as 'LOG'. | 31 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 21 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 374 - 383 |
| These nested if statements could be combined | 970 - 972 |
| These nested if statements could be combined | 1018 - 1025 |
| Avoid unused local variables such as 'fieldsForLookup'. | 1094 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 30 |
| Avoid unused private methods such as 'getBusinessObjectAuthorizationService()'. | 206 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.bo.SelectedObjectIds' | 25 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 134 - 136 |
| These nested if statements could be combined | 483 - 502 |
| These nested if statements could be combined | 556 - 558 |
| Avoid unused local variables such as 'attributeRefName'. | 613 |
| Avoid unused method parameters such as 'nestedObjectPrefix'. | 683 - 720 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocatorWeb' | 30 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 471 - 477 |
| These nested if statements could be combined | 494 - 497 |
| Avoid empty if statements | 527 - 535 |
| Avoid unused local variables such as 'cachedValues'. | 823 |
| These nested if statements could be combined | 1414 - 1452 |
| Avoid unused local variables such as 'catchme'. | 1426 |
| Avoid unused local variables such as 'catchme'. | 1504 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 140 - 142 |
| These nested if statements could be combined | 1098 - 1100 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.document.Document' | 18 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'docOrBoClass'. | 386 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 18 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 106 - 109 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 240 - 242 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.uif.util.ObjectPropertyUtils' | 30 |
| These nested if statements could be combined | 115 - 121 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 213 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 285 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'multipleValues'. | 60 |
| Avoid unused private fields such as 'lookupCriteriaEnabled'. | 61 |
| Avoid unused private fields such as 'supplementalActionsEnabled'. | 62 |
| Avoid unused private fields such as 'ddExtraButton'. | 63 |
| Avoid unused private fields such as 'headerBarEnabled'. | 64 |
| Avoid unused private fields such as 'disableSearchButtons'. | 65 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashSet' | 20 |
| Avoid unused imports such as 'java.util.List' | 21 |
| Avoid unused imports such as 'java.util.Set' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kns.uif.field.Field' | 29 |
| Avoid unused imports such as 'org.kuali.rice.kns.uif.field.GroupField' | 30 |
| Avoid unused imports such as 'org.kuali.rice.kns.uif.util.ComponentUtils' | 31 |
| These nested if statements could be combined | 184 - 188 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.springframework.util.MethodInvoker' | 26 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 238 - 245 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 69 - 71 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 42 - 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.ClassUtils' | 21 |
| Avoid unused private fields such as 'customValidatorClass'. | 75 |
| These nested if statements could be combined | 308 - 312 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSUtils' | 21 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.ObjectUtils' | 22 |
| Avoid unused imports such as 'org.springframework.util.MethodInvoker' | 23 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 306 - 311 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 66 - 68 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 96 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 81 - 83 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 108 - 111 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 182 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 45 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 87 - 90 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.services.CoreApiServiceLocator' | 5 |
| Avoid unused imports such as 'org.kuali.rice.core.framework.services.CoreFrameworkServiceLocator' | 6 |
| Avoid unused imports such as 'org.springframework.scheduling.quartz.SimpleTriggerBean' | 20 |
| Avoid unused imports such as 'org.springframework.util.MethodInvoker' | 21 |
| Avoid unused imports such as 'java.util.Set' | 30 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 208 - 210 |
| These nested if statements could be combined | 259 - 270 |
| These nested if statements could be combined | 737 - 746 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 223 - 225 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.uif.UifConstants.ViewStatus' | 34 |
| Avoid unused imports such as 'org.kuali.rice.kns.uif.container.PageGroup' | 35 |
| Avoid empty if statements | 704 - 706 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 75 - 77 |
| Avoid empty if statements | 78 - 80 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 95 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.uif.container.View' | 19 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 519 - 551 |
| These nested if statements could be combined | 547 - 549 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.HashMap' | 20 |
| Avoid unused imports such as 'org.hibernate.mapping.Component' | 27 |
| Avoid unused imports such as 'org.kuali.rice.kns.uif.control.TextControl' | 39 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.uif.core.Component' | 18 |
| Avoid empty catch blocks | 129 - 131 |
| Avoid empty catch blocks | 132 - 134 |
| These nested if statements could be combined | 153 - 155 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.util.KNSUtils' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kns.uif.util.ComponentUtils' | 24 |
| Avoid unused imports such as 'org.kuali.rice.kns.uif.util.ObjectPropertyUtils' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kns.util.ObjectUtils' | 26 |
| Avoid unused imports such as 'java.util.Map' | 28 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 23 |
| These nested if statements could be combined | 48 - 52 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 23 |
| No need to import a type that lives in the same package | 28 |
| No need to import a type that lives in the same package | 29 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'containsIntermediateNull(Object,String)'. | 681 |
| These nested if statements could be combined | 737 - 743 |
| Avoid empty if statements | 951 - 954 |
| These nested if statements could be combined | 1003 - 1005 |
| Avoid unused local variables such as 'propertyName'. | 1142 |
| Unnecessary final modifier in final class | 1392 - 1396 |
| Unnecessary final modifier in final class | 1405 - 1407 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 83 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.nio.channels.NonWritableChannelException' | 19 |
| Unnecessary final modifier in final class | 46 - 61 |
| Unnecessary final modifier in final class | 66 - 79 |
| Unnecessary final modifier in final class | 88 - 94 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' | 22 |
| Unnecessary final modifier in final class | 160 - 173 |
| Unnecessary final modifier in final class | 175 - 202 |
| Unnecessary final modifier in final class | 227 - 229 |
| Unnecessary final modifier in final class | 231 - 234 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 25 |
| No need to import a type that lives in the same package | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 19 |
| Avoid unused imports such as 'java.util.Iterator' | 20 |
| Avoid unused imports such as 'java.util.List' | 21 |
| Avoid unused imports such as 'java.util.ListIterator' | 22 |
| No need to import a type that lives in the same package | 26 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 242 - 244 |
| Avoid empty catch blocks | 244 - 246 |
| Avoid empty catch blocks | 297 - 299 |
| Avoid empty catch blocks | 302 - 304 |
| An empty statement (semicolon) not part of a loop | 595 |
| These nested if statements could be combined | 825 - 827 |
| These nested if statements could be combined | 915 - 917 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'params'. | 395 |
| Avoid unused local variables such as 'files'. | 413 |
| These nested if statements could be combined | 495 - 498 |
| These nested if statements could be combined | 546 - 549 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 19 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' | 22 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 66 - 68 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 112 - 114 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 18 |
| Avoid unused imports such as 'java.util.List' | 19 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 41 - 44 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 44 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.uif.UifConstants' | 27 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 80 - 82 |
| Avoid empty catch blocks | 153 - 155 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 206 - 208 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'sessionDocumentService'. | 78 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 154 - 157 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 284 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 26 |
| Avoid unused imports such as 'org.kuali.rice.kns.uif.UifConstants' | 28 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'OUTPUT_ENCRYPTION_WARNING'. | 77 |
| Avoid unused method parameters such as 'response'. | 242 - 253 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 532 |
| These nested if statements could be combined | 618 - 620 |
| Avoid unused method parameters such as 'response'. | 772 - 805 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 1156 - 1172 |
| Avoid empty catch blocks | 1295 - 1296 |
| Avoid unused local variables such as 'newNote'. | 1464 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 34 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'inactiveRecordDisplay'. | 75 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 350 - 352 |
| Violation | Line |
|---|---|
| Overriding method merely calls super | 427 - 429 |
| 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'. | 454 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'ddService'. | 147 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 226 |
| These nested if statements could be combined | 325 - 328 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'LOG'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 24 |
| Violation | Line |
|---|---|
| Avoid empty catch blocks | 185 - 187 |
| Avoid unused method parameters such as 'requestParams'. | 241 - 259 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 245 - 247 |
| These nested if statements could be combined | 267 - 269 |
| These nested if statements could be combined | 488 - 490 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 192 - 194 |
| Violation | Line |
|---|---|
| Avoid unused private methods such as 'getBusinessObjectAuthorizationService()'. | 63 |
| Avoid unused local variables such as 'row'. | 101 |
| These nested if statements could be combined | 354 - 673 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.dto.UserIdDTO' | 23 |
| Avoid unused imports such as 'org.kuali.rice.kew.exception.InvalidActionTakenException' | 25 |
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.lang.reflect.InvocationTargetException' | 18 |
| Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' | 26 |
| Avoid unused imports such as 'org.kuali.rice.core.api.mo.ModelObjectComplete' | 28 |
| Avoid unused private fields such as 'COULDNT_INVOKE_BUILDER_CREATE'. | 38 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.ArrayList' | 4 |
| Avoid unused imports such as 'java.util.Arrays' | 5 |
| Avoid unused imports such as 'java.util.Collections' | 7 |
| Avoid unused imports such as 'java.util.List' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| These nested if statements could be combined | 273 - 275 |
| These nested if statements could be combined | 319 - 321 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'rule'. | 208 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Set' | 5 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.rule.RuleDefinition' | 36 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.RepositoryConstants' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.RepositoryConstants.Namespaces' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 20 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.RepositoryConstants' | 22 |
| Violation | Line |
|---|---|
| Avoid unused constructor parameters such as 'termResolutionEngine'. | 26 - 46 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' | 9 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'parent'. | 176 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' | 20 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.type.KrmsTypeDefinition' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.type.KrmsTypeDefinition' | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.Term' | 18 |
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.term.TermDefinition' | 25 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'typeRepositoryService'. | 66 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionEngine' | 28 |
| Avoid unused local variables such as 'parameterDefinition'. | 109 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.api.repository.action.ActionDefinition' | 19 |
| Avoid duplicate imports such as 'org.kuali.rice.krms.api.repository.context.ContextDefinition' | 21 |
| Violation | Line |
|---|---|
| Avoid unused local variables such as 'boToUpdate'. | 86 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.swing.text.html.HTMLDocument.Iterator' | 26 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.namespace.QName' | 18 |
| Avoid unused imports such as 'org.kuali.rice.kns.service.BusinessObjectService' | 22 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Collection' | 20 |
| Avoid unused imports such as 'java.util.Collections' | 21 |
| Avoid unused imports such as 'java.util.HashMap' | 22 |
| Avoid unused imports such as 'java.util.Map' | 23 |
| Avoid unused imports such as 'org.apache.commons.collections.CollectionUtils' | 25 |
| Avoid unused imports such as 'org.apache.commons.lang.StringUtils' | 26 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 56 - 58 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.krms.framework.engine.AgendaTree' | 21 |
| Violation | Line |
|---|---|
| Avoid empty if statements | 58 - 60 |
| Overriding method merely calls super | 85 - 88 |
| Overriding method merely calls super | 110 - 113 |
| Overriding method merely calls super | 121 - 124 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.kns.service.BusinessObjectService' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.config.ConfigurationException' | 23 |
| Violation | Line |
|---|---|
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 57 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 72 |
| Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. | 74 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Avoid unused imports such as 'org.kuali.rice.ksb.api.KsbApiConstants' | 24 |
| Violation | Line |
|---|---|
| No need to import a type that lives in the same package | 23 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' | 5 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Avoid unused imports such as 'org.kuali.rice.ksb.api.KsbApiConstants' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 3 |
| Avoid unused imports such as 'org.kuali.rice.kns.bo.PersistableBusinessObjectBase' | 5 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 8 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 3 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 6 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.core.util.type.KualiPercent' | 11 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 20 |
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' | 28 |
| Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocatorInternal' | 29 |
| Avoid unused private fields such as 'LOG'. | 39 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 21 |
| Avoid unused private fields such as 'LOG'. | 37 |
| Avoid unused private fields such as 'vendorNumber'. | 41 |
| Avoid unused private fields such as 'vendorSoldToNumber'. | 64 |
| Avoid empty catch blocks | 177 - 179 |
| Avoid empty catch blocks | 252 - 254 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 18 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 20 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.LinkedHashMap' | 19 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'java.util.Date' | 18 |
| Overriding method merely calls super | 91 - 93 |
| Violation | Line |
|---|---|
| Avoid unused private fields such as 'rows'. | 34 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 22 |
| Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' | 22 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 24 |
| Violation | Line |
|---|---|
| Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' | 22 |
| Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' | 24 |