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

ItEr50S04ValidacionEProbasFuncionaisItEr49S04: Applying several refactorings to simplify the reload method

Browse source
This commit is contained in:
Óscar González Fernández 2010-03-10 00:41:38 +01:00
parent a56ca69d4c
commit 540294af87
1 changed files with 12 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
ganttzk/src/main/java/org/zkoss/ganttz/TaskList.java
View file

@ -318,10 +318,6 @@ public class TaskList extends XulElement implements AfterCompose {
response("adjust_height", new AuInvoke(TaskList.this, "adjust_height"));
}
public void hideTaskComponent(TaskComponent subtaskComponent) {
removeChild(subtaskComponent.getRow());
}
public void redrawDependencies() {
getGanttPanel().getDependencyList().redrawDependencies();
}
@ -359,27 +355,27 @@ public class TaskList extends XulElement implements AfterCompose {
addPendingTasks(tasksPendingToAdd, rowFor(task),
relocate);
}
if (predicate.accepts(task)) {
if (!visibleTasks.contains(task)) {
final boolean isShown = visibleTasks.contains(task);
if (predicate.accepts(task) != isShown) {
if (isShown) {
makeDisappear(task);
} else {
tasksPendingToAdd.add(task);
}
} else {
if (visibleTasks.contains(task)) {
TaskComponent taskComponent = find(task);
hideTaskComponent(taskComponent);
visibleTasks.remove(task);
task.setVisible(false);
}
}
if (task instanceof TaskContainer) {
reload(task.getTasks(), tasksPendingToAdd, relocate);
}
}
}
private void makeDisappear(Task task) {
TaskComponent taskComponent = find(task);
removeChild(taskComponent.getRow());
visibleTasks.remove(task);
task.setVisible(false);
}
private TaskRow rowFor(Task task) {
TaskComponent taskComponent = find(task);
return taskComponent == null ? null : taskComponent.getRow();