From 336cde9989ae0d3c1adff1d8ca4cd448ec5f03c0 Mon Sep 17 00:00:00 2001
From: Norbel AMBANUMBEN <aanorbel@gmail.com>
Date: Sat, 16 Mar 2024 13:59:23 +0100
Subject: [PATCH 1/3] feat: (OON Run v2) add missing post-load toast from
 install link flow https://github.com/ooni/run/issues/148

---
 .../ooniprobe/activity/adddescriptor/AddDescriptorActivity.kt   | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/src/main/java/org/openobservatory/ooniprobe/activity/adddescriptor/AddDescriptorActivity.kt b/app/src/main/java/org/openobservatory/ooniprobe/activity/adddescriptor/AddDescriptorActivity.kt
index 6fedc19fd..20a435156 100644
--- a/app/src/main/java/org/openobservatory/ooniprobe/activity/adddescriptor/AddDescriptorActivity.kt
+++ b/app/src/main/java/org/openobservatory/ooniprobe/activity/adddescriptor/AddDescriptorActivity.kt
@@ -10,6 +10,7 @@ import android.view.MenuInflater
 import android.view.MenuItem
 import android.widget.ImageView
 import android.widget.TextView
+import android.widget.Toast
 import androidx.activity.viewModels
 import androidx.appcompat.widget.Toolbar
 import androidx.databinding.BindingAdapter
