Merge pull request #526 from OneSignal/fix/methodSignatureAndroid

[FIX] Java Collection type parameters for Android

Author: shepherd-l

OneSignalExample/Assets/OneSignal/CHANGELOG.md

@@ -5,6 +5,9 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Fixed
+- Android `DeleteTags` and `RemoveTriggers` calls correctly use a Java array list instead of an array
+
 ## [3.0.3]
 ### Changed
 - Added support for OneSignal Android functionality `promptForPushNotifications`

com.onesignal.unity.android/Runtime/OneSignalAndroid.cs

@@ -170,7 +170,7 @@ public override void RemoveTrigger(string key)
             => _sdkClass.CallStatic("removeTriggerForKey", key);
 
         public override void RemoveTriggers(params string[] keys)
-            => _sdkClass.CallStatic("removeTriggersForKeys", Json.Serialize(keys));
+            => _sdkClass.CallStatic("removeTriggersForKeys", keys.ToArrayList());
 
         public override string GetTrigger(string key) {
             var triggerVal = _sdkClass.CallStatic<AndroidJavaObject>("getTriggerValueForKey", key);
@@ -210,7 +210,7 @@ public override async Task<bool> DeleteTag(string key) {
 
         public override async Task<bool> DeleteTags(params string[] keys) {
             var proxy = new ChangeTagsUpdateHandler();
-            _sdkClass.CallStatic("deleteTags", keys.ToList(), proxy);
+            _sdkClass.CallStatic("deleteTags", keys.ToArrayList(), proxy);
             return await proxy;
         }
 

com.onesignal.unity.android/Runtime/Utilities/AndroidJavaObjectExtensions.cs

@@ -111,5 +111,15 @@ public static AndroidJavaObject ToMap(this Dictionary<string, string> source) {
             return map;
         }
 
+        public static AndroidJavaObject ToArrayList(this string[] keys) {
+            AndroidJavaObject arrayList = new AndroidJavaObject("java.util.ArrayList");
+
+            foreach(string key in keys) {
+                arrayList.Call<bool>("add", key);
+            }
+
+            return arrayList;
+        }
+
     }
 }