From 89a9d8f45c1f6ba79eec598e3eb080c6ae8328c9 Mon Sep 17 00:00:00 2001 From: Albert S Date: Sat, 24 Apr 2021 17:35:27 +0200 Subject: [PATCH] LaunchServer: Fix argument order to FileManager Correct the order we pass the arguments to the FileManager so opening file:// URLs works. The path is a positional argument that was passed after the flags. We need to make sure the flags are passed before positional arguments. --- Userland/Services/LaunchServer/Launcher.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Userland/Services/LaunchServer/Launcher.cpp b/Userland/Services/LaunchServer/Launcher.cpp index b5e8fc5b95..63eb77c98a 100644 --- a/Userland/Services/LaunchServer/Launcher.cpp +++ b/Userland/Services/LaunchServer/Launcher.cpp @@ -283,9 +283,9 @@ bool Launcher::open_file_url(const URL& url) if (url.fragment().is_empty()) { fm_arguments.append(url.path()); } else { - fm_arguments.append(String::formatted("{}/{}", url.path(), url.fragment())); fm_arguments.append("-s"); fm_arguments.append("-r"); + fm_arguments.append(String::formatted("{}/{}", url.path(), url.fragment())); } return spawn("/bin/FileManager", fm_arguments); }