@@ -173,6 +174,7 @@ class AddDescriptorActivity : AbstractActivity() {
             // This observer is used to finish the activity when the descriptor is added.
             viewModel.finishActivity.observe(this) { shouldFinish ->
                 if (shouldFinish) {
+                    Toast.makeText(this@AddDescriptorActivity, "Link installed", Toast.LENGTH_LONG).show()
                     finish()
                 }
             }

From e9b68a09551b515f38200d7d2c3da3b08ce6a03f Mon Sep 17 00:00:00 2001
From: Norbel AMBANUMBEN <aanorbel@gmail.com>
Date: Sat, 16 Mar 2024 14:35:39 +0100
Subject: [PATCH 2/3] feat: (OONI Run V2) Add `Cancel` button on link install
 flow and Update Add link button text to Install Link
 https://github.com/ooni/run/issues/152 ,
 https://github.com/ooni/run/issues/149

---
 .../adddescriptor/AddDescriptorActivity.kt    | 29 +++++++---------
 .../res/layout/activity_add_descriptor.xml    | 34 ++++++++++++++++---
 app/src/main/res/menu/add_descriptor.xml      | 10 ------
 3 files changed, 41 insertions(+), 32 deletions(-)
 delete mode 100644 app/src/main/res/menu/add_descriptor.xml

diff --git a/app/src/main/java/org/openobservatory/ooniprobe/activity/adddescriptor/AddDescriptorActivity.kt b/app/src/main/java/org/openobservatory/ooniprobe/activity/adddescriptor/AddDescriptorActivity.kt
index 20a435156..450b3ae35 100644
--- a/app/src/main/java/org/openobservatory/ooniprobe/activity/adddescriptor/AddDescriptorActivity.kt
+++ b/app/src/main/java/org/openobservatory/ooniprobe/activity/adddescriptor/AddDescriptorActivity.kt
@@ -12,7 +12,6 @@ import android.widget.ImageView
 import android.widget.TextView
 import android.widget.Toast
 import androidx.activity.viewModels
-import androidx.appcompat.widget.Toolbar
 import androidx.databinding.BindingAdapter
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.ViewModelProvider
@@ -114,7 +113,7 @@ class AddDescriptorActivity : AbstractActivity() {
         setSupportActionBar(binding.toolbar)
         supportActionBar?.setDisplayHomeAsUpEnabled(false)
         supportActionBar?.setDisplayShowHomeEnabled(false)
-        supportActionBar?.title = "Add New Link"
+        supportActionBar?.title = "Install New Link"
         val descriptorExtra = if (VERSION.SDK_INT >= VERSION_CODES.TIRAMISU) {
             intent.getParcelableExtra(DESCRIPTOR, TestDescriptor::class.java)
         } else {
@@ -145,21 +144,17 @@ class AddDescriptorActivity : AbstractActivity() {
             for (i in 0 until adapter.groupCount) {
                 binding.expandableListView.expandGroup(i)
             }
-            val bottomBarOnMenuItemClickListener: Toolbar.OnMenuItemClickListener =
-                Toolbar.OnMenuItemClickListener { item ->
-                    when (item.itemId) {
-                        R.id.add_descriptor -> {
-                            viewModel.onAddButtonClicked(
-                                disabledAutorunNettests = adapter.nettests.filter { it.selected },
-                                automatedUpdates = binding.automaticUpdatesSwitch.isChecked
-                            )
-                            true
-                        }
-
-                        else -> false
-                    }
-                }
-            binding.bottomBar.setOnMenuItemClickListener(bottomBarOnMenuItemClickListener)
+
+            binding.btnInstallLink.setOnClickListener {
+                viewModel.onAddButtonClicked(
+                    disabledAutorunNettests = adapter.nettests.filter { it.selected },
+                    automatedUpdates = binding.automaticUpdatesSwitch.isChecked
+                )
+            }
+
+            binding.btnCancel.setOnClickListener {
+                finish()
+            }
 
             viewModel.selectedAllBtnStatus.observe(this) { state ->
                 binding.testsCheckbox.checkedState = state;
diff --git a/app/src/main/res/layout/activity_add_descriptor.xml b/app/src/main/res/layout/activity_add_descriptor.xml
index 36fbf7298..6d7aaaecf 100644
--- a/app/src/main/res/layout/activity_add_descriptor.xml
+++ b/app/src/main/res/layout/activity_add_descriptor.xml
@@ -162,13 +162,37 @@
             android:layout_gravity="bottom"
             android:theme="@style/Theme.MaterialComponents.Light.DarkActionBar.App.NoActionBar">
 
-            <androidx.appcompat.widget.Toolbar
-                android:id="@+id/bottomBar"
+            <LinearLayout
                 android:layout_width="match_parent"
-                android:layout_height="wrap_content"
+                android:layout_height="?attr/actionBarSize"
                 android:background="@color/color_gray0"
-                app:menu="@menu/add_descriptor"
-                app:titleTextAppearance="@style/TextAppearance.Widget.AppCompat.Toolbar.Title.App" />
+                android:gravity="center">
+
+                <Button
+                    android:id="@+id/btn_cancel"
+                    style="?android:attr/buttonBarButtonStyle"
+                    android:layout_width="0dp"
+                    android:layout_height="wrap_content"
+                    android:layout_weight="1"
+                    android:text="@string/Modal_Cancel"
+                    android:textColor="@color/color_base" />
+
+                <View
+                    android:layout_width="0dp"
+                    android:layout_height="wrap_content"
+                    android:layout_weight="2" />
+
+                <Button
+                    android:id="@+id/btn_install_link"
+                    style="?android:attr/buttonBarButtonStyle"
+                    android:layout_width="0dp"
+                    android:layout_height="wrap_content"
+                    android:layout_weight="1.5"
+                    android:elevation="0dp"
+                    android:text="Install Link"
+                    android:textColor="@color/color_base" />
+
+            </LinearLayout>
         </com.google.android.material.appbar.AppBarLayout>
     </androidx.coordinatorlayout.widget.CoordinatorLayout>
 </layout>
\ No newline at end of file
diff --git a/app/src/main/res/menu/add_descriptor.xml b/app/src/main/res/menu/add_descriptor.xml
deleted file mode 100644
index 632cc4035..000000000
--- a/app/src/main/res/menu/add_descriptor.xml
+++ /dev/null
@@ -1,10 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<menu xmlns:android="http://schemas.android.com/apk/res/android"
-    xmlns:app="http://schemas.android.com/apk/res-auto"
-    xmlns:tools="http://schemas.android.com/tools">
-    <item
-        android:id="@+id/add_descriptor"
-        app:showAsAction="ifRoom|withText"
-        tools:title="Add Link"
-        android:title="Add Link" />
-</menu>

From b43bc31ce35a766fe857dc9c9ebde6e24e0a0ed6 Mon Sep 17 00:00:00 2001
From: Norbel AMBANUMBEN <aanorbel@gmail.com>
Date: Sat, 16 Mar 2024 15:01:56 +0100
Subject: [PATCH 3/3] feat: (OONI Run v2) Add link cancellation toast
 https://github.com/ooni/run/issues/151

---
 .../ooniprobe/activity/oonirun/OoniRunV2Activity.kt   |  2 +-
 .../openobservatory/ooniprobe/common/TaskExecutor.kt  | 11 ++++++-----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/app/src/main/java/org/openobservatory/ooniprobe/activity/oonirun/OoniRunV2Activity.kt b/app/src/main/java/org/openobservatory/ooniprobe/activity/oonirun/OoniRunV2Activity.kt
index b5a092621..e832525af 100644
--- a/app/src/main/java/org/openobservatory/ooniprobe/activity/oonirun/OoniRunV2Activity.kt
+++ b/app/src/main/java/org/openobservatory/ooniprobe/activity/oonirun/OoniRunV2Activity.kt
@@ -85,7 +85,7 @@ class OoniRunV2Activity : AbstractActivity() {
         val executor = TaskExecutor()
         binding.cancelButton.setOnClickListener {
             executor.cancelTask()
-            finishWithError(message = getString(R.string.Modal_Cancel))
+            finishWithError(message = "Link installation cancelled")
         }
         executor.executeTask({
             try {
diff --git a/app/src/main/java/org/openobservatory/ooniprobe/common/TaskExecutor.kt b/app/src/main/java/org/openobservatory/ooniprobe/common/TaskExecutor.kt
index 8b821654d..57e5afdfb 100644
--- a/app/src/main/java/org/openobservatory/ooniprobe/common/TaskExecutor.kt
+++ b/app/src/main/java/org/openobservatory/ooniprobe/common/TaskExecutor.kt
@@ -44,8 +44,7 @@ typealias OnTaskComplete<R> = (R) -> Unit
 class TaskExecutor {
     private val executor = Executors.newSingleThreadExecutor()
     private val handler = Handler(Looper.getMainLooper())
-    private var future: Future<*>? = null
-
+    private lateinit var future: Future<*>
     /**
      * Executes a task in a separate thread and posts the result on the main thread.
      * @param task The task to be executed.
@@ -54,8 +53,10 @@ class TaskExecutor {
     fun <R> executeTask(task: Task<R>, onComplete: OnTaskComplete<R>) {
         future = executor.submit {
             val result = task.call()
-            handler.post {
-                onComplete(result)
+            if (!future.isCancelled) {
+                handler.post {
+                    onComplete(result)
+                }
             }
         }
     }
@@ -90,6 +91,6 @@ class TaskExecutor {
      * Cancels the currently running task.
      */
     fun cancelTask() {
-        future?.cancel(true)
+        this.future.cancel(true)
     }
 }
\ No newline at end of file