The following document contains the results of FindBugs Report
FindBugs Version is 1.3.9
Threshold is medium
Effort is min
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Possible null pointer dereference of cluSetInfo in org.kuali.student.lum.lu.assembly.CluSetManagementAssembler.upWrap(CluSetInfo) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 199 | Medium |
| Redundant nullcheck of saveValidationResults which is known to be null in org.kuali.student.lum.lu.assembly.CluSetManagementAssembler.saveCluSet(Data) | STYLE | RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE | 299 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to a in org.kuali.student.lum.lu.assembly.CluSetManagementIdTranslatorAssemblerFilter.translateIds(Data, AssemblerFilter$AssemblerManagerAccessable) | STYLE | DLS_DEAD_LOCAL_STORE | 60 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Field CourseConfigurer.groupName masks field in superclass org.kuali.student.common.ui.client.configurable.mvc.Configurer | CORRECTNESS | MF_CLASS_MASKS_FIELD | 113 | High |
| Field CourseConfigurer.modelDefinition masks field in superclass org.kuali.student.common.ui.client.configurable.mvc.Configurer | CORRECTNESS | MF_CLASS_MASKS_FIELD | 113 | Medium |
| Field CourseConfigurer.state masks field in superclass org.kuali.student.common.ui.client.configurable.mvc.Configurer | CORRECTNESS | MF_CLASS_MASKS_FIELD | 113 | Medium |
| Field CourseConfigurer.type masks field in superclass org.kuali.student.common.ui.client.configurable.mvc.Configurer | CORRECTNESS | MF_CLASS_MASKS_FIELD | 113 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.lum.lu.ui.course.client.configuration.CourseConfigurer$PersonList be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 775 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.lum.lu.ui.course.client.configuration.CourseConfigurer$ProposerPersonList be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 822 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.lu.ui.course.client.configuration.CourseSummaryConfigurer$8.setWidgetValue(HTML, DataModel, String) ignores return value of String.replace(CharSequence, CharSequence) | CORRECTNESS | RV_RETURN_VALUE_IGNORED | 829 | High |
| org.kuali.student.lum.lu.ui.course.client.configuration.CourseSummaryConfigurer$8.setWidgetValue(HTML, DataModel, String) ignores return value of String.trim() | CORRECTNESS | RV_RETURN_VALUE_IGNORED | 830 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to idType in org.kuali.student.lum.lu.ui.course.client.controllers.ViewCourseController.init(Callback) | STYLE | DLS_DEAD_LOCAL_STORE | 175 | Medium |
| Unread field: org.kuali.student.lum.lu.ui.course.client.controllers.ViewCourseController.REFERENCE_TYPE; should this field be static? | PERFORMANCE | SS_SHOULD_BE_STATIC | 82 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Invocation of toString on Throwable.getStackTrace() in org.kuali.student.lum.lu.ui.course.client.requirements.CourseRequirementsDataModel.getStmtTypeInfo(String) | CORRECTNESS | DMI_INVOKING_TOSTRING_ON_ARRAY | 284 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.lum.lu.ui.course.client.requirements.CourseRequirementsDataModel$5.onSuccess(Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 184 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class name org.kuali.student.lum.lu.ui.course.client.requirements.CourseRequirementsDataModel$requirementState doesn't start with an upper case letter | BAD_PRACTICE | NM_CLASS_NAMING_CONVENTION | 46 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.lum.lu.ui.course.client.requirements.CourseRequirementsManageView.originalReqCompNL | PERFORMANCE | URF_UNREAD_FIELD | 173 | Medium |
| Unused field: org.kuali.student.lum.lu.ui.course.client.requirements.CourseRequirementsManageView.originalLogicExpression | PERFORMANCE | UUF_UNUSED_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Private method org.kuali.student.lum.lu.ui.course.client.views.CurriculumHomeView.addIfPermitted(PermissionType, String) is never called | PERFORMANCE | UPM_UNCALLED_PRIVATE_METHOD | 78 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.lum.lu.ui.course.client.views.SelectVersionsView.metadataServiceAsync | PERFORMANCE | URF_UNREAD_FIELD | 40 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.lum.lu.ui.course.server.gwt.CourseRpcGwtServlet defines non-transient non-serializable instance field courseService | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Class org.kuali.student.lum.lu.ui.course.server.gwt.CourseRpcGwtServlet defines non-transient non-serializable instance field stateChangeService | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Class org.kuali.student.lum.lu.ui.course.server.gwt.CourseRpcGwtServlet defines non-transient non-serializable instance field statementService | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Method org.kuali.student.lum.lu.ui.course.server.gwt.CourseRpcGwtServlet.storeCourseStatements(String, Map, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 59 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to currVerPrevState in org.kuali.student.lum.lu.ui.course.server.gwt.CourseStateChangeServiceImpl.updateCourseVersionStates(CourseInfo, String, CourseInfo, String, boolean, Date) | STYLE | DLS_DEAD_LOCAL_STORE | 71 | Medium |
| org.kuali.student.lum.lu.ui.course.server.gwt.CourseStateChangeServiceImpl.changeState(String, String, Date) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 48 | Medium |
| Method org.kuali.student.lum.lu.ui.course.server.gwt.CourseStateChangeServiceImpl.updateCourseVersionStates(CourseInfo, String, CourseInfo, String, boolean, Date) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 103 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.lu.ui.main.client.controllers.ApplicationController.contentPanel isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 20 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Nullcheck of result at line 122 of value previously dereferenced in org.kuali.student.lum.lu.ui.main.client.widgets.ApplicationHeader$1.onSuccess(Map) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 121 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.lum.lu.ui.tools.client.configuration.CatalogBrowserConfigurer.formatMetadata(Metadata, String) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 109 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to cluRangeFieldEditDescriptor in org.kuali.student.lum.lu.ui.tools.client.configuration.CluSetsConfigurer.addClusetDetailsSections(SectionView, String) | STYLE | DLS_DEAD_LOCAL_STORE | 135 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.lum.lu.ui.tools.client.configuration.CluSetsConfigurer$CluSetEditOptionList be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 617 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.lum.lu.ui.tools.client.configuration.CluSetsConfigurer$ModelIdPlaceHolder be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 603 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.lu.ui.tools.client.configuration.CluSetsManagementController$9.exec(List) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 325 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.lu.ui.tools.client.configuration.ClusetView$1$1.onSuccess(Metadata) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 107 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.lum.lu.ui.tools.client.widgets.BrowsePanel$SelectButtonClickHandler be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 118 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.lum.lu.ui.tools.client.widgets.KSBrowser$ExecuteNextSearchCallback be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 88 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.lum.lu.ui.tools.client.widgets.KSBrowser$ViewCourseCallback be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 111 | Medium |
| Unread field: org.kuali.student.lum.lu.ui.tools.client.widgets.KSBrowser$ViewCourseCallback.controller | PERFORMANCE | URF_UNREAD_FIELD | 112 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.lu.ui.tools.server.gwt.CluSetDataService.isAuthorized(PermissionType, Map) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 73 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Possible null pointer dereference of cluSetInfo in org.kuali.student.lum.lu.ui.tools.server.gwt.CluSetManagementRpcGwtServlet.upWrap(CluSetInfo) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 147 | Medium |
| Method org.kuali.student.lum.lu.ui.tools.server.gwt.CluSetManagementRpcGwtServlet.getCluInformations(List) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 209 | Medium |
| Class org.kuali.student.lum.lu.ui.tools.server.gwt.CluSetManagementRpcGwtServlet defines non-transient non-serializable instance field lrcService | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Class org.kuali.student.lum.lu.ui.tools.server.gwt.CluSetManagementRpcGwtServlet defines non-transient non-serializable instance field luService | BAD_PRACTICE | SE_BAD_FIELD | Medium |
