Skip to content

Commit b1f576f

Browse files
committed
Fix resetting in layer dialog
1 parent 2a8d87f commit b1f576f

2 files changed

Lines changed: 13 additions & 9 deletions

File tree

alpha-gui/src/main/java/com/techsenger/alpha/gui/diagram/LayerDialogPresenter.java

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,6 +93,19 @@ protected void postInitialize() {
9393
}
9494

9595
protected void onReset() {
96+
// some pages can be non initialized, so we update all layers here
97+
for (var layer : this.layerConfigs) {
98+
for (var m : layer.getModules()) {
99+
m.setIncluded(false);
100+
m.setReads(false);
101+
m.setExports(false);
102+
m.setOpens(false);
103+
m.setRequires(false);
104+
m.setRequests(false);
105+
m.setColored(false);
106+
}
107+
}
108+
// and only now update all initialized pages
96109
for (var p : getComposer().getPageHost().getComposer().getPagePorts()) {
97110
LayerPagePort pagePort = (LayerPagePort) p;
98111
pagePort.reset();

alpha-gui/src/main/java/com/techsenger/alpha/gui/diagram/LayerPagePresenter.java

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -38,15 +38,6 @@ public LayerPagePresenter(V view, PageItem item, LayerConfig layer) {
3838

3939
@Override
4040
public void reset() {
41-
for (var m : layer.getModules()) {
42-
m.setIncluded(false);
43-
m.setReads(false);
44-
m.setExports(false);
45-
m.setOpens(false);
46-
m.setRequires(false);
47-
m.setRequests(false);
48-
m.setColored(false);
49-
}
5041
getView().updateModules();
5142
getView().clearSelectAll();
5243
}

0 commit comments

Comments
 (0)