com.onesignal.unity.ios 5.0.0 → 5.0.2

package/Runtime/Plugins/iOS/OneSignalUnityBridgeNotifications.mm

@@ -71,6 +71,8 @@ typedef void (*ClickListenerDelegate)(const char* notification, const char* resu
         [OneSignal.Notifications addPermissionObserver:self];
         [OneSignal.Notifications addForegroundLifecycleListener:self];
         [OneSignal.Notifications addClickListener:self];
+
+        _willDisplayEvents = [NSMutableDictionary new];
     }
 
     return self;
@@ -84,10 +86,10 @@ typedef void (*ClickListenerDelegate)(const char* notification, const char* resu
 
 - (void)onWillDisplayNotification:(OSNotificationWillDisplayEvent *)event {
     if (_willDisplayDelegate != nil) {
+        _willDisplayEvents[event.notification.notificationId] = event;
+
         NSString *stringNotification = [event.notification stringify];
         _willDisplayDelegate([stringNotification UTF8String]);
-
-        _willDisplayEvents[event.notification.notificationId] = event;
     }
 }
 

package/Runtime/Plugins/iOS/OneSignalUnityBridgeUser.mm

@@ -80,6 +80,7 @@ typedef void (*StateListenerDelegate)(const char* current, const char* previous)
 
 /*
  * Bridge methods
+ * We use strdup because Unity attempts to free the memory after we return the value
  */
 
 extern "C" {
@@ -156,11 +157,11 @@ extern "C" {
     }
 
     const char* _pushSubscriptionGetId() {
-        return [OneSignal.User.pushSubscription.id UTF8String];
+        return strdup([OneSignal.User.pushSubscription.id UTF8String]);
     }
 
     const char* _pushSubscriptionGetToken() {
-        return [OneSignal.User.pushSubscription.token UTF8String];
+        return strdup([OneSignal.User.pushSubscription.token UTF8String]);
     }
 
     bool _pushSubscriptionGetOptedIn() {

package/Runtime/Plugins/iOS/UIApplication+OneSignalUnity.mm

@@ -96,7 +96,7 @@ static bool swizzled = false;
 
 - (BOOL)oneSignalApplication:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions {
     [OneSignalWrapper setSdkType:@"unity"];
-    [OneSignalWrapper setSdkVersion:@"050000"];
+    [OneSignalWrapper setSdkVersion:@"050002"];
     [OneSignal setLaunchOptions:launchOptions];
 
     if ([self respondsToSelector:@selector(oneSignalApplication:didFinishLaunchingWithOptions:)])

package/Runtime/iOSNotificationsManager.cs

@@ -114,7 +114,7 @@ namespace OneSignalSDK.iOS.Notifications {
             {
                 // We use Send instead of Post because we need to *not* return to our caller until the
                 // event handlers have returned themselves. This allows a handler to call PreventDefault()
-                // which will get passed down to Android in InternalNotificationWillDisplayEventArgs.
+                // which will get passed down to iOS in InternalNotificationWillDisplayEventArgs.
                 UnityMainThreadDispatch.Send(state => handler(_instance, args));
             }
         }

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "com.onesignal.unity.ios",
   "displayName": "OneSignal Unity SDK - iOS",
-  "version": "5.0.0",
+  "version": "5.0.2",
   "unity": "2018.4",
   "description": "OneSignal is the market leader in customer engagement, powering mobile push, web push, email, and in-app messages.",
   "dependencies": {
-    "com.onesignal.unity.core": "5.0.0"
+    "com.onesignal.unity.core": "5.0.2"
   },
   "keywords": [
     "push-notifications",