correctly populate duration drop down in all cases

tuskyapp · Oct 28, 2024 · 6f09c93 · 6f09c93
1 parent 53969a4
commit 6f09c93
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/app/src/main/java/com/keylesspalace/tusky/components/filters/EditFilterActivity.kt b/app/src/main/java/com/keylesspalace/tusky/components/filters/EditFilterActivity.kt
@@ -137,6 +137,7 @@ class EditFilterActivity : BaseActivity() {
 
         if (originalFilter == null) {
             binding.filterActionWarn.isChecked = true
+            initializeDurationDropDown(false)
         } else {
             loadFilter()
         }
@@ -178,7 +179,11 @@ class EditFilterActivity : BaseActivity() {
     // Populate the UI from the filter's members
     private fun loadFilter() {
         viewModel.load(filter)
-        val durationNames = if (filter.expiresAt != null) {
+        initializeDurationDropDown(withNoChange = filter.expiresAt != null)
+    }
+
+    private fun initializeDurationDropDown(withNoChange: Boolean) {
+        val durationNames = if (withNoChange) {
             arrayOf(getString(R.string.duration_no_change)) + resources.getStringArray(R.array.filter_duration_names)
         } else {
             resources.getStringArray(R.array.filter_duration_names)