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

fix possible NPE in AbstractLoanScheduleGenerator

Browse Source
This commit is contained in:
Saransh Sharma
2020-06-01 21:53:02 +02:00
committed by Michael Vorburger ⛑️
parent aed3e41ca4
commit b6e6eba56f
1 changed files with 11 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/domain/AbstractLoanScheduleGenerator.java
+3
View File
@@ -518,6 +518,7 @@ public abstract class AbstractLoanScheduleGenerator implements LoanScheduleGener
currentPeriodParams.getPrincipalForThisPeriod().minus(principalProcessed)), currentPeriodParams.getPrincipalForThisPeriod().minus(principalProcessed)),
scheduleParams.getPeriodNumber() + 1, mc)); scheduleParams.getPeriodNumber() + 1, mc));
if (loanApplicationTerms.getAmortizationMethod().isEqualInstallment() if (loanApplicationTerms.getAmortizationMethod().isEqualInstallment()
&& fixedEmiAmount != null
&& fixedEmiAmount.compareTo(loanApplicationTerms.getFixedEmiAmount()) != 0) { && fixedEmiAmount.compareTo(loanApplicationTerms.getFixedEmiAmount()) != 0) {
currentPeriodParams.setEmiAmountChanged(true); currentPeriodParams.setEmiAmountChanged(true);
} }
@@ -637,8 +638,10 @@ public abstract class AbstractLoanScheduleGenerator implements LoanScheduleGener
scheduleParams.getTotalCumulativePrincipal().plus(currentPeriodParams.getPrincipalForThisPeriod()) scheduleParams.getTotalCumulativePrincipal().plus(currentPeriodParams.getPrincipalForThisPeriod())
.plus(currentPeriodParams.getEarlyPaidAmount()), scheduleParams.getPeriodNumber() + 1, mc)); .plus(currentPeriodParams.getEarlyPaidAmount()), scheduleParams.getPeriodNumber() + 1, mc));
if (loanApplicationTerms.getAmortizationMethod().isEqualInstallment() if (loanApplicationTerms.getAmortizationMethod().isEqualInstallment()
&& fixedEmiAmount != null
&& fixedEmiAmount.compareTo(loanApplicationTerms.getFixedEmiAmount()) != 0) { && fixedEmiAmount.compareTo(loanApplicationTerms.getFixedEmiAmount()) != 0) {
currentPeriodParams.setEmiAmountChanged(true); currentPeriodParams.setEmiAmountChanged(true);
currentPeriodParams.plusPrincipalForThisPeriod(currentPeriodParams.getEarlyPaidAmount());
} }
currentPeriodParams.plusPrincipalForThisPeriod(currentPeriodParams.getEarlyPaidAmount()); currentPeriodParams.plusPrincipalForThisPeriod(currentPeriodParams.getEarlyPaidAmount());
} }