From dcae1c609af92ab3b52f9898cf961326d8c6cf36 Mon Sep 17 00:00:00 2001
From: Ash <Glaxyinfinite@outlook.com>
Date: Wed, 17 Jan 2024 16:28:34 +0800
Subject: [PATCH] fix(fever): resolve issue with orphaned articles during sync

---
 .../reader/domain/service/FeverRssService.kt  | 32 ++++++++++++-------
 1 file changed, 21 insertions(+), 11 deletions(-)

diff --git a/app/src/main/java/me/ash/reader/domain/service/FeverRssService.kt b/app/src/main/java/me/ash/reader/domain/service/FeverRssService.kt
index 4431a34a8..e1d8f87fe 100644
--- a/app/src/main/java/me/ash/reader/domain/service/FeverRssService.kt
+++ b/app/src/main/java/me/ash/reader/domain/service/FeverRssService.kt
@@ -91,8 +91,8 @@ class FeverRssService @Inject constructor(
      * used as the starting mark for the next pull until the number of articles
      * obtained is 0 or their quantity exceeds 250, at which point the pulling process stops.
      *
-     * 1. Fetch the Fever groups
-     * 2. Fetch the Fever feeds (including favicons)
+     * 1. Fetch the Fever groups (may need to remove orphaned groups)
+     * 2. Fetch the Fever feeds (including favicons, may need to remove orphaned feeds)
      * 3. Fetch the Fever articles
      * 4. Synchronize read/unread and starred/un-starred items
      */
@@ -106,15 +106,20 @@ class FeverRssService @Inject constructor(
             val feverAPI = getFeverAPI()
 
             // 1. Fetch the Fever groups
-            groupDao.insertOrUpdate(
-                feverAPI.getGroups().groups?.map {
-                    Group(
-                        id = accountId.spacerDollar(it.id!!),
-                        name = it.title ?: context.getString(R.string.empty),
-                        accountId = accountId,
-                    )
-                } ?: emptyList()
-            )
+            val groups = feverAPI.getGroups().groups?.map {
+                Group(
+                    id = accountId.spacerDollar(it.id!!),
+                    name = it.title ?: context.getString(R.string.empty),
+                    accountId = accountId,
+                )
+            } ?: emptyList()
+            groupDao.insertOrUpdate(groups)
+            val groupIds = groups.map { it.id }
+            groupDao.queryAll(accountId).forEach {
+                if (!groupIds.contains(it.id)) {
+                    super.deleteGroup(it)
+                }
+            }
 
             // 2. Fetch the Fever feeds
             val feedsBody = feverAPI.getFeeds()
@@ -126,6 +131,11 @@ class FeverRssService @Inject constructor(
                     }
                 }
             }
+            feedDao.queryAll(accountId).forEach {
+                if (!feedsGroupsMap.contains(it.id.dollarLast())) {
+                    super.deleteFeed(it)
+                }
+            }
 
             // Fetch the Fever favicons
             val faviconsById = feverAPI.getFavicons().favicons?.associateBy { it.id } ?: emptyMap()