RenJuan/TASKPM
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Bug #1444: Renamed variable with a more meaningful name.

Browse source 
FEA: ItEr76S04BugFixing
This commit is contained in:
Jacobo Aragunde Pérez 2012-05-15 14:01:35 +02:00
parent 1b2be981de
commit 778776a804
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
libreplan-webapp/src/main/java/org/libreplan/web/planner/order/SaveCommandBuilder.java
View file

@ -398,7 +398,7 @@ public class SaveCommandBuilder {
private void removeTaskElementsWithTaskSourceNull() {
List<TaskElement> toRemove = taskElementDAO
.getTaskElementsNoMilestonesWithoutTaskSource();
List<TaskElement> toRemoveLater = new ArrayList<TaskElement>();
List<TaskElement> parentsWithChangesToSave = new ArrayList<TaskElement>();
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);
}
}