Merge pull request #437 from Goooler/dont-expose-mutable-state-flows

Avoid exposing mutable state flows in UI

Author: mlykotom

AdaptiveUiCodelab/app/src/main/java/com/example/reply/ui/ReplyHomeViewModel.kt

@@ -23,14 +23,15 @@ import com.example.reply.data.EmailsRepository
 import com.example.reply.data.EmailsRepositoryImpl
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.StateFlow
+import kotlinx.coroutines.flow.asStateFlow
 import kotlinx.coroutines.flow.catch
 import kotlinx.coroutines.launch
 
 class ReplyHomeViewModel(private val emailsRepository: EmailsRepository = EmailsRepositoryImpl()): ViewModel() {
 
     // UI state exposed to the UI
     private val _uiState = MutableStateFlow(ReplyHomeUIState(loading = true))
-    val uiState: StateFlow<ReplyHomeUIState> = _uiState
+    val uiState: StateFlow<ReplyHomeUIState> = _uiState.asStateFlow()
 
     init {
         observeEmails()

ThemingCodelab/app/src/main/java/com/example/reply/ui/ReplyHomeViewModel.kt

@@ -22,14 +22,15 @@ import com.example.reply.data.Email
 import com.example.reply.data.LocalEmailsDataProvider
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.StateFlow
+import kotlinx.coroutines.flow.asStateFlow
 import kotlinx.coroutines.flow.catch
 import kotlinx.coroutines.launch
 
 class ReplyHomeViewModel : ViewModel() {
 
     // UI state exposed to the UI
     private val _uiState = MutableStateFlow(ReplyHomeUIState(loading = true))
-    val uiState: StateFlow<ReplyHomeUIState> = _uiState
+    val uiState: StateFlow<ReplyHomeUIState> = _uiState.asStateFlow()
 
     init {
         initEmailList()