From 97298c81694b24eb006e5794656d15b71a57977c Mon Sep 17 00:00:00 2001 From: Marcel Hibbe Date: Fri, 27 Jan 2023 13:31:49 +0100 Subject: [PATCH] Avoid to set RecordingStartedState twice RecordingStartedState is set when the signaling message is received. There is no need to set this by CallStartRecordingObserver. Otherwise the toast "The call is being recorded" would have been shown twice when starting the recording. Signed-off-by: Marcel Hibbe --- .../com/nextcloud/talk/viewmodels/CallRecordingViewModel.kt | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/app/src/main/java/com/nextcloud/talk/viewmodels/CallRecordingViewModel.kt b/app/src/main/java/com/nextcloud/talk/viewmodels/CallRecordingViewModel.kt index e2373f5bb..f634672d5 100644 --- a/app/src/main/java/com/nextcloud/talk/viewmodels/CallRecordingViewModel.kt +++ b/app/src/main/java/com/nextcloud/talk/viewmodels/CallRecordingViewModel.kt @@ -119,9 +119,7 @@ class CallRecordingViewModel @Inject constructor(private val repository: CallRec } override fun onNext(startCallRecordingModel: StartCallRecordingModel) { - if (startCallRecordingModel.success) { - _viewState.value = RecordingStartedState - } + // unused atm. RecordingStartedState is set via setRecordingState which is triggered by signaling message. } override fun onError(e: Throwable) {