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.core.atp.entity.Atp.getEndDate() may expose internal representation by returning Atp.endDate | MALICIOUS_CODE | EI_EXPOSE_REP | 101 | Medium |
org.kuali.student.core.atp.entity.Atp.getStartDate() may expose internal representation by returning Atp.startDate | MALICIOUS_CODE | EI_EXPOSE_REP | 93 | Medium |
org.kuali.student.core.atp.entity.Atp.setEndDate(Date) may expose internal representation by storing an externally mutable object into Atp.endDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 105 | Medium |
org.kuali.student.core.atp.entity.Atp.setStartDate(Date) may expose internal representation by storing an externally mutable object into Atp.startDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 97 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.atp.entity.DateRange.getEndDate() may expose internal representation by returning DateRange.endDate | MALICIOUS_CODE | EI_EXPOSE_REP | 109 | Medium |
org.kuali.student.core.atp.entity.DateRange.getStartDate() may expose internal representation by returning DateRange.startDate | MALICIOUS_CODE | EI_EXPOSE_REP | 101 | Medium |
org.kuali.student.core.atp.entity.DateRange.setEndDate(Date) may expose internal representation by storing an externally mutable object into DateRange.endDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 113 | Medium |
org.kuali.student.core.atp.entity.DateRange.setStartDate(Date) may expose internal representation by storing an externally mutable object into DateRange.startDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 105 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.atp.entity.Milestone.getMilestoneDate() may expose internal representation by returning Milestone.milestoneDate | MALICIOUS_CODE | EI_EXPOSE_REP | 87 | Medium |
org.kuali.student.core.atp.entity.Milestone.setMilestoneDate(Date) may expose internal representation by storing an externally mutable object into Milestone.milestoneDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 91 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Method call in org.kuali.student.core.atp.service.impl.AtpServiceImpl.addDateRange(String, String, DateRangeInfo) passes null for nonnull parameter of AtpAssembler.toDateRangeInfo(DateRange) | CORRECTNESS | NP_NULL_PARAM_DEREF | 94 | Medium |
Method call in org.kuali.student.core.atp.service.impl.AtpServiceImpl.addMilestone(String, String, MilestoneInfo) passes null for nonnull parameter of AtpAssembler.toMilestoneInfo(Milestone) | CORRECTNESS | NP_NULL_PARAM_DEREF | 138 | Medium |
Method call in org.kuali.student.core.atp.service.impl.AtpServiceImpl.createAtp(String, String, AtpInfo) passes null for nonnull parameter of AtpAssembler.toAtpInfo(Atp) | CORRECTNESS | NP_NULL_PARAM_DEREF | 168 | Medium |
Method org.kuali.student.core.atp.service.impl.AtpServiceImpl.CheckMissingParameters(String[], Object[]) concatenates strings using + in a loop | PERFORMANCE | SBSC_USE_STRINGBUFFER_CONCATENATION | 102 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.comment.entity.Comment.getEffectiveDate() may expose internal representation by returning Comment.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 110 | Medium |
org.kuali.student.core.comment.entity.Comment.getExpirationDate() may expose internal representation by returning Comment.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 124 | Medium |
org.kuali.student.core.comment.entity.Comment.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into Comment.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 117 | Medium |
org.kuali.student.core.comment.entity.Comment.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into Comment.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 131 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.comment.entity.Tag.getEffectiveDate() may expose internal representation by returning Tag.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 118 | Medium |
org.kuali.student.core.comment.entity.Tag.getExpirationDate() may expose internal representation by returning Tag.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 132 | Medium |
org.kuali.student.core.comment.entity.Tag.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into Tag.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 125 | Medium |
org.kuali.student.core.comment.entity.Tag.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into Tag.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 139 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.document.entity.Document.getEffectiveDate() may expose internal representation by returning Document.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 138 | Medium |
org.kuali.student.core.document.entity.Document.getExpirationDate() may expose internal representation by returning Document.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 152 | Medium |
org.kuali.student.core.document.entity.Document.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into Document.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 145 | Medium |
org.kuali.student.core.document.entity.Document.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into Document.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 159 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.document.entity.DocumentCategory.getEffectiveDate() may expose internal representation by returning DocumentCategory.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 96 | Medium |
org.kuali.student.core.document.entity.DocumentCategory.getExpirationDate() may expose internal representation by returning DocumentCategory.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 110 | Medium |
org.kuali.student.core.document.entity.DocumentCategory.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into DocumentCategory.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 103 | Medium |
org.kuali.student.core.document.entity.DocumentCategory.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into DocumentCategory.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 117 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.document.entity.RefDocRelation.getEffectiveDate() may expose internal representation by returning RefDocRelation.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 110 | Medium |
org.kuali.student.core.document.entity.RefDocRelation.getExpirationDate() may expose internal representation by returning RefDocRelation.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 118 | Medium |
org.kuali.student.core.document.entity.RefDocRelation.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into RefDocRelation.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 114 | Medium |
org.kuali.student.core.document.entity.RefDocRelation.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into RefDocRelation.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 122 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.enumerationmanagement.entity.EnumeratedValue.getEffectiveDate() may expose internal representation by returning EnumeratedValue.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 106 | Medium |
org.kuali.student.core.enumerationmanagement.entity.EnumeratedValue.getExpirationDate() may expose internal representation by returning EnumeratedValue.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 114 | Medium |
org.kuali.student.core.enumerationmanagement.entity.EnumeratedValue.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into EnumeratedValue.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 110 | Medium |
org.kuali.student.core.enumerationmanagement.entity.EnumeratedValue.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into EnumeratedValue.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 118 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Method org.kuali.student.core.messages.service.impl.MessageServiceMock.getMessages(String, String) makes inefficient use of keySet iterator instead of entrySet iterator | PERFORMANCE | WMI_WRONG_MAP_ITERATOR | 177 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.organization.entity.Org.getEffectiveDate() may expose internal representation by returning Org.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 112 | Medium |
org.kuali.student.core.organization.entity.Org.getExpirationDate() may expose internal representation by returning Org.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 120 | Medium |
org.kuali.student.core.organization.entity.Org.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into Org.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 116 | Medium |
org.kuali.student.core.organization.entity.Org.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into Org.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 124 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.organization.entity.OrgHierarchy.getEffectiveDate() may expose internal representation by returning OrgHierarchy.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 110 | Medium |
org.kuali.student.core.organization.entity.OrgHierarchy.getExpirationDate() may expose internal representation by returning OrgHierarchy.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 118 | Medium |
org.kuali.student.core.organization.entity.OrgHierarchy.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into OrgHierarchy.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 114 | Medium |
org.kuali.student.core.organization.entity.OrgHierarchy.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into OrgHierarchy.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 122 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.organization.entity.OrgOrgRelation.getEffectiveDate() may expose internal representation by returning OrgOrgRelation.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 118 | Medium |
org.kuali.student.core.organization.entity.OrgOrgRelation.getExpirationDate() may expose internal representation by returning OrgOrgRelation.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 126 | Medium |
org.kuali.student.core.organization.entity.OrgOrgRelation.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into OrgOrgRelation.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 122 | Medium |
org.kuali.student.core.organization.entity.OrgOrgRelation.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into OrgOrgRelation.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 130 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.organization.entity.OrgPersonRelation.getEffectiveDate() may expose internal representation by returning OrgPersonRelation.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 100 | Medium |
org.kuali.student.core.organization.entity.OrgPersonRelation.getExpirationDate() may expose internal representation by returning OrgPersonRelation.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 108 | Medium |
org.kuali.student.core.organization.entity.OrgPersonRelation.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into OrgPersonRelation.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 104 | Medium |
org.kuali.student.core.organization.entity.OrgPersonRelation.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into OrgPersonRelation.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 112 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Method call in org.kuali.student.core.organization.service.impl.OrganizationServiceImpl.addPositionRestrictionToOrg(String, String, OrgPositionRestrictionInfo) passes null for nonnull parameter of OrganizationAssembler.toOrgPositionRestrictionInfo(OrgPositionRestriction) | CORRECTNESS | NP_NULL_PARAM_DEREF | 109 | Medium |
Method call in org.kuali.student.core.organization.service.impl.OrganizationServiceImpl.createOrgPersonRelation(String, String, String, OrgPersonRelationInfo) passes null for nonnull parameter of OrganizationAssembler.toOrgPersonRelationInfo(OrgPersonRelation) | CORRECTNESS | NP_NULL_PARAM_DEREF | 188 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
instanceof will always return true in org.kuali.student.core.personsearch.service.impl.PersonSearch.convertEntityToPerson(EntityDefault, Principal), since all RuntimeException are instances of RuntimeException | STYLE | BC_VACUOUS_INSTANCEOF | 80 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.proposal.entity.Proposal.getEffectiveDate() may expose internal representation by returning Proposal.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 154 | Medium |
org.kuali.student.core.proposal.entity.Proposal.getExpirationDate() may expose internal representation by returning Proposal.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 162 | Medium |
org.kuali.student.core.proposal.entity.Proposal.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into Proposal.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 158 | Medium |
org.kuali.student.core.proposal.entity.Proposal.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into Proposal.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 166 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.statement.entity.RefStatementRelation.getEffectiveDate() may expose internal representation by returning RefStatementRelation.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 123 | Medium |
org.kuali.student.core.statement.entity.RefStatementRelation.getExpirationDate() may expose internal representation by returning RefStatementRelation.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 131 | Medium |
org.kuali.student.core.statement.entity.RefStatementRelation.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into RefStatementRelation.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 127 | Medium |
org.kuali.student.core.statement.entity.RefStatementRelation.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into RefStatementRelation.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 135 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
org.kuali.student.core.statement.entity.ReqComponent.getEffectiveDate() may expose internal representation by returning ReqComponent.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP | 92 | Medium |
org.kuali.student.core.statement.entity.ReqComponent.getExpirationDate() may expose internal representation by returning ReqComponent.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP | 100 | Medium |
org.kuali.student.core.statement.entity.ReqComponent.setEffectiveDate(Date) may expose internal representation by storing an externally mutable object into ReqComponent.effectiveDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 96 | Medium |
org.kuali.student.core.statement.entity.ReqComponent.setExpirationDate(Date) may expose internal representation by storing an externally mutable object into ReqComponent.expirationDate | MALICIOUS_CODE | EI_EXPOSE_REP2 | 104 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Inconsistent synchronization of org.kuali.student.core.statement.naturallanguage.translators.NaturalLanguageTranslatorImpl.reqComponentTranslator; locked 75% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 50 | Medium |
Inconsistent synchronization of org.kuali.student.core.statement.naturallanguage.translators.NaturalLanguageTranslatorImpl.statementTranslator; locked 75% of time | MT_CORRECTNESS | IS2_INCONSISTENT_SYNC | 59 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Nullcheck of rootStatement at line 199 of value previously dereferenced in org.kuali.student.core.statement.naturallanguage.translators.StatementParser.traverseStatementTreeAndReduce(Statement, boolean, Statement) | CORRECTNESS | RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE | 181 | Medium |
Bug | Category | Details | Line | Priority |
---|---|---|---|---|
Potentially dangerous use of non-short-circuit logic in org.kuali.student.core.statement.service.assembler.StatementTreeViewAssembler.disassemble(StatementTreeViewInfo, BaseDTOAssemblyNode$NodeOperation) | STYLE | NS_DANGEROUS_NON_SHORT_CIRCUIT | 54 | High |