Repository: incubator-fineract Updated Branches: refs/heads/develop 37dfe1a0b -> a6c9d3d3e
fix for Junit test cases Project: http://git-wip-us.apache.org/repos/asf/incubator-fineract/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-fineract/commit/6e521f81 Tree: http://git-wip-us.apache.org/repos/asf/incubator-fineract/tree/6e521f81 Diff: http://git-wip-us.apache.org/repos/asf/incubator-fineract/diff/6e521f81 Branch: refs/heads/develop Commit: 6e521f81ac68e059b89351a4579038fae1381e1a Parents: 14499a2 Author: sachinkulkarni12 <[email protected]> Authored: Mon Mar 7 16:50:43 2016 +0530 Committer: sachinkulkarni12 <[email protected]> Committed: Mon Mar 7 16:50:43 2016 +0530 ---------------------------------------------------------------------- .../service/LoanReadPlatformServiceImpl.java | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-fineract/blob/6e521f81/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/service/LoanReadPlatformServiceImpl.java ---------------------------------------------------------------------- diff --git a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/service/LoanReadPlatformServiceImpl.java b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/service/LoanReadPlatformServiceImpl.java index b39aaf7..3f0e3a1 100755 --- a/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/service/LoanReadPlatformServiceImpl.java +++ b/fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/service/LoanReadPlatformServiceImpl.java @@ -1564,9 +1564,6 @@ public class LoanReadPlatformServiceImpl implements LoanReadPlatformService { LoanScheduleAccrualMapper mapper = new LoanScheduleAccrualMapper(); Date organisationStartDate = this.configurationDomainService.retrieveOrganisationStartDate(); - if(organisationStartDate == null){ - organisationStartDate = new Date(); - } final StringBuilder sqlBuilder = new StringBuilder(400); sqlBuilder .append("select ") @@ -1574,7 +1571,11 @@ public class LoanReadPlatformServiceImpl implements LoanReadPlatformService { .append(" where ((ls.fee_charges_amount <> if(ls.accrual_fee_charges_derived is null,0, ls.accrual_fee_charges_derived))") .append(" or ( ls.penalty_charges_amount <> if(ls.accrual_penalty_charges_derived is null,0,ls.accrual_penalty_charges_derived))") .append(" or ( ls.interest_amount <> if(ls.accrual_interest_derived is null,0,ls.accrual_interest_derived)))") - .append(" and ls.duedate > :organisationstartdate and loan.loan_status_id=:active and mpl.accounting_type=:type and loan.is_npa=0 and ls.duedate <= CURDATE() order by loan.id,ls.duedate"); + .append(" and loan.loan_status_id=:active and mpl.accounting_type=:type and loan.is_npa=0 and ls.duedate <= CURDATE() "); + if(organisationStartDate != null){ + sqlBuilder.append(" and ls.duedate > :organisationstartdate "); + } + sqlBuilder.append(" order by loan.id,ls.duedate "); Map<String, Object> paramMap = new HashMap<>(3); paramMap.put("active", LoanStatus.ACTIVE.getValue()); paramMap.put("type", AccountingRuleType.ACCRUAL_PERIODIC.getValue()); @@ -1588,9 +1589,6 @@ public class LoanReadPlatformServiceImpl implements LoanReadPlatformService { LoanSchedulePeriodicAccrualMapper mapper = new LoanSchedulePeriodicAccrualMapper(); Date organisationStartDate = this.configurationDomainService.retrieveOrganisationStartDate(); - if(organisationStartDate == null){ - organisationStartDate = new Date(); - } final StringBuilder sqlBuilder = new StringBuilder(400); sqlBuilder .append("select ") @@ -1598,8 +1596,12 @@ public class LoanReadPlatformServiceImpl implements LoanReadPlatformService { .append(" where ((ls.fee_charges_amount <> if(ls.accrual_fee_charges_derived is null,0, ls.accrual_fee_charges_derived))") .append(" or (ls.penalty_charges_amount <> if(ls.accrual_penalty_charges_derived is null,0,ls.accrual_penalty_charges_derived))") .append(" or (ls.interest_amount <> if(ls.accrual_interest_derived is null,0,ls.accrual_interest_derived)))") - .append(" and loan.loan_status_id=:active and mpl.accounting_type=:type and ls.duedate > :organisationstartdate and (loan.closedon_date <= :tilldate or loan.closedon_date is null)") - .append(" and loan.is_npa=0 and (ls.duedate <= :tilldate or (ls.duedate > :tilldate and ls.fromdate < :tilldate)) order by loan.id,ls.duedate"); + .append(" and loan.loan_status_id=:active and mpl.accounting_type=:type and (loan.closedon_date <= :tilldate or loan.closedon_date is null)") + .append(" and loan.is_npa=0 and (ls.duedate <= :tilldate or (ls.duedate > :tilldate and ls.fromdate < :tilldate)) "); + if(organisationStartDate != null){ + sqlBuilder.append(" and ls.duedate > :organisationstartdate "); + } + sqlBuilder.append(" order by loan.id,ls.duedate "); Map<String, Object> paramMap = new HashMap<>(4); paramMap.put("active", LoanStatus.ACTIVE.getValue()); paramMap.put("type", AccountingRuleType.ACCRUAL_PERIODIC.getValue());
