Fixes New menu forgets its "Hide accounts" setting

feature/navigation/drawer/src/main/kotlin/app/k9mail/feature/navigation/drawer/NavigationDrawerExternalContract.kt

@@ -1,13 +1,20 @@
 package app.k9mail.feature.navigation.drawer
 
+import kotlinx.coroutines.flow.Flow
+
 interface NavigationDrawerExternalContract {
 
     data class DrawerConfig(
         val showUnifiedFolders: Boolean,
         val showStarredCount: Boolean,
+        val showAccountSelector: Boolean,
     )
 
     fun interface DrawerConfigLoader {
-        fun loadDrawerConfig(): DrawerConfig
+        fun loadDrawerConfigFlow(): Flow<DrawerConfig>
+    }
+
+    fun interface DrawerConfigWriter {
+        fun writeDrawerConfig(drawerConfig: DrawerConfig)
     }
 }

feature/navigation/drawer/src/main/kotlin/app/k9mail/feature/navigation/drawer/NavigationDrawerModule.kt

@@ -4,6 +4,7 @@ import app.k9mail.feature.navigation.drawer.domain.DomainContract.UseCase
 import app.k9mail.feature.navigation.drawer.domain.usecase.GetDisplayAccounts
 import app.k9mail.feature.navigation.drawer.domain.usecase.GetDisplayFoldersForAccount
 import app.k9mail.feature.navigation.drawer.domain.usecase.GetDrawerConfig
+import app.k9mail.feature.navigation.drawer.domain.usecase.SaveDrawerConfig
 import app.k9mail.feature.navigation.drawer.domain.usecase.SyncAccount
 import app.k9mail.feature.navigation.drawer.domain.usecase.SyncAllAccounts
 import app.k9mail.feature.navigation.drawer.ui.DrawerViewModel
@@ -18,6 +19,11 @@ val navigationDrawerModule: Module = module {
             configProver = get(),
         )
     }
