Update observer to include null external id

OneSignal · Feb 16, 2024 · b735e32 · b735e32
1 parent c383ccf
commit b735e32
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 9 deletions.
diff --git a/src/android/com/onesignal/cordova/OneSignalObserverController.java b/src/android/com/onesignal/cordova/OneSignalObserverController.java
@@ -101,14 +101,13 @@ public void onUserStateChange(UserChangedState state) {
   private static JSONObject createUserIds(UserState user) {
     JSONObject userIds = new JSONObject();
     try {
-      if (!user.getExternalId().isEmpty()) {
-        userIds.put("externalId", user.getExternalId());
-      }
-      if (!user.getOnesignalId().isEmpty()) {
-        userIds.put("onesignalId", user.getOnesignalId());
-      }
+        String externalId = user.getExternalId();
+        String onesignalId = user.getOnesignalId();
+
+        userIds.put("externalId", externalId != null && !externalId.isEmpty() ? externalId : JSONObject.NULL);
+        userIds.put("onesignalId", onesignalId != null && !onesignalId.isEmpty() ? onesignalId : JSONObject.NULL);
     } catch (JSONException e) {
-      e.printStackTrace();
+        e.printStackTrace();
     }
     return userIds;
   }

diff --git a/src/ios/OneSignalPush.m b/src/ios/OneSignalPush.m
@@ -173,11 +173,17 @@ - (void)onUserStateDidChangeWithState:(OSUserChangedState * _Nonnull)state {
     NSMutableDictionary *result = [NSMutableDictionary new];
 
     NSMutableDictionary *currentObject = [NSMutableDictionary new];
-    if (onesignalId.length > 0) {
+
+    if (onesignalId && ![onesignalId isEqualToString:@""]) {
         currentObject[@"onesignalId"] = onesignalId;
+    } else {
+        currentObject[@"onesignalId"] = [NSNull null];
     }
-    if (externalId.length > 0) {
+
+    if (externalId && ![externalId isEqualToString:@""]) {
         currentObject[@"externalId"] = externalId;
+    } else {
+        currentObject[@"externalId"] = [NSNull null];
     }
 
     result[@"current"] = currentObject;