From d23a2ed11f372586c9d0699c7f52474189b22aa8 Mon Sep 17 00:00:00 2001 From: Acclorite <140836141+Acclorite@users.noreply.github.com> Date: Fri, 23 Aug 2024 22:30:16 +0300 Subject: [PATCH] =?UTF-8?q?=F0=9F=9B=A0=EF=B8=8F=20Fix=20crash=20when=20de?= =?UTF-8?q?leting=20color=20preset?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fixed crash when deleting color preset (IndexOutOfBounds) --- .../presentation/screens/settings/data/SettingsViewModel.kt | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/ua/acclorite/book_story/presentation/screens/settings/data/SettingsViewModel.kt b/app/src/main/java/ua/acclorite/book_story/presentation/screens/settings/data/SettingsViewModel.kt index 18041475..8d5cc3bf 100644 --- a/app/src/main/java/ua/acclorite/book_story/presentation/screens/settings/data/SettingsViewModel.kt +++ b/app/src/main/java/ua/acclorite/book_story/presentation/screens/settings/data/SettingsViewModel.kt @@ -280,6 +280,7 @@ class SettingsViewModel @Inject constructor( viewModelScope.launch { cancelColorPresetJobs() deleteColorPresetJob = launch { + if (_state.value.colorPresets.size == 1) return@launch val colorPreset = getColorPresetById(event.id) ?: return@launch yield() @@ -298,7 +299,10 @@ class SettingsViewModel @Inject constructor( yield() deleteColorPreset.execute(colorPreset) - onSelectColorPreset(getColorPresets.execute()[nextPosition]) + val nextColorPreset = getColorPresets.execute().getOrNull(nextPosition) + ?: return@launch + + onSelectColorPreset(nextColorPreset) val colorPresets = getColorPresets.execute() _state.update {