Use notifications for error reporting

4 files changed, 92 insertions, 9 deletions

diff --git a/app/src/main/java/org/pacien/tincapp/context/App.kt b/app/src/main/java/org/pacien/tincapp/context/App.kt
index 2d9151e..88308ba 100644
--- a/app/src/main/java/org/pacien/tincapp/context/App.kt
+++ b/app/src/main/java/org/pacien/tincapp/context/App.kt
@@ -25,8 +25,6 @@ import android.net.Uri
 import android.os.Build
 import android.os.Handler
 import android.support.annotation.StringRes
-import android.support.v7.app.AlertDialog
-import android.view.WindowManager
 import org.pacien.tincapp.BuildConfig
 import org.pacien.tincapp.R
 import org.slf4j.LoggerFactory
@@ -59,16 +57,13 @@ class App : Application() {
     private var appContext: Context? = null
     private var handler: Handler? = null
 
+    val notificationManager: AppNotificationManager by lazy { AppNotificationManager(appContext!!) }
+
     fun getContext() = appContext!!
     fun getResources() = getContext().resources!!
 
-    fun alert(@StringRes title: Int, msg: String, manualLink: String? = null) = handler!!.post {
-      AlertDialog.Builder(getContext(), R.style.Theme_AppCompat_Dialog)
-        .setTitle(title).setMessage(msg)
-        .apply { if (manualLink != null) setNeutralButton(R.string.action_open_manual) { _, _ -> openURL(manualLink) } }
-        .setPositiveButton(R.string.action_close) { _, _ -> Unit }
-        .create().apply { window.setType(WindowManager.LayoutParams.TYPE_SYSTEM_ERROR) }.show()
-    }
+    fun alert(@StringRes title: Int, msg: String, manualLink: String? = null) =
+      notificationManager.notifyError(appContext!!.getString(title), msg, manualLink)
 
     fun openURL(url: String) {
       val intent = Intent(Intent.ACTION_VIEW, Uri.parse(url))
diff --git a/app/src/main/java/org/pacien/tincapp/context/AppNotificationManager.kt b/app/src/main/java/org/pacien/tincapp/context/AppNotificationManager.kt
new file mode 100644
index 0000000..0f9092a
--- /dev/null
+++ b/app/src/main/java/org/pacien/tincapp/context/AppNotificationManager.kt
@@ -0,0 +1,84 @@
+/*
+ * tinc app, an Android binding and user interface for the tinc mesh VPN daemon
+ * Copyright (C) 2017-2018 Pacien TRAN-GIRARD
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+package org.pacien.tincapp.context
+
+import android.app.NotificationChannel
+import android.app.NotificationManager
+import android.app.PendingIntent
+import android.content.Context
+import android.content.Intent
+import android.net.Uri
+import android.os.Build
+import android.support.annotation.RequiresApi
+import android.support.v4.app.NotificationCompat
+import android.support.v4.app.NotificationManagerCompat
+import org.pacien.tincapp.R
+
+/**
+ * @author pacien
+ */
+class AppNotificationManager(private val context: Context) {
+  companion object {
+    private const val CHANNEL_ID = "org.pacien.tincapp.notification.channels.error"
+    private const val ERROR_NOTIFICATION_ID = 0
+  }
+
+  init {
+    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) registerChannel()
+  }
+
+  fun notifyError(title: String, message: String, manualLink: String? = null) {
+    val notification = NotificationCompat.Builder(context, CHANNEL_ID)
+      .setSmallIcon(R.drawable.ic_warning_primary_24dp)
+      .setContentTitle(title)
+      .setContentText(message)
+      .setStyle(NotificationCompat.BigTextStyle().bigText(message))
+      .setHighPriority()
+      .setAutoCancel(true)
+      .apply { if (manualLink != null) setManualLink(manualLink) }
+      .build()
+
+    NotificationManagerCompat.from(context)
+      .notify(ERROR_NOTIFICATION_ID, notification)
+  }
+
+  fun dismissAll() {
+    NotificationManagerCompat.from(context).cancelAll()
+  }
+
+  @RequiresApi(Build.VERSION_CODES.O)
+  private fun registerChannel() {
+    val name = context.getString(R.string.notification_channel_error_name)
+    val importance = NotificationManager.IMPORTANCE_HIGH
+    val channel = NotificationChannel(CHANNEL_ID, name, importance)
+    val notificationManager = context.getSystemService(NotificationManager::class.java)
+    notificationManager.createNotificationChannel(channel)
+  }
+
+  private fun NotificationCompat.Builder.setHighPriority() = apply {
+    priority = NotificationCompat.PRIORITY_MAX
+    if (Build.VERSION.SDK_INT < Build.VERSION_CODES.O) setDefaults(NotificationCompat.DEFAULT_SOUND) // force heads-up notification
+  }
+
+  private fun NotificationCompat.Builder.setManualLink(manualLink: String) = apply {
+    val intent = Intent(Intent.ACTION_VIEW, Uri.parse(manualLink))
+    val pendingIntent = PendingIntent.getActivity(context, 0, intent, 0)
+    addAction(R.drawable.ic_help_primary_24dp, context.getString(R.string.action_open_manual), pendingIntent)
+  }
+}
diff --git a/app/src/main/java/org/pacien/tincapp/service/TincVpnService.kt b/app/src/main/java/org/pacien/tincapp/service/TincVpnService.kt
index 30d7c8b..6ed9a34 100644
--- a/app/src/main/java/org/pacien/tincapp/service/TincVpnService.kt
+++ b/app/src/main/java/org/pacien/tincapp/service/TincVpnService.kt
@@ -197,6 +197,8 @@ class TincVpnService : VpnService() {
     fun isConnected() = !(daemon?.isDone ?: true)
 
     fun connect(netName: String, passphrase: String? = null) {
+      App.notificationManager.dismissAll()
+
       App.getContext().startService(
         Intent(App.getContext(), TincVpnService::class.java)
           .setAction(Actions.ACTION_CONNECT)
diff --git a/app/src/main/java/org/pacien/tincapp/utils/ProgressModal.kt b/app/src/main/java/org/pacien/tincapp/utils/ProgressModal.kt
index 379e3af..6bf02d5 100644
--- a/app/src/main/java/org/pacien/tincapp/utils/ProgressModal.kt
+++ b/app/src/main/java/org/pacien/tincapp/utils/ProgressModal.kt
@@ -27,6 +27,8 @@ import android.widget.TextView
 import org.pacien.tincapp.R
 
 /**
+ * An indefinite progress dialog replacing the deprecated `android.app.ProgressDialog`.
+ *
  * @author pacien
  */
 object ProgressModal {