Check ID instead of entire source object when toggling source selection

msasikanth · Apr 30, 2024 · 8cee489 · 8cee489
1 parent 15cf446
commit 8cee489
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/shared/src/commonMain/kotlin/dev/sasikanth/rss/reader/feeds/FeedsPresenter.kt b/shared/src/commonMain/kotlin/dev/sasikanth/rss/reader/feeds/FeedsPresenter.kt
@@ -291,8 +291,10 @@ class FeedsPresenter(
     private fun onToggleSourceSelection(source: Source) {
       _state.update {
         val selectedFeeds = _state.value.selectedSources
-        if (selectedFeeds.contains(source)) {
-          it.copy(selectedSources = selectedFeeds - setOf(source))
+        if (selectedFeeds.any { selectedFeed -> selectedFeed.id == source.id }) {
+          it.copy(
+            selectedSources = selectedFeeds.filterNot { feed -> feed.id == source.id }.toSet()
+          )
         } else {
           it.copy(selectedSources = selectedFeeds + setOf(source))
         }