You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@fineract.apache.org by GitBox <gi...@apache.org> on 2022/07/27 17:39:47 UTC

[GitHub] [fineract] rrpawar96 commented on a diff in pull request #2436: FINERACT-1330:fixPayCharge

rrpawar96 commented on code in PR #2436:
URL: https://github.com/apache/fineract/pull/2436#discussion_r931333116


##########
fineract-provider/src/main/java/org/apache/fineract/portfolio/savings/domain/SavingsAccountTransactionRepository.java:
##########
@@ -44,4 +44,8 @@ List<SavingsAccountTransaction> findTransactionsAfterPivotDate(@Param("savingsAc
 
     @Query("select sat from SavingsAccountTransaction sat where sat.refNo = :refNo")
     List<SavingsAccountTransaction> findAllTransactionByRefNo(@Param("refNo") String refNo);
+
+    @Query("select sat from SavingsAccountTransaction sat where sat.savingsAccount.id = :savingsId and sat.dateOf <= :transactionDate and sat.reversed=0 ORDER BY sat.dateOf DESC")

Review Comment:
   > `sat.reversed=0` should be `sat.reversed=false`
   
   @galovics Comment Addressed, Please review.



-- 
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: commits-unsubscribe@fineract.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org