🔨 Update DefaultUserrepository Codebase as cache not working before

Author: Mark TBL

app/src/main/java/com/coxtunes/androidarchitecturekotlin2021/common/di/AppModule.kt

@@ -12,7 +12,9 @@ import dagger.Provides
 import dagger.hilt.InstallIn
 import dagger.hilt.components.SingletonComponent
 import kotlinx.coroutines.CoroutineDispatcher
+import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.SupervisorJob
 import okhttp3.OkHttpClient
 import okhttp3.logging.HttpLoggingInterceptor
 import retrofit2.Retrofit
@@ -68,12 +70,19 @@ object AppModule {
         return retrofit.create(UsersApi::class.java)
     }
 
+
+    @Provides
+    fun provideCoroutinesScope(dispatcher: DispatcherProvider): CoroutineScope {
+        return CoroutineScope(SupervisorJob() + dispatcher.default)
+
+    }
+
     @Singleton
     @Provides
     fun provideUserRepository(
         remoteDataSource: UsersRemoteDataSource,
-        dispatcher: DispatcherProvider
+        externalScope: CoroutineScope
     ): UserRepository {
-        return DefaultUserRepository(remoteDataSource, dispatcher)
+        return DefaultUserRepository(remoteDataSource, externalScope)
     }
 }

app/src/main/java/com/coxtunes/androidarchitecturekotlin2021/data/repository/DefaultUserRepository.kt

@@ -1,21 +1,21 @@
 package com.coxtunes.androidarchitecturekotlin2021.data.repository
+
 import com.coxtunes.androidarchitecturekotlin2021.common.responseSealed.Resource
-import com.coxtunes.androidarchitecturekotlin2021.common.utility.DispatcherProvider
 import com.coxtunes.androidarchitecturekotlin2021.data.remoteDataSource.UsersRemoteDataSource
 import com.coxtunes.androidarchitecturekotlin2021.domain.repository.UserRepository
 import com.coxtunes.androidarchitecturekotlin2021.domain.viewobjects.UsersViewItems
+import kotlinx.coroutines.CoroutineScope
+import kotlinx.coroutines.async
 import kotlinx.coroutines.sync.Mutex
 import kotlinx.coroutines.sync.withLock
-import kotlinx.coroutines.withContext
 import javax.inject.Inject
 
 /**
  * @Author: Nayeem Shiddiki Abir
- * @Date: 28-Dec-21
  */
 class DefaultUserRepository @Inject constructor(
     private val remoteDataSource: UsersRemoteDataSource,
-    private val dispatcher: DispatcherProvider
+    private val externalScope: CoroutineScope
 ) : UserRepository {
 
     // Mutex to make writes to cached values thread-safe.
@@ -25,15 +25,17 @@ class DefaultUserRepository @Inject constructor(
     private var latestUsersList: Resource<List<UsersViewItems>> = Resource.Empty()
 
     override suspend fun getUsersList(refresh: Boolean): Resource<List<UsersViewItems>> {
-        withContext(dispatcher.io) {
-            val itemsRes = remoteDataSource.getUsers()
-
-            // Thread-safe write to latestNews
-            latestUsersMutex.withLock {
-                this@DefaultUserRepository.latestUsersList = itemsRes
-            }
+        return if (latestUsersList.data == null) {
+            externalScope.async {
+                remoteDataSource.getUsers().also { userlist ->
+                    // Thread-safe write to latestNews
+                    latestUsersMutex.withLock {
+                        this@DefaultUserRepository.latestUsersList = userlist
+                    }
+                }
+            }.await()
+        } else {
+            latestUsersMutex.withLock { latestUsersList }
         }
-        return latestUsersMutex.withLock { latestUsersList }
     }
-
 }