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 |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.ProgramClientConstants.CREDENTIAL_PROGRAM should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 36 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.ProgramUtils.df isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 27 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field org.kuali.student.lum.program.client.core.CoreManager.eventBus from instance method new org.kuali.student.lum.program.client.core.CoreManager() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 30 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field org.kuali.student.lum.program.client.credential.CredentialManager.eventBus from instance method new org.kuali.student.lum.program.client.credential.CredentialManager() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 30 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.AddSpecializationEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 11 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.AfterSaveEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 13 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.ChangeViewEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 11 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.MetadataLoadedEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 14 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.ModelLoadedEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 12 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.ProgramViewEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 11 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.RemoveSpecializationEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 11 | High |
| Unread field: org.kuali.student.lum.program.client.events.RemoveSpecializationEvent.id | PERFORMANCE | URF_UNREAD_FIELD | 16 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.SpecializationCreatedEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 11 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.SpecializationSaveEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 12 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.SpecializationUpdateEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 13 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.StateChangeEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 12 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.StoreRequirementIDsEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 10 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.StoreSpecRequirementIDsEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 10 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.events.UpdateEvent.TYPE isn't final but should be | MALICIOUS_CODE | MS_SHOULD_BE_FINAL | 13 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Incorrect lazy initialization and update of static field org.kuali.student.lum.program.client.major.ActionType.values in org.kuali.student.lum.program.client.major.ActionType.getValues() | MT_CORRECTNESS | LI_LAZY_INIT_UPDATE_STATIC | 30 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Write to static field org.kuali.student.lum.program.client.major.MajorManager.eventBus from instance method new org.kuali.student.lum.program.client.major.MajorManager() | STYLE | ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD | 41 | High |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.lum.program.client.major.edit.MajorInformationEditConfiguration$DiplomaBinding be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 136 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Method org.kuali.student.lum.program.client.requirements.ProgramRequirementsDataModel$5.onSuccess(Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 247 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| The class name org.kuali.student.lum.program.client.requirements.ProgramRequirementsDataModel$requirementState doesn't start with an upper case letter | BAD_PRACTICE | NM_CLASS_NAMING_CONVENTION | 48 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.lum.program.client.requirements.ProgramRequirementsManageView.originalReqCompNL | PERFORMANCE | URF_UNREAD_FIELD | 182 | Medium |
| Unused field: org.kuali.student.lum.program.client.requirements.ProgramRequirementsManageView.originalLogicExpression | PERFORMANCE | UUF_UNUSED_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| org.kuali.student.lum.program.client.requirements.ProgramRequirementsSummaryView.tempStmtTreeID should be package protected | MALICIOUS_CODE | MS_PKGPROTECT | 67 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.lum.program.client.variation.VariationController.majorController | PERFORMANCE | URF_UNREAD_FIELD | 44 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to row in org.kuali.student.lum.program.client.variation.edit.VariationEditController$3.onEvent(ModelLoadedEvent) | STYLE | DLS_DEAD_LOCAL_STORE | 87 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Should org.kuali.student.lum.program.client.variation.edit.VariationInformationEditConfiguration$DiplomaBinding be a _static_ inner class? | PERFORMANCE | SIC_INNER_SHOULD_BE_STATIC | 114 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.lum.program.client.versions.ProgramSelectVersionsView.metadataServiceAsync | PERFORMANCE | URF_UNREAD_FIELD | 38 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.lum.program.client.widgets.ProgramSideBar.viewContext | PERFORMANCE | URF_UNREAD_FIELD | 68 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Class org.kuali.student.lum.program.server.MajorDisciplineRpcServlet defines non-transient non-serializable instance field programService | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Class org.kuali.student.lum.program.server.MajorDisciplineRpcServlet defines non-transient non-serializable instance field statementService | BAD_PRACTICE | SE_BAD_FIELD | Medium | |
| Method org.kuali.student.lum.program.server.MajorDisciplineRpcServlet.storeProgramRequirements(Map, Map) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 49 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unread field: org.kuali.student.lum.program.server.transform.CoreCredentialProgramFilter.programService | PERFORMANCE | URF_UNREAD_FIELD | 71 | Medium |
| Unused field: org.kuali.student.lum.program.server.transform.CoreCredentialProgramFilter.metadataService | PERFORMANCE | UUF_UNUSED_FIELD | Medium |
