Use shared coroutine scope in cleaning settings

MihaiCristianCondrea · MihaiCristianCondrea · commit f2d17a645c42 · 2025-08-17T23:29:53.000+03:00
diff --git a/app/src/main/kotlin/com/d4rk/cleaner/app/settings/cleaning/ui/CleaningSettingsList.kt b/app/src/main/kotlin/com/d4rk/cleaner/app/settings/cleaning/ui/CleaningSettingsList.kt
@@ -10,6 +10,7 @@ import androidx.compose.foundation.shape.RoundedCornerShape
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.collectAsState
 import androidx.compose.runtime.getValue
+import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.draw.clip
 import androidx.compose.ui.platform.LocalContext
@@ -22,14 +23,14 @@ import com.d4rk.android.libs.apptoolkit.core.utils.constants.ui.SizeConstants
 import com.d4rk.cleaner.R
 import com.d4rk.cleaner.app.auto.AutoCleanScheduler
 import com.d4rk.cleaner.core.data.datastore.DataStore
-import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.launch
 import org.koin.compose.koinInject
 
 @Composable
 fun CleaningSettingsList(paddingValues: PaddingValues) {
     val context = LocalContext.current
+    val scope = rememberCoroutineScope()
     val dataStore: DataStore = koinInject()
     val genericFilter: Boolean by dataStore.genericFilter.collectAsState(initial = true)
     val deleteEmptyFolders: Boolean by dataStore.deleteEmptyFolders.collectAsState(initial = false)
@@ -69,7 +70,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_generic_filter),
                     checked = genericFilter,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveGenericFilter(isChecked)
                     }
                 }
@@ -80,7 +81,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     title = stringResource(id = R.string.delete_empty_folders),
                     checked = deleteEmptyFolders,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteEmptyFolders(isChecked)
                     }
                 }
@@ -92,7 +93,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_archive_filter),
                     checked = deleteArchives,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteArchives(isChecked)
                     }
                 }
@@ -104,7 +105,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_corpse_files),
                     checked = deleteCorpseFiles,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteCorpseFiles(isChecked)
                     }
                 }
@@ -116,7 +117,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_apk_files),
                     checked = deleteApkFiles,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteApkFiles(isChecked)
                     }
                 }
@@ -128,7 +129,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_windows_files),
                     checked = windowsExtensions,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteWindowsFiles(isChecked)
                     }
                 }
@@ -140,7 +141,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_office_files),
                     checked = officeExtensions,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteOfficeFiles(isChecked)
                     }
                 }
@@ -152,7 +153,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_font_files),
                     checked = fontExtensions,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteFontFiles(isChecked)
                     }
                 }
@@ -164,7 +165,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_other_files),
                     checked = otherExtensions,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteOtherFiles(isChecked)
                     }
                 }
@@ -185,7 +186,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_audio),
                     checked = deleteAudioFiles,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteAudioFiles(isChecked)
                     }
                 }
@@ -197,7 +198,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_video),
                     checked = deleteVideoFiles,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteVideoFiles(isChecked)
                     }
                 }
@@ -209,7 +210,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_images),
                     checked = deleteImageFiles,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteImageFiles(isChecked)
                     }
                 }
@@ -221,7 +222,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_invalid_media),
                     checked = deleteInvalidMedia,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteInvalidMedia(isChecked)
                     }
                 }
@@ -242,7 +243,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_show_hidden_files),
                     checked = showHiddenFiles,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveShowHiddenFiles(isChecked)
                     }
                 }
@@ -254,7 +255,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     summary = stringResource(id = R.string.summary_preference_settings_delete_duplicates),
                     checked = deleteDuplicateFiles,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveDeleteDuplicateFiles(isChecked)
                     }
                 }
@@ -265,7 +266,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     title = stringResource(id = R.string.preference_streak_reminder),
                     checked = streakReminderEnabled,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveStreakReminderEnabled(isChecked)
                     }
                 }
