Improved the getting bookmarks for current book.

kiwix · Jan 27, 2025 · 053b6e0 · 053b6e0
1 parent b1c0658
commit 053b6e0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/core/src/main/java/org/kiwix/kiwixmobile/core/dao/LibkiwixBookmarks.kt b/core/src/main/java/org/kiwix/kiwixmobile/core/dao/LibkiwixBookmarks.kt
@@ -129,10 +129,10 @@ class LibkiwixBookmarks @Inject constructor(
     } ?: emptyList()
   }
 
-  fun bookmarkUrlsForCurrentBook(zimFileReader: ZimFileReader): Flowable<List<String>> =
+  fun bookmarkUrlsForCurrentBook(zimId: String): Flowable<List<String>> =
     flowableBookmarkList()
       .map { bookmarksList ->
-        bookmarksList.filter { it.zimId == zimFileReader.id }
+        bookmarksList.filter { it.zimId == zimId }
           .map(LibkiwixBookmarkItem::bookmarkUrl)
       }
       .subscribeOn(Schedulers.io())

diff --git a/core/src/main/java/org/kiwix/kiwixmobile/core/main/CoreReaderFragment.kt b/core/src/main/java/org/kiwix/kiwixmobile/core/main/CoreReaderFragment.kt
@@ -1819,7 +1819,7 @@ abstract class CoreReaderFragment :
   protected fun setUpBookmarks(zimFileReader: ZimFileReader) {
     safeDispose()
     bookmarkingDisposable = Flowable.combineLatest(
-      libkiwixBookmarks?.bookmarkUrlsForCurrentBook(zimFileReader),
+      libkiwixBookmarks?.bookmarkUrlsForCurrentBook(zimFileReader.id),
       webUrlsProcessor,
       List<String?>::contains
     )