nil check on notificationIds before converting it into a string (#573)

* nil check on notificationIds before converting it into a string
* Release 2.12.2

* Prevent side effects by not assigning object directly

* Updated binary

Author: mikechoch

OneSignal.podspec

@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
   s.name             = "OneSignal"
-  s.version          = "2.12.1"
+  s.version          = "2.12.2"
   s.summary          = "OneSignal push notification library for mobile apps."
   s.homepage         = "https://onesignal.com"
   s.license          = { :type => 'MIT', :file => 'LICENSE' }

iOS_SDK/OneSignalSDK/Framework/OneSignal.framework/Versions/A/OneSignal

@@ -51,16 +51,20 @@ - (NSDictionary * _Nonnull)jsonRepresentation {
     let json = [NSMutableDictionary new];
 
     json[@"session"] = OS_SESSION_TO_STRING(self.session);
+    json[@"id"] = self.name;
+    json[@"timestamp"] = @(self.timestamp.intValue);
+    json[@"weight"] = self.weight.stringValue;
+    
+    if (!self.notificationIds) {
+        json[@"notification_ids"] = [NSArray new];
+        return json;
+    }
 
     NSError *error;
     NSData *jsonNotificationIds = [NSJSONSerialization dataWithJSONObject:self.notificationIds options:NSJSONWritingPrettyPrinted error:&error];
     NSString *stringNotificationIds = [[NSString alloc] initWithData:jsonNotificationIds encoding:NSUTF8StringEncoding];
     json[@"notification_ids"] = stringNotificationIds;
 
-    json[@"id"] = self.name;
-    json[@"timestamp"] = @(self.timestamp.intValue);
-    json[@"weight"] = self.weight.stringValue;
-    
     return json;
 }
 

iOS_SDK/OneSignalSDK/Source/OSOutcomeEvent.m

@@ -132,7 +132,7 @@ @interface OneSignal (SessionStatusDelegate)
 
 @implementation OneSignal
 
-NSString* const ONESIGNAL_VERSION = @"021201";
+NSString* const ONESIGNAL_VERSION = @"021202";
 static NSString* mSDKType = @"native";
 static BOOL coldStartFromTapOnNotification = NO;