fix: Don't print message in open in headless mode (#247)

unioslo · Nov 6, 2024 · f493ba3 · f493ba3
1 parent 09a2fab
commit f493ba3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/zabbix_cli/commands/cli.py b/zabbix_cli/commands/cli.py
@@ -135,7 +135,6 @@ def open_config_dir(
         print_path(directory)
     else:
         open_directory(directory, command=open_command, force=force)
-        success(f"Opened {directory}")
 
 
 @app.command("debug", hidden=True, rich_help_panel=HELP_PANEL)

diff --git a/zabbix_cli/utils/fs.py b/zabbix_cli/utils/fs.py
@@ -13,6 +13,8 @@
 from zabbix_cli.exceptions import ZabbixCLIError
 from zabbix_cli.exceptions import ZabbixCLIFileError
 from zabbix_cli.exceptions import ZabbixCLIFileNotFoundError
+from zabbix_cli.output.console import print_path
+from zabbix_cli.output.console import success
 
 logger = logging.getLogger(__name__)
 
@@ -59,13 +61,11 @@ def open_directory(
     elif sys.platform == "darwin":
         subprocess.run([command or "open", spath])
     else:  # Linux and Unix
-        if not os.environ.get("DISPLAY"):
-            from zabbix_cli.output.console import print_path
-
+        if not os.environ.get("DISPLAY") and not force:
             print_path(directory)
-            if not force:
-                return
+            return
         subprocess.run([command or "xdg-open", spath])
+    success(f"Opened {directory}")
 
 
 def mkdir_if_not_exists(path: Path) -> None: