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 private fields such as 'LOG'. No need to import a type that lives in the same package These nested if statements could be combined Avoid empty catch blocks Avoid unused local variables such as 'memberPrefix'. Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'javax.xml.bind.annotation.XmlRootElement' Avoid unused private fields such as 'fetchLimit'. Avoid unused imports such as 'java.sql.Date' Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' Avoid unused imports such as 'org.kuali.rice.core.jaxb.SqlDateAdapter' No need to import a type that lives in the same package Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' Avoid unused imports such as 'org.kuali.rice.core.jaxb.SqlDateAdapter' Avoid empty catch blocks Always check the return of one of the navigation method (next,previous,first,last) of a ResultSet. 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 private fields such as 'CONFIG_CODED_DEFAULTS'. These nested if statements could be combined Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'org.kuali.rice.core.exception.RiceRuntimeException' Avoid unused imports such as 'org.kuali.rice.core.util.ClassLoaderUtils' Avoid unused private fields such as 'LOG'. Overriding method merely calls super Avoid unused local variables such as 'baseProperties'. Avoid unused imports such as 'java.sql.SQLException' Avoid unused imports such as 'java.util.Map' Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid unused local variables such as 'bd'. Avoid unused method parameters such as 'allowWildcards'. Avoid unused private fields such as 'bindParamCount'. Avoid unused method parameters such as 'parentCriteria'. Avoid unused imports such as 'java.util.List' 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 empty catch blocks No need to import a type that lives in the same package No need to import a type that lives in the same package Overriding method merely calls super 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 Overriding method merely calls super No need to import a type that lives in the same package Avoid unused private fields such as 'suppressedServices'. Avoid empty catch blocks Avoid unused private fields such as 'name'. Avoid unused private fields such as 'LOG'. 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 imports such as 'org.kuali.rice.core.config.Config' Avoid unused imports such as 'org.kuali.rice.core.exception.RiceRuntimeException' 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 unused local variables such as 'supportedVersion'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'java.io.IOException' Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'org.xml.sax.XMLFilter' These nested if statements could be combined Avoid unused imports such as 'org.xml.sax.SAXException' Avoid unused imports such as 'org.xml.sax.SAXParseException' Avoid unused imports such as 'java.util.LinkedList' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'org.kuali.rice.core.lifecycle.Lifecycle' Avoid unused imports such as 'org.kuali.rice.core.ojb.BaseOjbConfigurer' Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. Avoid unused local variables such as 'mode'. Avoid unused private fields such as 'LOG'. No need to import a type that lives in the same package Avoid empty catch blocks Avoid empty if statements Avoid unused imports such as 'javax.xml.namespace.QName' Avoid unused imports such as 'org.kuali.rice.core.config.ConfigContext' Avoid unused imports such as 'org.kuali.rice.core.resourceloader.GlobalResourceLoader' Avoid unused imports such as 'org.kuali.rice.ken.service.KENAPIService' Avoid unused imports such as 'org.kuali.rice.ken.service.KENServiceConstants' Avoid unused private fields such as 'KEW_CHANNEL'. Avoid unused local variables such as 'error'. Ensure you override both equals() and hashCode() Avoid unused imports such as 'javax.persistence.CascadeType' Avoid unused imports such as 'javax.persistence.JoinColumns' Avoid unused private fields such as 'notification'. Avoid unused private fields such as 'notification'. Avoid unused private fields such as 'LOG'. 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 imports such as 'java.util.HashMap' Avoid unused imports such as 'java.util.LinkedHashMap' Avoid unused imports such as 'org.kuali.rice.ken.exception.ErrorList' Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'java.util.LinkedHashMap' Avoid unused imports such as 'org.kuali.rice.ken.exception.ErrorList' Avoid empty if statements Avoid unused imports such as 'org.kuali.rice.kew.dto.NetworkIdDTO' Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' Avoid unused imports such as 'org.xml.sax.SAXException' 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 unused imports such as 'org.apache.ojb.broker.query.QueryByCriteria' Avoid unused imports such as 'org.apache.ojb.broker.query.QueryFactory' Avoid unused imports such as 'org.kuali.rice.kew.doctype.bo.DocumentType' 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 empty catch blocks Avoid empty catch blocks Avoid unused private fields such as 'ROUTEHEADERID_PROP'. Avoid unused private fields such as 'DOCTITLE_PROP'. Avoid unused private fields such as 'ACTIONITEM_ROUTEHEADERID_INVALID_ERRKEY'. 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 unused method parameters such as 'request'. Avoid unused imports such as 'java.util.Iterator' Avoid unused method parameters such as 'request'. Avoid unused imports such as 'org.apache.struts.action.ActionErrors' Avoid empty catch blocks Avoid unused private fields such as 'groupName'. Avoid unused imports such as 'org.kuali.rice.kim.service.KIMServiceLocator' 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.engine.RouteContext' Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' These nested if statements could be combined Avoid unused imports such as 'java.util.Iterator' 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 These nested if statements could be combined Avoid unused local variables such as 'actionTaken'. Avoid unused imports such as 'java.util.Iterator' These nested if statements could be combined Avoid unused imports such as 'java.util.Iterator' These nested if statements could be combined Avoid unused imports such as 'java.util.Iterator' Avoid unused imports such as 'java.util.Iterator' Avoid unused method parameters such as 'successful'. 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 instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid empty if statements Avoid empty catch blocks Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'org.kuali.rice.kew.util.Utilities' Avoid unused imports such as 'java.util.HashSet' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'java.util.Set' Avoid unused imports such as 'org.kuali.rice.kew.engine.node.KeyValuePair' Avoid unused imports such as 'java.util.Map' 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 fields such as 'CREATE_DATE_FIELD_STRING'. Avoid unused private fields such as 'APPROVE_DATE_FIELD_STRING'. Avoid unused private fields such as 'FINALIZATION_DATE_FIELD_STRING'. Avoid unused private fields such as 'LAST_STATUS_UPDATE_DATE'. Avoid unused private fields such as 'STATUS_TRANSITION_DATE_FIELD_STRING'. 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 local variables such as 'pList'. 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 unused imports such as 'org.apache.struts.action.ActionForm' 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 unused imports such as 'javax.persistence.GeneratedValue' Avoid unused imports such as 'javax.persistence.SequenceGenerator' Avoid unused imports such as 'javax.persistence.FetchType' Avoid unused imports such as 'javax.persistence.JoinColumn' Avoid unused imports such as 'javax.persistence.ManyToOne' Avoid unused imports such as 'org.kuali.rice.kew.doctype.bo.DocumentType' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'java.io.IOException' Avoid unused imports such as 'javax.xml.parsers.ParserConfigurationException' Avoid unused imports such as 'org.kuali.rice.core.reflect.ObjectDefinition' Avoid unused imports such as 'org.kuali.rice.core.resourceloader.GlobalResourceLoader' Avoid unused imports such as 'org.xml.sax.SAXException' 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'. Avoid unused imports such as 'org.kuali.rice.kew.util.Utilities' These nested if statements could be combined Overriding method merely calls super Avoid unused imports such as 'org.kuali.rice.kew.rule.RuleDelegation' 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 imports such as 'org.kuali.rice.kew.doctype.bo.DocumentType' Avoid unused imports such as 'org.kuali.rice.kew.notes.Note' Avoid unused imports such as 'java.text.SimpleDateFormat' Avoid empty if statements This usage of the Collection.toArray() method will throw a ClassCastException. 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.kew.service.KEWServiceLocator' Avoid unused imports such as 'org.kuali.rice.kew.user.AuthenticationUserId' Avoid duplicate imports such as 'org.kuali.rice.kim.bo.entity.KimPrincipal' 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 imports such as 'org.kuali.rice.kew.dto.NetworkIdDTO' Avoid unused local variables such as 'requestParser'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'LOG'. 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 'routeHeaderId'. 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 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 unused imports such as 'javax.persistence.PrePersist' 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 'doc'. No need to import a type that lives in the same package Avoid unused private methods such as 'constructSubject(FeedbackForm)'. Avoid unused private methods such as 'constructEmailBody(FeedbackForm)'. Avoid unused method parameters such as 'request'. Avoid unused local variables such as 'errors'. Avoid unused imports such as 'org.kuali.rice.core.util.RiceConstants' Avoid unused imports such as 'org.kuali.rice.kns.exception.ValidationException' Avoid unused imports such as 'org.apache.struts.action.ActionForm' Avoid unused private fields such as 'logger'. Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'java.util.List' 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 Avoid unused imports such as 'javax.persistence.CascadeType' Avoid unused local variables such as 'bytesWritten'. 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 method parameters such as 'request'. Avoid unused imports such as 'org.kuali.rice.kew.util.KEWConstants' Avoid unused imports such as 'org.kuali.rice.kew.util.Utilities' Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' Overriding method merely calls super Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid unused imports such as 'org.kuali.rice.core.config.Config' Avoid unused imports such as 'org.kuali.rice.core.config.ConfigContext' Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.KimPrincipal' 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 unused method parameters such as 'request'. Avoid unused imports such as 'org.kuali.rice.kew.exception.WorkflowServiceErrorImpl' Avoid unused imports such as 'org.kuali.rice.kew.util.KEWConstants' Avoid unused private fields such as 'LOG'. 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 unused imports such as 'org.apache.struts.action.ActionMessages' Avoid unused imports such as 'org.kuali.rice.kew.web.KeyValue' Avoid unused imports such as 'org.kuali.rice.kns.util.GlobalVariables' Avoid unused method parameters such as 'request'. Avoid unused private fields such as 'removeReplaceDocument'. Avoid unused private fields such as 'removeReplaceDocument'. Do not use if statements that are always true or always false Avoid unused private fields such as 'INVALID_REPLACEMENT_USER_ID_MSG'. Avoid unused method parameters such as 'request'. Avoid unused imports such as 'org.kuali.rice.kew.web.WorkflowRoutingForm' Avoid unused private fields such as 'LOG'. 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 imports such as 'org.apache.commons.lang.StringUtils' Avoid unused imports such as 'org.kuali.rice.kew.user.UserUtils' Avoid unused imports such as 'org.kuali.rice.kew.web.session.UserSession' Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.dto.KimEntityDefaultInfo' Avoid unused imports such as 'org.kuali.rice.core.jpa.annotations.Sequence' Avoid unused imports such as 'javax.persistence.IdClass' Avoid unused method parameters such as 'routeContext'. Avoid unused imports such as 'org.apache.ojb.broker.query.Criteria' Avoid unused imports such as 'org.apache.ojb.broker.query.QueryFactory' Avoid unused imports such as 'org.apache.ojb.broker.query.ReportQueryByCriteria' Avoid empty if statements Avoid empty catch blocks Avoid unused local variables such as 'principal'. Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAttributes' Avoid unused imports such as 'org.kuali.rice.kim.bo.types.dto.AttributeSet' Avoid unused imports such as 'org.kuali.rice.kim.service.KIMServiceLocator' Avoid unused imports such as 'org.kuali.rice.kim.util.KimConstants' Avoid unused imports such as 'org.kuali.rice.kns.exception.AuthorizationException' These nested if statements could be combined Avoid unused method parameters such as 'request'. Avoid unused imports such as 'org.apache.commons.lang.StringUtils' Avoid unused private fields such as 'LOG'. Avoid empty if statements 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)'. 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 unused imports such as 'javax.persistence.ManyToOne' Avoid unused imports such as 'javax.persistence.Transient' Ensure you override both equals() and hashCode() Avoid unused imports such as 'javax.persistence.GeneratedValue' Avoid unused imports such as 'javax.persistence.GenerationType' Avoid unused imports such as 'javax.persistence.SequenceGenerator' Avoid unused imports such as 'javax.persistence.Transient' Avoid unused imports such as 'org.kuali.rice.kns.util.Guid' Ensure you override both equals() and hashCode() 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.kew.util.KEWPropertyConstants' Avoid unused private fields such as 'ROLE_REVIEWER_PROPERTY_NAME'. Avoid unused private fields such as 'DOC_TYP_LOOKUPABLE'. Avoid unused private fields such as 'RULE_TEMPLATE_LOOKUPABLE'. Avoid unused private fields such as 'WORKGROUP_LOOKUPABLE'. Avoid unused private fields such as 'PERSON_LOOKUPABLE'. Avoid unused private fields such as 'WORKGROUP_ID_PROPERTY_NAME'. Avoid unused local variables such as 'i'. Avoid unused local variables such as 'errors'. Avoid unused local variables such as 'attributeLabel'. Avoid unused local variables such as 'attributeLabel'. Avoid unused local variables such as 'attributeLabel'. Avoid unused private fields such as 'RULE_TEMPLATE_ID_PROPERTY_NAME'. Avoid unused private fields such as 'DOC_TYP_LOOKUPABLE'. Avoid unused private fields such as 'RULE_TEMPLATE_LOOKUPABLE'. Avoid unused private fields such as 'WORKGROUP_LOOKUPABLE'. Avoid unused private fields such as 'PERSON_LOOKUPABLE'. Avoid unused local variables such as 'i'. Avoid unused local variables such as 'delegationWizard'. Avoid unused local variables such as 'isDelegateRule'. Avoid unused local variables such as 'attributeLabel'. Avoid unused local variables such as 'attributeLabel'. Avoid unused local variables such as 'attributeLabel'. Avoid unused imports such as 'javax.persistence.Transient' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. 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 private methods such as 'getWorkgroupResponsibilitySubQuery(Set)'. Avoid unused imports such as 'org.kuali.rice.core.jpa.criteria.Criteria' Avoid unused imports such as 'org.kuali.rice.core.jpa.criteria.QueryByCriteria' 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 These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'initiatorPrincipal'. Avoid unused method parameters such as 'isDelegate'. Avoid unused private methods such as 'makeLookupPathParam(ActionMapping,HttpServletRequest)'. Avoid unused method parameters such as 'request'. Avoid unused imports such as 'java.util.Properties' Avoid unused imports such as 'org.apache.struts.action.ActionForm' Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' Avoid unused imports such as 'org.kuali.rice.kns.util.UrlFactory' These nested if statements could be combined Avoid unused imports such as 'java.util.ArrayList' 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 Avoid unused private fields such as 'LOG'. 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 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 unused local variables such as 'principalId'. 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 imports such as 'org.apache.commons.lang.StringUtils' Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.KimEntityPrivacyPreferences' Avoid unused imports such as 'org.kuali.rice.kns.util.ObjectUtils' Avoid unused private fields such as 'RESTRICTED_DATA_MASK'. Avoid empty catch blocks Avoid unused private fields such as 'LOG'. No need to import a type that lives in the same package Avoid unused imports such as 'org.kuali.rice.core.config.SimpleConfig' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'org.kuali.rice.kew.user.UserUtils' Avoid unused local variables such as 'pk'. Avoid unused imports such as 'java.util.Map' Avoid unused method parameters such as 'nodeTypeName'. These nested if statements could be combined Avoid empty if statements Avoid unused method parameters such as 'attributes'. Avoid unused private fields such as 'DEFAULT_GROUP_DESCRIPTION'. Avoid unused private methods such as 'isBlank(Object)'. Avoid unused private fields such as 'DEFAULT_GROUP_DESCRIPTION'. Avoid unused private fields such as 'groupAttributes'. 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'. 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 '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 Avoid unused imports such as 'org.kuali.rice.kew.util.KEWConstants' Avoid unused private methods such as 'exportRuleDelegations(Element,RuleBaseValues)'. Avoid unused imports such as 'org.jdom.output.Format.TextMode' Avoid unused imports such as 'org.kuali.rice.kns.exception.ExportNotSupportedException' Avoid unused private methods such as 'dateMask()'. Avoid unused imports such as 'org.kuali.rice.kim.util.KimCommonUtils' Avoid unused imports such as 'org.kuali.rice.kim.bo.reference.EmploymentStatus' Avoid unused imports such as 'org.kuali.rice.kim.bo.reference.EmploymentType' Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.dto.KimEntityDefaultInfo' Avoid unused imports such as 'org.kuali.rice.kim.util.KimCommonUtils' 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 'org.kuali.rice.kim.bo.reference.dto.AddressTypeInfo' Avoid unused imports such as 'java.util.Set' Avoid unused imports such as 'org.apache.commons.lang.builder.EqualsBuilder' Avoid unused imports such as 'org.apache.commons.lang.builder.HashCodeBuilder' Avoid unused imports such as 'org.kuali.rice.kim.bo.Role' Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAbstractMemberImpl' Avoid unused imports such as 'java.sql.Date' Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' Avoid unused imports such as 'javax.persistence.FetchType' Avoid unused imports such as 'javax.persistence.JoinColumn' Avoid unused imports such as 'javax.persistence.ManyToOne' Avoid unused imports such as 'org.kuali.rice.kim.bo.reference.CitizenshipStatus' Avoid unused imports such as 'org.kuali.rice.kim.bo.reference.impl.CitizenshipStatusImpl' Avoid unused imports such as 'org.kuali.rice.kns.bo.PersistableBusinessObjectBase' Avoid unused imports such as 'org.kuali.rice.kns.bo.PersistableBusinessObjectBase' Avoid unused imports such as 'java.sql.Date' Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'org.kuali.rice.kns.bo.PersistableBusinessObjectBase' Avoid unused imports such as 'javax.persistence.Id' Avoid unused imports such as 'org.kuali.rice.kim.service.impl.UiDocumentServiceImpl' Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.dto.KimPrincipalInfo' Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' 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 'java.util.LinkedList' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'org.kuali.rice.core.lifecycle.Lifecycle' Avoid unused imports such as 'org.kuali.rice.kim.bo.entity.impl.KimPrincipalImpl' Avoid empty try blocks Avoid unused imports such as 'org.apache.ojb.broker.PersistenceBroker' Avoid unused imports such as 'org.apache.ojb.broker.PersistenceBrokerException' Avoid unused imports such as 'org.kuali.rice.kim.util.KimCommonUtils' 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'. 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 Avoid empty if statements Avoid unused imports such as 'org.kuali.rice.kns.lookup.Lookupable' Avoid unused imports such as 'java.util.Collections.sort' Avoid unused imports such as 'java.util.ListIterator' Avoid unused imports such as 'org.kuali.rice.kim.bo.group.dto.GroupInfo' Avoid unused imports such as 'org.kuali.rice.kim.bo.ui.KimAttributeDataComparator' Avoid unused local variables such as 'skipPropTypeCheck'. Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.GenericPermission' Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.GroupImpl' Avoid unused imports such as 'org.kuali.rice.kim.bo.types.impl.KimTypeImpl' Avoid unused private fields such as 'LOG'. Avoid unused local variables such as 'kimTypeId'. Avoid unused private methods such as 'setupAttributeRows()'. Avoid unused local variables such as 'attributeRows'. Overriding method merely calls super Avoid unused imports such as 'org.kuali.rice.kim.rule.ui.AddMemberRule' Avoid unused imports such as 'org.kuali.rice.kim.rule.ui.AddDelegationMemberRule' Avoid unused imports such as 'org.kuali.rice.kim.rule.ui.AddPersonDocumentRoleQualifierRule' Avoid unused imports such as 'org.kuali.rice.kim.bo.ui.PersonDocumentGroup' Avoid unused imports such as 'org.kuali.rice.kns.document.Document' Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' Avoid unused local variables such as 'i'. Avoid unused local variables such as 'document'. Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAttributes' Avoid unused imports such as 'org.kuali.rice.kim.document.IdentityManagementRoleDocument' Avoid unused imports such as 'org.kuali.rice.kim.util.KimConstants' Avoid unused imports such as 'org.kuali.rice.kim.bo.ui.KimDocumentRoleMember' Avoid unused imports such as 'org.kuali.rice.kim.bo.ui.RoleDocumentDelegation' Avoid unused local variables such as 'i'. These nested if statements could be combined Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'EXEC_INTERVAL_SECS'. Avoid unused private fields such as 'MAX_WRITE_QUEUE_SIZE'. These nested if statements could be combined 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 Avoid unused imports such as 'java.util.Iterator' Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' Avoid unused imports such as 'org.kuali.rice.kim.bo.role.impl.ResponsibilityAttributeDataImpl' Avoid unused imports such as 'org.kuali.rice.kim.bo.role.impl.RoleMemberAttributeDataImpl' Avoid unused imports such as 'org.kuali.rice.kim.bo.types.dto.KimTypeAttributeInfo' Avoid unused imports such as 'org.kuali.rice.kim.service.ResponsibilityUpdateService' Avoid unused imports such as 'org.kuali.rice.kns.service.BusinessObjectService' Avoid unused imports such as 'org.kuali.rice.kns.service.SequenceAccessorService' Avoid unused imports such as 'org.kuali.rice.ksb.cache.RiceCacheAdministrator' Avoid unused imports such as 'org.kuali.rice.core.exception.RiceRemoteServiceConnectionException' Avoid unused imports such as 'org.kuali.rice.core.resourceloader.GlobalResourceLoader' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' 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 'getPrincipalImpl(String)'. 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 imports such as 'java.util.List' Avoid unused imports such as 'org.kuali.rice.kim.bo.role.dto.DelegateInfo' 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 These nested if statements could be combined Avoid empty catch blocks Avoid empty catch blocks Avoid unused private fields such as 'LAST_ACCCESS_TIMEOUT_FIELD'. Avoid unused private fields such as 'kimTypeServiceCache'. 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 unused imports such as 'org.kuali.rice.kns.question.ConfirmationQuestion' Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.RoleImpl' Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.GroupImpl' Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' Avoid empty if statements These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'org.apache.commons.lang.StringUtils' Avoid unused private fields such as 'persistenceStructureService'. Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'java.util.Set' Avoid unused imports such as 'java.util.HashSet' Avoid unused imports such as 'org.apache.commons.lang.StringUtils' Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'javax.persistence.Column' Avoid unused imports such as 'javax.persistence.Id' Avoid unused imports such as 'javax.persistence.CascadeType' Avoid unused imports such as 'javax.persistence.Version' Avoid unused imports such as 'javax.persistence.CascadeType' 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 These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'javax.persistence.Version' Avoid unused imports such as 'javax.persistence.TemporalType' Avoid unused imports such as 'javax.persistence.Temporal' Avoid unused imports such as 'javax.persistence.Id' Avoid unused imports such as 'javax.persistence.CascadeType' Avoid unused private fields such as 'attachmentLink'. Avoid unused imports such as 'javax.persistence.Version' Avoid unused imports such as 'javax.persistence.CascadeType' 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 unused imports such as 'org.kuali.rice.kns.bo.ExternalizableBusinessObject' 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 private fields such as 'buildingCode'. Avoid unused private fields such as 'buildingRoomNumber'. Avoid unused imports such as 'javax.persistence.Version' Avoid unused imports such as 'javax.persistence.TemporalType' Avoid unused imports such as 'javax.persistence.Temporal' Avoid unused imports such as 'javax.persistence.Id' Avoid unused imports such as 'javax.persistence.CascadeType' Overriding method merely calls super Overriding method merely calls super Overriding method merely calls super 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 private fields such as 'LOG'. 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 unused imports such as 'java.util.HashMap' Avoid unused imports such as 'org.apache.commons.beanutils.BeanUtils' Avoid unused imports such as 'org.kuali.rice.kim.bo.Group' Avoid unused imports such as 'org.kuali.rice.kim.service.KIMServiceLocator' Avoid unused imports such as 'org.kuali.rice.kns.bo.AdHocRoutePerson' Avoid unused imports such as 'org.kuali.rice.kns.bo.AdHocRouteWorkgroup' Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'org.kuali.rice.kim.bo.Group' Avoid unused imports such as 'org.kuali.rice.kim.service.KIMServiceLocator' Avoid unused imports such as 'org.kuali.rice.kns.bo.AdHocRoutePerson' Avoid unused imports such as 'org.kuali.rice.kns.bo.AdHocRouteWorkgroup' Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'java.sql.Date' Avoid unused imports such as 'org.apache.ojb.broker.query.QueryFactory' Avoid empty catch blocks Avoid unused imports such as 'java.sql.Date' Avoid unused imports such as 'org.kuali.rice.kns.bo.DocumentHeader' Avoid unused private fields such as 'LOG'. 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 'javax.persistence.EntityManager' Avoid unused imports such as 'javax.persistence.PersistenceContext' 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 private fields such as 'LOG'. Avoid unused private fields such as 'DOCUMENT_AD_HOC_SERVICE_NAME'. Avoid unused imports such as 'org.kuali.rice.core.config.ConfigurationException' Avoid unused imports such as 'org.kuali.rice.kns.dao.impl.PersistenceDaoJpa' Avoid unused local variables such as 'entityManager'. Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'org.apache.commons.lang.StringUtils' Avoid unused imports such as 'org.apache.commons.logging.LogFactory' Avoid unused imports such as 'org.kuali.rice.kew.attribute.Attribute' Avoid unused imports such as 'java.util.Collections' Avoid unused imports such as 'java.util.HashSet' Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.exception.CompletionException' Avoid unused imports such as 'org.kuali.rice.kns.service.ModuleService' Avoid unused local variables such as 'indexName'. These nested if statements could be combined Avoid empty catch blocks 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 Avoid unused imports such as 'org.apache.commons.lang.StringUtils' 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.kns.util.KNSConstants' An empty statement (semicolon) not part of a loop Ensure you override both equals() and hashCode() Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.mask.MaskFormatterLiteral' Avoid unused private fields such as 'LOG'. Avoid unused local variables such as 'regexString'. Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'org.kuali.rice.kns.exception.ValidationException' 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 Avoid unused private fields such as 'LOG'. These nested if statements could be combined Avoid empty catch blocks Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.PersonImpl' Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'java.util.List' Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'java.util.Set' Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAttributes' Avoid unused imports such as 'org.kuali.rice.kim.util.KimConstants' Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' Avoid unused imports such as 'java.util.Set' Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' 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.kns.bo.SelectedObjectIds' Avoid unused imports such as 'org.kuali.rice.kns.exception.AuthorizationException' Avoid unused private methods such as 'getBusinessObjectAuthorizationService()'. Avoid unused local variables such as 'elementClass'. Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'org.kuali.rice.kns.bo.SelectedObjectIds' Avoid unused imports such as 'org.kuali.rice.kns.bo.PersistableBusinessObject' Avoid unused imports such as 'org.kuali.rice.kns.util.KNSPropertyConstants' Avoid unused private fields such as 'kualiConfigurationService'. Avoid empty catch blocks 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'. Avoid unused imports such as 'org.kuali.rice.kns.service.RiceApplicationConfigurationService' Avoid unused private fields such as 'parameterService'. Avoid unused private fields such as 'parameterService'. These nested if statements could be combined Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'org.kuali.rice.kns.bo.SelectedObjectIds' Avoid unused imports such as 'org.kuali.rice.kns.exception.AuthorizationException' Avoid unused constructor parameters such as 'parameterNamesapce'. 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 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 Avoid unused local variables such as 'fieldValues'. These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kns.util.ObjectUtils' Avoid unused imports such as 'org.kuali.rice.kns.util.ObjectUtils' Avoid unused imports such as 'org.kuali.rice.kns.util.ObjectUtils' Avoid unused private methods such as 'logEvent()'. Avoid unused imports such as 'org.kuali.rice.kns.document.Document' Avoid unused local variables such as 'docOrBoClass'. Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'org.kuali.rice.kim.bo.types.dto.AttributeSet' Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.mask.Mask' Avoid unused imports such as 'java.util.Collection' Avoid unused imports such as 'org.kuali.rice.kns.exception.ClassNotPersistableException' Avoid unused imports such as 'org.kuali.rice.kns.exception.ClassNotPersistableException' Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' An empty statement (semicolon) not part of a loop Avoid unused imports such as 'org.kuali.rice.kns.dao.NoteDao' 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 Avoid unused local variables such as 'namespaceCode'. Avoid unused local variables such as 'componentName'. Avoid unused local variables such as 'propertyName'. 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 'attributeRefName'. These nested if statements could be combined Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid empty catch blocks Avoid unused method parameters such as 'refFieldName'. 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 catch blocks Avoid empty catch blocks 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'. Avoid unused imports such as 'org.apache.commons.lang.StringUtils' Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' Avoid unused imports such as 'org.kuali.rice.kew.util.Utilities' Avoid unused imports such as 'org.kuali.rice.kim.service.KIMServiceLocator' Avoid unused imports such as 'org.kuali.rice.kns.datadictionary.mask.Mask' 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 methods such as 'checkParameterArgument(Parameter,String)'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'EMPTY_COLLECTION'. Avoid unused private fields such as 'dataDictionaryService'. Avoid unused private fields such as 'persistenceStructureService'. Avoid unused imports such as 'java.util.Arrays' Avoid unused imports such as 'java.util.Collections' Avoid unused imports such as 'java.util.Iterator' Avoid unused imports such as 'java.util.Set' These nested if statements could be combined Avoid unused method parameters such as 'businessObject'. These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'LOG'. Avoid unused imports such as 'org.apache.log4j.Logger' Avoid unused local variables such as 'key'. Avoid unused imports such as 'org.apache.log4j.Logger' Avoid unused imports such as 'org.apache.log4j.Logger' Avoid unused imports such as 'org.kuali.rice.kns.util.KNSConstants' Avoid unused private fields such as 'LOG'. These nested if statements could be combined Avoid unused local variables such as 'refPkNames'. Avoid unused imports such as 'org.kuali.rice.kns.service.KualiModuleService' These nested if statements could be combined Avoid unused imports such as 'javax.persistence.CascadeType' Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'java.util.Arrays' Avoid unused imports such as 'java.util.Collections' Avoid unused imports such as 'org.apache.commons.collections.ListUtils' Avoid unused private fields such as 'dateTimeService'. These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'difference'. Avoid unused local variables such as 'difference'. Avoid empty catch blocks Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' Avoid unused imports such as 'org.kuali.rice.kns.service.KualiModuleService' Avoid unused imports such as 'org.kuali.rice.kns.service.ModuleService' Avoid unused imports such as 'java.util.Set' Avoid unused imports such as 'org.kuali.rice.kns.web.format.Formatter' 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'. Avoid unused imports such as 'java.util.LinkedList' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'org.kuali.rice.kew.engine.RouteContext' 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.WorkflowException' Avoid unused imports such as 'java.util.Collection' Avoid unused imports such as 'java.util.Iterator' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'java.util.ListIterator' No need to import a type that lives in the same package Avoid unused local variables such as 'entryKey'. These nested if statements could be combined These nested if statements could be combined 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 Avoid unused private fields such as 'S_TRUE'. Avoid unused private fields such as 'S_FALSE'. Avoid empty catch blocks Avoid empty catch blocks Avoid unused method parameters such as 'origSource'. 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 These nested if statements could be combined Avoid unused local variables such as 'params'. Avoid unused local variables such as 'files'. Avoid unused method parameters such as 'request'. Avoid unused private methods such as 'getMultipartHandler(HttpServletRequest,ActionForm)'. Avoid empty if statements These nested if statements could be combined These nested if statements could be combined Avoid empty catch blocks Avoid unused local variables such as 'copySize'. Avoid empty catch blocks Avoid unused constructor parameters such as 'broker'. Avoid unused constructor parameters such as 'prop'. Avoid unused local variables such as 'cache'. Avoid unused imports such as 'java.util.Collection' Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' Avoid empty if statements These nested if statements could be combined Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'java.util.Collection' Avoid unused imports such as 'org.kuali.rice.kns.bo.BusinessObject' Avoid unused imports such as 'org.kuali.rice.kns.exception.PropertiesException' Avoid unused imports such as 'org.kuali.rice.kns.exception.PropertiesException' Avoid empty catch blocks Avoid unused imports such as 'org.springframework.transaction.annotation.Transactional' 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.core.config.SimpleConfig' Avoid unused imports such as 'java.util.Collection' No need to import a type that lives in the same package These nested if statements could be combined Avoid empty catch blocks Avoid unused imports such as 'java.util.Arrays' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'org.kuali.rice.kns.util.RiceKeyConstants' 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 Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused method parameters such as 'form'. 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'. Avoid unused imports such as 'org.kuali.rice.kns.exception.ValidationException' Avoid unused imports such as 'org.kuali.rice.kns.rule.event.SaveDocumentEvent' Avoid unnecessary temporaries when converting primitives to Strings These nested if statements could be combined Avoid empty catch blocks Avoid unused imports such as 'org.kuali.rice.kew.service.KEWServiceLocator' 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 unused imports such as 'org.kuali.rice.kns.datadictionary.BusinessObjectEntry' Avoid empty if statements These nested if statements could be combined Avoid unused method parameters such as 'request'. Avoid unused local variables such as 'refreshCaller'. 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 local variables such as 'refreshCaller'. Avoid unused imports such as 'java.util.Properties' 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 These nested if statements could be combined Avoid unused method parameters such as 'response'. Avoid unused imports such as 'java.util.Map' No need to import a type that lives in the same package Avoid unused private fields such as 'targetName'. Avoid unused imports such as 'org.kuali.rice.kns.service.DataDictionaryService' Avoid unused private fields such as 'inactiveRecordDisplay'. 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 'docTypeName'. 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 unused imports such as 'org.kuali.rice.kns.web.format.EncryptionFormatter' Avoid empty catch blocks Avoid unused method parameters such as 'requestParams'. Avoid unused private methods such as 'getAllParametersForMultipartRequest(HttpServletRequest,MultipartRequestHandler)'. Avoid unused imports such as 'org.kuali.rice.kns.service.KNSServiceLocator' Avoid unused imports such as 'org.kuali.rice.kns.util.ExternalizableBusinessObjectUtils' Avoid empty catch blocks Avoid unused imports such as 'org.apache.struts.taglib.TagUtils' Avoid unused imports such as 'org.apache.struts.taglib.TagUtils' Avoid unused imports such as 'org.apache.struts.taglib.TagUtils' 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 methods such as 'getBusinessObjectAuthorizationService()'. Avoid unused local variables such as 'row'. These nested if statements could be combined 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 imports such as 'org.kuali.rice.kew.dto.UserIdDTO' Avoid unused imports such as 'org.kuali.rice.kew.exception.InvalidActionTakenException' Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' Avoid unused imports such as 'org.kuali.rice.kew.exception.ResourceUnavailableException' Avoid unused imports such as 'org.kuali.rice.kew.exception.InvalidActionTakenException' 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 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 unused imports such as 'java.util.Properties' Avoid unused imports such as 'org.apache.log4j.Logger' Avoid unused imports such as 'org.apache.ws.security.components.crypto.Crypto' Avoid unused imports such as 'org.apache.ws.security.components.crypto.Merlin' Avoid unused imports such as 'org.apache.ws.security.handler.RequestData' Avoid unused imports such as 'org.apache.ws.security.handler.WSHandlerConstants' Avoid unused imports such as 'org.kuali.rice.core.config.ConfigContext' Avoid unused imports such as 'org.kuali.rice.core.exception.RiceRuntimeException' Avoid unused imports such as 'org.kuali.rice.core.util.ClassLoaderUtils' Avoid unused imports such as 'org.kuali.rice.ksb.config.wss4j.CryptoPasswordCallbackHandler' Avoid unused imports such as 'org.kuali.rice.ksb.messaging.ServiceInfo' Avoid unused imports such as 'java.util.Properties' Avoid unused imports such as 'org.apache.log4j.Logger' Avoid unused imports such as 'org.apache.ws.security.components.crypto.Crypto' Avoid unused imports such as 'org.apache.ws.security.components.crypto.Merlin' Avoid unused imports such as 'org.apache.ws.security.handler.RequestData' Avoid unused imports such as 'org.apache.ws.security.handler.WSHandlerConstants' Avoid unused imports such as 'org.kuali.rice.core.config.ConfigContext' Avoid unused imports such as 'org.kuali.rice.core.exception.RiceRuntimeException' Avoid unused imports such as 'org.kuali.rice.core.util.ClassLoaderUtils' Avoid unused imports such as 'org.kuali.rice.ksb.config.wss4j.CryptoPasswordCallbackHandler' Avoid unused imports such as 'org.kuali.rice.ksb.messaging.ServiceInfo' Overriding method merely calls super 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. Overriding method merely calls super These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'java.net.URLEncoder' Avoid unused local variables such as 'i'. 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.core.util.OrmUtils' 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. 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 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 instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused private fields such as 'serviceInfo'. Avoid unused private fields such as 'serviceLocator'. Avoid unused local variables such as 'server'. Avoid unused imports such as 'org.kuali.rice.core.config.Config' 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 'serviceInfo'. Avoid unused private fields such as 'LOG'. Avoid unused private fields such as 'urlPathHelper'. Avoid empty catch blocks Avoid unused imports such as 'org.kuali.rice.kim.bo.Person' Avoid unused imports such as 'org.kuali.rice.kim.bo.impl.KimAttributes' Avoid unused local variables such as 'routeQueueId'. These nested if statements could be combined Don't create instances of already existing BigInteger and BigDecimal (ZERO, ONE, TEN) 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 An empty statement (semicolon) not part of a loop Avoid empty catch blocks Overriding method merely calls super Avoid empty catch blocks Avoid unused imports such as 'javax.persistence.CascadeType' Avoid unused imports such as 'uk.ltd.getahead.dwr.util.LoggingOutput' Avoid unused private fields such as 'rows'.