From c738331acdb673eae00b107c415eb9d659c92c6b Mon Sep 17 00:00:00 2001
From: crschnick <crschnick@xpipe.io>
Date: Thu, 20 Mar 2025 10:18:49 +0000
Subject: [PATCH] Disable terminal logging when not supported

---
 app/src/main/java/io/xpipe/app/prefs/TerminalCategory.java | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/app/src/main/java/io/xpipe/app/prefs/TerminalCategory.java b/app/src/main/java/io/xpipe/app/prefs/TerminalCategory.java
index af2524357..5e7e3f8a4 100644
--- a/app/src/main/java/io/xpipe/app/prefs/TerminalCategory.java
+++ b/app/src/main/java/io/xpipe/app/prefs/TerminalCategory.java
@@ -15,6 +15,7 @@ import io.xpipe.app.terminal.ExternalTerminalType;
 import io.xpipe.app.terminal.TerminalLauncher;
 import io.xpipe.app.util.*;
 
+import javafx.application.Platform;
 import javafx.beans.binding.Bindings;
 import javafx.geometry.Insets;
 import javafx.geometry.Pos;
@@ -58,6 +59,10 @@ public class TerminalCategory extends AppPrefsCategory {
             var feature = LicenseProvider.get().getFeature("logging");
             if (newValue && !feature.isSupported()) {
                 try {
+                    // Disable it again so people don't forget that they left it on
+                    Platform.runLater(() -> {
+                        prefs.enableTerminalLogging.set(false);
+                    });
                     feature.throwIfUnsupported();
                 } catch (LicenseRequiredException ex) {
                     ErrorEvent.fromThrowable(ex).handle();