No need to import a type that lives in the same package Avoid unused imports such as 'org.kuali.rice.kim.impl.role.RoleBo' Avoid unused imports such as 'org.kuali.rice.kim.impl.role.RoleDao' Avoid unused imports such as 'org.kuali.rice.kim.impl.services.KimImplServiceLocator' Avoid unused imports such as 'org.kuali.rice.krad.util.KRADConstants' Avoid duplicate imports such as 'org.kuali.rice.core.api.criteria.PredicateFactory.and' Avoid duplicate imports such as 'org.kuali.rice.core.api.criteria.PredicateFactory.lessThan' Avoid unused private fields such as 'criteriaLookupService'. These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kim.api.identity.principal.EntityNamePrincipalName' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'java.util.Map' 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.core.api.criteria.Predicate' Avoid unused imports such as 'org.kuali.rice.core.api.criteria.PredicateUtils' Avoid unused imports such as 'org.kuali.rice.core.api.criteria.QueryByCriteria' Avoid unused imports such as 'org.kuali.rice.kim.api.KimConstants' Avoid unused imports such as 'org.kuali.rice.kim.api.group.Group' Avoid unused imports such as 'org.kuali.rice.kim.api.group.GroupQueryResults' Avoid unused imports such as 'org.kuali.rice.kim.api.group.GroupService' Avoid unused imports such as 'org.kuali.rice.kim.impl.group.GroupBo' Avoid unused imports such as 'java.sql.Timestamp' Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'java.util.Calendar' Avoid unused imports such as 'java.util.HashMap' Avoid unused local variables such as 'externalIdentifierCriteria'. These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' These nested if statements could be combined Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' Avoid unused method parameters such as 'namespaceCode'. Avoid unused method parameters such as 'namespaceCode'. Avoid unused method parameters such as 'qualification'. Avoid unused imports such as 'org.kuali.rice.kim.api.responsibility.Responsibility' Avoid unused imports such as 'org.kuali.rice.kim.impl.common.attribute.KimAttributeDataBo' Avoid unused imports such as 'org.kuali.rice.kim.impl.permission.PermissionTemplateBo' Avoid unused imports such as 'org.kuali.rice.kim.api.role.RoleMember' Avoid unused imports such as 'org.kuali.rice.kim.api.role.RoleMembership' Avoid unused imports such as 'org.kuali.rice.kim.api.role.Role' Avoid unused imports such as 'org.kuali.rice.kim.api.role.RoleMember' Avoid unused imports such as 'org.kuali.rice.kim.api.role.RoleMembership' Avoid unused private methods such as 'getPrincipalIdsForPrincipalName(String)'. Avoid unused private methods such as 'hasCoreRoleMemberCriteria(Map)'. Avoid unused private methods such as 'hasExtraRoleMemberCriteria(Map)'. Avoid unused private methods such as 'getRoleMembersRoles(String,String)'. Avoid unused private methods such as 'getRoleMembersGroupIds(String,String)'. These nested if statements could be combined Avoid unused imports such as 'org.joda.time.DateTime' Avoid unused imports such as 'org.kuali.rice.kim.api.responsibility.Responsibility' Avoid unused imports such as 'org.kuali.rice.kim.impl.responsibility.ResponsibilityAttributeBo' Avoid unused imports such as 'org.kuali.rice.kim.impl.responsibility.ResponsibilityBo' These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'delegateMember'. Avoid unused private methods such as 'deleteNullMemberAttributeData(List)'. Avoid unused private methods such as 'deleteNullDelegationMemberAttributeData(List)'.