From eef554f9517dd18a7cfd66ef9e06f9d48384d9e5 Mon Sep 17 00:00:00 2001
From: Manuel Rego Casasnovas <mrego@igalia.com>
Date: Tue, 15 Feb 2011 09:20:38 +0100
Subject: [PATCH] Added warning log message when fake advance is NULL in
 AdvanceAssignmentPlanningModel.

FEA: ItEr70S04BugFixing
---
 .../planner/advances/AdvanceAssignmentPlanningModel.java | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/advances/AdvanceAssignmentPlanningModel.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/advances/AdvanceAssignmentPlanningModel.java
index 81c7671bb..641bb7420 100644
--- a/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/advances/AdvanceAssignmentPlanningModel.java
+++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/planner/advances/AdvanceAssignmentPlanningModel.java
@@ -22,6 +22,8 @@
 package org.navalplanner.web.planner.advances;
 
 
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 import org.navalplanner.business.advance.entities.AdvanceAssignment;
 import org.navalplanner.business.advance.entities.AdvanceMeasurement;
 import org.navalplanner.business.advance.entities.DirectAdvanceAssignment;
@@ -48,6 +50,9 @@ import org.zkoss.ganttz.extensions.IContextWithPlannerTask;
 public class AdvanceAssignmentPlanningModel implements
         IAdvanceAssignmentPlanningModel {
 
+    private static final Log LOG = LogFactory
+            .getLog(AdvanceAssignmentPlanningModel.class);
+
     @Autowired
     private ITaskElementDAO taskElementDAO;
 
@@ -124,6 +129,10 @@ public class AdvanceAssignmentPlanningModel implements
                     .calculateFakeDirectAdvanceAssignment(advance);
             if (fakedDirect != null) {
                 forceLoadAdvanceConsolidatedValues(fakedDirect);
+            } else {
+                LOG
+                        .warn("Fake direct advance assignment shouldn't be NULL for type '"
+                                + advance.getAdvanceType().getUnitName() + "'");
             }
         }
     }