Merge pull request #5 from DispatcherInc/fix/catalyst-instance-crash

Try fix the Catalyst instance crash

android/src/main/java/com/dispatcher/rnbranch/RNBranchModule.java

@@ -235,9 +235,22 @@ private Branch.BranchLinkShareListener init(Callback callback) {
   }
 
   public void sendRNEvent(String eventName, @Nullable WritableMap params) {
-    getReactApplicationContext()
-        .getJSModule(DeviceEventManagerModule.RCTDeviceEventEmitter.class)
-        .emit(eventName, params);
+    // This should avoid the crash in getJSModule() at startup
+    // Todo: It would be even better to wait until the catalyst instance has became available, but its not clear
+    // how to do that.
+    // See also: https://github.com/walmartreact/react-native-orientation-listener/issues/8
+    // We also wait 100ms here before sending the event so the catalyst instance should be available.
+
+    try {
+        Thread.sleep(100);  
+        if (getReactApplicationContext().hasActiveCatalystInstance()) {
+            getReactApplicationContext()
+                .getJSModule(DeviceEventManagerModule.RCTDeviceEventEmitter.class)
+                .emit(eventName, params);
+        } 
+    } catch(InterruptedException ex) {
+        Thread.currentThread().interrupt();
+    }   
   }
 
   private static Object getReadableMapObjectForKey(ReadableMap readableMap, String key) {
@@ -368,4 +381,4 @@ private static WritableArray convertJsonToArray(JSONArray jsonArray) throws JSON
     }
     return array;
   }
-}
+}

package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-branch",
-  "version": "0.0.9",
+  "version": "0.0.10",
   "description": "Native Wrapper around Branch Metrics native SDKs",
   "main": "index.js",
   "scripts": {