From 4bfe3a4774a6f241110e0e29ead5d60e849c3ef0 Mon Sep 17 00:00:00 2001 From: Marcel Hibbe Date: Tue, 5 Dec 2023 16:20:28 +0100 Subject: [PATCH] avoid error when declining recording consent When declining recording consent, 404 might be returned when leaving the call (because the call was not joined before). It might not be the best option to handle this via onError, but for the moment (18.0.0 release) it's the most robust/lowest-risk solution without to change some state handling to check if the call was joined). finish was added which makes sense anyway, but for declining recording consent the error snackbar was removed. Signed-off-by: Marcel Hibbe --- .../main/java/com/nextcloud/talk/activities/CallActivity.kt | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/app/src/main/java/com/nextcloud/talk/activities/CallActivity.kt b/app/src/main/java/com/nextcloud/talk/activities/CallActivity.kt index 8a6c92466d..225f0620c6 100644 --- a/app/src/main/java/com/nextcloud/talk/activities/CallActivity.kt +++ b/app/src/main/java/com/nextcloud/talk/activities/CallActivity.kt @@ -2049,7 +2049,7 @@ class CallActivity : CallBaseActivity() { } override fun onNext(genericOverall: GenericOverall) { - if (!switchToRoomToken.isEmpty()) { + if (switchToRoomToken.isNotEmpty()) { val intent = Intent(context, ChatActivity::class.java) intent.addFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP) val bundle = Bundle() @@ -2070,8 +2070,8 @@ class CallActivity : CallBaseActivity() { } override fun onError(e: Throwable) { - Snackbar.make(binding!!.root, R.string.nc_common_error_sorry, Snackbar.LENGTH_LONG).show() - Log.e(TAG, "Error while leaving the call", e) + Log.w(TAG, "Something went wrong when leaving the call", e) + finish() } override fun onComplete() {