Merge pull request #272 from Aga-C/fix-mark-as-read-notification

tibbi · web-flow · commit 76e83810806d · 2022-01-19T15:43:27.000+01:00
Fixed disabled mark as read in notification
diff --git a/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt b/app/src/main/kotlin/com/simplemobiletools/smsmessenger/extensions/Context.kt
@@ -717,7 +717,7 @@ fun Context.showMessageNotification(address: String, body: String, threadId: Lon
         putExtra(THREAD_ID, threadId)
     }
 
-    val markAsReadPendingIntent = PendingIntent.getBroadcast(this, 0, markAsReadIntent, PendingIntent.FLAG_CANCEL_CURRENT)
+    val markAsReadPendingIntent = PendingIntent.getBroadcast(this, threadId.hashCode(), markAsReadIntent, PendingIntent.FLAG_UPDATE_CURRENT)
     var replyAction: NotificationCompat.Action? = null
 
     if (isNougatPlus()) {

Original file line number	Diff line number	Diff line change
`@@ -717,7 +717,7 @@ fun Context.showMessageNotification(address: String, body: String, threadId: Lon`
`717`	`717`	`putExtra(THREAD_ID, threadId)`
`718`	`718`	`}`
`719`	`719`
`720`		`- val markAsReadPendingIntent = PendingIntent.getBroadcast(this, 0, markAsReadIntent, PendingIntent.FLAG_CANCEL_CURRENT)`
	`720`	`+ val markAsReadPendingIntent = PendingIntent.getBroadcast(this, threadId.hashCode(), markAsReadIntent, PendingIntent.FLAG_UPDATE_CURRENT)`
`721`	`721`	`var replyAction: NotificationCompat.Action? = null`
`722`	`722`
`723`	`723`	`if (isNougatPlus()) {`