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 |
|---|---|---|---|---|
| edu.sampleu.bookstore.bo.Book.getPublicationDate() may expose internal representation by returning Book.publicationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 84 | Medium |
| edu.sampleu.bookstore.bo.Book.setPublicationDate(Date) may expose internal representation by storing an externally mutable object into Book.publicationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 88 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Dead store to addresses in edu.sampleu.bookstore.maintenance.AuthorMaintainable.saveBusinessObject() | STYLE | DLS_DEAD_LOCAL_STORE | 32 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to edu.sampleu.kew.krad.form.IngesterForm in edu.sampleu.kew.krad.controller.IngesterController.start(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 70 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to edu.sampleu.kew.krad.form.StatsForm in edu.sampleu.kew.krad.controller.StatsController.start(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 58 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| edu.sampleu.kew.krad.form.StatsForm.getBeginningDate() may expose internal representation by returning StatsForm.beginningDate | MALICIOUS_CODE | EI_EXPOSE_REP | 249 | Medium |
| edu.sampleu.kew.krad.form.StatsForm.getEndingDate() may expose internal representation by returning StatsForm.endingDate | MALICIOUS_CODE | EI_EXPOSE_REP | 257 | Medium |
| edu.sampleu.kew.krad.form.StatsForm.setBeginningDate(Date) may expose internal representation by storing an externally mutable object into StatsForm.beginningDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 253 | Medium |
| edu.sampleu.kew.krad.form.StatsForm.setEndingDate(Date) may expose internal representation by storing an externally mutable object into StatsForm.endingDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 261 | Medium |
| Class edu.sampleu.kew.krad.form.StatsForm defines non-transient non-serializable instance field stats | BAD_PRACTICE | SE_BAD_FIELD | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| edu.sampleu.travel.bo.TravelAccount.getCreateDate() may expose internal representation by returning TravelAccount.createDate | MALICIOUS_CODE | EI_EXPOSE_REP | 55 | Medium |
| edu.sampleu.travel.bo.TravelAccount.setCreateDate(Date) may expose internal representation by storing an externally mutable object into TravelAccount.createDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 122 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| edu.sampleu.travel.dto.TravelAccountInfo.getCreateDate() may expose internal representation by returning TravelAccountInfo.createDate | MALICIOUS_CODE | EI_EXPOSE_REP | 81 | Medium |
| edu.sampleu.travel.dto.TravelAccountInfo.setCreateDate(Date) may expose internal representation by storing an externally mutable object into TravelAccountInfo.createDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 85 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to edu.sampleu.travel.krad.form.UILayoutTestForm in edu.sampleu.travel.krad.controller.UILayoutTestController.start(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 50 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| Unchecked/unconfirmed cast from org.kuali.rice.krad.web.form.UifFormBase to edu.sampleu.travel.krad.form.UITestForm in edu.sampleu.travel.krad.controller.UITestController.start(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) | STYLE | BC_UNCONFIRMED_CAST | 57 | Medium |
| Method edu.sampleu.travel.krad.controller.UITestController.start(UifFormBase, BindingResult, HttpServletRequest, HttpServletResponse) invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead | PERFORMANCE | DM_NUMBER_CTOR | 65 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| edu.sampleu.travel.service.FiscalOfficerInfoMaintainableImpl is Serializable; consider declaring a serialVersionUID | BAD_PRACTICE | SE_NO_SERIALVERSIONID | 30 | Medium |
| Bug | Category | Details | Line | Priority |
|---|---|---|---|---|
| edu.sampleu.travel.service.FiscalOfficerServiceImpl.FOI_SKIP should be both final and package protected | MALICIOUS_CODE | MS_FINAL_PKGPROTECT | 42 | Medium |
