fix imageLoader, remote images and logging

Author: stslex

core/core/src/main/java/st/slex/csplashscreen/core/core/Logger.kt

@@ -28,4 +28,17 @@ object Logger {
         val currentTag = "$DEFAULT_TAG:${tag.orEmpty()}"
         Log.d(currentTag, message)
     }
-}
+
+    fun i(
+        message: String,
+        tag: String? = null,
+        throwable: Throwable? = null,
+    ) {
+        if (BuildConfig.DEBUG.not()) return
+        val currentTag = "$DEFAULT_TAG:${tag.orEmpty()}"
+        Log.i(currentTag, message, throwable)
+    }
+
+    fun tag(tag: String): LoggerTag = LoggerTag(tag)
+}
+

core/core/src/main/java/st/slex/csplashscreen/core/core/LoggerTag.kt

@@ -0,0 +1,32 @@
+package st.slex.csplashscreen.core.core
+
+class LoggerTag(
+    private val tag: String
+) {
+
+    fun e(throwable: Throwable, message: String? = null) {
+        Logger.e(
+            throwable = throwable,
+            tag = tag,
+            message = message
+        )
+    }
+
+    fun d(message: String) {
+        Logger.d(
+            message = message,
+            tag = tag
+        )
+    }
+
+    fun i(
+        message: String,
+        throwable: Throwable? = null,
+    ) {
+        Logger.i(
+            message = message,
+            tag = tag,
+            throwable = throwable
+        )
+    }
+}

core/image/src/main/kotlin/st/slex/scplashscreen/core/image/AppImageRequest.kt

@@ -6,12 +6,14 @@ import androidx.compose.runtime.remember
 import androidx.compose.ui.platform.LocalContext
 import coil3.request.CachePolicy
 import coil3.request.ImageRequest
