MIFOSX-262: slight changes to anuruddha's commit to use joda date classes over java.util.
This commit is contained in:
+8
-4
@@ -141,6 +141,14 @@ public class Office extends AbstractPersistable<Long> {
|
||||
return actualChanges;
|
||||
}
|
||||
|
||||
public boolean isOpeningDateBefore(final LocalDate baseDate) {
|
||||
return getOpeningLocalDate().isBefore(baseDate);
|
||||
}
|
||||
|
||||
public boolean isOpeningDateAfter(final LocalDate activationLocalDate) {
|
||||
return getOpeningLocalDate().isAfter(activationLocalDate);
|
||||
}
|
||||
|
||||
private LocalDate getOpeningLocalDate() {
|
||||
LocalDate openingLocalDate = null;
|
||||
if (this.openingDate != null) {
|
||||
@@ -217,8 +225,4 @@ public class Office extends AbstractPersistable<Long> {
|
||||
|
||||
return match;
|
||||
}
|
||||
|
||||
public boolean isBefore(final Date baseDate) {
|
||||
return this.openingDate.before(baseDate);
|
||||
}
|
||||
}
|
||||
+9
-10
@@ -356,19 +356,18 @@ public final class Client extends AbstractPersistable<Long> {
|
||||
baseDataValidator.reset().parameter("middlename").value(this.middlename).ignoreIfNull().notExceedingLengthOf(50);
|
||||
baseDataValidator.reset().parameter("lastname").value(this.lastname).notBlank().notExceedingLengthOf(50);
|
||||
}
|
||||
|
||||
if(this.activationDate!=null)
|
||||
{
|
||||
if(!office.isBefore(this.activationDate)){
|
||||
final String defaultUserMessage = "Client activation date cannot be a date before the Office activation date";
|
||||
final ApiParameterError error = ApiParameterError.parameterError("error.msg.clients.activationDate.cannot.be.before.Office.activatoin.date",
|
||||
defaultUserMessage, "activationDate", this.activationDate);
|
||||
|
||||
if (this.activationDate != null) {
|
||||
if (office.isOpeningDateAfter(getActivationLocalDate())) {
|
||||
final String defaultUserMessage = "Client activation date cannot be a date before the office opening date.";
|
||||
final ApiParameterError error = ApiParameterError.parameterError(
|
||||
"error.msg.clients.activationDate.cannot.be.before.office.activation.date", defaultUserMessage, "activationDate",
|
||||
getActivationLocalDate());
|
||||
dataValidationErrors.add(error);
|
||||
}
|
||||
|
||||
|
||||
}
|
||||
if (!dataValidationErrors.isEmpty()) { throw new PlatformApiDataValidationException("validation.msg.validation.errors.exist",
|
||||
"Validation errors exist.", dataValidationErrors); }
|
||||
if (!dataValidationErrors.isEmpty()) { throw new PlatformApiDataValidationException(dataValidationErrors); }
|
||||
}
|
||||
|
||||
private void deriveDisplayName() {
|
||||
|
||||
Reference in New Issue
Block a user