Update proguard-rules.pro

app/proguard-rules.pro

@@ -14,8 +14,18 @@
 
 # Uncomment this to preserve the line number information for
 # debugging stack traces.
-#-keepattributes SourceFile,LineNumberTable
+-keepattributes SourceFile,LineNumberTable
 
 # If you keep the line number information, uncomment this to
 # hide the original source file name.
-#-renamesourcefileattribute SourceFile
+-renamesourcefileattribute SourceFile
+
+# For SerializedName
+
+-keepattributes *Annotation*
+
+-keepclassmembers,allowobfuscation class * {
+  @com.google.gson.annotations.SerializedName <fields>;
+}
+
+-keep,allowobfuscation @interface com.google.gson.annotations.SerializedName

app/src/main/java/com/pixelized/biblib/repository/book/BookUtils.kt

@@ -29,21 +29,22 @@ suspend fun loadNewBooks(
     cache: IAPICacheRepository,
 ): Boolean {
     val cached = cache.new
-    val updated = client.service.new()
+    val response: BookListResponse = client.service.new()
 
-    if (updated.isError) {
+    return when {
-        Log.e("loadNewBooks", updated.message ?: "")
+        response.isError -> {
-        throw NewBookFetchException(updated.message)
+            Log.e("loadNewBooks", response.message ?: "")
+            throw NewBookFetchException(response.message)
         }
-
+        cached != response -> {
-    return if (cached != updated) {
+            cache.new = response
-        cache.new = updated
             true
-    } else {
+        }
+        else -> {
             false
         }
     }
-
+}
 
 
 /**
@@ -51,31 +52,35 @@ suspend fun loadNewBooks(
  * @param client the client used to fetch the data.
  * @param cache the cache used to read the new book data.
  * @param repository the repository to save the book data.
+ * @return factory the factory use to convert dto to bo.
  * @return this method will return true the books have been correctly fetch, otherwise false.
  */
 suspend fun loadAllBooks(
     client: IBibLibClient,
     cache: IAPICacheRepository,
     repository: IBookRepository,
+    factory: BookFactory = BookFactory(),
 ): Boolean {
-    val response: BookListResponse = client.service.list().also { response ->
+    val response: BookListResponse = client.service.list()
-        if (response.isError) {
+
+    return when {
+        response.isError -> {
             Log.e("loadAllBooks", response.message ?: "")
             throw BookFetchException(response.message)
         }
-
+        else -> {
             val newIds = cache.new?.data?.map { it.book_id } ?: listOf()
-        val factory = BookFactory()
+            val books = response.data?.map { dto ->
-        val books = response.data?.mapNotNull { dto ->
                 val isNew = newIds.contains(dto.book_id)
                 val index = newIds.indexOf(dto.book_id)
-            try {
                 factory.fromListResponseToBook(dto, isNew, index)
-            } catch (exception: Exception) {
+            }
-                null
+            if (books.isNullOrEmpty()) {
+                Log.e("loadAllBooks", "Book list is empty")
+                throw BookFetchException("Book list is empty")
+            }
+            books.let { data -> repository.update(data) }
+            true
         }
     }
-        books?.let { data -> repository.update(data) }
-    }
-    return response.isError.not()
 }

app/src/main/java/com/pixelized/biblib/utils/exception/BookFetchException.kt

@@ -1,3 +1,3 @@
 package com.pixelized.biblib.utils.exception
 
-class BookFetchException(message: String?): RuntimeException("New Book fetch failed.\n$message")
+class BookFetchException(message: String?): RuntimeException("Book fetch failed.\n$message")