From 488b3958a892ecc439c3727cf2e9ae0c6b5c0ac8 Mon Sep 17 00:00:00 2001 From: Manuel Rego Casasnovas Date: Mon, 7 Mar 2011 10:08:14 +0100 Subject: [PATCH] [Bug #916] Fixed issue checking if list of constraints is empty and not only if it is null. FEA: ItEr72S04BugFixing --- .../ganttz/data/criticalpath/CriticalPathCalculator.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ganttzk/src/main/java/org/zkoss/ganttz/data/criticalpath/CriticalPathCalculator.java b/ganttzk/src/main/java/org/zkoss/ganttz/data/criticalpath/CriticalPathCalculator.java index 6dc33c939..4323b9500 100644 --- a/ganttzk/src/main/java/org/zkoss/ganttz/data/criticalpath/CriticalPathCalculator.java +++ b/ganttzk/src/main/java/org/zkoss/ganttz/data/criticalpath/CriticalPathCalculator.java @@ -311,13 +311,14 @@ public class CriticalPathCalculator> { .getStartConstraintsFor(task); List> endConstraints = graph .getEndConstraintsFor(task); - if ((startConstraints == null) && (endConstraints == null)) { + if ((startConstraints == null || startConstraints.isEmpty()) + && (endConstraints == null || endConstraints.isEmpty())) { return null; } - if (startConstraints == null) { + if (startConstraints == null || startConstraints.isEmpty()) { return Constraint.coalesce(endConstraints); } - if (endConstraints == null) { + if (endConstraints == null || endConstraints.isEmpty()) { return Constraint.coalesce(startConstraints); } startConstraints.addAll(endConstraints);