extracting the personalization complete emitting to a dedicated function

element-hq · Mar 10, 2022 · 305c52b · 305c52b
1 parent 7e4941b
commit 305c52b
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/vector/src/main/java/im/vector/app/features/onboarding/OnboardingViewModel.kt b/vector/src/main/java/im/vector/app/features/onboarding/OnboardingViewModel.kt
@@ -159,7 +159,7 @@ class OnboardingViewModel @AssistedInject constructor(
             OnboardingAction.ClearHomeServerHistory        -> handleClearHomeServerHistory()
             is OnboardingAction.UpdateDisplayName          -> updateDisplayName(action.displayName)
             OnboardingAction.UpdateDisplayNameSkipped      -> handleDisplayNameStepComplete()
-            OnboardingAction.UpdateProfilePictureSkipped   -> _viewEvents.post(OnboardingViewEvents.OnPersonalizationComplete)
+            OnboardingAction.UpdateProfilePictureSkipped   -> completePersonalization()
             OnboardingAction.PersonalizeProfile            -> handlePersonalizeProfile()
             is OnboardingAction.ProfilePictureSelected     -> handleProfilePictureSelected(action)
             OnboardingAction.SaveSelectedProfilePicture    -> updateProfilePicture()
@@ -954,7 +954,7 @@ class OnboardingViewModel @AssistedInject constructor(
         withPersonalisationState {
             when {
                 it.supportsChangingProfilePicture -> _viewEvents.post(OnboardingViewEvents.OnChooseProfilePicture)
-                else                              -> _viewEvents.post(OnboardingViewEvents.OnPersonalizationComplete)
+                else                              -> completePersonalization()
             }
         }
     }
@@ -1000,6 +1000,10 @@ class OnboardingViewModel @AssistedInject constructor(
     }
 
     private fun onProfilePictureSaved() {
+        completePersonalization()
+    }
+
+    private fun completePersonalization() {
         _viewEvents.post(OnboardingViewEvents.OnPersonalizationComplete)
     }
 }