From 0f4e3e4ee26430b3cf7aacf5a1a97ce2327f0785 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jacobo=20Aragunde=20P=C3=A9rez?= Date: Thu, 10 Nov 2011 19:44:12 +0100 Subject: [PATCH] [Bug #1141] Remove Task.reloadResourcesTextIfChange(). We think it's not necessary to reload the resources text when the start date of a task is updated, and that's what was being done here. This change caused a invalidate() on the ZK object of the task, which destroyed the old object and replaced it with a new one. Because of that, the setter we implemented to fix bug #1141 didn't work. FEA: ItEr75S04BugFixing --- ganttzk/src/main/java/org/zkoss/ganttz/data/Task.java | 8 -------- 1 file changed, 8 deletions(-) diff --git a/ganttzk/src/main/java/org/zkoss/ganttz/data/Task.java b/ganttzk/src/main/java/org/zkoss/ganttz/data/Task.java index 2c3bad395..6d8e60ac2 100644 --- a/ganttzk/src/main/java/org/zkoss/ganttz/data/Task.java +++ b/ganttzk/src/main/java/org/zkoss/ganttz/data/Task.java @@ -96,7 +96,6 @@ public abstract class Task implements ITaskFundamentalProperties { fundamentalPropertiesListeners.firePropertyChange("beginDate", previousStart, fundamentalProperties.getBeginDate()); fireEndDate(previousEnd); - reloadResourcesTextIfChange(newStart, previousStart); } @Override @@ -204,13 +203,6 @@ public abstract class Task implements ITaskFundamentalProperties { newStart); } - private void reloadResourcesTextIfChange(GanttDate newDate, - GanttDate previousDate) { - if (!ObjectUtils.equals(newDate, previousDate)) { - reloadResourcesText(); - } - } - public void fireChangesForPreviousValues(GanttDate previousStart, GanttDate previousEnd) { dependenciesEnforcerHook.setStartDate(previousStart, previousStart,