refactor: Rollback old optimisation that was not the source of the bug

Infomaniak · Dec 18, 2024 · 7b826b3 · 7b826b3
1 parent 5720aa7
commit 7b826b3
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/kDriveCore/Data/Models/Drive/Drive.swift b/kDriveCore/Data/Models/Drive/Drive.swift
@@ -183,9 +183,12 @@ public final class Drive: Object, Codable {
             return []
         }
 
-        // If File is not managed by Realm: cannot use the `.sorted(by:)` method :(
-        // Also the Realm sort can crash if managed by realm
-        let fileCategoriesIds = file.categories.sorted { $0.addedAt.compare($1.addedAt) == .orderedAscending }.map(\.categoryId)
+        let fileCategoriesIds: [Int]
+        if file.isManagedByRealm {
+            fileCategoriesIds = Array(file.categories.sorted(by: \.addedAt, ascending: true)).map(\.categoryId)
+        } else {
+            fileCategoriesIds = file.categories.sorted { $0.addedAt.compare($1.addedAt) == .orderedAscending }.map(\.categoryId)
+        }
         let filteredCategories = categories.filter("id IN %@", fileCategoriesIds)
 
         // Sort the categories