+import st.slex.csplashscreen.core.core.Logger
 
 object AppImageRequest {
 
     @Composable
     fun createImageRequestBuilder(url: String): ImageRequest.Builder {
         val context = LocalContext.current
+        Logger.tag("AppImageRequest").d("createImageRequestBuilder: $url")
         return remember {
             createImageRequestBuilder(context, url)
                 .placeholderMemoryCacheKey(url)

core/network/src/main/java/st/slex/csplashscreen/core/network/model/remote/user/RemoteUserLinksModel.kt

@@ -10,6 +10,6 @@ data class RemoteUserLinksModel(
     @SerialName("photos") val photos: String,
     @SerialName("likes") val likes: String,
     @SerialName("portfolio") val portfolio: String,
-    @SerialName("following") val following: String,
-    @SerialName("followers") val followers: String
+    @SerialName("following") val following: String?,
+    @SerialName("followers") val followers: String?
 )

core/ui/src/main/java/st/slex/csplashscreen/core/ui/components/ImageComponent.kt

@@ -3,9 +3,11 @@ package st.slex.csplashscreen.core.ui.components
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.layout.ContentScale
+import androidx.compose.ui.platform.LocalContext
 import coil3.compose.AsyncImage
+import coil3.request.CachePolicy
+import coil3.request.ImageRequest
 import coil3.request.crossfade
-import st.slex.scplashscreen.core.image.AppImageRequest.createImageRequestBuilder
 
 @Composable
 fun ImageComponent(
@@ -15,7 +17,14 @@ fun ImageComponent(
 ) {
     AsyncImage(
         modifier = modifier,
-        model = createImageRequestBuilder(url)
+        model = ImageRequest.Builder(LocalContext.current)
+            .data(url)
+            .placeholderMemoryCacheKey(url)
+            .memoryCacheKey(url)
+            .diskCacheKey(url)
+            .networkCachePolicy(CachePolicy.ENABLED)
+            .diskCachePolicy(CachePolicy.ENABLED)
+            .memoryCachePolicy(CachePolicy.ENABLED)
             .crossfade(true)
             .build(),
         contentDescription = null,

core/ui/src/main/java/st/slex/csplashscreen/core/ui/mvi/Store.kt

@@ -35,21 +35,27 @@ abstract class Store<S : State, E : Event, A : Action, N : Navigation>(
     initialState: S
 ) : ViewModel() {
 
+    private val logger = Logger.tag(this::class.simpleName ?: "Store")
+
     private val _state: MutableStateFlow<S> = MutableStateFlow(initialState)
     val state: StateFlow<S>
         get() = _state.asStateFlow()
 
     val event: MutableSharedFlow<E> = MutableSharedFlow()
 
-    abstract fun sendAction(action: A)
+    open fun sendAction(action: A) {
+        i("action: $action")
+    }
 
     fun sendEvent(event: E) {
+        i("event: $event")
         viewModelScope.launch(appDispatcher.default) {
             this@Store.event.emit(event)
         }
     }
 
     protected fun navigate(event: N) {
+        i("navigate: $event")
         router(event)
     }
 
@@ -106,4 +112,16 @@ abstract class Store<S : State, E : Event, A : Action, N : Navigation>(
         }
         .onEach(each)
         .launchIn(viewModelScope)
+
+    protected fun d(msg: String) {
+        logger.d(msg)
+    }
+
+    protected fun e(throwable: Throwable, msg: String? = null) {
+        logger.e(throwable, msg)
+    }
+
+    protected fun i(msg: String, throwable: Throwable? = null) {
+        logger.i(msg, throwable)
+    }
 }

feature/home/src/main/java/st/slex/csplashscreen/feature/home/domain/HomeInteractorImpl.kt

@@ -1,6 +1,7 @@
 package st.slex.csplashscreen.feature.home.domain
 
 import st.slex.csplashscreen.core.collection.data.CollectionsRepository
+import st.slex.csplashscreen.core.core.Logger
 import st.slex.csplashscreen.core.network.model.mapToDomain
 import st.slex.csplashscreen.core.network.model.toDomain
 import st.slex.csplashscreen.core.network.model.ui.CollectionDomainModel
@@ -31,4 +32,7 @@ class HomeInteractorImpl(
             pageSize = pageSize
         )
         .toDomain()
+        .also {
+            Logger.tag("DEBUG_HOME").d("getAllPhotos: $it")
+        }
 }

feature/home/src/main/java/st/slex/csplashscreen/feature/home/navigation/HomeGraph.kt

@@ -34,6 +34,14 @@ fun NavGraphBuilder.homeGraph(
             // TODO NOT IMPLEMENTED YET
         }
 
+        LaunchedEffect(photos.loadState) {
+            store.sendAction(Action.PhotosPagingState(photos.loadState))
+        }
+
+        LaunchedEffect(photos.loadState) {
+            store.sendAction(Action.CollectionsPagingState(photos.loadState))
+        }
+
         MainScreen(
             modifier = modifier,
             navToProfile = remember {

feature/home/src/main/java/st/slex/csplashscreen/feature/home/ui/presenter/HomeStore.kt

@@ -1,5 +1,7 @@
 package st.slex.csplashscreen.feature.home.ui.presenter
 
+import androidx.paging.CombinedLoadStates
+import androidx.paging.LoadState
 import androidx.paging.Pager
 import androidx.paging.PagingConfig
 import androidx.paging.PagingData
@@ -28,20 +30,25 @@ class HomeStore(
     initialState = State.INIT
 ) {
 
-    private val collections: StateFlow<PagingData<CollectionModel>> =
-        Pager(config = config) { PagingSource(interactor::getAllCollections) }
+    private val collections: StateFlow<PagingData<CollectionModel>>
+        get() = Pager(config = config) { PagingSource(interactor::getAllCollections) }
             .state { collection -> collection.toPresentation() }
 
-    private val photos: StateFlow<PagingData<PhotoModel>> =
-        Pager(config = config) { PagingSource(interactor::getAllPhotos) }
+    private val photos: StateFlow<PagingData<PhotoModel>>
+        get() = Pager(config = config) {
+            PagingSource(interactor::getAllPhotos)
+        }
             .state { image -> image.toPresentation() }
 
     override fun sendAction(action: Action) {
+        super.sendAction(action)
         when (action) {
             is Action.Init -> actionInit()
             is Action.OnCollectionClick -> actionCollectionClick(action)
             is Action.OnImageClick -> actionImageClick(action)
             is Action.OnUserClick -> actionUserClick(action)
+            is Action.CollectionsPagingState -> actionCollectionPagingState(action)
+            is Action.PhotosPagingState -> actionPhotosPagingState(action)
         }
     }
 
@@ -70,6 +77,27 @@ class HomeStore(
         navigate(Navigation.User(action.username))
     }
 
+    private fun actionCollectionPagingState(action: Action.CollectionsPagingState) {
+        action.combinedLoadStates.checkError()?.let { e(it) }
+    }
+
+    private fun actionPhotosPagingState(action: Action.PhotosPagingState) {
+        action.combinedLoadStates.checkError()?.let { e(it) }
+    }
+
+    private fun CombinedLoadStates.checkError(): Throwable? {
+        if (refresh is LoadState.Error) {
+            return (refresh as LoadState.Error).error
+        }
+        if (append is LoadState.Error) {
+            return (append as LoadState.Error).error
+        }
+        if (prepend is LoadState.Error) {
+            return (prepend as LoadState.Error).error
+        }
+        return null
+    }
+
     companion object {
 
         private val config = PagingConfig(

feature/home/src/main/java/st/slex/csplashscreen/feature/home/ui/presenter/HomeStoreComponent.kt

@@ -1,6 +1,7 @@
 package st.slex.csplashscreen.feature.home.ui.presenter
 
 import androidx.compose.runtime.Stable
+import androidx.paging.CombinedLoadStates
 import androidx.paging.PagingData
 import st.slex.csplashscreen.core.collection.ui.model.CollectionModel
 import st.slex.csplashscreen.core.photos.ui.model.PhotoModel
@@ -63,6 +64,16 @@ interface HomeStoreComponent : StoreComponent {
         data class OnImageClick(
             val uuid: String
         ) : Action
+
+        @Stable
+        data class PhotosPagingState(
+            val combinedLoadStates: CombinedLoadStates
+        ) : Action
+
+        @Stable
+        data class CollectionsPagingState(
+            val combinedLoadStates: CombinedLoadStates
+        ) : Action
     }
 }