+    single<UseCase.SaveDrawerConfig> {
+        SaveDrawerConfig(
+            drawerConfigWriter = get(),
+        )
+    }
 
     single<UseCase.GetDisplayAccounts> {
         GetDisplayAccounts(
@@ -50,6 +56,7 @@ val navigationDrawerModule: Module = module {
     viewModel {
         DrawerViewModel(
             getDrawerConfig = get(),
+            saveDrawerConfig = get(),
             getDisplayAccounts = get(),
             getDisplayFoldersForAccount = get(),
             syncAccount = get(),

feature/navigation/drawer/src/main/kotlin/app/k9mail/feature/navigation/drawer/domain/DomainContract.kt

@@ -12,6 +12,10 @@ internal interface DomainContract {
             operator fun invoke(): Flow<DrawerConfig>
         }
 
+        fun interface SaveDrawerConfig {
+            operator fun invoke(drawerConfig: DrawerConfig): Flow<Unit>
+        }
+
         fun interface GetDisplayAccounts {
             operator fun invoke(): Flow<List<DisplayAccount>>
         }

feature/navigation/drawer/src/main/kotlin/app/k9mail/feature/navigation/drawer/domain/usecase/GetDrawerConfig.kt

@@ -4,15 +4,11 @@ import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract.Dra
 import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract.DrawerConfigLoader
 import app.k9mail.feature.navigation.drawer.domain.DomainContract.UseCase
 import kotlinx.coroutines.flow.Flow
-import kotlinx.coroutines.flow.flow
 
 internal class GetDrawerConfig(
     private val configProver: DrawerConfigLoader,
 ) : UseCase.GetDrawerConfig {
     override operator fun invoke(): Flow<DrawerConfig> {
-        // TODO This needs to be updated when the config changes
-        return flow {
-            emit(configProver.loadDrawerConfig())
-        }
+        return configProver.loadDrawerConfigFlow()
     }
 }

feature/navigation/drawer/src/main/kotlin/app/k9mail/feature/navigation/drawer/domain/usecase/SaveDrawerConfig.kt

@@ -0,0 +1,17 @@
+package app.k9mail.feature.navigation.drawer.domain.usecase
+
+import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract
+import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract.DrawerConfigWriter
+import app.k9mail.feature.navigation.drawer.domain.DomainContract.UseCase
+import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.flow.flow
+
+internal class SaveDrawerConfig(
+    private val drawerConfigWriter: DrawerConfigWriter,
+) : UseCase.SaveDrawerConfig {
+    override fun invoke(drawerConfig: NavigationDrawerExternalContract.DrawerConfig): Flow<Unit> {
+        return flow {
+            emit(drawerConfigWriter.writeDrawerConfig(drawerConfig))
+        }
+    }
+}

feature/navigation/drawer/src/main/kotlin/app/k9mail/feature/navigation/drawer/ui/DrawerContent.kt

@@ -41,14 +41,14 @@ internal fun DrawerContent(
                 AccountView(
                     account = selectedAccount,
                     onClick = { onEvent(Event.OnAccountViewClick(selectedAccount)) },
-                    showAvatar = state.showAccountSelector,
+                    showAvatar = state.config.showAccountSelector,
                 )
 
                 DividerHorizontal()
             }
             Row {
                 AnimatedVisibility(
-                    visible = state.showAccountSelector,
+                    visible = state.config.showAccountSelector,
                 ) {
                     AccountList(
                         accounts = state.accounts,
@@ -76,7 +76,7 @@ internal fun DrawerContent(
                     SettingList(
                         onAccountSelectorClick = { onEvent(Event.OnAccountSelectorClick) },
                         onManageFoldersClick = { onEvent(Event.OnManageFoldersClick) },
-                        showAccountSelector = state.showAccountSelector,
+                        showAccountSelector = state.config.showAccountSelector,
                     )
                 }
             }

feature/navigation/drawer/src/main/kotlin/app/k9mail/feature/navigation/drawer/ui/DrawerContract.kt

@@ -17,12 +17,12 @@ internal interface DrawerContract {
         val config: DrawerConfig = DrawerConfig(
             showUnifiedFolders = false,
             showStarredCount = false,
+            showAccountSelector = true,
         ),
         val accounts: ImmutableList<DisplayAccount> = persistentListOf(),
         val selectedAccountId: String? = null,
         val folders: ImmutableList<DisplayFolder> = persistentListOf(),
         val selectedFolderId: String? = null,
-        val showAccountSelector: Boolean = true,
         val isLoading: Boolean = false,
     )
 

feature/navigation/drawer/src/main/kotlin/app/k9mail/feature/navigation/drawer/ui/DrawerViewModel.kt

@@ -20,12 +20,14 @@ import kotlinx.coroutines.flow.collectLatest
 import kotlinx.coroutines.flow.distinctUntilChanged
 import kotlinx.coroutines.flow.filterNotNull
 import kotlinx.coroutines.flow.flatMapLatest
+import kotlinx.coroutines.flow.launchIn
 import kotlinx.coroutines.flow.map
 import kotlinx.coroutines.launch
 
 @Suppress("MagicNumber", "TooManyFunctions")
 internal class DrawerViewModel(
     private val getDrawerConfig: UseCase.GetDrawerConfig,
+    private val saveDrawerConfig: UseCase.SaveDrawerConfig,
     private val getDisplayAccounts: UseCase.GetDisplayAccounts,
     private val getDisplayFoldersForAccount: UseCase.GetDisplayFoldersForAccount,
     private val syncAccount: UseCase.SyncAccount,
@@ -113,7 +115,11 @@ internal class DrawerViewModel(
                 )
             }
 
-            Event.OnAccountSelectorClick -> updateState { it.copy(showAccountSelector = it.showAccountSelector.not()) }
+            Event.OnAccountSelectorClick -> {
+                saveDrawerConfig(
+                    state.value.config.copy(showAccountSelector = state.value.config.showAccountSelector.not()),
+                ).launchIn(viewModelScope)
+            }
             Event.OnManageFoldersClick -> emitEffect(Effect.OpenManageFolders)
             Event.OnSettingsClick -> emitEffect(Effect.OpenSettings)
             Event.OnSyncAccount -> onSyncAccount()

feature/navigation/drawer/src/test/kotlin/app/k9mail/feature/navigation/drawer/domain/usecase/GetDrawerConfigTest.kt

@@ -1,32 +1,32 @@
 package app.k9mail.feature.navigation.drawer.domain.usecase
 
 import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract.DrawerConfig
+import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract.DrawerConfigLoader
 import assertk.assertThat
 import assertk.assertions.isEqualTo
 import kotlin.test.Test
 import kotlinx.coroutines.flow.first
+import kotlinx.coroutines.flow.flowOf
 import kotlinx.coroutines.test.runTest
+import org.mockito.Mockito.mock
+import org.mockito.kotlin.whenever
 
 internal class GetDrawerConfigTest {
 
     @Test
     fun `should get drawer config`() = runTest {
+        val configProver: DrawerConfigLoader = mock()
         val drawerConfig = DrawerConfig(
             showUnifiedFolders = true,
             showStarredCount = true,
+            showAccountSelector = true,
         )
 
-        val testSubject = GetDrawerConfig(
-            configProver = { drawerConfig },
-        )
+        val testSubject = GetDrawerConfig(configProver = configProver)
+        whenever(configProver.loadDrawerConfigFlow()).thenReturn(flowOf(drawerConfig))
 
         val result = testSubject().first()
 
-        assertThat(result).isEqualTo(
-            DrawerConfig(
-                showUnifiedFolders = true,
-                showStarredCount = true,
-            ),
-        )
+        assertThat(result).isEqualTo(drawerConfig)
     }
 }

feature/navigation/drawer/src/test/kotlin/app/k9mail/feature/navigation/drawer/ui/DrawerStateTest.kt

@@ -18,12 +18,12 @@ internal class DrawerStateTest {
                 config = DrawerConfig(
                     showUnifiedFolders = false,
                     showStarredCount = false,
+                    showAccountSelector = true,
                 ),
                 accounts = persistentListOf(),
                 selectedAccountId = null,
                 folders = persistentListOf(),
                 selectedFolderId = null,
-                showAccountSelector = true,
                 isLoading = false,
             ),
         )

feature/navigation/drawer/src/test/kotlin/app/k9mail/feature/navigation/drawer/ui/DrawerViewModelTest.kt

@@ -363,14 +363,16 @@ internal class DrawerViewModelTest {
     fun `should change state when OnAccountSelectorClick event is received`() = runMviTest {
         val testSubject = createTestSubject()
         val turbines = turbinesWithInitialStateCheck(testSubject, State())
+        val drawerConfigWithAccountSelectorEnabled = createDrawerConfig(showAccountSelector = true)
+        val drawerConfigWithAccountSelectorDisabled = createDrawerConfig(showAccountSelector = false)
 
         testSubject.event(Event.OnAccountSelectorClick)
 
-        assertThat(turbines.awaitStateItem()).isEqualTo(State(showAccountSelector = false))
+        assertThat(turbines.awaitStateItem()).isEqualTo(State(config = drawerConfigWithAccountSelectorDisabled))
 
         testSubject.event(Event.OnAccountSelectorClick)
 
-        assertThat(turbines.awaitStateItem()).isEqualTo(State(showAccountSelector = true))
+        assertThat(turbines.awaitStateItem()).isEqualTo(State(config = drawerConfigWithAccountSelectorEnabled))
     }
 
     @Test
@@ -414,16 +416,19 @@ internal class DrawerViewModelTest {
             },
             syncAccount = { syncAccountFlow },
             syncAllAccounts = { syncAllAccounts },
+            saveDrawerConfig = { flow { emit(Unit) } },
         )
     }
 
     private fun createDrawerConfig(
         showUnifiedInbox: Boolean = false,
         showStarredCount: Boolean = false,
+        showAccountSelector: Boolean = true,
     ): DrawerConfig {
         return DrawerConfig(
             showUnifiedFolders = showUnifiedInbox,
             showStarredCount = showStarredCount,
+            showAccountSelector = showAccountSelector,
         )
     }
 

legacy/common/src/main/java/com/fsck/k9/feature/FeatureModule.kt

@@ -13,6 +13,10 @@ val featureLauncherModule = module {
     }
 
     single<NavigationDrawerExternalContract.DrawerConfigLoader> {
-        NavigationDrawerConfigLoader()
+        NavigationDrawerConfigLoader(get())
+    }
+
+    single<NavigationDrawerExternalContract.DrawerConfigWriter> {
+        NavigationDrawerConfigWriter(get())
     }
 }

legacy/common/src/main/java/com/fsck/k9/feature/NavigationDrawerConfigLoader.kt

@@ -2,13 +2,11 @@ package com.fsck.k9.feature
 
 import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract.DrawerConfig
 import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract.DrawerConfigLoader
-import com.fsck.k9.K9
+import com.fsck.k9.preferences.DrawerConfigManager
+import kotlinx.coroutines.flow.Flow
 
-class NavigationDrawerConfigLoader : DrawerConfigLoader {
-    override fun loadDrawerConfig(): DrawerConfig {
-        return DrawerConfig(
-            showUnifiedFolders = K9.isShowUnifiedInbox,
-            showStarredCount = K9.isShowStarredCount,
-        )
+class NavigationDrawerConfigLoader(private val drawerConfigManager: DrawerConfigManager) : DrawerConfigLoader {
+    override fun loadDrawerConfigFlow(): Flow<DrawerConfig> {
+        return drawerConfigManager.getConfigFlow()
     }
 }

legacy/common/src/main/java/com/fsck/k9/feature/NavigationDrawerConfigWriter.kt

@@ -0,0 +1,13 @@
+package com.fsck.k9.feature
+
+import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract
+import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract.DrawerConfig
+import com.fsck.k9.preferences.DrawerConfigManager
+
+class NavigationDrawerConfigWriter(
+    private val drawerConfigManager: DrawerConfigManager,
+) : NavigationDrawerExternalContract.DrawerConfigWriter {
+    override fun writeDrawerConfig(drawerConfig: DrawerConfig) {
+        drawerConfigManager.save(drawerConfig)
+    }
+}

legacy/core/build.gradle.kts

@@ -36,6 +36,7 @@ dependencies {
     implementation(libs.timber)
     implementation(libs.mime4j.core)
     implementation(libs.mime4j.dom)
+    implementation(projects.feature.navigation.drawer)
 
     testApi(projects.core.testing)
     testApi(projects.core.android.testing)

legacy/core/src/main/java/com/fsck/k9/K9.kt

@@ -200,6 +200,9 @@ object K9 : KoinComponent {
     @JvmStatic
     var isShowUnifiedInbox = false
 
+    @JvmStatic
+    var isShowAccountSelector = true
+
     @JvmStatic
     var isShowStarredCount = false
 
@@ -329,6 +332,7 @@ object K9 : KoinComponent {
         isShowAnimations = storage.getBoolean("animations", true)
         isUseVolumeKeysForNavigation = storage.getBoolean("useVolumeKeysForNavigation", false)
         isShowUnifiedInbox = storage.getBoolean("showUnifiedInbox", false)
+        isShowAccountSelector = storage.getBoolean("showAccountSelector", true)
         isShowStarredCount = storage.getBoolean("showStarredCount", false)
         isMessageListSenderAboveSubject = storage.getBoolean("messageListSenderAboveSubject", false)
         isShowMessageListStars = storage.getBoolean("messageListStars", true)
@@ -424,6 +428,7 @@ object K9 : KoinComponent {
         editor.putEnum("messageListDensity", messageListDensity)
         editor.putBoolean("messageListSenderAboveSubject", isMessageListSenderAboveSubject)
         editor.putBoolean("showUnifiedInbox", isShowUnifiedInbox)
+        editor.putBoolean("showAccountSelector", isShowAccountSelector)
         editor.putBoolean("showStarredCount", isShowStarredCount)
         editor.putBoolean("messageListStars", isShowMessageListStars)
         editor.putInt("messageListPreviewLines", messageListPreviewLines)

legacy/core/src/main/java/com/fsck/k9/preferences/DrawerConfigManager.kt

@@ -0,0 +1,6 @@
+package com.fsck.k9.preferences
+
+import app.k9mail.feature.navigation.drawer.NavigationDrawerExternalContract.DrawerConfig
+import app.k9mail.legacy.preferences.ConfigManager
+
+interface DrawerConfigManager: ConfigManager<DrawerConfig>

legacy/core/src/main/java/com/fsck/k9/preferences/GeneralSettingsDescriptions.java

@@ -174,6 +174,9 @@ class GeneralSettingsDescriptions {
                 new V(69, new BooleanSetting(true)),
                 new V(101, new BooleanSetting(false))
         ));
+        s.put("isShowAccountSelector", Settings.versions(
+            new V(102, new BooleanSetting(true))
+        ));
         s.put("sortTypeEnum", Settings.versions(
                 new V(10, new EnumSetting<>(SortType.class, Account.DEFAULT_SORT_TYPE))
         ));

legacy/core/src/main/java/com/fsck/k9/preferences/KoinModule.kt

@@ -26,6 +26,12 @@ val preferencesModule = module {
             coroutineScope = get(named("AppCoroutineScope")),
         )
     } bind GeneralSettingsManager::class
+    single {
+        RealDrawerConfigManager(
+            preferences = get(),
+            coroutineScope = get(named("AppCoroutineScope")),
+        )
+    } bind DrawerConfigManager::class
 
     factory { SettingsFileParser() }