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


##########
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/service/ProgressiveLoanInterestRefundServiceImpl.java:
##########
@@ -118,9 +118,8 @@ private boolean 
isTransactionNeededForInterestRefundCalculations(LoanTransaction
 
     @Override
     @Transactional(readOnly = true, propagation = Propagation.REQUIRES_NEW)
-    public Money 
totalInterestByTransactions(LoanRepaymentScheduleTransactionProcessor 
processor, final Long loanId,
+    public Money 
totalInterestByTransactions(LoanRepaymentScheduleTransactionProcessor 
processor, final Loan loan,

Review Comment:
   We cannot do this.  The changes on the loan entity might be saved outside of 
this transactions which is unwanted behaviour!



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