Prevent init onHostResume if already initialized for that activity co…

…ntext

Previously we were entering into the init function and logging the error "Already initialized..." which was reported as a bug.

android/src/main/java/com/geektime/rnonesignalandroid/RNOneSignal.java

@@ -154,22 +154,15 @@ public RNOneSignal(ReactApplicationContext reactContext) {
    }
 
    // Initialize OneSignal only once when an Activity is available.
-   // React creates an instance of this class to late for OneSignal to get the current Activity
+   // React creates an instance of this class too late for OneSignal to get the current Activity
    // based on registerActivityLifecycleCallbacks it uses to listen for the first Activity.
    // However it seems it is also to soon to call getCurrentActivity() from the reactContext as well.
    // This will normally succeed when onHostResume fires instead.
    private void initOneSignal() {
+      oneSignalInitDone = true;
       OneSignal.sdkType = "react";
       Context context = mReactApplicationContext.getCurrentActivity();
 
-      if (oneSignalInitDone) {
-         Log.e("OneSignal", "Already initialized the OneSignal React-Native SDK");
-         return;
-      }
-
-      oneSignalInitDone = true;
-
-
       if (context == null) {
          // in some cases, especially when react-native-navigation is installed,
          // the activity can be null, so we can initialize with the context instead
@@ -844,7 +837,14 @@ public void onHostPause() {
 
    @Override
    public void onHostResume() {
-      initOneSignal();
+      // Initialize OneSignal only once when an Activity is available.
+      // React creates an instance of this class too late for OneSignal to get the current Activity
+      // based on registerActivityLifecycleCallbacks it uses to listen for the first Activity.
+      // However it seems it is also to soon to call getCurrentActivity() from the reactContext as well.
+      // This will normally succeed when onHostResume fires instead.
+      if (!oneSignalInitDone) {
+        initOneSignal();
+      }
    }
 
    @Override