Skip to content

Commit 07660f6

Browse files
authored
Merge pull request #1986 from OneSignal/Add_default_value_for_notification_types
Add default value to legacy notification types
2 parents 6c6b12c + cab7cb1 commit 07660f6

File tree

1 file changed

+4
-3
lines changed
  • OneSignalSDK/onesignal/core/src/main/java/com/onesignal/internal

1 file changed

+4
-3
lines changed

OneSignalSDK/onesignal/core/src/main/java/com/onesignal/internal/OneSignalImp.kt

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import com.onesignal.common.IDManager
99
import com.onesignal.common.OneSignalUtils
1010
import com.onesignal.common.modeling.ModelChangeTags
1111
import com.onesignal.common.modules.IModule
12+
import com.onesignal.common.safeInt
1213
import com.onesignal.common.safeString
1314
import com.onesignal.common.services.IServiceProvider
1415
import com.onesignal.common.services.ServiceBuilder
@@ -277,7 +278,7 @@ internal class OneSignalImp : IOneSignal, IServiceProvider {
277278

278279
if (legacyUserSyncString != null) {
279280
val legacyUserSyncJSON = JSONObject(legacyUserSyncString)
280-
val notificationTypes = legacyUserSyncJSON.getInt("notification_types")
281+
val notificationTypes = legacyUserSyncJSON.safeInt("notification_types")
281282

282283
val pushSubscriptionModel = SubscriptionModel()
283284
pushSubscriptionModel.id = legacyPlayerId
@@ -286,8 +287,8 @@ internal class OneSignalImp : IOneSignal, IServiceProvider {
286287
notificationTypes != SubscriptionStatus.NO_PERMISSION.value && notificationTypes != SubscriptionStatus.UNSUBSCRIBE.value
287288
pushSubscriptionModel.address =
288289
legacyUserSyncJSON.safeString("identifier") ?: ""
289-
pushSubscriptionModel.status = SubscriptionStatus.fromInt(notificationTypes)
290-
?: SubscriptionStatus.NO_PERMISSION
290+
if (notificationTypes != null)
291+
pushSubscriptionModel.status = SubscriptionStatus.fromInt(notificationTypes) ?: SubscriptionStatus.NO_PERMISSION
291292

292293
pushSubscriptionModel.sdk = OneSignalUtils.SDK_VERSION
293294
pushSubscriptionModel.deviceOS = Build.VERSION.RELEASE

0 commit comments

Comments
 (0)