varsvat commented on a change in pull request #122:
URL: https://github.com/apache/fineract-cn-mobile/pull/122#discussion_r593412185



##########
File path: 
app/src/main/java/org/apache/fineract/ui/online/loanaccounts/plannedpayment/PlannedPaymentPresenter.java
##########
@@ -56,7 +56,7 @@ public void detachView() {
     @Override
     public void fetchPlannedPayment(String productIdentifier, String 
caseIdentifier,
             Integer pageIndex, String initialDisbursalDate, Boolean loadmore) {
-        this.loadmore = loadmore;
+//        this.loadmore = loadmore;

Review comment:
       Actually I got your point and have already tried that approach also, but 
the solution you are suggesting will always pass false as a parameter to the 
fetchPlannedPayment method, which did not seem right to me as it will then 
always manipulate 
[these](https://github.com/apache/fineract-cn-mobile/blob/592099e802aee546a145f8c9e96a3aecd149312b/app/src/main/java/org/apache/fineract/ui/online/loanaccounts/plannedpayment/PlannedPaymentPresenter.java#L79)
 conditional statements in the same way.




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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to