[fineract] branch develop updated: FINERACT-27 - Extend holiday period to 30 days

[aw...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

awasum pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/fineract.git


The following commit(s) were added to refs/heads/develop by this push:
     new 161d518  FINERACT-27 - Extend holiday period to 30 days
161d518 is described below

commit 161d5184553208455d875088461fabf883340287
Author: Yemdjih Kaze Nasser <ka...@outlook.com>
AuthorDate: Thu Nov 26 07:24:25 2020 +0100

    FINERACT-27 - Extend holiday period to 30 days
---
 .../service/HolidayWritePlatformServiceJpaRepositoryImpl.java     | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/fineract-provider/src/main/java/org/apache/fineract/organisation/holiday/service/HolidayWritePlatformServiceJpaRepositoryImpl.java b/fineract-provider/src/main/java/org/apache/fineract/organisation/holiday/service/HolidayWritePlatformServiceJpaRepositoryImpl.java
index 68ec371..df4cb85 100644
--- a/fineract-provider/src/main/java/org/apache/fineract/organisation/holiday/service/HolidayWritePlatformServiceJpaRepositoryImpl.java
+++ b/fineract-provider/src/main/java/org/apache/fineract/organisation/holiday/service/HolidayWritePlatformServiceJpaRepositoryImpl.java
@@ -233,16 +233,16 @@ public class HolidayWritePlatformServiceJpaRepositoryImpl implements HolidayWrit
             }
 
             // validate repaymentsRescheduledTo date
-            // 1. should be within a 7 days date range.
+            // 1. should be within a 30 days date range.
             // 2. Alternative date should not be an exist holiday.//TBD
             // 3. Holiday should not be on an repaymentsRescheduledTo date of
             // another holiday.//TBD
 
-            // restricting repaymentsRescheduledTo date to be within 7 days
+            // restricting repaymentsRescheduledTo date to be within 30 days
             // range
             // before or after from date and to date.
-            if (repaymentsRescheduledTo.isBefore(fromDate.minusDays(7)) || repaymentsRescheduledTo.isAfter(toDate.plusDays(7))) {
-                defaultUserMessage = "Repayments Rescheduled to date must be within 7 days before or after from and to dates";
+            if (repaymentsRescheduledTo.isBefore(fromDate.minusDays(30)) || repaymentsRescheduledTo.isAfter(toDate.plusDays(30))) {
+                defaultUserMessage = "Repayments Rescheduled to date must be within 30 days before or after from and to dates";
                 throw new HolidayDateException("repayments.rescheduled.to.must.be.within.range", defaultUserMessage, fromDate.toString(),
                         toDate.toString(), repaymentsRescheduledTo.toString());
             }