New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FINERACT-1724: Fee's are not included in total balance amount, when we have multiple disbursements on first day #3150
FINERACT-1724: Fee's are not included in total balance amount, when we have multiple disbursements on first day #3150
Conversation
if (installmentNumber == null && charge.isDueForCollectionFromAndUpToAndIncluding(startDate, installment.getDueDate())) { | ||
|
||
boolean isDue = installment.isFirstPeriod() | ||
? charge.isDueForCollectionFromIncludingAndUpToAndIncluding(startDate, installment.getDueDate()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably yes.
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/Loan.java
Outdated
Show resolved
Hide resolved
...anaccount/domain/transactionprocessor/AbstractLoanRepaymentScheduleTransactionProcessor.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left comments regarding 2 cases, where we just check if loanCharge
is due or not.
The startDate
in both these cases is disbursementDate
and endDate
is installment.getDueDate()
, so only installment.dueDate
parameter is sufficient.
Can there be 1 IT for this flow?
Thank you for your review! I have replied to your comments and i am working on the IT test case. |
f3aadf7
to
aba2cc4
Compare
…e have multiple disbursements on first day
aba2cc4
to
62b2faf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Describe the changes made and why they were made.
Ignore if these details are present on the associated Apache Fineract JIRA ticket.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!
Write the commit message as per https://github.com/apache/fineract/#pull-requests
Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.
Create/update unit or integration tests for verifying the changes made.
Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.
Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes
Submission is not a "code dump". (Large changes can be made "in repository" via a branch. Ask on the developer mailing list for guidance, if required.)
FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.