fix: Wait for cleanRealm before fetching

Infomaniak · Oct 30, 2024 · f10ead0 · f10ead0
1 parent 38e2692
commit f10ead0
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 16 deletions.
diff --git a/Mail/Views/Settings/General/SettingsThreadModeView.swift b/Mail/Views/Settings/General/SettingsThreadModeView.swift
@@ -31,6 +31,7 @@ struct ThreadModeSettingUpdate: Identifiable {
 
 struct SettingsThreadModeView: View {
     @LazyInjectService private var matomo: MatomoUtils
+    @LazyInjectService private var accountManager: AccountManager
 
     @State private var selectedValue: ThreadMode
     @ModalState(wrappedValue: nil, context: ContextKeys.settings) private var threadModeSettingUpdate: ThreadModeSettingUpdate?
@@ -73,7 +74,10 @@ struct SettingsThreadModeView: View {
                     secondaryButtonTitle: MailResourcesStrings.Localizable.buttonCancel
                 ) {
                     selectedValue = threadModeUpdate.newSetting
-                    UserDefaults.shared.threadMode = selectedValue
+                    Task {
+                        accountManager.updateConversationSettings()
+                        UserDefaults.shared.threadMode = selectedValue
+                    }
                 }
             }
         }

diff --git a/Mail/Views/Settings/SettingsView.swift b/Mail/Views/Settings/SettingsView.swift
@@ -219,9 +219,6 @@ struct SettingsView: View {
                 }
             }
         }
-        .onChange(of: threadMode) { _ in
-            accountManager.updateConversationSettings()
-        }
         .background(MailResourcesAsset.backgroundColor.swiftUIColor)
         .navigationBarTitle(MailResourcesStrings.Localizable.settingsTitle, displayMode: .inline)
         .backButtonDisplayMode(.minimal)

diff --git a/MailCore/Cache/MailboxManager/MailboxManager+DB.swift b/MailCore/Cache/MailboxManager/MailboxManager+DB.swift
@@ -20,20 +20,18 @@ import Foundation
 
 public extension MailboxManager {
     func cleanRealm() {
-        Task {
-            try? writeTransaction { writableRealm in
-                let threads = writableRealm.objects(Thread.self)
-                writableRealm.delete(threads)
+        try? writeTransaction { writableRealm in
+            let threads = writableRealm.objects(Thread.self)
+            writableRealm.delete(threads)
 
-                let messages = writableRealm.objects(Message.self)
-                writableRealm.delete(messages)
+            let messages = writableRealm.objects(Message.self)
+            writableRealm.delete(messages)
 
-                let folders = writableRealm.objects(Folder.self)
-                for folder in folders {
-                    folder.cursor = nil
-                    folder.resetHistoryInfo()
-                    folder.computeUnreadCount()
-                }
+            let folders = writableRealm.objects(Folder.self)
+            for folder in folders {
+                folder.cursor = nil
+                folder.resetHistoryInfo()
+                folder.computeUnreadCount()
             }
         }
     }