@@ -276,7 +277,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                         title = stringResource(id = R.string.preference_show_streak_card),
                         checked = false,
                     ) { isChecked ->
-                        CoroutineScope(Dispatchers.IO).launch {
+                        scope.launch(Dispatchers.IO) { 
                             dataStore.saveShowStreakCard(isChecked)
                             if (isChecked) dataStore.saveStreakHideUntil(0L)
                         }
@@ -305,7 +306,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {
                     title = stringResource(id = R.string.enable_automatic_cleaning),
                     checked = autoCleanEnabled,
                 ) { isChecked ->
-                    CoroutineScope(Dispatchers.IO).launch {
+                    scope.launch(Dispatchers.IO) { 
                         dataStore.saveAutoCleanEnabled(isChecked)
                         if (isChecked) AutoCleanScheduler.schedule(
                             context,

Original file line number	Diff line number	Diff line change
`@@ -10,6 +10,7 @@ import androidx.compose.foundation.shape.RoundedCornerShape`
`10`	`10`	`import androidx.compose.runtime.Composable`
`11`	`11`	`import androidx.compose.runtime.collectAsState`
`12`	`12`	`import androidx.compose.runtime.getValue`
	`13`	`+import androidx.compose.runtime.rememberCoroutineScope`
`13`	`14`	`import androidx.compose.ui.Modifier`
`14`	`15`	`import androidx.compose.ui.draw.clip`
`15`	`16`	`import androidx.compose.ui.platform.LocalContext`
`@@ -22,14 +23,14 @@ import com.d4rk.android.libs.apptoolkit.core.utils.constants.ui.SizeConstants`
`22`	`23`	`import com.d4rk.cleaner.R`
`23`	`24`	`import com.d4rk.cleaner.app.auto.AutoCleanScheduler`
`24`	`25`	`import com.d4rk.cleaner.core.data.datastore.DataStore`
`25`		`-import kotlinx.coroutines.CoroutineScope`
`26`	`26`	`import kotlinx.coroutines.Dispatchers`
`27`	`27`	`import kotlinx.coroutines.launch`
`28`	`28`	`import org.koin.compose.koinInject`
`29`	`29`
`30`	`30`	`@Composable`
`31`	`31`	`fun CleaningSettingsList(paddingValues: PaddingValues) {`
`32`	`32`	`val context = LocalContext.current`
	`33`	`+ val scope = rememberCoroutineScope()`
`33`	`34`	`val dataStore: DataStore = koinInject()`
`34`	`35`	`val genericFilter: Boolean by dataStore.genericFilter.collectAsState(initial = true)`
`35`	`36`	`val deleteEmptyFolders: Boolean by dataStore.deleteEmptyFolders.collectAsState(initial = false)`
`@@ -69,7 +70,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`69`	`70`	`summary = stringResource(id = R.string.summary_preference_settings_generic_filter),`
`70`	`71`	`checked = genericFilter,`
`71`	`72`	`) { isChecked ->`
`72`		`- CoroutineScope(Dispatchers.IO).launch {`
	`73`	`+ scope.launch(Dispatchers.IO) {`
`73`	`74`	`dataStore.saveGenericFilter(isChecked)`
`74`	`75`	`}`
`75`	`76`	`}`
`@@ -80,7 +81,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`80`	`81`	`title = stringResource(id = R.string.delete_empty_folders),`
`81`	`82`	`checked = deleteEmptyFolders,`
`82`	`83`	`) { isChecked ->`
`83`		`- CoroutineScope(Dispatchers.IO).launch {`
	`84`	`+ scope.launch(Dispatchers.IO) {`
`84`	`85`	`dataStore.saveDeleteEmptyFolders(isChecked)`
`85`	`86`	`}`
`86`	`87`	`}`
`@@ -92,7 +93,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`92`	`93`	`summary = stringResource(id = R.string.summary_preference_settings_archive_filter),`
`93`	`94`	`checked = deleteArchives,`
`94`	`95`	`) { isChecked ->`
`95`		`- CoroutineScope(Dispatchers.IO).launch {`
	`96`	`+ scope.launch(Dispatchers.IO) {`
`96`	`97`	`dataStore.saveDeleteArchives(isChecked)`
`97`	`98`	`}`
`98`	`99`	`}`
`@@ -104,7 +105,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`104`	`105`	`summary = stringResource(id = R.string.summary_preference_settings_delete_corpse_files),`
`105`	`106`	`checked = deleteCorpseFiles,`
`106`	`107`	`) { isChecked ->`
`107`		`- CoroutineScope(Dispatchers.IO).launch {`
	`108`	`+ scope.launch(Dispatchers.IO) {`
`108`	`109`	`dataStore.saveDeleteCorpseFiles(isChecked)`
`109`	`110`	`}`
`110`	`111`	`}`
`@@ -116,7 +117,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`116`	`117`	`summary = stringResource(id = R.string.summary_preference_settings_delete_apk_files),`
`117`	`118`	`checked = deleteApkFiles,`
`118`	`119`	`) { isChecked ->`
`119`		`- CoroutineScope(Dispatchers.IO).launch {`
	`120`	`+ scope.launch(Dispatchers.IO) {`
`120`	`121`	`dataStore.saveDeleteApkFiles(isChecked)`
`121`	`122`	`}`
`122`	`123`	`}`
`@@ -128,7 +129,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`128`	`129`	`summary = stringResource(id = R.string.summary_preference_settings_delete_windows_files),`
`129`	`130`	`checked = windowsExtensions,`
`130`	`131`	`) { isChecked ->`
`131`		`- CoroutineScope(Dispatchers.IO).launch {`
	`132`	`+ scope.launch(Dispatchers.IO) {`
`132`	`133`	`dataStore.saveDeleteWindowsFiles(isChecked)`
`133`	`134`	`}`
`134`	`135`	`}`
`@@ -140,7 +141,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`140`	`141`	`summary = stringResource(id = R.string.summary_preference_settings_delete_office_files),`
`141`	`142`	`checked = officeExtensions,`
`142`	`143`	`) { isChecked ->`
`143`		`- CoroutineScope(Dispatchers.IO).launch {`
	`144`	`+ scope.launch(Dispatchers.IO) {`
`144`	`145`	`dataStore.saveDeleteOfficeFiles(isChecked)`
`145`	`146`	`}`
`146`	`147`	`}`
`@@ -152,7 +153,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`152`	`153`	`summary = stringResource(id = R.string.summary_preference_settings_delete_font_files),`
`153`	`154`	`checked = fontExtensions,`
`154`	`155`	`) { isChecked ->`
`155`		`- CoroutineScope(Dispatchers.IO).launch {`
	`156`	`+ scope.launch(Dispatchers.IO) {`
`156`	`157`	`dataStore.saveDeleteFontFiles(isChecked)`
`157`	`158`	`}`
`158`	`159`	`}`
`@@ -164,7 +165,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`164`	`165`	`summary = stringResource(id = R.string.summary_preference_settings_delete_other_files),`
`165`	`166`	`checked = otherExtensions,`
`166`	`167`	`) { isChecked ->`
`167`		`- CoroutineScope(Dispatchers.IO).launch {`
	`168`	`+ scope.launch(Dispatchers.IO) {`
`168`	`169`	`dataStore.saveDeleteOtherFiles(isChecked)`
`169`	`170`	`}`
`170`	`171`	`}`
`@@ -185,7 +186,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`185`	`186`	`summary = stringResource(id = R.string.summary_preference_settings_delete_audio),`
`186`	`187`	`checked = deleteAudioFiles,`
`187`	`188`	`) { isChecked ->`
`188`		`- CoroutineScope(Dispatchers.IO).launch {`
	`189`	`+ scope.launch(Dispatchers.IO) {`
`189`	`190`	`dataStore.saveDeleteAudioFiles(isChecked)`
`190`	`191`	`}`
`191`	`192`	`}`
`@@ -197,7 +198,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`197`	`198`	`summary = stringResource(id = R.string.summary_preference_settings_delete_video),`
`198`	`199`	`checked = deleteVideoFiles,`
`199`	`200`	`) { isChecked ->`
`200`		`- CoroutineScope(Dispatchers.IO).launch {`
	`201`	`+ scope.launch(Dispatchers.IO) {`
`201`	`202`	`dataStore.saveDeleteVideoFiles(isChecked)`
`202`	`203`	`}`
`203`	`204`	`}`
`@@ -209,7 +210,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`209`	`210`	`summary = stringResource(id = R.string.summary_preference_settings_delete_images),`
`210`	`211`	`checked = deleteImageFiles,`
`211`	`212`	`) { isChecked ->`
`212`		`- CoroutineScope(Dispatchers.IO).launch {`
	`213`	`+ scope.launch(Dispatchers.IO) {`
`213`	`214`	`dataStore.saveDeleteImageFiles(isChecked)`
`214`	`215`	`}`
`215`	`216`	`}`
`@@ -221,7 +222,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`221`	`222`	`summary = stringResource(id = R.string.summary_preference_settings_delete_invalid_media),`
`222`	`223`	`checked = deleteInvalidMedia,`
`223`	`224`	`) { isChecked ->`
`224`		`- CoroutineScope(Dispatchers.IO).launch {`
	`225`	`+ scope.launch(Dispatchers.IO) {`
`225`	`226`	`dataStore.saveDeleteInvalidMedia(isChecked)`
`226`	`227`	`}`
`227`	`228`	`}`
`@@ -242,7 +243,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`242`	`243`	`summary = stringResource(id = R.string.summary_preference_settings_show_hidden_files),`
`243`	`244`	`checked = showHiddenFiles,`
`244`	`245`	`) { isChecked ->`
`245`		`- CoroutineScope(Dispatchers.IO).launch {`
	`246`	`+ scope.launch(Dispatchers.IO) {`
`246`	`247`	`dataStore.saveShowHiddenFiles(isChecked)`
`247`	`248`	`}`
`248`	`249`	`}`
`@@ -254,7 +255,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`254`	`255`	`summary = stringResource(id = R.string.summary_preference_settings_delete_duplicates),`
`255`	`256`	`checked = deleteDuplicateFiles,`
`256`	`257`	`) { isChecked ->`
`257`		`- CoroutineScope(Dispatchers.IO).launch {`
	`258`	`+ scope.launch(Dispatchers.IO) {`
`258`	`259`	`dataStore.saveDeleteDuplicateFiles(isChecked)`
`259`	`260`	`}`
`260`	`261`	`}`
`@@ -265,7 +266,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`265`	`266`	`title = stringResource(id = R.string.preference_streak_reminder),`
`266`	`267`	`checked = streakReminderEnabled,`
`267`	`268`	`) { isChecked ->`
`268`		`- CoroutineScope(Dispatchers.IO).launch {`
	`269`	`+ scope.launch(Dispatchers.IO) {`
`269`	`270`	`dataStore.saveStreakReminderEnabled(isChecked)`
`270`	`271`	`}`
`271`	`272`	`}`
`@@ -276,7 +277,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`276`	`277`	`title = stringResource(id = R.string.preference_show_streak_card),`
`277`	`278`	`checked = false,`
`278`	`279`	`) { isChecked ->`
`279`		`- CoroutineScope(Dispatchers.IO).launch {`
	`280`	`+ scope.launch(Dispatchers.IO) {`
`280`	`281`	`dataStore.saveShowStreakCard(isChecked)`
`281`	`282`	`if (isChecked) dataStore.saveStreakHideUntil(0L)`
`282`	`283`	`}`
`@@ -305,7 +306,7 @@ fun CleaningSettingsList(paddingValues: PaddingValues) {`
`305`	`306`	`title = stringResource(id = R.string.enable_automatic_cleaning),`
`306`	`307`	`checked = autoCleanEnabled,`
`307`	`308`	`) { isChecked ->`
`308`		`- CoroutineScope(Dispatchers.IO).launch {`
	`309`	`+ scope.launch(Dispatchers.IO) {`
`309`	`310`	`dataStore.saveAutoCleanEnabled(isChecked)`
`310`	`311`	`if (isChecked) AutoCleanScheduler.schedule(`
`311`	`312`	`context,`