diff --git a/mifosng-provider/src/main/java/org/mifosplatform/portfolio/client/domain/Client.java b/mifosng-provider/src/main/java/org/mifosplatform/portfolio/client/domain/Client.java index 0e1322129..a8d7b366b 100644 --- a/mifosng-provider/src/main/java/org/mifosplatform/portfolio/client/domain/Client.java +++ b/mifosng-provider/src/main/java/org/mifosplatform/portfolio/client/domain/Client.java @@ -313,18 +313,6 @@ public final class Client extends AbstractPersistable { throw new PlatformApiDataValidationException(dataValidationErrors); } - if (isDateInTheFuture(activationLocalDate)) { - - final String defaultUserMessage = "Activation date cannot be in the future."; - final ApiParameterError error = ApiParameterError.parameterError("error.msg.clients.activationDate.in.the.future", - defaultUserMessage, ClientApiConstants.activationDateParamName, activationLocalDate); - - final List dataValidationErrors = new ArrayList(); - dataValidationErrors.add(error); - - throw new PlatformApiDataValidationException(dataValidationErrors); - } - this.activationDate = activationLocalDate.toDate(); this.activatedBy = currentUser; this.officeJoiningDate = this.activationDate; @@ -472,6 +460,16 @@ public final class Client extends AbstractPersistable { } private void validateActivationDate(final List dataValidationErrors) { + + if (getActivationLocalDate() != null && isDateInTheFuture(getActivationLocalDate())) { + + final String defaultUserMessage = "Activation date cannot be in the future."; + final ApiParameterError error = ApiParameterError.parameterError("error.msg.clients.activationDate.in.the.future", + defaultUserMessage, ClientApiConstants.activationDateParamName, getActivationLocalDate()); + + dataValidationErrors.add(error); + } + if (this.activationDate != null) { if (this.office.isOpeningDateAfter(getActivationLocalDate())) { final String defaultUserMessage = "Client activation date cannot be a date before the office opening date.";