chore: Migrate LiveData to Flow

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

Author: AndyScherzinger

app/src/main/java/com/nextcloud/talk/location/components/GeocodingScreen.kt

@@ -22,8 +22,8 @@ import androidx.compose.material3.Scaffold
 import androidx.compose.material3.TopAppBar
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
+import androidx.compose.runtime.collectAsState
 import androidx.compose.runtime.getValue
-import androidx.compose.runtime.livedata.observeAsState
 import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
 import androidx.compose.runtime.saveable.rememberSaveable
@@ -42,15 +42,15 @@ import fr.dudie.nominatim.model.Address
 
 @Composable
 fun GeocodingScreen(viewModel: GeoCodingViewModel, onBack: () -> Unit, onAddressSelected: (Address) -> Unit) {
-    val results by viewModel.getGeocodingResultsLiveData().observeAsState(viewModel.getGeocodingResults())
+    val results by viewModel.geocodingResults.collectAsState()
     val initialQuery = viewModel.getQuery()
     var textFieldValue by rememberSaveable(stateSaver = TextFieldValue.Saver) {
         mutableStateOf(TextFieldValue(initialQuery, TextRange(initialQuery.length)))
     }
     val keyboardController = LocalSoftwareKeyboardController.current
 
     LaunchedEffect(Unit) {
-        if (viewModel.getQuery().isNotEmpty() && viewModel.getGeocodingResults().isEmpty()) {
+        if (viewModel.getQuery().isNotEmpty() && results.isEmpty()) {
             viewModel.searchLocation()
         }
     }

app/src/main/java/com/nextcloud/talk/viewmodels/GeoCodingViewModel.kt

@@ -7,33 +7,30 @@
 package com.nextcloud.talk.viewmodels
 
 import android.util.Log
-import androidx.lifecycle.LiveData
-import androidx.lifecycle.MutableLiveData
 import androidx.lifecycle.ViewModel
 import fr.dudie.nominatim.client.TalkJsonNominatimClient
 import fr.dudie.nominatim.model.Address
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.flow.MutableStateFlow
+import kotlinx.coroutines.flow.StateFlow
 import kotlinx.coroutines.launch
 import okhttp3.OkHttpClient
 import java.io.IOException
 
 class GeoCodingViewModel : ViewModel() {
-    private val geocodingResultsLiveData = MutableLiveData<List<Address>>()
+    private val _geocodingResults = MutableStateFlow<List<Address>>(emptyList())
+    val geocodingResults: StateFlow<List<Address>> = _geocodingResults
     private val nominatimClient: TalkJsonNominatimClient
     private val okHttpClient: OkHttpClient = OkHttpClient.Builder().build()
-    private var geocodingResults: List<Address> = ArrayList()
     private var query: String = ""
-    fun getGeocodingResultsLiveData(): LiveData<List<Address>> = geocodingResultsLiveData
 
     fun getQuery(): String = query
 
     fun setQuery(query: String) {
         this.query = query
     }
 
-    fun getGeocodingResults(): List<Address> = geocodingResults
-
     init {
         nominatimClient = TalkJsonNominatimClient(
             "https://nominatim.openstreetmap.org/",
@@ -47,8 +44,7 @@ class GeoCodingViewModel : ViewModel() {
             CoroutineScope(Dispatchers.IO).launch {
                 try {
                     val results = nominatimClient.search(query) as ArrayList<Address>
-                    geocodingResults = results
-                    geocodingResultsLiveData.postValue(results)
+                    _geocodingResults.value = results
                 } catch (e: IOException) {
                     Log.e(TAG, "Failed to get geocoded addresses", e)
                 }