androidApp/src/main/java/bou/amine/apps/readerforselfossv2/android/HomeActivity.kt

@@ -1017,6 +1017,7 @@ class HomeActivity : AppCompatActivity(), SearchView.OnQueryTextListener, DIAwar
                     Toast.makeText(this, R.string.refresh_in_progress, Toast.LENGTH_SHORT).show()
                     // TODO: Use Dispatchers.IO
                     CoroutineScope(Dispatchers.Main).launch {
+                        repository.offlineOverride = false
                         val updatedRemote = repository.updateRemote()
                         if (updatedRemote) {
                             // TODO: Send toast messages from the repository

shared/src/commonMain/kotlin/bou/amine/apps/readerforselfossv2/repository/RepositoryImpl.kt

@@ -178,9 +178,13 @@ class Repository(private val api: SelfossApi, private val apiDetails: ApiDetails
         return success
     }
 
-    suspend fun markAsReadById(id: Int): Boolean =
-        isNetworkAvailable() && api.markAsRead(id.toString())?.isSuccess == true
-
+    suspend fun markAsReadById(id: Int): Boolean {
+        var success = false
+        if (isNetworkAvailable()) {
+            success = api.markAsRead(id.toString())?.isSuccess == true
+        }
+        return success
+    }
 
     suspend fun unmarkAsRead(item: SelfossModel.Item): Boolean {
         val success = unmarkAsReadById(item.id)
@@ -191,8 +195,13 @@ class Repository(private val api: SelfossApi, private val apiDetails: ApiDetails
         return success
     }
 
-    suspend fun unmarkAsReadById(id: Int): Boolean =
-        isNetworkAvailable() && api.unmarkAsRead(id.toString())?.isSuccess == true
+    suspend fun unmarkAsReadById(id: Int): Boolean {
+        var success = false
+        if (isNetworkAvailable()) {
+            success = api.unmarkAsRead(id.toString())?.isSuccess == true
+        }
+        return success
+    }
 
     suspend fun starr(item: SelfossModel.Item): Boolean {
         val success = starrById(item.id)
@@ -203,8 +212,13 @@ class Repository(private val api: SelfossApi, private val apiDetails: ApiDetails
         return success
     }
 
-    suspend fun starrById(id: Int): Boolean =
-        isNetworkAvailable() && api.starr(id.toString())?.isSuccess == true
+    suspend fun starrById(id: Int): Boolean {
+        var success = false
+        if (isNetworkAvailable()) {
+            success = api.starr(id.toString())?.isSuccess == true
+        }
+        return success
+    }
 
     suspend fun unstarr(item: SelfossModel.Item): Boolean {
         val success = unstarrById(item.id)
@@ -215,15 +229,22 @@ class Repository(private val api: SelfossApi, private val apiDetails: ApiDetails
         return success
     }
 
-    suspend fun unstarrById(id: Int): Boolean =
-        isNetworkAvailable() && api.unstarr(id.toString())?.isSuccess == true
+    suspend fun unstarrById(id: Int): Boolean {
+        var success = false
+        if (isNetworkAvailable()) {
+            success = api.unstarr(id.toString())?.isSuccess == true
+        }
+        return success
+    }
 
 
     suspend fun markAllAsRead(items: ArrayList<SelfossModel.Item>): Boolean {
         var success = false
+        if (isNetworkAvailable()) {
+            success = api.markAllAsRead(items.map { it.id.toString() })?.isSuccess == true
+        }
 
-        if (isNetworkAvailable() && api.markAllAsRead(items.map { it.id.toString() })?.isSuccess == true) {
-            success = true
+        if (success) {
             for (item in items) {
                 markAsReadLocally(item)
             }
@@ -298,8 +319,13 @@ class Repository(private val api: SelfossApi, private val apiDetails: ApiDetails
         return success
     }
 
-    suspend fun updateRemote(): Boolean =
-        isNetworkAvailable() && api.update()?.isSuccess == true
+    suspend fun updateRemote(): Boolean {
+        var response = false
+        if (isConnectionAvailable.value) {
+            response = api.update()?.isSuccess == true
+        }
+        return response
+    }
 
     suspend fun login(): Boolean {
         var result = false