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


##########
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanRepository.java:
##########
@@ -84,11 +84,13 @@ public interface LoanRepository extends JpaRepository<Loan, 
Long>, JpaSpecificat
 
     // should follow the logic of 
`FIND_ALL_NON_CLOSED_LOANS_BY_LAST_CLOSED_BUSINESS_DATE` query
     String FIND_OLDEST_COB_PROCESSED_LOAN = "select loan.id, 
loan.lastClosedBusinessDate from Loan loan where loan.loanStatus in 
(100,200,300,303,304) and loan.lastClosedBusinessDate = (select 
min(l.lastClosedBusinessDate) from Loan l where l"
-            + ".loanStatus in (100,200,300,303,304) and 
l.lastClosedBusinessDate <> :cobBusinessDate)";
+            + ".loanStatus in (100,200,300,303,304) and 
l.lastClosedBusinessDate < :cobBusinessDate)";
 
-    String FIND_ALL_NON_CLOSED_LOANS_BEHIND_BY_LOAN_IDS = "select loan.id, 
loan.lastClosedBusinessDate from Loan loan where loan.id IN :loanIds and 
loan.loanStatus in (100,200,300,303,304) and (loan.lastClosedBusinessDate <> 
:cobBusinessDate or "
+    String FIND_ALL_NON_CLOSED_LOANS_BEHIND_OR_NULL_BY_LOAN_IDS = "select 
loan.id, loan.lastClosedBusinessDate from Loan loan where loan.id IN :loanIds 
and loan.loanStatus in (100,200,300,303,304) and (loan.lastClosedBusinessDate < 
:cobBusinessDate or "

Review Comment:
   Will be covered in new PR



##########
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/LoanRepository.java:
##########
@@ -84,11 +84,13 @@ public interface LoanRepository extends JpaRepository<Loan, 
Long>, JpaSpecificat
 
     // should follow the logic of 
`FIND_ALL_NON_CLOSED_LOANS_BY_LAST_CLOSED_BUSINESS_DATE` query
     String FIND_OLDEST_COB_PROCESSED_LOAN = "select loan.id, 
loan.lastClosedBusinessDate from Loan loan where loan.loanStatus in 
(100,200,300,303,304) and loan.lastClosedBusinessDate = (select 
min(l.lastClosedBusinessDate) from Loan l where l"
-            + ".loanStatus in (100,200,300,303,304) and 
l.lastClosedBusinessDate <> :cobBusinessDate)";
+            + ".loanStatus in (100,200,300,303,304) and 
l.lastClosedBusinessDate < :cobBusinessDate)";
 
-    String FIND_ALL_NON_CLOSED_LOANS_BEHIND_BY_LOAN_IDS = "select loan.id, 
loan.lastClosedBusinessDate from Loan loan where loan.id IN :loanIds and 
loan.loanStatus in (100,200,300,303,304) and (loan.lastClosedBusinessDate <> 
:cobBusinessDate or "
+    String FIND_ALL_NON_CLOSED_LOANS_BEHIND_OR_NULL_BY_LOAN_IDS = "select 
loan.id, loan.lastClosedBusinessDate from Loan loan where loan.id IN :loanIds 
and loan.loanStatus in (100,200,300,303,304) and (loan.lastClosedBusinessDate < 
:cobBusinessDate or "
             + "loan.lastClosedBusinessDate is null)";
 
+    String FIND_ALL_NON_CLOSED_LOANS_BEHIND_BY_LOAN_IDS = "select loan.id, 
loan.lastClosedBusinessDate from Loan loan where loan.id IN :loanIds and 
loan.loanStatus in (100,200,300,303,304) and loan.lastClosedBusinessDate < 
:cobBusinessDate";

Review Comment:
   Will be covered in new 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