From d52e61413d398769bc6f347c60bff387f1858a5d Mon Sep 17 00:00:00 2001
From: Benoit Marty <benoit@matrix.org>
Date: Tue, 4 Jan 2022 14:32:03 +0100
Subject: [PATCH] Ask for user confirmation before deleting Widget

---
 .../java/im/vector/app/features/widgets/WidgetFragment.kt | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/vector/src/main/java/im/vector/app/features/widgets/WidgetFragment.kt b/vector/src/main/java/im/vector/app/features/widgets/WidgetFragment.kt
index afe34a9b7f..90eb9491a6 100644
--- a/vector/src/main/java/im/vector/app/features/widgets/WidgetFragment.kt
+++ b/vector/src/main/java/im/vector/app/features/widgets/WidgetFragment.kt
@@ -160,7 +160,7 @@ class WidgetFragment @Inject constructor() :
                 return@withState true
             }
             R.id.action_delete          -> {
-                viewModel.handle(WidgetAction.DeleteWidget)
+                deleteWidget()
                 return@withState true
             }
             R.id.action_refresh         -> if (state.formattedURL.complete) {
@@ -172,7 +172,7 @@ class WidgetFragment @Inject constructor() :
                 return@withState true
             }
             R.id.action_revoke          -> if (state.status == WidgetStatus.WIDGET_ALLOWED) {
-                viewModel.handle(WidgetAction.RevokeWidget)
+                revokeWidget()
                 return@withState true
             }
         }
@@ -306,7 +306,7 @@ class WidgetFragment @Inject constructor() :
         )
     }
 
-    fun deleteWidget() {
+    private fun deleteWidget() {
         MaterialAlertDialogBuilder(requireContext())
                 .setMessage(R.string.widget_delete_message_confirmation)
                 .setPositiveButton(R.string.remove) { _, _ ->
@@ -316,7 +316,7 @@ class WidgetFragment @Inject constructor() :
                 .show()
     }
 
-    fun revokeWidget() {
+    private fun revokeWidget() {
         viewModel.handle(WidgetAction.RevokeWidget)
     }
 }