adamsaghy commented on code in PR #2882:
URL: https://github.com/apache/fineract/pull/2882#discussion_r1073533927


##########
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/Loan.java:
##########
@@ -4158,20 +4157,12 @@ public LocalDate 
getExpectedDisbursedOnLocalDateForTemplate() {
         return expectedDisbursementDate;
     }
 
-    /*
-     * Reason for derving
-     */
-
     public BigDecimal getDisburseAmountForTemplate() {
-        BigDecimal principal = 
this.loanRepaymentScheduleDetail.getPrincipal().getAmount();

Review Comment:
   I see. Thanks for letting me know.
   
   Would you please raise a new story and PR with this change? 
   We should focus to fix one issue / PR, it makes confusing and complex if 
more than one issue is addressed in one PR.



-- 
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]

Reply via email to