From 8eaf4e6324397fe954c0cdc61b4fdc161719d1fe Mon Sep 17 00:00:00 2001
From: Claudio Cambra <claudio.cambra@nextcloud.com>
Date: Wed, 27 Dec 2023 18:27:18 +0800
Subject: [PATCH] Move domain manager acquisition into separate function

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>
---
 src/gui/macOS/fileproviderxpc_mac.mm | 25 ++++++++++++++++++-------
 1 file changed, 18 insertions(+), 7 deletions(-)

diff --git a/src/gui/macOS/fileproviderxpc_mac.mm b/src/gui/macOS/fileproviderxpc_mac.mm
index 2a4f27766b..b530f4aac4 100644
--- a/src/gui/macOS/fileproviderxpc_mac.mm
+++ b/src/gui/macOS/fileproviderxpc_mac.mm
@@ -33,15 +33,11 @@ namespace Mac {
 
 Q_LOGGING_CATEGORY(lcFileProviderXPC, "nextcloud.gui.macos.fileprovider.xpc", QtInfoMsg)
 
-FileProviderXPC::FileProviderXPC(QObject *parent)
-    : QObject{parent}
+namespace XPCUtils
 {
-}
 
-void FileProviderXPC::start()
+NSArray<NSFileProviderManager *> *getDomainManagers()
 {
-    qCInfo(lcFileProviderXPC) << "Starting file provider XPC";
-
     dispatch_group_t group = dispatch_group_create();
     __block NSMutableArray<NSFileProviderManager *> *managers = NSMutableArray.array;
 
@@ -68,9 +64,24 @@ void FileProviderXPC::start()
 
     if (managers.count == 0) {
         qCWarning(lcFileProviderXPC) << "No domains found";
-        return;
     }
 
+    return managers.copy;
+}
+
+} // namespace XPCUtils
+
+FileProviderXPC::FileProviderXPC(QObject *parent)
+    : QObject{parent}
+{
+}
+
+void FileProviderXPC::start()
+{
+    qCInfo(lcFileProviderXPC) << "Starting file provider XPC";
+
+    const auto managers = XPCUtils::getDomainManagers();
+    dispatch_group_t group = dispatch_group_create();
     __block NSMutableArray<NSURL *> *urls = NSMutableArray.array;
 
     for (NSFileProviderManager *const manager in managers) {