Refactor: Make LocationViewModel methods non-suspend

The suspend modifier has been removed from the `saveToDatabase`, `deleteFromDatabase`, and `updateCurrentLocation` methods in `LocationViewModel.kt`.

Author: JoshLudahl

app/src/main/java/com/warbler/ui/location/LocationViewModel.kt

@@ -17,67 +17,67 @@ import javax.inject.Inject
 
 @HiltViewModel
 class LocationViewModel
-@Inject
-constructor(
-    private val locationRepository: LocationRepository,
-    private val locationNetworkRepository: LocationNetworkRepository,
-) : ViewModel() {
-    private val _locationList = MutableStateFlow<Resource<List<LocationEntity>>>(Resource.Loading)
-    val locationList: StateFlow<Resource<List<LocationEntity>>>
-        get() = _locationList
+    @Inject
+    constructor(
+        private val locationRepository: LocationRepository,
+        private val locationNetworkRepository: LocationNetworkRepository,
+    ) : ViewModel() {
+        private val _locationList = MutableStateFlow<Resource<List<LocationEntity>>>(Resource.Loading)
+        val locationList: StateFlow<Resource<List<LocationEntity>>>
+            get() = _locationList
 
-    private val _locationSearchList =
-        MutableStateFlow<Resource<List<LocationEntity>>>(
-            Resource.Loading,
-        )
-    val locationSearchList: StateFlow<Resource<List<LocationEntity>>>
-        get() = _locationSearchList
+        private val _locationSearchList =
+            MutableStateFlow<Resource<List<LocationEntity>>>(
+                Resource.Loading,
+            )
+        val locationSearchList: StateFlow<Resource<List<LocationEntity>>>
+            get() = _locationSearchList
 
-    init {
-        viewModelScope.launch {
-            locationRepository
-                .getAllLocationsFromDatabase()
-                .catch {
-                    Log.d("LocationViewModel", "Error: ${it.message}")
-                    _locationList.value = Resource.Error(message = it.message)
-                }.collect {
-                    Log.d("LocationViewModel", "Success: $it")
-                    _locationList.value = Resource.Success(it)
-                }
+        init {
+            viewModelScope.launch {
+                locationRepository
+                    .getAllLocationsFromDatabase()
+                    .catch {
+                        Log.d("LocationViewModel", "Error: ${it.message}")
+                        _locationList.value = Resource.Error(message = it.message)
+                    }.collect {
+                        Log.d("LocationViewModel", "Success: $it")
+                        _locationList.value = Resource.Success(it)
+                    }
+            }
         }
-    }
 
-    suspend fun saveToDatabase(location: LocationEntity) {
-        viewModelScope.launch {
-            locationRepository.saveLocationToDatabaseAndSetAsCurrent(location)
+        fun saveToDatabase(location: LocationEntity) {
+            viewModelScope.launch {
+                locationRepository.saveLocationToDatabaseAndSetAsCurrent(location)
+            }
         }
-    }
 
-    suspend fun deleteFromDatabase(location: LocationEntity) {
-        Log.d("LocationViewModel", "deleteFromDatabase launching coroutine...")
-        viewModelScope.launch {
-            locationRepository.deleteLocation(location)
+        fun deleteFromDatabase(location: LocationEntity) {
+            Log.d("LocationViewModel", "deleteFromDatabase launching coroutine...")
+            viewModelScope.launch {
+                locationRepository.deleteLocation(location)
+            }
         }
-    }
 
-    suspend fun updateCurrentLocation(location: LocationEntity) {
-        viewModelScope.launch(Dispatchers.IO) {
-            locationRepository.updateToCurrentLocation(location)
+        fun updateCurrentLocation(location: LocationEntity) {
+            viewModelScope.launch(Dispatchers.IO) {
+                locationRepository.updateToCurrentLocation(location)
+            }
         }
-    }
 
-    fun searchForLocation(query: String) {
-        Log.d("LocationViewModel", "searchForLocation launching coroutine...")
-        viewModelScope.launch {
-            Log.d("LocationViewModel", "searchForLocation Attempting to search for: $query")
-            locationNetworkRepository
-                .getLocationsFromGeoService(query)
-                .catch { error ->
-                    Log.d("LocationViewModel", "searchForLocation error: ${error.message}")
-                }.collect {
-                    Log.i("LocationViewModel", "searchForLocation success: ${it.size}")
-                    _locationSearchList.value = Resource.Success(it)
-                }
+        fun searchForLocation(query: String) {
+            Log.d("LocationViewModel", "searchForLocation launching coroutine...")
+            viewModelScope.launch {
+                Log.d("LocationViewModel", "searchForLocation Attempting to search for: $query")
+                locationNetworkRepository
+                    .getLocationsFromGeoService(query)
+                    .catch { error ->
+                        Log.d("LocationViewModel", "searchForLocation error: ${error.message}")
+                    }.collect {
+                        Log.i("LocationViewModel", "searchForLocation success: ${it.size}")
+                        _locationSearchList.value = Resource.Success(it)
+                    }
+            }
         }
     }
-}