From 9532c2b71f43bc2b1fe6550ab2428384ba5b43ec Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=93scar=20Gonz=C3=A1lez=20Fern=C3=A1ndez?= Date: Fri, 11 Feb 2011 16:08:12 +0100 Subject: [PATCH] Avoid exception if effortAssigned is zero Add a warning if this happens and the totalEffort is not zero. FEA: ItEr70S04BugFixingx --- .../navalplanner/web/resourceload/LoadPeriodGenerator.java | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/resourceload/LoadPeriodGenerator.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/resourceload/LoadPeriodGenerator.java index 76b24ffc9..722a9d83b 100644 --- a/navalplanner-webapp/src/main/java/org/navalplanner/web/resourceload/LoadPeriodGenerator.java +++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/resourceload/LoadPeriodGenerator.java @@ -212,6 +212,11 @@ abstract class LoadPeriodGenerator { if (totalEffort.isZero()) { return effortAssigned.isZero() ? 0 : Integer.MAX_VALUE; } + if (effortAssigned.isZero()) { + LOG.warn("total effort is " + totalEffort + + " but effortAssigned is zero"); + return Integer.MAX_VALUE; + } Fraction fraction = totalEffort.divivedBy(effortAssigned); Fraction percentage = fraction.multiplyBy(Fraction.getFraction(100, 1)); return percentage.intValue();