From 9f7e428e4ae865ac556d49cd0c550db8d3dfb856 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=93scar=20Gonz=C3=A1lez=20Fern=C3=A1ndez?= Date: Mon, 24 Jan 2011 18:42:07 +0100 Subject: [PATCH] [Bug #816] Fix bug The problem lied in that the @NotNull annotation was applied to a field. This is dangerous because if the object that is being validated is a proxy, the value is always null. Now the annotation was moved to the getter. FEA: ItEr69S04BugFixing --- .../business/planner/entities/ResourceAllocation.java | 2 +- 1 file changed, 1 insertion(+), 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 0f1101c1e..771d95cf8 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 @@ -443,7 +443,6 @@ public abstract class ResourceAllocation extends private AssignmentFunction assignmentFunction; @OnCopy(Strategy.SHARE) - @NotNull private ResourcesPerDay resourcesPerDay; private Integer intendedTotalHours; @@ -1418,6 +1417,7 @@ public abstract class ResourceAllocation extends return calculateResourcesPerDayFromAssignments(getConsolidatedAssignments()); } + @NotNull public ResourcesPerDay getResourcesPerDay() { if (resourcesPerDay == null) { return ResourcesPerDay.amount(0);