From 8a684caf1b0045386ca149ce089f65e5b9386291 Mon Sep 17 00:00:00 2001
From: crschnick <crschnick@xpipe.io>
Date: Mon, 1 Jan 2024 15:57:35 +0000
Subject: [PATCH] Require macos applications to be in some applications
 directory

---
 .../io/xpipe/app/prefs/ExternalApplicationType.java    | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/io/xpipe/app/prefs/ExternalApplicationType.java b/app/src/main/java/io/xpipe/app/prefs/ExternalApplicationType.java
index 681dee958..a5155b6df 100644
--- a/app/src/main/java/io/xpipe/app/prefs/ExternalApplicationType.java
+++ b/app/src/main/java/io/xpipe/app/prefs/ExternalApplicationType.java
@@ -65,13 +65,9 @@ public abstract class ExternalApplicationType implements PrefsChoiceValue {
                         }
                     }).sorted(Comparator.comparingInt(value -> value.length())).toList();
 
-                    // Prefer app in proper applications directory
-                    var app = valid.stream().filter(s -> s.contains("/Applications")).findFirst();
-                    if (app.isPresent()) {
-                        return app.map(Path::of);
-                    }
-
-                    return valid.stream().findFirst().map(Path::of);
+                    // Require app in proper applications directory
+                    var app = valid.stream().filter(s -> s.contains("Applications")).findFirst();
+                    return app.map(Path::of);
                 }
             } catch (Exception e) {
                 ErrorEvent.fromThrowable(e).omit().handle();