Don't unnecessarily fail to provide parentItemIdentifier in NextcloudFileProviderFile...

Signed-off-by: Claudio Cambra <claudio.cambra@nextcloud.com>

diff --git a/shell_integration/MacOSX/NextcloudIntegration/FileProviderExt/NextcloudFileProviderFilesUtils.swift b/shell_integration/MacOSX/NextcloudIntegration/FileProviderExt/NextcloudFileProviderFilesUtils.swift
index eaa4ce0dec6ca5a88991870a9fad16688b0cb1b4..0ffa48029d5b791f13aeff91a7eb9b254eaf3d65 100644 (file)
--- a/shell_integration/MacOSX/NextcloudIntegration/FileProviderExt/NextcloudFileProviderFilesUtils.swift
+++ b/shell_integration/MacOSX/NextcloudIntegration/FileProviderExt/NextcloudFileProviderFilesUtils.swift
@@ -91,15 +91,22 @@ func createFileOrDirectoryLocally(metadata: NextcloudItemMetadataTable) {
 
 func parentItemIdentifierFromMetadata(_ metadata: NextcloudItemMetadataTable) -> NSFileProviderItemIdentifier? {
     let homeServerFilesUrl = metadata.urlBase + "/remote.php/dav/files/" + metadata.userId
-    let ncDatabase = NextcloudFilesDatabaseManager.shared
 
-    guard let itemParentDirectory = ncDatabase.parentDirectoryMetadataForItem(metadata) else { return nil }
-    if itemParentDirectory.serverUrl == homeServerFilesUrl {
+    if metadata.serverUrl == homeServerFilesUrl {
         return .rootContainer
-    } else if let parentDirectoryMetadata = ncDatabase.itemMetadataFromOcId(itemParentDirectory.ocId) {
+    }
+
+    let ncDatabase = NextcloudFilesDatabaseManager.shared
+    guard let itemParentDirectory = ncDatabase.parentDirectoryMetadataForItem(metadata) else {
+        NSLog("Could not get item parent directory metadata for metadata with ocId: %@ and fileName: %@, returning nil", metadata.ocId, metadata.fileName)
+        return nil
+    }
+
+    if let parentDirectoryMetadata = ncDatabase.itemMetadataFromOcId(itemParentDirectory.ocId) {
         return NSFileProviderItemIdentifier(parentDirectoryMetadata.ocId)
     }
 
+    NSLog("Could not get item parent directory item metadata for metadata with ocId: %@ and fileName: %@, returning nil", metadata.ocId, metadata.fileName)
     return nil
 }