From dd4f39abb4079793dc3455b00f181a8616fa8c0e Mon Sep 17 00:00:00 2001
From: MohitMali <futuretechnoworld.mohit@gmail.com>
Date: Tue, 7 Nov 2023 18:46:31 +0530
Subject: [PATCH] Added logs while reading reading the bookmarks/library data,
 and added logs before writing the library in file.

---
 .../kiwixmobile/core/dao/LibkiwixBookmarks.kt | 33 +++++++++++++++++--
 1 file changed, 31 insertions(+), 2 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
index 0a1a61365a..d368e59df9 100644
--- a/core/src/main/java/org/kiwix/kiwixmobile/core/dao/LibkiwixBookmarks.kt
+++ b/core/src/main/java/org/kiwix/kiwixmobile/core/dao/LibkiwixBookmarks.kt
@@ -65,11 +65,11 @@ class LibkiwixBookmarks @Inject constructor(
   }
 
   private val bookmarkFile: File by lazy {
-    File("$bookmarksFolderPath/bookmark.txt")
+    File("$bookmarksFolderPath/bookmark.xml")
   }
 
   private val libraryFile: File by lazy {
-    File("$bookmarksFolderPath/library.txt")
+    File("$bookmarksFolderPath/library.xml")
   }
 
   init {
@@ -83,6 +83,21 @@ class LibkiwixBookmarks @Inject constructor(
     if (!bookmarkFile.isFileExist()) bookmarkFile.createNewFile()
     // set up manager to read the bookmarks from this file
     manager.readBookmarkFile(bookmarkFile.canonicalPath)
+
+    library.booksIds
+      .asSequence()
+      .map(library::getBookById)
+      .forEach {
+        Log.e(
+          TAG,
+          "readLibraryFromFile: " +
+            "reading books from a file via the manager.readFile() method current books size" +
+            " current books size in the library is = ${library.booksIds.size}\n" +
+            "book path = ${it.path}\n" +
+            "book title = ${it.title}\n" +
+            "book id = ${it.name}\n",
+        )
+      }
   }
 
   fun bookmarks(): Flowable<List<Page>> =
@@ -160,6 +175,20 @@ class LibkiwixBookmarks @Inject constructor(
    * to prevent potential data loss and ensures that the library holds the updated ZIM file paths and favicons.
    */
   private fun writeBookMarksAndSaveLibraryToFile() {
+    library.booksIds
+      .asSequence()
+      .map(library::getBookById)
+      .forEach {
+        Log.e(
+          TAG,
+          "writeBookMarksAndSaveLibraryToFile:" +
+            " Trying to write a file with the library.writeFile() method." +
+            " current books size in the library is = ${library.booksIds.size}\n" +
+            "book path = ${it.path}\n" +
+            "book title = ${it.title}\n" +
+            "book id = ${it.name}\n",
+        )
+      }
     // Save the library, which contains ZIM file paths and favicons, to a file.
     library.writeToFile(libraryFile.canonicalPath)