Merge pull request #2009 from OneSignal/v5-migration-onesignal-id-missing

jinliu9508 · web-flow · commit b2961117e86c · 2024-03-01T15:57:44.000-05:00
Migration: Missing onesignalId during initWithContext
diff --git a/OneSignalSDK/onesignal/core/src/main/java/com/onesignal/internal/OneSignalImp.kt b/OneSignalSDK/onesignal/core/src/main/java/com/onesignal/internal/OneSignalImp.kt
@@ -289,6 +289,8 @@ internal class OneSignalImp : IOneSignal, IServiceProvider {
                             legacyUserSyncJSON.safeString("identifier") ?: ""
                         if (notificationTypes != null) {
                             pushSubscriptionModel.status = SubscriptionStatus.fromInt(notificationTypes) ?: SubscriptionStatus.NO_PERMISSION
+                        } else {
+                            pushSubscriptionModel.status = SubscriptionStatus.SUBSCRIBED
                         }
 
                         pushSubscriptionModel.sdk = OneSignalUtils.SDK_VERSION

Original file line number	Diff line number	Diff line change
`@@ -289,6 +289,8 @@ internal class OneSignalImp : IOneSignal, IServiceProvider {`
`289`	`289`	`legacyUserSyncJSON.safeString("identifier") ?: ""`
`290`	`290`	`if (notificationTypes != null) {`
`291`	`291`	`pushSubscriptionModel.status = SubscriptionStatus.fromInt(notificationTypes) ?: SubscriptionStatus.NO_PERMISSION`
	`292`	`+ } else {`
	`293`	`+ pushSubscriptionModel.status = SubscriptionStatus.SUBSCRIBED`
`292`	`294`	`}`
`293`	`295`
`294`	`296`	`pushSubscriptionModel.sdk = OneSignalUtils.SDK_VERSION`