From fc40838fbdb3f27e8044fb8955da4e82ec737b2c Mon Sep 17 00:00:00 2001 From: crschnick Date: Tue, 7 Apr 2026 10:47:16 +0000 Subject: [PATCH] Fix NPE --- .../menu/impl/OpenTerminalInDirectoryMenuProvider.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/io/xpipe/app/browser/menu/impl/OpenTerminalInDirectoryMenuProvider.java b/app/src/main/java/io/xpipe/app/browser/menu/impl/OpenTerminalInDirectoryMenuProvider.java index bf9e0e495..d30887909 100644 --- a/app/src/main/java/io/xpipe/app/browser/menu/impl/OpenTerminalInDirectoryMenuProvider.java +++ b/app/src/main/java/io/xpipe/app/browser/menu/impl/OpenTerminalInDirectoryMenuProvider.java @@ -30,7 +30,9 @@ public class OpenTerminalInDirectoryMenuProvider implements BrowserMenuLeafProvi ? List.of(model.getCurrentDirectory().getPath()) : Collections.singletonList((FilePath) null); for (var dir : dirs) { - var name = model.getFileSystem().supportsTerminalWorkingDirectory() ? dir.toString() : null; + // IntelliJ says the null check is not needed, but it is! + // noinspection all + var name = model.getFileSystem().supportsTerminalWorkingDirectory() && dir != null ? dir.toString() : null; model.openTerminalAsync( name, model.getFileSystem().supportsTerminalWorkingDirectory() ? dir : null,