From 97390ce668b89e03da7a49b2764024caf0dee503 Mon Sep 17 00:00:00 2001 From: Manuel Rego Casasnovas Date: Thu, 4 Aug 2011 11:32:11 +0200 Subject: [PATCH] [Bug #1121] Fix issue adding @AssertTrue on AdvanceType entity * Now entity checks that defaultMaxValue is greater than unitPrecision * Default value for defaultMaxValue was set to 100 (as 0 doesn't have too much sense) FEA: ItEr75S04BugFixing --- .../business/advance/entities/AdvanceType.java | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/navalplanner-business/src/main/java/org/navalplanner/business/advance/entities/AdvanceType.java b/navalplanner-business/src/main/java/org/navalplanner/business/advance/entities/AdvanceType.java index cb253eb11..732437e51 100644 --- a/navalplanner-business/src/main/java/org/navalplanner/business/advance/entities/AdvanceType.java +++ b/navalplanner-business/src/main/java/org/navalplanner/business/advance/entities/AdvanceType.java @@ -68,7 +68,7 @@ public class AdvanceType extends BaseEntity implements IHumanIdentifiable{ private String unitName; @NotNull - private BigDecimal defaultMaxValue = BigDecimal.ZERO; + private BigDecimal defaultMaxValue = new BigDecimal(100); @NotNull private boolean updatable = true; @@ -262,4 +262,13 @@ public class AdvanceType extends BaseEntity implements IHumanIdentifiable{ return true; } } + + @AssertTrue(message = "the default max value must be greater than the precision value") + public boolean checkDefaultMaxValueGreaterThanPrecision() { + if (defaultMaxValue.compareTo(unitPrecision) == -1) { + return false; + } + return true; + } + }