Skip to content

Commit

Permalink
CSSTUDIO-2482 Bugfix: getPaneOrSplit() did not return the correct val…
Browse files Browse the repository at this point in the history
…ue for instances of SplitDock.
  • Loading branch information
abrahamwolk committed Jul 17, 2024
1 parent a2c4de2 commit 3feaa38
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions core/ui/src/main/java/org/phoebus/ui/docking/DockStage.java
Original file line number Diff line number Diff line change
Expand Up @@ -342,16 +342,21 @@ public static BorderPane getLayout(final Stage stage)
public static Node getPaneOrSplit(final Stage stage)
{
Node container = getLayout(stage).getCenter();
if (container instanceof SplitPane) {
if (container instanceof DockPane || container instanceof SplitDock) {
// Note: the check for "instanceof SplitDock" must occur
// before "instanceof SplitPane" (the next clause),
// since the class SplitDock extends the class
// SplitPane. (Otherwise, the wrong code is run
// for instances of SplitDock.)
return container;
}
else if (container instanceof SplitPane) {
SplitPane splitPane = (SplitPane) container;
var maybeDockPaneOrSplitDock = splitPane.getItems().stream().filter(item -> item instanceof DockPane || item instanceof SplitDock).findFirst();
if (maybeDockPaneOrSplitDock.isPresent()) {
return maybeDockPaneOrSplitDock.get();
}
}
if (container instanceof DockPane ||
container instanceof SplitDock)
return container;
throw new IllegalStateException("Expect DockPane or SplitDock, got " + container);
}

Expand Down

0 comments on commit 3feaa38

Please sign in to comment.