CodeMirror/xpipe
1
0
Fork 0
mirror of https://github.com/xpipe-io/xpipe.git synced 2024-10-02 18:17:27 +13:00
Code Issues Projects Releases Packages Wiki Activity

Fix NPE if no terminal was set

Browse source
This commit is contained in:
crschnick 2024-01-04 09:11:34 +00:00
parent 6c9fddebe6
commit fcba5d1c54
1 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
app/src/main/java/io/xpipe/app/browser/action/MultiExecuteAction.java
View file

@ -12,10 +12,6 @@ import java.util.List;
public abstract class MultiExecuteAction implements BranchAction { public abstract class MultiExecuteAction implements BranchAction {
protected String filesArgument(List<BrowserEntry> entries) {
return entries.size() == 1 ? entries.get(0).getOptionallyQuotedFileName() : "(" + entries.size() + ")";
}
protected abstract String createCommand(ShellControl sc, OpenFileSystemModel model, BrowserEntry entry); protected abstract String createCommand(ShellControl sc, OpenFileSystemModel model, BrowserEntry entry);
@Override @Override
@ -37,9 +33,15 @@ public abstract class MultiExecuteAction implements BranchAction {
false); false);
} }
@Override
public boolean isApplicable(OpenFileSystemModel model, List<BrowserEntry> entries) {
return AppPrefs.get().terminalType().getValue() != null;
}
@Override @Override
public String getName(OpenFileSystemModel model, List<BrowserEntry> entries) { public String getName(OpenFileSystemModel model, List<BrowserEntry> entries) {
return "in " + AppPrefs.get().terminalType().getValue().toTranslatedString(); var t = AppPrefs.get().terminalType().getValue();
return "in " + (t != null ? t.toTranslatedString() : "?");
} }
}, },
new LeafAction() { new LeafAction() {