fix: stable mark all feed list, fixed #245

Signed-off-by: Innei <i@innei.in>
RSSNext · Aug 30, 2024 · c9d098f · c9d098f
1 parent 4287b5d
commit c9d098f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/renderer/src/store/subscription/store.ts b/src/renderer/src/store/subscription/store.ts
@@ -153,16 +153,18 @@ class SubscriptionActions {
   async markReadByFeedIds(...args: [string[], FeedViewType?, MarkReadFilter?]) {
     const [feedIds, view, filter] = args
 
+    const stableFeedIds = feedIds.concat()
+
     doMutationAndTransaction(
       () =>
         apiClient.reads.all.$post({
           json: {
-            feedIdList: feedIds,
+            feedIdList: stableFeedIds,
             ...filter,
           },
         }),
       async () => {
-        for (const feedId of feedIds) {
+        for (const feedId of stableFeedIds) {
           // We can not process this logic in local, so skip it. and then we will fetch the unread count from server.
           !filter && feedUnreadActions.updateByFeedId(feedId, 0)
           entryActions.patchManyByFeedId(feedId, { read: true }, filter)

diff --git a/src/renderer/src/store/unread/index.ts b/src/renderer/src/store/unread/index.ts
@@ -34,9 +34,10 @@ class FeedUnreadActions {
       for (const [key, value] of data) {
         state.data[key] = value
       }
-      FeedUnreadService.updateFeedUnread(data)
       return { ...state }
     })
+
+    FeedUnreadService.updateFeedUnread(data)
   }
 
   async fetchUnreadByView(view: FeedViewType | undefined) {