fixed mark read being slow

mlemgroup · Nov 2, 2023 · e484c8e · e484c8e
1 parent 4095e7b
commit e484c8e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/Mlem/Models/Content/Inbox/MentionModel.swift b/Mlem/Models/Content/Inbox/MentionModel.swift
@@ -145,17 +145,16 @@ extension MentionModel {
 
         // state fake
         await setPersonMention(APIPersonMention(from: personMention, read: !personMention.read))
-        await unreadTracker.toggleMentionRead(originalState: originalPersonMention.read)
 
         // call API and either update with latest info or revert state fake on fail
         do {
             let newMessage = try await inboxRepository.markMentionRead(id: personMention.id, isRead: personMention.read)
+            await unreadTracker.toggleMentionRead(originalState: originalPersonMention.read)
             await reinit(from: newMessage)
         } catch {
             hapticManager.play(haptic: .failure, priority: .high)
             errorHandler.handle(error)
             await setPersonMention(originalPersonMention)
-            await unreadTracker.toggleMentionRead(originalState: !originalPersonMention.read)
         }
     }
 

diff --git a/Mlem/Models/Content/Inbox/MessageModel.swift b/Mlem/Models/Content/Inbox/MessageModel.swift
@@ -57,17 +57,16 @@ class MessageModel: ContentIdentifiable, ObservableObject {
 
         // state fake
         await setPrivateMessage(APIPrivateMessage(from: privateMessage, read: !privateMessage.read))
-        await unreadTracker.toggleMessageRead(originalState: originalPrivateMessage.read)
 
         // call API and either update with latest info or revert state fake on fail
         do {
             let newMessage = try await inboxRepository.markMessageRead(id: privateMessage.id, isRead: privateMessage.read)
+            await unreadTracker.toggleMessageRead(originalState: originalPrivateMessage.read)
             await reinit(from: newMessage)
         } catch {
             hapticManager.play(haptic: .failure, priority: .high)
             errorHandler.handle(error)
             await setPrivateMessage(originalPrivateMessage)
-            await unreadTracker.toggleMessageRead(originalState: !originalPrivateMessage.read)
         }
     }
 

diff --git a/Mlem/Models/Content/Inbox/ReplyModel.swift b/Mlem/Models/Content/Inbox/ReplyModel.swift
@@ -146,17 +146,16 @@ extension ReplyModel {
 
         // state fake
         await setCommentReply(APICommentReply(from: commentReply, read: !commentReply.read))
-        await unreadTracker.toggleReplyRead(originalState: originalCommentReply.read)
 
         // call API and either update with latest info or revert state fake on fail
         do {
             let newReply = try await inboxRepository.markReplyRead(id: commentReply.id, isRead: commentReply.read)
+            await unreadTracker.toggleReplyRead(originalState: originalCommentReply.read)
             await reinit(from: newReply)
         } catch {
             hapticManager.play(haptic: .failure, priority: .high)
             errorHandler.handle(error)
             await setCommentReply(originalCommentReply)
-            await unreadTracker.toggleReplyRead(originalState: !originalCommentReply.read)
         }
     }