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 |
|---|---|---|---|---|
| Nullcheck of courseInfo at line 180 of value previously dereferenced in org.kuali.student.lum.workflow.CoursePostProcessorBase.retireCourseByProposalCopyAndSave(String, CourseInfo, ProposalInfo) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 165 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.workflow.CourseStateChangeServiceImpl.changeState(String, String, String) invokes inefficient Boolean constructor; use Boolean.valueOf(...) instead | PERFORMANCE | DM_BOOLEAN_CTOR | 63 | Medium |
| Method org.kuali.student.lum.workflow.CourseStateChangeServiceImpl.updateCourseVersionStates(CourseInfo, String, CourseInfo, String, boolean, String) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 200 | Medium |
| Call to equals() comparing different types in org.kuali.student.lum.workflow.CourseStateChangeServiceImpl.updateCourseVersionStates(CourseInfo, String, CourseInfo, String, boolean, String) | CORRECTNESS | EC_UNRELATED_TYPES | 163 | High |
| Possible null pointer dereference of newState in org.kuali.student.lum.workflow.CourseStateChangeServiceImpl.changeState(String, String, String) | CORRECTNESS | NP_NULL_ON_SOME_PATH | 55 | Medium |
| Method call in org.kuali.student.lum.workflow.CourseStateChangeServiceImpl.changeState(String, String, String) passes null for nonnull parameter of activateCourse(CourseInfo, String) | CORRECTNESS | NP_NULL_PARAM_DEREF | 53 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Nullcheck of actionTaken at line 75 of value previously dereferenced in org.kuali.student.lum.workflow.KualiStudentPostProcessorBase.doActionTaken(ActionTakenEvent) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 73 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.lum.workflow.MajorDisciplineStateChangeServiceImpl.updatePreviousVersions(MajorDisciplineInfo, String, String, String) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 177 | Medium |
| Suspicious comparison of Long references in org.kuali.student.lum.workflow.MajorDisciplineStateChangeServiceImpl.updatePreviousVersions(MajorDisciplineInfo, String, String, String) | CORRECTNESS | RC_REF_COMPARISON | 184 | High |
