Просмотр исходного кода

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 <dev@mhibbe.de>
Marcel Hibbe 2 лет назад
Родитель
Сommit
97298c8169

+ 1 - 3
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) {