From 04bdd1937bec071f827b98886d22a9c2d84d617e Mon Sep 17 00:00:00 2001 From: Manuel Rego Casasnovas Date: Tue, 9 Oct 2012 09:29:41 +0200 Subject: [PATCH] Revert "Bug #1320: Fix issue changing methods to get constraints for a task" This reverts commit 8a98bf466623cf37da782b898ce91ecb6cc30ef0 because of the solution for bug #1320 was causing a new bug #1540. --- .../web/planner/TaskElementAdapter.java | 41 +++++++------------ 1 file changed, 14 insertions(+), 27 deletions(-) diff --git a/libreplan-webapp/src/main/java/org/libreplan/web/planner/TaskElementAdapter.java b/libreplan-webapp/src/main/java/org/libreplan/web/planner/TaskElementAdapter.java index 0a9651f17..08e3e1326 100644 --- a/libreplan-webapp/src/main/java/org/libreplan/web/planner/TaskElementAdapter.java +++ b/libreplan-webapp/src/main/java/org/libreplan/web/planner/TaskElementAdapter.java @@ -120,7 +120,6 @@ public class TaskElementAdapter { public static List> getStartConstraintsFor( TaskElement taskElement, LocalDate orderInitDate) { - List> constraints = new ArrayList>(); if (taskElement instanceof ITaskPositionConstrained) { ITaskPositionConstrained task = (ITaskPositionConstrained) taskElement; TaskPositionConstraint startConstraint = task @@ -130,31 +129,25 @@ public class TaskElementAdapter { switch (constraintType) { case AS_SOON_AS_POSSIBLE: if (orderInitDate != null) { - constraints.add(biggerOrEqualThan(toGantt(orderInitDate))); + return Collections + .singletonList(biggerOrEqualThan(toGantt(orderInitDate))); } - break; + return Collections.emptyList(); case START_IN_FIXED_DATE: - constraints.add(equalTo(toGantt(startConstraint - .getConstraintDate()))); - break; + return Collections + .singletonList(equalTo(toGantt(startConstraint + .getConstraintDate()))); case START_NOT_EARLIER_THAN: - constraints.add(biggerOrEqualThan(toGantt(startConstraint - .getConstraintDate()))); - break; + return Collections + .singletonList(biggerOrEqualThan(toGantt(startConstraint + .getConstraintDate()))); } } - if (!taskElement.isRoot() && taskElement instanceof TaskGroup) { - for (TaskElement child : ((TaskGroup) taskElement).getChildren()) { - constraints - .addAll(getStartConstraintsFor(child, orderInitDate)); - } - } - return constraints; + return Collections.emptyList(); } public static List> getEndConstraintsFor( TaskElement taskElement, LocalDate deadline) { - List> constraints = new ArrayList>(); if (taskElement instanceof ITaskPositionConstrained) { ITaskPositionConstrained task = (ITaskPositionConstrained) taskElement; TaskPositionConstraint endConstraint = task.getPositionConstraint(); @@ -162,21 +155,15 @@ public class TaskElementAdapter { switch (type) { case AS_LATE_AS_POSSIBLE: if (deadline != null) { - constraints.add(lessOrEqualThan(toGantt(deadline))); + return Collections + .singletonList(lessOrEqualThan(toGantt(deadline))); } - break; case FINISH_NOT_LATER_THAN: GanttDate date = toGantt(endConstraint.getConstraintDate()); - constraints.add(lessOrEqualThan(date)); - break; + return Collections.singletonList(lessOrEqualThan(date)); } } - if (!taskElement.isRoot() && taskElement instanceof TaskGroup) { - for (TaskElement child : ((TaskGroup) taskElement).getChildren()) { - constraints.addAll(getEndConstraintsFor(child, deadline)); - } - } - return constraints; + return Collections.emptyList(); } public static GanttDate toGantt(IntraDayDate date) {