From 87213ac5eb7a6b557cf58609fce03c2e0ef1e36a Mon Sep 17 00:00:00 2001 From: pacien Date: Mon, 6 Aug 2018 02:30:00 +0200 Subject: Use notifications for error reporting --- .../main/java/org/pacien/tincapp/context/App.kt | 13 ++-- .../tincapp/context/AppNotificationManager.kt | 84 ++++++++++++++++++++++ .../org/pacien/tincapp/service/TincVpnService.kt | 2 + .../java/org/pacien/tincapp/utils/ProgressModal.kt | 2 + 4 files changed, 92 insertions(+), 9 deletions(-) create mode 100644 app/src/main/java/org/pacien/tincapp/context/AppNotificationManager.kt (limited to 'app/src/main/java/org') 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 . + */ + +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 { -- cgit v1.2.3