CBSNexora/cbsnexora-backend
4
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

FINERACT-2080: Refactoring Loan entity

Browse Source 
FINERACT-2080: Refactoring Loan entity
This commit is contained in:
Harsh-Srivastav123
2025-03-14 18:11:52 +00:00
committed by Adam Saghy
parent 35aa6e1f4f
commit c236ee04ef
1 changed files with 19 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
fineract-loan/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/Loan.java
+19 -11
View File
@@ -1585,10 +1585,16 @@ public class Loan extends AbstractAuditableWithUTCDateTimeCustom<Long> {
} }
public boolean doPostLoanTransactionChecks(final LocalDate transactionDate, final LoanLifecycleStateMachine loanLifecycleStateMachine) { public boolean doPostLoanTransactionChecks(final LocalDate transactionDate, final LoanLifecycleStateMachine loanLifecycleStateMachine) {
boolean statusChanged = checkAndHandleLoanStatus(transactionDate, loanLifecycleStateMachine);
resetOverpaidDateIfNeeded();
return statusChanged;
}
private boolean checkAndHandleLoanStatus(final LocalDate transactionDate, final LoanLifecycleStateMachine loanLifecycleStateMachine) {
boolean statusChanged = false; boolean statusChanged = false;
boolean isOverpaid = MathUtil.isGreaterThanZero(totalOverpaid); boolean isOverpaid = MathUtil.isGreaterThanZero(totalOverpaid);
if (isOverpaid) { if (isOverpaid) {
// FIXME - kw - update account balance to negative amount.
handleLoanOverpayment(transactionDate, loanLifecycleStateMachine); handleLoanOverpayment(transactionDate, loanLifecycleStateMachine);
statusChanged = true; statusChanged = true;
} else if (this.summary.isRepaidInFull(getCurrency())) { } else if (this.summary.isRepaidInFull(getCurrency())) {
@@ -1597,10 +1603,14 @@ public class Loan extends AbstractAuditableWithUTCDateTimeCustom<Long> {
} else { } else {
loanLifecycleStateMachine.transition(LoanEvent.LOAN_REPAYMENT_OR_WAIVER, this); loanLifecycleStateMachine.transition(LoanEvent.LOAN_REPAYMENT_OR_WAIVER, this);
} }
return statusChanged;
}
private void resetOverpaidDateIfNeeded() {
if (MathUtil.isEmpty(totalOverpaid)) { if (MathUtil.isEmpty(totalOverpaid)) {
this.overpaidOnDate = null; this.overpaidOnDate = null;
} }
return statusChanged;
} }
private void handleLoanRepaymentInFull(final LocalDate transactionDate, final LoanLifecycleStateMachine loanLifecycleStateMachine) { private void handleLoanRepaymentInFull(final LocalDate transactionDate, final LoanLifecycleStateMachine loanLifecycleStateMachine) {
@@ -2453,14 +2463,12 @@ public class Loan extends AbstractAuditableWithUTCDateTimeCustom<Long> {
} }
public LocalDate getLastUserTransactionDate() { public LocalDate getLastUserTransactionDate() {
LocalDate currentTransactionDate = getDisbursementDate(); return this.loanTransactions.stream().filter(this::isUserTransaction).map(LoanTransaction::getTransactionDate)
for (final LoanTransaction previousTransaction : this.loanTransactions) { .filter(date -> DateUtils.isBefore(getDisbursementDate(), date)).max(LocalDate::compareTo).orElse(getDisbursementDate());
if (!(previousTransaction.isReversed() || previousTransaction.isAccrualRelated() || previousTransaction.isIncomePosting()) }
&& DateUtils.isBefore(currentTransactionDate, previousTransaction.getTransactionDate())) {
currentTransactionDate = previousTransaction.getTransactionDate(); private boolean isUserTransaction(LoanTransaction transaction) {
} return !(transaction.isReversed() || transaction.isAccrualRelated() || transaction.isIncomePosting());
}
return currentTransactionDate;
} }
public LocalDate getLastRepaymentDate() { public LocalDate getLastRepaymentDate() {
@@ -3564,7 +3572,7 @@ public class Loan extends AbstractAuditableWithUTCDateTimeCustom<Long> {
public boolean isChargeOffOnDate(final LocalDate onDate) { public boolean isChargeOffOnDate(final LocalDate onDate) {
final LoanTransaction chargeOffTransaction = findChargedOffTransaction(); final LoanTransaction chargeOffTransaction = findChargedOffTransaction();
return (chargeOffTransaction == null) ? false : (chargeOffTransaction.getDateOf().compareTo(onDate) <= 0); return chargeOffTransaction != null && chargeOffTransaction.getDateOf().compareTo(onDate) <= 0;
} }
public boolean hasMonetaryActivityAfter(final LocalDate transactionDate) { public boolean hasMonetaryActivityAfter(final LocalDate transactionDate) {