Merge pull request #1931 from OneSignal/notification-click-not-fired-without-IAM-module

jinliu9508 · jinliu9508 · commit f7527d5d237a · 2024-02-06T11:26:05.000-05:00
Notification click not fired without iam module
diff --git a/OneSignalSDK/onesignal/notifications/src/main/java/com/onesignal/notifications/internal/lifecycle/impl/NotificationLifecycleService.kt b/OneSignalSDK/onesignal/notifications/src/main/java/com/onesignal/notifications/internal/lifecycle/impl/NotificationLifecycleService.kt
@@ -87,7 +87,7 @@ internal class NotificationLifecycleService(
         activity: Activity,
         data: JSONObject,
     ): Boolean {
-        var canOpen = false
+        var canOpen = extOpenedCallback.hasSubscribers
         intLifecycleCallback.suspendingFire { canOpen = it.canOpenNotification(activity, data) }
         return canOpen
     }

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ internal class NotificationLifecycleService(`
`87`	`87`	`activity: Activity,`
`88`	`88`	`data: JSONObject,`
`89`	`89`	`): Boolean {`
`90`		`- var canOpen = false`
	`90`	`+ var canOpen = extOpenedCallback.hasSubscribers`
`91`	`91`	`intLifecycleCallback.suspendingFire { canOpen = it.canOpenNotification(activity, data) }`
`92`	`92`	`return canOpen`
`93`	`93`	`}`