From c2fe6696708c33ed927b8cdd344cddcb253e1788 Mon Sep 17 00:00:00 2001 From: Adam Brown Date: Thu, 10 Mar 2022 17:24:15 +0000 Subject: [PATCH] extracting the personalization complete emitting to a dedicated function --- .../vector/app/features/onboarding/OnboardingViewModel.kt | 8 ++++++-- 1 file changed, 6 insertions(+), 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 index f97f4fbfea..36020fbe61 100644 --- 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) } }