Avoid unused private fields such as 'fetchLimit'. Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' These nested if statements could be combined Avoid unused imports such as 'org.apache.commons.logging.Log' Avoid unused imports such as 'org.apache.commons.logging.LogFactory' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused private fields such as 'LAST_ACCCESS_TIMEOUT_FIELD'. Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' Avoid unused imports such as 'org.kuali.rice.core.api.config.ConfigurationException' Avoid empty catch blocks These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' No need to import a type that lives in the same package Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' These nested if statements could be combined Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! Do not hard code IPv4 or IPv6 addresses, even 127.0.0.1 ! Avoid empty catch blocks Avoid unused local variables such as 'copySize'. Avoid empty catch blocks Avoid unused imports such as 'org.apache.commons.lang.SerializationException' Avoid empty catch blocks Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid empty catch blocks Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. These nested if statements could be combined Avoid empty catch blocks Avoid unused imports such as 'javax.servlet.http.HttpServletRequest' These nested if statements could be combined Overriding method merely calls super Overriding method merely calls super Overriding method merely calls super Overriding method merely calls super Overriding method merely calls super Overriding method merely calls super Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. These nested if statements could be combined Avoid unused method parameters such as 'parentCriteria'. Avoid unused method parameters such as 'allowWildcards'. Avoid unused local variables such as 'otherClass'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused private fields such as 'JPA_PARENT_INITIAL_ALIAS'. Avoid unused private fields such as 'ALIAS_PREFIX_LEN'. Avoid unused local variables such as 'valueLen'. Avoid empty if statements Avoid empty catch blocks Avoid empty catch blocks Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal An operation on an Immutable object (BigDecimal or BigInteger) won't change the object itself Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal Avoid unused local variables such as 'converted'. These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Overriding method merely calls super Avoid unused private fields such as 'S_TRUE'. Avoid unused private fields such as 'S_FALSE'. These nested if statements could be combined These nested if statements could be combined Avoid creating BigDecimal with a decimal (float/double) literal. Use a String literal Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid unused imports such as 'java.util.HashSet' Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.DataDefinition' Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.PropertyDefinition' Avoid unused private fields such as 'LOG'. Avoid empty catch blocks Avoid empty if statements Overriding method merely calls super Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused local variables such as 'keys'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused method parameters such as 'criteria'. Avoid empty catch blocks Too many static imports may lead to messy code Avoid unused method parameters such as 'successful'. Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' Avoid unused imports such as 'org.kuali.rice.core.api.resourceloader.ResourceLoader' Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.BaseLifecycle' Too many static imports may lead to messy code Avoid empty catch blocks Avoid unused private fields such as 'parameterService'. Avoid unused private fields such as 'parameterService'. These nested if statements could be combined Avoid unused method parameters such as 'parameters'. Avoid unused local variables such as 'toReturn'. Avoid unused private fields such as 'fieldName'. Avoid unused private fields such as 'fieldType'. Avoid unused local variables such as 'sourceFilePath'. Avoid unused local variables such as 'length'. Avoid empty while statements These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kew.dto.RouteHeaderDTO' Avoid unused imports such as 'com.thoughtworks.xstream.XStream' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid empty if statements Avoid unused method parameters such as 'currentPage'. Avoid unused imports such as 'org.apache.commons.lang.StringUtils' Avoid unused private fields such as 'edlContext'. Avoid unused private fields such as 'LOG'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused local variables such as 'requestParser'. Avoid unused imports such as 'org.kuali.rice.core.api.config.module.RunMode' Avoid unused imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' Avoid unused imports such as 'org.kuali.rice.core.framework.persistence.jpa.OrmUtils' Avoid unused imports such as 'org.kuali.rice.kew.lifecycle.EmbeddedLifeCycle' Avoid unused imports such as 'java.util.Collections' Avoid unused imports such as 'java.util.LinkedList' Avoid unused private methods such as 'getClientProtocol()'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' Avoid empty if statements Too many static imports may lead to messy code Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' Avoid empty catch blocks Avoid empty if statements Avoid unused private fields such as 'KEW_CHANNEL'. Avoid unused local variables such as 'error'. Ensure you override both equals() and hashCode() Avoid unused private fields such as 'notification'. Avoid unused private fields such as 'notification'. Avoid unused private fields such as 'LOG'. Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.LifecycleBean' Avoid unused imports such as 'org.springframework.beans.factory.BeanInitializationException' Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'dataSource'. Avoid unused private fields such as 'NOTIFICATION_MESSAGE_DELIVERY_DISPATCH_SERVICE'. Avoid unused private fields such as 'NOTIFICATION_MESSAGE_DELIVERER_REGISTRY_SERVICE'. Avoid unused private fields such as 'NOTIFICATION_EMAIL_SERVICE'. Avoid unused private fields such as 'NOTIFICATION_CONFIG'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused local variables such as 'lockedDeliveries'. Avoid unused private fields such as 'LOG'. Avoid empty if statements Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' Avoid unused imports such as 'org.xml.sax.SAXException' Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' Avoid unused imports such as 'java.util.Collection' Avoid unused private fields such as 'prefsService'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'businessObjectDao'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused method parameters such as 'content'. Avoid unused method parameters such as 'notification'. Avoid unused local variables such as 'firstMessageDelivery'. Avoid unused private fields such as 'userPreferenceService'. Avoid unused private fields such as 'notificationWorkflowDocumentService'. These nested if statements could be combined Avoid unused private fields such as 'notificationChannelService'. Avoid unused private fields such as 'LOG'. Unnecessary final modifier in final class Unnecessary final modifier in final class Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused method parameters such as 'response'. Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'org.springframework.web.servlet.view.RedirectView' Avoid unused method parameters such as 'response'. These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.ken.deliverer.NotificationMessageDeliverer' Avoid unused imports such as 'org.kuali.rice.ken.exception.ErrorList' An empty statement (semicolon) not part of a loop These nested if statements could be combined Avoid unused private methods such as 'toDocumentRouteHeaderValueActionListExtension(DocumentRouteHeaderValue)'. Avoid empty catch blocks Avoid empty catch blocks Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused local variables such as 'index'. Avoid empty catch blocks These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.krad.util.GlobalVariables' These nested if statements could be combined Avoid unused local variables such as 'allDeactivated'. Avoid unused local variables such as 'matchedArs'. Avoid unused local variables such as 'arGroups'. These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'java.util.Iterator' Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'actionTaken'. These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.core.api.impex.xml.XmlIngesterService' Avoid unused local variables such as 'parameters'. An empty statement (semicolon) not part of a loop Avoid unused local variables such as 'pkNames'. Avoid empty catch blocks Avoid empty if statements An empty statement (semicolon) not part of a loop Avoid empty catch blocks Avoid unused local variables such as 'returnKeys'. Avoid unused local variables such as 'pkNames'. Avoid unused local variables such as 'user'. Avoid empty try blocks Avoid empty catch blocks Avoid empty if statements An empty statement (semicolon) not part of a loop Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.ResourceLoader' Avoid duplicate imports such as 'org.kuali.rice.core.impl.resourceloader.RiceResourceLoaderFactory' Avoid unused imports such as 'org.kuali.rice.kim.config.KIMThinClientConfigurer' Avoid unused imports such as 'org.kuali.rice.ksb.messaging.config.KSBThinClientConfigurer' Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.BaseLifecycle' Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' Avoid empty if statements Avoid empty catch blocks Avoid unused imports such as 'java.util.Map' Avoid unused constructor parameters such as 'headerBarDisplayed'. Avoid unused constructor parameters such as 'searchCriteriaDisplayed'. Avoid unused private fields such as 'LOG'. Avoid unused private methods such as 'parseAndConvertDateToRange(String)'. Avoid unused local variables such as 'queryTableColumnName'. These nested if statements could be combined Avoid unused private methods such as 'tokenizeCriteria(String)'. Avoid unused private methods such as 'validateNumber(List)'. Avoid unused private fields such as 'LOG'. Avoid unnecessary return statements These nested if statements could be combined These nested if statements could be combined Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. Avoid unused local variables such as 'documentSecurityService'. These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused private methods such as 'validateNetworkId(List)'. Avoid unused method parameters such as 'principalName'. Avoid unused private methods such as 'validatePersonByPrincipalName(String)'. Avoid unused private methods such as 'getValidDocumentType(String)'. Avoid unused private methods such as 'getValidDocumentTypeOld(String)'. Avoid unused private methods such as 'tokenizeCriteria(String)'. Avoid empty if statements These nested if statements could be combined Avoid unused method parameters such as 'dataType'. Avoid unused method parameters such as 'quickfinderService'. Avoid unused private methods such as 'getBooleanWithPotentialOverrides(String,NamedNodeMap,NamedNodeMap,NamedNodeMap)'. These nested if statements could be combined Avoid unused local variables such as 'rangeDefinitionAttributes'. Avoid empty if statements Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'org.xml.sax.SAXException' Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' Avoid unused imports such as 'java.io.IOException' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. These nested if statements could be combined Avoid unused local variables such as 'docType'. These nested if statements could be combined Overriding method merely calls super These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'extensions'. Avoid unused local variables such as 'ruleTemplate'. This usage of the Collection.toArray() method will throw a ClassCastException. Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' Avoid unused method parameters such as 'nextNodeInstance'. Avoid unused method parameters such as 'nodeInstance'. Avoid unused method parameters such as 'nodeInstanceId'. Avoid unused method parameters such as 'nodeInstanceId'. Avoid unused local variables such as 'documentId'. Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' These nested if statements could be combined Overriding method merely calls super These nested if statements could be combined Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'org.kuali.rice.kew.actionrequest.ActionRequestValue' Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' These nested if statements could be combined Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' Avoid unused imports such as 'org.kuali.rice.kew.engine.node.Branch' Avoid unused local variables such as 'i'. Avoid unused local variables such as 'documentType'. Avoid empty if statements Avoid unused method parameters such as 'helper'. Avoid unused method parameters such as 'helper'. These nested if statements could be combined Avoid empty catch blocks Avoid empty if statements Avoid unused local variables such as 'errors'. Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceIllegalArgumentException' Avoid unused private fields such as 'logger'. Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.BaseLifecycle' These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'java.util.logging.Logger' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused method parameters such as 'request'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid duplicate imports such as 'org.kuali.rice.core.impl.resourceloader.BaseWrappingResourceLoader' Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' Overriding method merely calls super Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.ResourceLoader' Avoid unused private fields such as 'DEFAULT_COLOR'. Avoid unused private fields such as 'DEFAULT_ACTION_LIST_SIZE'. Avoid unused private fields such as 'DEFAULT_REFRESH_RATE'. Avoid unused local variables such as 'stagingDirectory'. Avoid empty if statements Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' Avoid unused private fields such as 'searchableAttributeValues'. Avoid unused method parameters such as 'routeContext'. Avoid empty if statements Avoid empty if statements Avoid empty catch blocks Avoid unused imports such as 'java.util.Iterator' Avoid unused local variables such as 'principal'. These nested if statements could be combined Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' Avoid unused private fields such as 'LOG'. Avoid empty if statements Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' Avoid unused method parameters such as 'ruleTemplateName'. Avoid unused private fields such as 'LOG'. Avoid unused private methods such as 'makeActionRequests(ActionRequestFactory,RouteContext,RuleBaseValues,DocumentRouteHeaderValue,ActionRequestValue,RuleDelegation)'. Avoid empty catch blocks Avoid unused private methods such as 'parseParameters(String)'. Ensure you override both equals() and hashCode() Avoid unused local variables such as 'documentContent'. Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'org.kuali.rice.krad.util.GlobalVariables' Avoid unused imports such as 'org.kuali.rice.krad.util.GlobalVariables' Avoid unused private fields such as 'LOG'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. These nested if statements could be combined Avoid unused private methods such as 'getWorkgroupOrCriteria(Collection)'. Avoid unused private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. Avoid unused imports such as 'java.util.Iterator' Avoid unused private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. Avoid empty if statements These nested if statements could be combined Avoid empty if statements Avoid empty if statements These nested if statements could be combined Avoid unused method parameters such as 'isDelegate'. Avoid unused method parameters such as 'request'. These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kim.bo.role.dto.KimPermissionInfo' Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' Avoid unused local variables such as 'o'. An empty statement (semicolon) not part of a loop Avoid unused local variables such as 'quickfinderService'. Avoid empty if statements These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'principalId'. Avoid unused imports such as 'java.util.Map' Avoid unnecessary temporaries when converting primitives to Strings Avoid unnecessary temporaries when converting primitives to Strings Avoid unnecessary temporaries when converting primitives to Strings Avoid unnecessary temporaries when converting primitives to Strings Avoid unnecessary temporaries when converting primitives to Strings Avoid empty catch blocks Avoid empty catch blocks Avoid unused method parameters such as 'request'. These nested if statements could be combined Avoid unused private methods such as 'getNewOptionIdForActionList()'. Avoid unused imports such as 'javax.sql.DataSource' Avoid unused imports such as 'java.sql.ResultSet' Avoid unused local variables such as 'result'. Avoid empty catch blocks Avoid empty catch blocks Avoid unused private fields such as 'LOG'. Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' Avoid unused private fields such as 'filterConfig'. These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'filterConfig'. Avoid unused method parameters such as 'nodeTypeName'. These nested if statements could be combined Avoid unused private fields such as 'DEFAULT_GROUP_DESCRIPTION'. Avoid unused private fields such as 'groupAttributes'. Avoid empty if statements Too many static imports may lead to messy code Avoid empty if statements Avoid empty if statements Avoid unused private methods such as 'checkForDuplicateRuleDelegations(List)'. Avoid unused local variables such as 'delegations'. Avoid unused method parameters such as 'attributes'. Too many static imports may lead to messy code These nested if statements could be combined Avoid unused method parameters such as 'nodeType'. Avoid unused method parameters such as 'nodeType'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unnecessary temporaries when converting primitives to Strings Too many static imports may lead to messy code Too many static imports may lead to messy code Too many static imports may lead to messy code Too many static imports may lead to messy code Too many static imports may lead to messy code Avoid unused private methods such as 'exportRuleDelegations(Element,RuleBaseValues)'. The null check here is misplaced; if the variable is null therell be a NullPointerException These nested if statements could be combined Avoid unused imports such as 'java.sql.Date' Avoid unused imports such as 'java.sql.Date' Avoid unused imports such as 'java.sql.Date' These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kim.api.permission.Permission' Avoid unused private fields such as 'LOG'. Avoid unused local variables such as 'spacesPattern'. These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'i'. Avoid unused imports such as 'org.kuali.rice.kim.framework.role.RoleEbo' Avoid unused imports such as 'org.kuali.rice.kim.impl.role.RoleBo' These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined No need to import a type that lives in the same package Avoid empty if statements Avoid unused private fields such as 'LOG'. Avoid unused method parameters such as 'namespaceCode'. Avoid unused method parameters such as 'namespaceCode'. Avoid unused method parameters such as 'qualification'. These nested if statements could be combined Avoid empty catch blocks These nested if statements could be combined These nested if statements could be combined Avoid empty catch blocks Avoid empty catch blocks Avoid empty if statements Avoid empty if statements Avoid unused local variables such as 'roleBo'. Avoid unused private fields such as 'LOG'. These nested if statements could be combined Avoid unused private methods such as 'getAttrDefnId(AttributeDefinition)'. Avoid unused imports such as 'org.kuali.rice.kim.bo.role.dto.KimPermissionInfo' Avoid unused local variables such as 'i'. Avoid unused local variables such as 'document'. Avoid unused local variables such as 'i'. Avoid unused imports such as 'org.kuali.rice.kim.impl.responsibility.ResponsibilityInternalService' Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' Avoid unused imports such as 'org.kuali.rice.krad.service.SequenceAccessorService' These nested if statements could be combined Avoid unused private fields such as 'LOG'. These nested if statements could be combined Avoid unused local variables such as 'externalIdentifierCriteria'. These nested if statements could be combined These nested if statements could be combined Avoid unused private methods such as 'getPrincipalImpl(String)'. Avoid unused local variables such as 'empInfo'. These nested if statements could be combined Avoid empty if statements Avoid empty if statements Avoid empty if statements These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'LOG'. Avoid unused private methods such as 'getAttributeInfo(List,String)'. Avoid empty if statements These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocatorWeb' No need to import a type that lives in the same package Avoid unused private fields such as 'dataDictionaryService'. Avoid unused private fields such as 'documentHelperService'. These nested if statements could be combined These nested if statements could be combined Avoid empty catch blocks Avoid unused local variables such as 'namespaceCode'. Avoid unused local variables such as 'componentName'. Avoid unused local variables such as 'propertyName'. Avoid unused local variables such as 'attributeRefName'. Avoid unused imports such as 'org.kuali.rice.krad.util.ObjectUtils' Avoid empty catch blocks Avoid empty catch blocks These nested if statements could be combined Avoid unused method parameters such as 'businessObject'. Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' Avoid unused method parameters such as 'form'. These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'targetName'. These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'ojbCollectionHelper'. Avoid empty catch blocks Avoid empty catch blocks Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'org.kuali.rice.krad.service.BusinessObjectDictionaryService' Avoid empty catch blocks Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused local variables such as 'fk'. Avoid unused local variables such as 'foreignPK'. Avoid empty catch blocks Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'businessObjectDao'. Avoid unused private fields such as 'LOG'. Avoid unused local variables such as 'START_INDEX'. Avoid unused local variables such as 'START_INDEX'. Avoid unused local variables such as 'START_INDEX'. Avoid unused local variables such as 'START_INDEX'. Avoid unused local variables such as 'entityManager'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'org.kuali.rice.krad.document.Document' These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid empty if statements These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid empty catch blocks These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'maintenanceDocumentEntry'. Avoid empty catch blocks Avoid unused method parameters such as 'refFieldName'. Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' Avoid unused private fields such as 'LOG'. These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'LOG'. This usage of the Collection.toArray() method will throw a ClassCastException. Avoid unused method parameters such as 'documentHeaderId'. These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.krad.service.BusinessObjectDictionaryService' Avoid unused private fields such as 'persistenceService'. Avoid unused local variables such as 'formatter'. These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused local variables such as 'user'. These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'LOG'. These nested if statements could be combined Avoid unused local variables such as 'refPkNames'. These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'dateTimeService'. Avoid unused method parameters such as 'clazz'. Avoid unused method parameters such as 'key'. Avoid unused method parameters such as 'strValue'. Avoid unused local variables such as 'caseInsensitive'. Avoid unused method parameters such as 'clazz'. These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' Avoid unused private fields such as 'cachedObjects'. Avoid unused local variables such as 'key'. These nested if statements could be combined Avoid empty catch blocks Avoid empty catch blocks Avoid unused method parameters such as 'origSource'. Avoid unused imports such as 'org.kuali.rice.krad.exception.PropertiesException' Avoid unused imports such as 'org.kuali.rice.krad.exception.PropertiesException' Unnecessary final modifier in final class Unnecessary final modifier in final class Unnecessary final modifier in final class Unnecessary final modifier in final class Unnecessary final modifier in final class Avoid empty catch blocks Avoid unused local variables such as 'boEntry'. Avoid unused local variables such as 'docContent'. Avoid unused local variables such as 'name'. These nested if statements could be combined Avoid unused private methods such as 'getLastXPath(Element)'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kew.exception.InvalidActionTakenException' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid empty catch blocks Avoid empty catch blocks Avoid unused imports such as 'org.kuali.rice.ksb.api.registry.ServiceRegistry' Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' Overriding method merely calls super Avoid unused imports such as 'org.kuali.rice.ksb.util.KSBConstants.Config' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'org.kuali.rice.ksb.impl.registry.ServiceInfoBo' Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' Avoid duplicate imports such as 'org.kuali.rice.core.api.reflect.ObjectDefinition' These nested if statements could be combined These nested if statements could be combined Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused private fields such as 'transactionManager'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. These nested if statements could be combined Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid importing anything from the package 'java.lang' Avoid empty catch blocks These nested if statements could be combined Avoid empty catch blocks Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) Avoid empty catch blocks These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.ksb.api.messaging.MessageHelper' These nested if statements could be combined Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'org.kuali.rice.core.impl.resourceloader.SpringResourceLoader' An empty statement (semicolon) not part of a loop Avoid empty catch blocks Avoid duplicate imports such as 'org.kuali.rice.core.api.resourceloader.GlobalResourceLoader' Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' Avoid duplicate imports such as 'org.kuali.rice.core.api.lifecycle.Lifecycle' Overriding method merely calls super Avoid empty catch blocks Avoid empty catch blocks Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'org.kuali.rice.kew.api.action.InvalidActionTakenException' Avoid unused imports such as 'org.kuali.rice.kew.api.doctype.IllegalDocumentTypeException' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' Avoid unused private fields such as 'DISAPPROVE'. Avoid unused private fields such as 'USE_KEW_SUPERUSER_DOCHANDLER'. Avoid unused private fields such as 'ALLOW_UNREQUESTED_ACTION'. Avoid unused private fields such as 'DEFAULT_APPROVE'. Avoid unused private fields such as 'INITIATOR_MUST_ROUTE'. Avoid unused private fields such as 'INITIATOR_MUST_SAVE'. Avoid unused private fields such as 'INITIATOR_MUST_CANCEL'. Avoid unused private fields such as 'INITIATOR_MUST_BLANKET_APPROVE'. Avoid unused private fields such as 'LOOK_FUTURE'. Avoid unused private fields such as 'SEND_NOTIFICATION_ON_SU_APPROVE'. Avoid unused private fields such as 'SUPPORTS_QUICK_INITIATE'. Avoid unused private fields such as 'NOTIFY_ON_SAVE'. Avoid unused private fields such as 'DOCUMENT_STATUS_POLICY'. Avoid unused private fields such as 'ALLOW_SU_POSTPROCESSOR_OVERRIDE'. Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kew.api.KewApiConstants' Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' No need to import a type that lives in the same package Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.Type' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.TypeContract' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'org.apache.commons.lang.ObjectUtils' No need to import a type that lives in the same package Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.affiliation.EntityAffiliationTypeContract' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.entity.Entity' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'org.joda.time.DateTime' Avoid unused imports such as 'java.util.Date' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.affiliation.EntityAffiliation' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.affiliation.EntityAffiliationContract' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.employment.EntityEmployment' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.entity.EntityContract' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.external.EntityExternalIdentifier' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.external.EntityExternalIdentifierContract' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.privacy.EntityPrivacyPreferences' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.type.EntityTypeDataContract' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.type.EntityTypeDataDefault' Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'java.util.List' No need to import a type that lives in the same package Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.address.EntityAddressContract' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.affiliation.EntityAffiliation' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.external.EntityExternalIdentifier' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.name.EntityName' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.phone.EntityPhoneContract' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.principal.Principal' Avoid unused imports such as 'org.kuali.rice.kim.api.identity.privacy.EntityPrivacyPreferences' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' These nested if statements could be combined Avoid unused private fields such as 'CACHE_MAX_AGE_SECONDS'. These nested if statements could be combined These nested if statements could be combined Overriding method merely calls super Overriding method merely calls super Overriding method merely calls super Overriding method merely calls super Overriding method merely calls super Overriding method merely calls super Overriding method merely calls super These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'delegateMember'. Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.DataDictionaryException' Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.InactivationBlockingDefinition' Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.exception.ClassValidationException' Avoid unused imports such as 'org.kuali.rice.krad.document.authorization.DocumentAuthorizer' Avoid unused imports such as 'org.kuali.rice.krad.document.authorization.DocumentPresentationController' These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.DataDictionary' Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.ReferenceDefinition' Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.exception.AttributeValidationException' Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.exception.ClassValidationException' Avoid unused imports such as 'org.kuali.rice.krad.document.authorization.MaintenanceDocumentAuthorizer' Ensure you override both equals() and hashCode() These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'fieldsForLookup'. Avoid unused imports such as 'org.kuali.rice.krad.util.GlobalVariables' Avoid unused private methods such as 'getBusinessObjectAuthorizationService()'. Avoid unused private fields such as 'LOG'. These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'attributeRefName'. Avoid unused method parameters such as 'nestedObjectPrefix'. These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'org.hibernate.dialect.Oracle10gDialect' Avoid empty if statements These nested if statements could be combined Avoid empty if statements Avoid unused local variables such as 'cachedValues'. These nested if statements could be combined Avoid unused local variables such as 'catchme'. Avoid unused local variables such as 'catchme'. Avoid empty catch blocks These nested if statements could be combined These nested if statements could be combined Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. These nested if statements could be combined Avoid unused private methods such as 'containsIntermediateNull(Object,String)'. These nested if statements could be combined Avoid empty if statements These nested if statements could be combined Avoid unused local variables such as 'propertyName'. Unnecessary final modifier in final class Unnecessary final modifier in final class Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'org.kuali.rice.krad.util.AuditCluster' Avoid unused imports such as 'org.kuali.rice.krad.util.MessageMap' Avoid unused imports such as 'java.util.HashMap' Unnecessary final modifier in final class Unnecessary final modifier in final class Unnecessary final modifier in final class Unnecessary final modifier in final class Avoid unused private methods such as 'getMaintenanceDocumentService()'. Avoid unused private methods such as 'getWorkflowDocumentService()'. Avoid unused private methods such as 'getKualiConfigurationService()'. Avoid unused private methods such as 'getKualiExceptionIncidentService()'. Avoid unused local variables such as 'params'. Avoid unused local variables such as 'files'. These nested if statements could be combined These nested if statements could be combined Avoid empty if statements Avoid unused private fields such as 'OUTPUT_ENCRYPTION_WARNING'. Avoid unused method parameters such as 'response'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. These nested if statements could be combined Avoid unused method parameters such as 'response'. These nested if statements could be combined Avoid empty catch blocks Avoid unused local variables such as 'newNote'. Avoid unused local variables such as 'label'. Avoid unused local variables such as 'objectDescription'. Avoid unused local variables such as 'helpForm'. Avoid unused local variables such as 'kualiInquirable'. Avoid empty if statements These nested if statements could be combined Avoid unused method parameters such as 'request'. Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.DocumentEntry' These nested if statements could be combined Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks These nested if statements could be combined Avoid unused local variables such as 'documentAuthorizer'. Avoid unused local variables such as 'newBo'. Avoid unused local variables such as 'maintenanceDocumentAuthorizer'. Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' Avoid unused private fields such as 'inactiveRecordDisplay'. Avoid unused imports such as 'org.kuali.rice.krad.service.DocumentDictionaryService' These nested if statements could be combined Overriding method merely calls super Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'docTypeName'. Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowException' Avoid unused imports such as 'org.kuali.rice.krad.service.KRADServiceLocator' Avoid unused local variables such as 'ddService'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. These nested if statements could be combined Avoid unused private fields such as 'LOG'. Avoid empty catch blocks Avoid unused method parameters such as 'requestParams'. Avoid empty catch blocks These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid unused private methods such as 'getBusinessObjectAuthorizationService()'. Avoid unused local variables such as 'row'. These nested if statements could be combined Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid empty catch blocks Avoid unused imports such as 'org.apache.commons.lang.ObjectUtils' Avoid duplicate imports such as 'org.kuali.rice.kim.api.services.KimApiServiceLocator' Avoid duplicate imports such as 'org.kuali.rice.kim.bo.Person' Avoid duplicate imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' Avoid unused private fields such as 'persistenceStructureService'. Avoid unused imports such as 'org.kuali.rice.krad.datadictionary.exception.CompletionException' Avoid unused local variables such as 'indexName'. These nested if statements could be combined Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks These nested if statements could be combined Avoid duplicate imports such as 'java.util.Map' Avoid unused imports such as 'org.kuali.rice.krad.bo.BusinessObject' Avoid unused local variables such as 'componentBeans'. No need to import a type that lives in the same package An empty statement (semicolon) not part of a loop Avoid unused local variables such as 'regexString'. Avoid empty catch blocks Overriding method merely calls super Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Overriding method merely calls super Overriding method merely calls super Avoid unused private fields such as 'entryName'. Avoid unused imports such as 'org.kuali.rice.krad.exception.ValidationException' Avoid unused imports such as 'java.util.Map' These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'java.io.FileNotFoundException' These nested if statements could be combined Avoid empty if statements Avoid unused private fields such as 'LOG'. These nested if statements could be combined Avoid unused private fields such as 'LOG'. No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package No need to import a type that lives in the same package Avoid unused imports such as 'java.util.StringTokenizer' Avoid empty catch blocks Avoid unused imports such as 'org.kuali.rice.krad.uif.container.Group' Avoid unused imports such as 'org.kuali.rice.krad.uif.container.LookupView' Avoid unused imports such as 'org.kuali.rice.krad.uif.container.View' Avoid unused imports such as 'org.kuali.rice.krad.uif.field.GeneratedField' Avoid unused imports such as 'org.kuali.rice.krad.web.controller.MaintenanceDocumentController' Avoid unused local variables such as 'pkNames'. Avoid unused local variables such as 'user'. Avoid unused local variables such as 'searchUsingOnlyPrimaryKeyValues'. Avoid unused local variables such as 'searchPropertyValue'. Avoid unused local variables such as 'dataObjectExample'. These nested if statements could be combined Avoid unused local variables such as 'lookupView'. Avoid unused local variables such as 'docOrBoClass'. These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid empty catch blocks These nested if statements could be combined Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused private fields such as 'multipleValues'. Avoid unused private fields such as 'lookupCriteriaEnabled'. Avoid unused private fields such as 'supplementalActionsEnabled'. Avoid unused private fields such as 'disableSearchButtons'. Avoid unused imports such as 'org.kuali.rice.krad.service.DocumentDictionaryService' These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.krad.uif.field.AttributeField' Avoid unused imports such as 'org.kuali.rice.krad.uif.util.ViewModelUtils' Overriding method merely calls super Avoid unused private fields such as 'customValidatorClass'. An empty statement (semicolon) not part of a loop These nested if statements could be combined Overriding method merely calls super Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. These nested if statements could be combined These nested if statements could be combined Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused private fields such as 'LOG'. These nested if statements could be combined Avoid empty if statements Avoid empty if statements Avoid empty if statements Avoid empty if statements Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'java.util.ListIterator' These nested if statements could be combined These nested if statements could be combined Avoid empty catch blocks Avoid empty catch blocks These nested if statements could be combined Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'org.apache.commons.lang.StringUtils' Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'org.apache.commons.lang.StringUtils' Unnecessary final modifier in final class Unnecessary final modifier in final class Unnecessary final modifier in final class Unnecessary final modifier in final class Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks An empty statement (semicolon) not part of a loop These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined These nested if statements could be combined Avoid empty catch blocks These nested if statements could be combined Avoid unused private fields such as 'sessionDocumentService'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'org.kuali.rice.krad.uif.UifParameters' These nested if statements could be combined Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' Avoid unused imports such as 'java.lang.reflect.InvocationTargetException' Avoid unused imports such as 'org.kuali.rice.core.api.exception.RiceRuntimeException' Avoid unused imports such as 'org.kuali.rice.core.api.mo.ModelObjectComplete' Avoid unused private fields such as 'COULDNT_INVOKE_BUILDER_CREATE'. Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'java.util.Arrays' Avoid unused imports such as 'java.util.Collections' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused private fields such as 'rule'. Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'java.util.Set' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.krms.api.repository.rule.RuleDefinition' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.krms.api.repository.RepositoryConstants' Avoid unused imports such as 'org.kuali.rice.krms.api.repository.RepositoryConstants.Namespaces' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.krms.api.repository.RepositoryConstants' Avoid unused constructor parameters such as 'termResolutionEngine'. Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' Avoid unused local variables such as 'parent'. Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionException' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'org.kuali.rice.krms.api.repository.type.KrmsTypeDefinition' Avoid unused imports such as 'org.kuali.rice.krms.api.repository.type.KrmsTypeDefinition' Avoid unused imports such as 'org.kuali.rice.krms.api.engine.Term' Avoid unused imports such as 'org.kuali.rice.krms.api.repository.term.TermDefinition' Avoid unused private fields such as 'typeRepositoryService'. Avoid unused imports such as 'org.kuali.rice.krms.api.engine.TermResolutionEngine' Avoid unused local variables such as 'parameterDefinition'. An empty statement (semicolon) not part of a loop Avoid unused method parameters such as 'request'. Avoid unused private methods such as 'treeToInOrderList(AgendaItemBo,List)'. An empty statement (semicolon) not part of a loop Avoid empty if statements Avoid unused imports such as 'org.kuali.rice.krad.bo.BusinessObject' Avoid unused imports such as 'org.kuali.rice.krad.bo.PersistableBusinessObject' Avoid unused imports such as 'org.kuali.rice.krad.util.KRADConstants' Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.AgendaBo' Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.ContextBo' Overriding method merely calls super Avoid empty if statements Avoid unused imports such as 'org.kuali.rice.krms.impl.repository.KrmsAttributeDefinitionBo' Avoid unused imports such as 'org.kuali.rice.core.api.config.ConfigurationException' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.ksb.api.KsbApiConstants' Avoid unused imports such as 'org.kuali.rice.core.api.CoreConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.ksb.api.KsbApiConstants' Avoid unused imports such as 'java.util.LinkedHashMap' Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'vendorNumber'. Avoid unused private fields such as 'vendorSoldToNumber'. Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid unused local variables such as 'message'. Avoid empty if statements Avoid empty catch blocks Avoid empty catch blocks Avoid unused imports such as 'org.kuali.rice.krad.util.GlobalVariables' Overriding method merely calls super Overriding method merely calls super Avoid unused private fields such as 'rows'. Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm' Avoid unused imports such as 'org.kuali.rice.shareddata.api.SharedDataConstants' Avoid unused imports such as 'javax.xml.bind.annotation.XmlNsForm'