From ab5792f59ffdca4008f35b54350ffb03e6e10c57 Mon Sep 17 00:00:00 2001
From: John Preston <johnprestonmail@gmail.com>
Date: Tue, 8 Nov 2022 18:21:39 +0400
Subject: [PATCH] Revert "Fix crash when asking download path in sandbox
 environment"

This reverts commit a4b044304706709eef09dee57f7b3d39ac6c01fe.
---
 Telegram/SourceFiles/core/file_utilities.cpp | 42 +++++++++-----------
 1 file changed, 19 insertions(+), 23 deletions(-)

diff --git a/Telegram/SourceFiles/core/file_utilities.cpp b/Telegram/SourceFiles/core/file_utilities.cpp
index 9d90b4848e..576d8bc02b 100644
--- a/Telegram/SourceFiles/core/file_utilities.cpp
+++ b/Telegram/SourceFiles/core/file_utilities.cpp
@@ -9,7 +9,6 @@ https://github.com/telegramdesktop/tdesktop/blob/master/LEGAL
 
 #include "boxes/abstract_box.h"
 #include "storage/localstorage.h"
-#include "storage/storage_account.h"
 #include "base/platform/base_platform_info.h"
 #include "base/platform/base_platform_file_utilities.h"
 #include "platform/platform_file_utilities.h"
@@ -174,32 +173,29 @@ QString DefaultDownloadPathFolder(not_null<Main::Session*> session) {
 }
 
 QString DefaultDownloadPath(not_null<Main::Session*> session) {
-	static auto Choosing = false;
-	const auto realDefaultPath = QStandardPaths::writableLocation(
+	if (KSandbox::isInside() && Core::App().settings().downloadPath().isEmpty()) {
+		QStringList files;
+		QByteArray remoteContent;
+		const auto success = Platform::FileDialog::Get(
+			nullptr,
+			files,
+			remoteContent,
+			tr::lng_download_path_choose(tr::now),
+			QString(),
+			FileDialog::internal::Type::ReadFolder,
+			QString());
+		if (success && !files.isEmpty() && !files[0].isEmpty()) {
+			const auto result = files[0].endsWith('/') ? files[0] : (files[0] + '/');
+			Core::App().settings().setDownloadPath(result);
+			Core::App().saveSettings();
+			return result;
+		}
+	}
+	return QStandardPaths::writableLocation(
 		QStandardPaths::DownloadLocation)
 		+ '/'
 		+ DefaultDownloadPathFolder(session)
 		+ '/';
-	if (KSandbox::isInside() && Core::App().settings().downloadPath().isEmpty()) {
-		if (Choosing) {
-			return session->local().tempDirectory();
-		}
-		Choosing = true;
-		FileDialog::GetFolder(
-			nullptr,
-			tr::lng_download_path_choose(tr::now),
-			realDefaultPath,
-			[](const QString &result) {
-				Core::App().settings().setDownloadPath(result.endsWith('/')
-					? result
-					: (result + '/'));
-				Core::App().saveSettings();
-				Choosing = false;
-			},
-			[] { Choosing = false; });
-		return session->local().tempDirectory();
-	}
-	return realDefaultPath;
 }
 
 namespace internal {