From cd065cbc14a32bfc58e9b669128beda52e4bd4b7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=93scar=20Gonz=C3=A1lez=20Fern=C3=A1ndez?=
 <ogonzalez@igalia.com>
Date: Fri, 1 Oct 2010 00:00:24 +0200
Subject: [PATCH] Protect against effortDuration being null.

Use always getter to protect against this possibility. The problem
must lie in a IntraDayDate created as result of a migration such that
the column for effortDuration is null. Otherwise a newly created
IntraDayDate never has a null effortDuration.

FEA: ItEr61S05BugFixing
---
 .../org/navalplanner/business/workingday/IntraDayDate.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/navalplanner-business/src/main/java/org/navalplanner/business/workingday/IntraDayDate.java b/navalplanner-business/src/main/java/org/navalplanner/business/workingday/IntraDayDate.java
index 7dbe94599..a98b5ea3d 100644
--- a/navalplanner-business/src/main/java/org/navalplanner/business/workingday/IntraDayDate.java
+++ b/navalplanner-business/src/main/java/org/navalplanner/business/workingday/IntraDayDate.java
@@ -82,7 +82,7 @@ public class IntraDayDate implements Comparable<IntraDayDate> {
     }
 
     public boolean isStartOfDay() {
-        return effortDuration.isZero();
+        return getEffortDuration().isZero();
     }
 
     @Override
@@ -118,7 +118,7 @@ public class IntraDayDate implements Comparable<IntraDayDate> {
     public int compareTo(IntraDayDate other) {
         int result = date.compareTo(other.date);
         if (result == 0) {
-            result = effortDuration.compareTo(other.effortDuration);
+            result = getEffortDuration().compareTo(other.getEffortDuration());
         }
         return result;
     }