check permission before post notification

ichenhe · Nov 11, 2023 · ab64181 · ab64181
1 parent 3856624
commit ab64181
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 3 deletions.
diff --git a/app/src/main/java/cc/chenhe/qqnotifyevo/core/InnerNotificationProcessor.kt b/app/src/main/java/cc/chenhe/qqnotifyevo/core/InnerNotificationProcessor.kt
@@ -1,5 +1,7 @@
 package cc.chenhe.qqnotifyevo.core
 
+import android.Manifest
+import android.annotation.SuppressLint
 import android.app.Notification
 import android.content.Context
 import android.service.notification.StatusBarNotification
@@ -9,6 +11,7 @@ import androidx.core.content.pm.ShortcutInfoCompat
 import androidx.core.content.pm.ShortcutManagerCompat
 import cc.chenhe.qqnotifyevo.utils.NotifyChannel
 import cc.chenhe.qqnotifyevo.utils.Tag
+import cc.chenhe.qqnotifyevo.utils.hasPermission
 import kotlinx.coroutines.CoroutineScope
 import timber.log.Timber
 import java.util.*
@@ -60,8 +63,11 @@ class InnerNotificationProcessor(
     }
 
     private fun sendNotification(context: Context, tag: Tag, id: Int, notification: Notification) {
-        NotificationManagerCompat.from(context).notify(id, notification)
-        addNotifyId(tag, id)
+        @SuppressLint("MissingPermission")
+        if (context.hasPermission(Manifest.permission.POST_NOTIFICATIONS)) {
+            NotificationManagerCompat.from(context).notify(id, notification)
+            addNotifyId(tag, id)
+        }
     }
 
     override fun renewQzoneNotification(

diff --git a/app/src/main/java/cc/chenhe/qqnotifyevo/core/NevoNotificationProcessor.kt b/app/src/main/java/cc/chenhe/qqnotifyevo/core/NevoNotificationProcessor.kt
@@ -1,5 +1,7 @@
 package cc.chenhe.qqnotifyevo.core
 
+import android.Manifest
+import android.annotation.SuppressLint
 import android.app.Notification
 import android.app.PendingIntent
 import android.content.Context
@@ -51,7 +53,8 @@ class NevoNotificationProcessor(context: Context, scope: CoroutineScope) :
             // 目前关联账号的消息都会合并
             return
         }
-        if (nevoMultiMsgTip(ctx)) {
+        @SuppressLint("MissingPermission")
+        if (nevoMultiMsgTip(ctx) && ctx.hasPermission(Manifest.permission.POST_NOTIFICATIONS)) {
             val dontShow = PendingIntent.getBroadcast(
                 ctx, REQ_MULTI_MSG_DONT_SHOW,
                 Intent(ctx, StaticReceiver::class.java).also {

diff --git a/app/src/main/java/cc/chenhe/qqnotifyevo/utils/ContextUtils.kt b/app/src/main/java/cc/chenhe/qqnotifyevo/utils/ContextUtils.kt
@@ -3,6 +3,8 @@ package cc.chenhe.qqnotifyevo.utils
 import android.app.Activity
 import android.content.Context
 import android.content.ContextWrapper
+import android.content.pm.PackageManager
+import androidx.core.content.ContextCompat
 
 /**
  * @throws IllegalStateException  can not find [Activity]
@@ -17,3 +19,6 @@ fun Context.getActivity(): Activity {
     }
     throw IllegalStateException("can not find Activity from current context")
 }
+
+fun Context.hasPermission(permission: String): Boolean =
+    ContextCompat.checkSelfPermission(this, permission) == PackageManager.PERMISSION_GRANTED