Merge pull request #3616 from ControlSystemStudio/CSSTUDIO-3537

shroffk · web-flow · commit 9a9d761f1a3d · 2025-10-31T09:26:02.000-04:00
CSSTUDIO-3537 Bugfix: Fix check for non-zero-length array (For ESS-specific navigator)
diff --git a/core/ui/src/main/java/org/phoebus/ui/docking/DockPane.java b/core/ui/src/main/java/org/phoebus/ui/docking/DockPane.java
@@ -658,7 +658,7 @@ else if (dock_parent instanceof SplitPane) // "dock_parent instanceof SplitPane"
             final SplitPane parent = (SplitPane) dock_parent;
             // Remove this dock pane from BorderPane
             Optional<Double> dividerPosition;
-            if (parent.getDividerPositions().length > 1) {
+            if (parent.getDividerPositions().length > 0) {
                 dividerPosition = Optional.of(parent.getDividerPositions()[0]);
             }
             else {

Original file line number	Diff line number	Diff line change
`@@ -658,7 +658,7 @@ else if (dock_parent instanceof SplitPane) // "dock_parent instanceof SplitPane"`
`658`	`658`	`final SplitPane parent = (SplitPane) dock_parent;`
`659`	`659`	`// Remove this dock pane from BorderPane`
`660`	`660`	`Optional<Double> dividerPosition;`
`661`		`- if (parent.getDividerPositions().length > 1) {`
	`661`	`+ if (parent.getDividerPositions().length > 0) {`
`662`	`662`	`dividerPosition = Optional.of(parent.getDividerPositions()[0]);`
`663`	`663`	`}`
`664`	`664`	`else {`