Fixes #13312: Delegate onConfigurationChanged to the ViewModel

#13312
signalapp · Dec 8, 2023 · 3627679 · 3627679
1 parent 71f9474
commit 3627679
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/app/src/main/java/org/thoughtcrime/securesms/conversationlist/ConversationListFragment.java b/app/src/main/java/org/thoughtcrime/securesms/conversationlist/ConversationListFragment.java
@@ -601,7 +601,7 @@ public boolean onOptionsItemSelected(@NonNull MenuItem item) {
   @Override
   public void onConfigurationChanged(@NonNull Configuration newConfig) {
     super.onConfigurationChanged(newConfig);
-    onMegaphoneChanged(viewModel.getMegaphone());
+    viewModel.onConfigurationChanged();
   }
 
   private ContactSearchConfiguration mapSearchStateToConfiguration(@NonNull ContactSearchState state) {

diff --git a/app/src/main/java/org/thoughtcrime/securesms/conversationlist/ConversationListViewModel.kt b/app/src/main/java/org/thoughtcrime/securesms/conversationlist/ConversationListViewModel.kt
@@ -202,6 +202,12 @@ class ConversationListViewModel(
     }
   }
 
+  fun onConfigurationChanged() {
+    megaphoneRepository.getNextMegaphone { next ->
+      store.update { it.copy(megaphone = next ?: Megaphone.NONE) }
+    }
+  }
+
   private data class ConversationListState(
     val conversations: List<Conversation> = emptyList(),
     val megaphone: Megaphone = Megaphone.NONE,