From 78fc9ba51ecf0d44732581e94d29be8d1493ae2f Mon Sep 17 00:00:00 2001
From: Diego Pino Garcia <dpino@igalia.com>
Date: Tue, 22 Mar 2011 10:43:15 +0100
Subject: [PATCH] [Bug #943] Temporal fix, LOG error instead of launching
 exception

FEA: ItEr73S04BugFixing
---
 .../business/planner/entities/ResourceAllocation.java    | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

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 c22a0ce5f..701dea260 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
@@ -39,6 +39,8 @@ import java.util.Map.Entry;
 import java.util.Set;
 
 import org.apache.commons.lang.Validate;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.hibernate.validator.Min;
 import org.hibernate.validator.NotNull;
 import org.joda.time.LocalDate;
@@ -78,6 +80,8 @@ import org.navalplanner.business.workingday.ResourcesPerDay;
 public abstract class ResourceAllocation<T extends DayAssignment> extends
         BaseEntity {
 
+    private static final Log LOG = LogFactory.getLog(ResourceAllocation.class);
+
     public static <T extends ResourceAllocation<?>> List<T> getSatisfied(
             Collection<T> resourceAllocations) {
         Validate.notNull(resourceAllocations);
@@ -241,8 +245,9 @@ public abstract class ResourceAllocation<T extends DayAssignment> extends
                 List<ResourcesPerDayModification> allocations) {
             for (ResourcesPerDayModification r : allocations) {
                 if (isZero(r.getGoal().getAmount())) {
-                    throw new IllegalArgumentException(
-                            "all resources per day must be no zero");
+                    // FIXME: Temporal fix for bug 943, log error instead of
+                    // launching exception
+                    LOG.error("all resources per day must be no zero");
                 }
             }
         }