🐛 Fix bug where callbacks could only get invoked once

Author: salmatarzi

android/src/main/java/com/instabug/reactlibrary/RNInstabugReactnativeModule.java

@@ -909,7 +909,7 @@ public void setPreInvocationHandler(final Callback preInvocationHandler) {
             Runnable preInvocationRunnable = new Runnable() {
                 @Override
                 public void run() {
-                    preInvocationHandler.invoke();
+                    sendEvent(getReactApplicationContext(), "IBGpreInvocationHandler", null);
                 }
             };
             mInstabug.setPreInvocation(preInvocationRunnable);
@@ -933,7 +933,7 @@ public void setPreSendingHandler(final Callback preSendingHandler) {
             Runnable preSendingRunnable = new Runnable() {
                 @Override
                 public void run() {
-                    preSendingHandler.invoke();
+                    sendEvent(getReactApplicationContext(), "IBGpreSendingHandler", null);
                 }
             };
             mInstabug.setPreSendingRunnable(preSendingRunnable);
@@ -957,7 +957,10 @@ public void setPostInvocationHandler(final Callback postInvocationHandler) {
             mInstabug.setOnSdkDismissedCallback(new OnSdkDismissedCallback() {
                 @Override
                 public void onSdkDismissed(DismissType issueState, Bug.Type bugType) {
-                    postInvocationHandler.invoke();
+                    WritableMap params = Arguments.createMap();
+                    params.putString("issueState",issueState.toString());
+                    params.putString("bugType",bugType.toString());
+                    sendEvent(getReactApplicationContext(), "IBGpostInvocationHandler", params);
                 }
             });
 
@@ -1022,7 +1025,7 @@ public void setWillShowSurveyHandler(final Callback willShowSurveyHandler) {
             Runnable willShowSurveyRunnable = new Runnable() {
                 @Override
                 public void run() {
-                    willShowSurveyHandler.invoke();
+                    sendEvent(getReactApplicationContext(), "IBGWillShowSurvey", null);
                 }
             };
             mInstabug.setPreShowingSurveyRunnable(willShowSurveyRunnable);
@@ -1044,7 +1047,7 @@ public void setDidDismissSurveyHandler(final Callback didDismissSurveyHandler) {
             Runnable didDismissSurveyRunnable = new Runnable() {
                 @Override
                 public void run() {
-                    didDismissSurveyHandler.invoke();
+                    sendEvent(getReactApplicationContext(), "IBGDidDismissSurvey", null);
                 }
             };
             mInstabug.setAfterShowingSurveyRunnable(didDismissSurveyRunnable);
@@ -1115,7 +1118,7 @@ public void setOnNewMessageHandler(final Callback onNewMessageHandler) {
             Runnable onNewMessageRunnable = new Runnable() {
                 @Override
                 public void run() {
-                    onNewMessageHandler.invoke();
+                    sendEvent(getReactApplicationContext(), "IBGonNewMessageHandler", null);
                 }
             };
             mInstabug.setNewMessageHandler(onNewMessageRunnable);

index.js

@@ -95,7 +95,8 @@ module.exports = {
                 'IBGpreSendingHandler',
                 preSendingHandler
             );
-
+        } else {
+            DeviceEventEmitter.addListener('IBGpreSendingHandler', preSendingHandler);
         }
 
         Instabug.setPreSendingHandler(preSendingHandler);
@@ -115,6 +116,8 @@ module.exports = {
                 'IBGpreInvocationHandler',
                 preInvocationHandler
             );
+        } else {
+            DeviceEventEmitter.addListener('IBGpreInvocationHandler', preInvocationHandler);
         }
 
         Instabug.setPreInvocationHandler(preInvocationHandler);
@@ -137,6 +140,10 @@ module.exports = {
                     postInvocationHandler(payload['dismissType'], payload['reportType']);
                 }
             );
+        } else {
+            DeviceEventEmitter.addListener('IBGpostInvocationHandler', function(payload) {
+                postInvocationHandler(payload.issueState, payload.bugType);
+            });
         }
 
         Instabug.setPostInvocationHandler(postInvocationHandler);
@@ -398,9 +405,11 @@ module.exports = {
                 'IBGonNewMessageHandler',
                 onNewMessageHandler
             );
+        } else {
+            DeviceEventEmitter.addListener('IBGonNewMessageHandler', onNewMessageHandler);
         }
 
-        Instabug.setOnNewMessageHandler(onNewMessgaeHandler);
+        Instabug.setOnNewMessageHandler(onNewMessageHandler);
 
     },
 
@@ -680,6 +689,8 @@ module.exports = {
                 'IBGWillShowSurvey',
                 willShowSurveyHandler
             );
+        } else {
+            DeviceEventEmitter.addListener('IBGWillShowSurvey', willShowSurveyHandler);
         }
 
         Instabug.setWillShowSurveyHandler(willShowSurveyHandler);
@@ -699,6 +710,8 @@ module.exports = {
                 'IBGDidDismissSurvey',
                 didDismissSurveyHandler
             );
+        } else {
+            DeviceEventEmitter.addListener('IBGDidDismissSurvey', didDismissSurveyHandler);
         }
 
         Instabug.setDidDismissSurveyHandler(didDismissSurveyHandler);