bharathcgowda commented on code in PR #6546:
URL: https://github.com/apache/fineract/pull/6546#discussion_r4397556588
##########
fineract-loan/src/main/java/org/apache/fineract/portfolio/loanaccount/loanschedule/domain/DefaultScheduledDateGenerator.java:
##########
@@ -124,6 +124,11 @@ public LocalDate generateNextRepaymentDate(final LocalDate
lastRepaymentDate, fi
LocalDate seedDate = null;
String reccuringString = null;
Calendar currentCalendar = loanApplicationTerms.getLoanCalendar();
+ if
(loanApplicationTerms.getRepaymentPeriodFrequencyType().isSemiMonthly()) {
+ // The two due days are fixed points of the calendar month, so
the next date is the first of them
+ // after the previous one rather than an offset added to it.
+ return SemiMonthlyScheduleDates.next(lastRepaymentDate,
loanApplicationTerms.semiMonthlyDueDays());
Review Comment:
When `allowPartialPeriodInterestCalculation` is enabled for `same as
repayment`, interest should be calculated only for the number of days between
disbursement and repayment.
A repayment made the day after disbursement is valid under this
configuration, but interest should still be calculated for that one day.
However, if `minimumDaysBetweenDisbursalAndFirstRepayment` is configured,
the system should prioritize and enforce this setting.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
