remove try/catch block from getTags

The try/catch block is unnecessary since creating a new JSONObject from a Map will not throw a JSONException.
OneSignal · Dec 5, 2023 · f5d1e27 · f5d1e27
1 parent c076c28
commit f5d1e27
Showing 1 changed file with 5 additions and 9 deletions.
diff --git a/android/src/main/java/com/onesignal/rnonesignalandroid/RNOneSignal.java b/android/src/main/java/com/onesignal/rnonesignalandroid/RNOneSignal.java
@@ -577,16 +577,12 @@ public void removeTags(ReadableArray tagKeys) {
 
     @ReactMethod
     public void getTags(Promise promise) {
-        try {
-            Map<String, String> tags = OneSignal.getUser().getTags();
-            WritableMap writableTags = Arguments.createMap();
-            for (Map.Entry<String, String> entry : tags.entrySet()) {
-                writableTags.putString(entry.getKey(), entry.getValue());
-            }
-            promise.resolve(writableTags);
-        } catch (Throwable t) {
-            promise.reject(t.getMessage());
+        Map<String, String> tags = OneSignal.getUser().getTags();
+        WritableMap writableTags = Arguments.createMap();
+        for (Map.Entry<String, String> entry : tags.entrySet()) {
+            writableTags.putString(entry.getKey(), entry.getValue());
         }
+        promise.resolve(writableTags); 
     }
 
     @ReactMethod