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 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 Avoid empty catch blocks Avoid empty if statements Avoid empty catch blocks 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 method parameters such as 'objStructure'. Avoid unused method parameters such as 'valResults'. Avoid unused method parameters such as 'valResults'. Avoid unused private methods such as 'processTypeStateCaseConstraint(List)'. These nested if statements could be combined Avoid unused method parameters such as 'bcb'. 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 methods such as '_toString()'. 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 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 Avoid empty catch blocks Avoid unused private methods such as 'calcNotSoSimpleName(String)'. 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 These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'BASE_KUALI_TYPE'. Avoid unused private fields such as 'BASE_KUALI_STATE'. Avoid unused private fields such as 'BASE_KUALI_EFFECTIVE_DATE'. Avoid unused private fields such as 'BASE_KUALI_EXPIRATION_DATE'. Avoid unused method parameters such as 'processed'. Avoid unused local variables such as 'subClazz'. 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 'and'. These nested if statements could be combined These nested if statements could be combined Avoid unused method parameters such as 'attribute'. Avoid unused method parameters such as 'attribute'. Avoid unused method parameters such as 'ac'. These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'pattern'. Avoid unused local variables such as 'i'. Avoid unused method parameters such as 'dataType'. Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid unused private fields such as 'os'. Avoid unused private methods such as 'pad(String,int)'. Avoid unused private methods such as 'escapeWiki(String)'. Avoid unused private fields such as 'searchInfoTypeMap'. Avoid unused private fields such as 'queryMap'. Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid unused imports such as 'org.kuali.student.common.ui.client.demo.HeaderDemo' Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.table.scroll.TableDemoPanel' 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 Do not use if statements that are always true or always false Overriding method merely calls super These nested if statements could be combined These nested if statements could be combined Avoid unused constructor parameters such as 'style'. Avoid unused private fields such as 'fieldMap'. 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 imports such as 'com.google.gwt.user.client.Window' Overriding method merely calls super Avoid unused private fields such as 'title'. Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.FieldDescriptor' Avoid unused imports such as 'com.google.gwt.user.client.ui.Hidden' Avoid unused private fields such as 'timer'. Avoid empty catch blocks Too many static imports may lead to messy code Avoid unused imports such as 'com.google.gwt.event.dom.client.BlurEvent' Avoid unused imports such as 'com.google.gwt.event.dom.client.FocusEvent' Avoid unused private fields such as 'PANEL_CONTENT_REMOVE_LINK'. Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyDownEvent' Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyDownHandler' Avoid unused private fields such as 'mouseOver'. Avoid unused private fields such as 'appUrl'. Avoid unused private fields such as 'title'. Avoid unused private fields such as 'title'. Avoid unused private fields such as 'margin'. Avoid empty if statements Avoid unused constructor parameters such as 'fieldTitle'. Avoid unused constructor parameters such as 'widget'. Avoid unused constructor parameters such as 'fieldTitle'. Avoid unused constructor parameters such as 'key'. Avoid unused constructor parameters such as 'widget'. Avoid unused private fields such as 'lineCount'. Avoid unused private fields such as 'fieldArea'. Avoid unused private fields such as 'top'. Avoid unused private fields such as 'buttonPanel'. These nested if statements could be combined Avoid unused private fields such as 'allowedKeyPress'. Avoid unused private fields such as 'hasWatermark'. Avoid unused private fields such as 'watermarkShowing'. Avoid unused private fields such as 'watermarkText'. 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 'w'. 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 'ids'. Avoid unused private fields such as 'buttonLabel'. Avoid unused local variables such as 'value'. These nested if statements could be combined Avoid unused private fields such as 'key'. 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 An empty statement (semicolon) not part of a loop Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. The null check here is misplaced; if the variable is null therell be a NullPointerException The null check here is misplaced; if the variable is null therell be a NullPointerException Avoid unused method parameters such as 'searchId'. Avoid unused method parameters such as 'query'. Avoid empty if statements Avoid unused private methods such as 'createSummaryTableFromDataModel()'. 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 constructor parameters such as 'style'. Overriding method merely calls super Avoid unused imports such as 'org.kuali.student.core.exceptions.MissingParameterException' Avoid duplicate imports such as 'org.kuali.rice.kim.service.IdentityManagementService' Avoid unused method parameters such as 'request'. These nested if statements could be combined Avoid unused imports such as 'java.util.Properties' Avoid unused imports such as 'org.kuali.student.common.messagebuilder.impl.exceptions.MessageBuilderException' An empty statement (semicolon) not part of a loop Avoid empty finally blocks Avoid empty finally blocks Avoid empty finally blocks Avoid empty finally blocks Avoid empty finally blocks Avoid empty finally blocks Avoid empty finally blocks Avoid empty finally blocks Avoid empty finally blocks These nested if statements could be combined Avoid empty finally blocks Avoid empty finally blocks Avoid empty finally blocks Avoid unused local variables such as 'LP9'. Avoid unused local variables such as 'RP11'. Avoid unused local variables such as 'LP9_tree'. Avoid unused local variables such as 'RP11_tree'. Avoid empty finally blocks Avoid unused imports such as 'org.kuali.student.common.messagebuilder.impl.exceptions.MessageBuilderException' 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 'java.util.ArrayList' Avoid unused imports such as 'java.util.Date' Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'java.util.Map' Avoid unused imports such as 'javax.xml.bind.annotation.XmlAttribute' Avoid unused imports such as 'javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter' Avoid unused imports such as 'org.kuali.student.core.dto.MetaInfo' Avoid unused imports such as 'org.kuali.student.core.dto.RichTextInfo' Avoid unused imports such as 'org.kuali.student.core.ws.binding.JaxbAttributeMapListAdapter' Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'org.kuali.student.core.dto.MetaInfo' Avoid unused imports such as 'java.util.ArrayList' Avoid unused imports such as 'java.util.List' Avoid unused imports such as 'org.kuali.student.core.exceptions.CircularRelationshipException' Overriding method merely calls super Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Overriding method merely calls super Overriding method merely calls super An empty statement (semicolon) not part of a loop Avoid unused imports such as 'org.kuali.student.core.exceptions.DoesNotExistException' Overriding method merely calls super Avoid unused method parameters such as 'value'. 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.kim.service.IdentityService' Overriding method merely calls super Overriding method merely calls super Avoid unused private fields such as 'statementTypeId'. Avoid unused private fields such as 'reqComponentTypeId'. Avoid unused private fields such as 'statementTypeId'. Avoid unused private fields such as 'childStatementTypeId'. Avoid unused imports such as 'javax.persistence.ManyToOne' Avoid unused imports such as 'java.util.HashSet' Avoid unused imports such as 'java.util.Set' Avoid unused imports such as 'org.kuali.student.core.exceptions.DoesNotExistException' Avoid unused imports such as 'org.kuali.student.core.statement.dto.ReqCompFieldTypeInfo' Avoid unused imports such as 'org.kuali.student.core.statement.dto.ReqComponentTypeInfo' Avoid unused imports such as 'org.kuali.student.core.statement.service.StatementService' Avoid empty catch blocks Do not use if statements that are always true or always false Avoid empty catch blocks Avoid empty catch blocks Avoid unused private fields such as 'viewEnum'. Avoid unused private fields such as 'viewName'. Avoid unused private fields such as 'referenceAttributes'. Avoid unused private fields such as 'buttonPanel'. Avoid empty if statements Avoid unused method parameters such as 'deletionParentKeys'. Avoid unused method parameters such as 'swappableFieldsDefinition'. Avoid unused private fields such as 'NO_SELECTION_TEXT'. Avoid empty catch blocks Avoid unused private fields such as 'ruleChanged'. Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSPlaceholder' Avoid unused private fields such as 'separator1'. Avoid unused private fields such as 'separator2'. Avoid unused private fields such as 'isOperatorChecked'. These nested if statements could be combined These nested if statements could be combined Avoid unused local variables such as 'token'. 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 'getDeeperNode(List)'. Avoid unused private methods such as 'findNodeIndex(List,int)'. Avoid unused private methods such as 'hasParenthesis(List)'. Avoid unused local variables such as 'rf'. Avoid unused imports such as 'org.kuali.student.core.proposal.ProposalConstants' Avoid unused private fields such as 'titlePanel'. Avoid unused private fields such as 'title'. Avoid unused private fields such as 'proposalNameHeader'. Avoid unused private fields such as 'decisionTypeKey'. Avoid unused local variables such as 'columnIndex'. Too many static imports may lead to messy code Avoid unused local variables such as 'validationResults'. Avoid empty if statements Avoid empty if statements Avoid unused local variables such as 'result'. Too many static imports may lead to messy code These nested if statements could be combined Too many static imports may lead to messy code Avoid unused local variables such as 'validationResults'. Avoid unused local variables such as 'result'. Avoid empty if statements Avoid empty if statements Too many static imports may lead to messy code Avoid unused local variables such as 'positions'. Avoid unused local variables such as 'relations'. Avoid unused local variables such as 'validationResults'. Avoid unused local variables such as 'relationData'. Avoid unused local variables such as 'positionData'. Avoid unused local variables such as 'relationData'. These nested if statements could be combined Avoid unused private methods such as 'copyIfExists(Data,T,String)'. Avoid unused private fields such as 'WITH_DIVIDER'. Avoid unused private methods such as 'initSection(SectionTitle,boolean)'. Avoid unused private methods such as 'getH3Title(String)'. Avoid unused private fields such as 'orgPosTypeList'. Avoid unused local variables such as 'lbl'. Avoid unused local variables such as 'orgName'. These nested if statements could be combined Avoid duplicate imports such as 'org.kuali.rice.kim.service.IdentityManagementService' Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.ModelChangeEvent' Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.ModelChangeHandler' Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.ModelChangeEvent.Action' 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.ArrayList' Avoid unused imports such as 'java.util.Date' Avoid unused imports such as 'java.util.Date' 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 empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks These nested if statements could be combined 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 empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid unused local variables such as 'clu'. Avoid unused imports such as 'org.kuali.student.common.util.MessageUtils' Overriding method merely calls super Avoid unused private methods such as 'getIncludingLos(String)'. Avoid empty catch blocks Avoid empty catch blocks Overriding method merely calls super Overriding method merely calls super 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 empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid unused local variables such as 'credentialClu'. 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 empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid empty catch blocks Avoid unused imports such as 'org.kuali.student.core.exceptions.InvalidParameterException' Avoid unused imports such as 'org.kuali.student.core.exceptions.MissingParameterException' Avoid unused imports such as 'org.kuali.student.core.exceptions.OperationFailedException' Avoid empty catch blocks Avoid unused local variables such as 'statement'. Avoid empty catch blocks These nested if statements could be combined Avoid empty catch blocks Avoid empty catch blocks Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' Avoid unused imports such as 'org.kuali.student.lum.program.client.major.edit.MajorEditController' Avoid unused private fields such as 'id'. Avoid unused imports such as 'org.kuali.student.lum.program.client.ProgramStatus' Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' Avoid unused private fields such as 'originalReqCompNL'. Avoid unused private fields such as 'originalLogicExpression'. Avoid unnecessary return statements Overriding method merely calls super Avoid unused private fields such as 'majorController'. Avoid unused method parameters such as 'data'. Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityConfiguration' Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.MultiplicitySection' Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.Section' Avoid unused imports such as 'org.kuali.student.core.assembly.data.Metadata' Avoid unused imports such as 'org.kuali.student.core.assembly.data.QueryPath' Avoid unused imports such as 'org.kuali.student.lum.program.client.widgets.EditableHeader' Avoid unused imports such as 'org.kuali.student.core.assembly.data.ModelDefinition' Avoid unused private fields such as 'initialized'. Avoid unused private fields such as 'viewContext'. Avoid unused method parameters such as 'versionaLabel'. Avoid unused private methods such as 'createHistoryPanel()'. Avoid unused private methods such as 'getStatus()'. Avoid unused private fields such as 'metadataService'. Avoid unused private fields such as 'mapper'. Avoid unused private fields such as 'credPgmMetadata'. 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 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 unused imports such as 'javax.xml.namespace.QName' Avoid unused imports such as 'org.kuali.rice.core.resourceloader.GlobalResourceLoader' Avoid unused imports such as 'org.kuali.rice.kew.role.QualifierResolver' These nested if statements could be combined Avoid unused imports such as 'java.util.HashMap' Avoid unused imports such as 'org.kuali.student.core.assembly.data.Data.StringKey' Avoid unused imports such as 'java.util.Date' Avoid unused constructor parameters such as 'metadata'. Avoid unused imports such as 'java.util.Date' Avoid unused imports such as 'com.google.gwt.user.client.rpc.AsyncCallback' Overriding method merely calls super Avoid unused local variables such as 'name'. Avoid unused local variables such as 'typeKey'. Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyUpEvent' Avoid unused imports such as 'com.google.gwt.event.dom.client.KeyUpHandler' Avoid unused imports such as 'com.google.gwt.user.client.Window' Avoid unused imports such as 'com.google.gwt.user.client.ui.TextArea' Avoid unused imports such as 'org.kuali.student.core.dto.MetaInfo' No need to import a type that lives in the same package Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' 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. Ensure you override both equals() and hashCode() Avoid empty if statements Avoid unused imports such as 'java.util.Set' Avoid unused imports such as 'com.google.gwt.user.client.Window' 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 'searchParams'. 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. These nested if statements could be combined Avoid unused method parameters such as 'enabled'. Avoid unused method parameters such as 'enabled'. Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.layout.VerticalFlowPanel' Avoid unused private fields such as 'searchRpcServiceAsync'. Avoid unused imports such as 'org.kuali.student.common.ui.client.configurable.mvc.sections.SwapSection' Avoid unused imports such as 'org.kuali.student.common.ui.client.event.SectionUpdateEvent' Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' Avoid unused imports such as 'org.kuali.student.lum.program.dto.MajorDisciplineInfo' Avoid unused local variables such as 'a'. These nested if statements could be combined These nested if statements could be combined Avoid unused imports such as 'com.google.gwt.user.client.ui.Label' Avoid unused imports such as 'com.google.gwt.user.client.ui.VerticalPanel' Avoid unused constructor parameters such as 'fieldWidget'. Avoid unused constructor parameters such as 'modelWidgetBinding'. These nested if statements could be combined These nested if statements could be combined Avoid unused private fields such as 'STATEMENTS_PATH'. Avoid unused private fields such as 'ID_TRANSLATION'. Overriding method merely calls super Avoid unused private fields such as 'lastId1'. Avoid unused private fields such as 'lastId2'. Avoid unused method parameters such as 'id1Model'. Avoid unused local variables such as 'idType'. Avoid unused local variables such as 'viewContextId'. Avoid unused private methods such as 'getCurrentVersion(ModelRequestCallback,Callback)'. Avoid unused private methods such as 'getQuitButton()'. Avoid unused private methods such as 'createActionSubmitSuccessHandler()'. Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.breadcrumb.BreadcrumbManager' Avoid unused imports such as 'org.kuali.student.common.ui.client.mvc.history.HistoryManager' Avoid unused private fields such as 'statementRpcServiceAsync'. Avoid unused private fields such as 'originalReqCompNL'. Avoid unused private fields such as 'originalLogicExpression'. Avoid unused imports such as 'org.kuali.student.common.ui.client.util.WindowTitleUtils' Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.headers.KSDocumentHeader' Avoid unused constructor parameters such as 'controller'. Avoid unused constructor parameters such as 'modelId'. Avoid unused constructor parameters such as 'name'. Avoid unused constructor parameters such as 'viewEnum'. Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSErrorDialog' Avoid unused private methods such as 'addIfPermitted(PermissionType,String)'. Avoid unused method parameters such as 'searchType'. Avoid empty if statements Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.widgets.CourseWorkflowActionList' Avoid empty if statements Avoid empty if statements Avoid unused method parameters such as 'newState'. Avoid unused constructor parameters such as 'onSubmitSuccessHandler'. Avoid empty if statements 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 'currVerPrevState'. Avoid unused imports such as 'org.kuali.student.lum.lo.dto.LoInfo' Avoid unused imports such as 'java.util.Date' Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.headers.KSHeader' Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSFooter' Avoid unused imports such as 'org.kuali.student.lum.lu.ui.course.client.controllers.CurriculumHomeController.LUMViews' Avoid unused private fields such as 'appUrl'. Avoid unused private fields such as 'codeServer'. Avoid unused private methods such as 'getLabel(String)'. Avoid unused private fields such as 'container'. Overriding method merely calls super Avoid unused local variables such as 'cluRangeFieldEditDescriptor'. Avoid unused private methods such as 'createCluSetSection()'. Avoid unused private methods such as 'editCluSetSection()'. Avoid unused private methods such as 'viewCluSetSection()'. Avoid unused method parameters such as 'modelDefinition'. 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 Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.layout.VerticalFlowPanel' Avoid unused imports such as 'org.kuali.student.common.ui.client.widgets.KSErrorDialog' Avoid unused imports such as 'org.kuali.student.common.ui.shared.IdAttributes' Avoid unused imports such as 'org.kuali.student.core.assembly.data.AssemblyException' Avoid unused imports such as 'org.kuali.student.core.assembly.transform.MetadataFilter' Avoid unused imports such as 'org.kuali.student.core.assembly.transform.TransformFilter' Avoid instantiating Boolean objects; reference Boolean.TRUE or Boolean.FALSE or call Boolean.valueOf() instead. These nested if statements could be combined Overriding method merely calls super Overriding method merely calls super Avoid unused imports such as 'org.apache.xpath.XPathAPI' Avoid unused imports such as 'org.w3c.dom.NodeList' Avoid unused local variables such as 'principal'. Avoid empty if statements Avoid unused imports such as 'java.io.StringWriter' Avoid unused imports such as 'javax.xml.transform.Transformer' Avoid unused imports such as 'javax.xml.transform.TransformerFactory' Avoid unused imports such as 'javax.xml.transform.dom.DOMSource' Avoid unused imports such as 'javax.xml.transform.stream.StreamResult'