From 95b3be84861945e12f2dba2340641aede8164fd9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=93scar=20Gonz=C3=A1lez=20Fern=C3=A1ndez?= Date: Mon, 28 Feb 2011 13:59:40 +0100 Subject: [PATCH] [Bug #895] Fix bug The problem was that the resource allocation was a proxy and mistakenly the validation annotation was applied to the field. FEA: ItEr71S04BugFixing --- .../business/planner/entities/ResourceAllocation.java | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourceAllocation.java b/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourceAllocation.java index 2626ac4da..69e92499b 100644 --- a/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourceAllocation.java +++ b/navalplanner-business/src/main/java/org/navalplanner/business/planner/entities/ResourceAllocation.java @@ -454,7 +454,6 @@ public abstract class ResourceAllocation extends private AssignmentFunction assignmentFunction; @OnCopy(Strategy.SHARE) - @NotNull private ResourcesPerDay resourcesPerDay; private Integer intendedTotalHours; @@ -1545,6 +1544,13 @@ public abstract class ResourceAllocation extends return calculateResourcesPerDayFromAssignments(getConsolidatedAssignments()); } + // just called for validation purposes. It must be public, otherwise if it's + // a proxy the call is not intercepted. + @NotNull + public ResourcesPerDay getRawResourcesPerDay() { + return resourcesPerDay; + } + public ResourcesPerDay getResourcesPerDay() { if (resourcesPerDay == null) { return ResourcesPerDay.amount(0);