diff --git a/libreplan-webapp/src/main/java/org/libreplan/web/planner/order/SaveCommandBuilder.java b/libreplan-webapp/src/main/java/org/libreplan/web/planner/order/SaveCommandBuilder.java index 50cd0a8e8..83b39055a 100644 --- a/libreplan-webapp/src/main/java/org/libreplan/web/planner/order/SaveCommandBuilder.java +++ b/libreplan-webapp/src/main/java/org/libreplan/web/planner/order/SaveCommandBuilder.java @@ -398,7 +398,7 @@ public class SaveCommandBuilder { private void removeTaskElementsWithTaskSourceNull() { List toRemove = taskElementDAO .getTaskElementsNoMilestonesWithoutTaskSource(); - List toRemoveLater = new ArrayList(); + List parentsWithChangesToSave = new ArrayList(); for (TaskElement taskElement : toRemove) { try { taskElementDAO.remove(taskElement.getId()); @@ -406,7 +406,7 @@ public class SaveCommandBuilder { TaskGroup parent = taskElement.getParent(); if (parent != null && !toRemove.contains(parent)) { parent.remove(taskElement); - toRemoveLater.add(parent); + parentsWithChangesToSave.add(parent); } LOG.info("TaskElement removed because of TaskSource was null. " @@ -417,7 +417,7 @@ public class SaveCommandBuilder { // so if it's not in the database there isn't any problem } } - for (TaskElement taskElement : toRemoveLater) { + for (TaskElement taskElement : parentsWithChangesToSave) { taskElementDAO.save(taskElement); } }