From 4f03486675d599ad2c0ef22bdd4b67ae9e5ef71a Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Tue, 21 May 2024 09:30:55 +0200
Subject: [PATCH] `distributors` and `distributorNames` do not need to be
 mutable.

---
 .../impl/advanced/AdvancedSettingsPresenter.kt        | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/features/preferences/impl/src/main/kotlin/io/element/android/features/preferences/impl/advanced/AdvancedSettingsPresenter.kt b/features/preferences/impl/src/main/kotlin/io/element/android/features/preferences/impl/advanced/AdvancedSettingsPresenter.kt
index dc76298ae5..21d84567e1 100644
--- a/features/preferences/impl/src/main/kotlin/io/element/android/features/preferences/impl/advanced/AdvancedSettingsPresenter.kt
+++ b/features/preferences/impl/src/main/kotlin/io/element/android/features/preferences/impl/advanced/AdvancedSettingsPresenter.kt
@@ -62,16 +62,17 @@ class AdvancedSettingsPresenter @Inject constructor(
         var showChangeThemeDialog by remember { mutableStateOf(false) }
 
         // List of PushProvider -> Distributor
-        var distributors by remember { mutableStateOf<List<Pair<PushProvider, Distributor>>>(emptyList()) }
-        var distributorNames by remember { mutableStateOf<List<String>>(emptyList()) }
-        LaunchedEffect(Unit) {
-            distributors = pushService.getAvailablePushProviders()
+        val distributors = remember {
+            pushService.getAvailablePushProviders()
                 .flatMap { pushProvider ->
                     pushProvider.getDistributors().map { distributor ->
                         pushProvider to distributor
                     }
                 }
-            distributorNames = distributors.map { it.second.name }
+        }
+        // List of Distributor names
+        val distributorNames = remember {
+            distributors.map { it.second.name }
         }
 
         var currentDistributorName by remember { mutableStateOf<AsyncAction<String>>(AsyncAction.Uninitialized) }