Assasinate LocalBroadcastReceiver in favor of a simple EventBus implementation

This is needed since some of the data we need to send in the events
can not be serialised and added to a Bundle.

android/app/src/main/java/com/reactnativenavigation/events/Event.java

@@ -0,0 +1,5 @@
+package com.reactnativenavigation.events;
+
+public interface Event {
+    String getType();
+}

android/app/src/main/java/com/reactnativenavigation/events/EventBus.java

@@ -0,0 +1,31 @@
+package com.reactnativenavigation.events;
+
+import java.util.ArrayList;
+import java.util.List;
+
+public enum EventBus {
+    instance;
+
+    List<Subscriber> subscribers;
+
+    EventBus() {
+        subscribers = new ArrayList<>();
+    }
+
+    public void register(Subscriber subscriber) {
+        if (subscribers.contains(subscriber)) {
+            throw new RuntimeException("Subscriber already registered");
+        }
+        subscribers.add(subscriber);
+    }
+
+    public void unregister(Subscriber subscriber) {
+        subscribers.remove(subscriber);
+    }
+
+    public void post(Event event) {
+        for (Subscriber subscriber : subscribers) {
+            subscriber.onEvent(event);
+        }
+    }
+}

android/app/src/main/java/com/reactnativenavigation/events/LocalBroadcastEvent.java

@@ -1,16 +0,0 @@
-package com.reactnativenavigation.events;
-
-import android.content.Intent;
-import android.support.v4.content.LocalBroadcastManager;
-
-import com.reactnativenavigation.NavigationApplication;
-
-public abstract class LocalBroadcastEvent {
-
-    public abstract Intent getIntent();
-
-    public void send() {
-        LocalBroadcastManager.getInstance(NavigationApplication.instance).sendBroadcast(getIntent());
-    }
-
-}

android/app/src/main/java/com/reactnativenavigation/events/ScreenChangeBroadcast.java

@@ -1,12 +0,0 @@
-package com.reactnativenavigation.events;
-
-import android.content.Intent;
-
-public class ScreenChangeBroadcast extends LocalBroadcastEvent {
-    public static final String ACTION = "screenChange";
-
-    @Override
-    public Intent getIntent() {
-        return new Intent(ACTION);
-    }
-}

android/app/src/main/java/com/reactnativenavigation/events/ScreenChangeBroadcastReceiver.java

@@ -1,40 +0,0 @@
-package com.reactnativenavigation.events;
-
-import android.content.BroadcastReceiver;
-import android.content.Context;
-import android.content.Intent;
-import android.content.IntentFilter;
-import android.support.v4.content.LocalBroadcastManager;
-
-import com.reactnativenavigation.NavigationApplication;
-
-public class ScreenChangeBroadcastReceiver extends BroadcastReceiver {
-    private OnScreenChangeListener onTabSelectedListener;
-
-    public interface OnScreenChangeListener {
-        void onScreenChangeListener();
-    }
-
-    public ScreenChangeBroadcastReceiver(OnScreenChangeListener onTabSelectedListener) {
-        this.onTabSelectedListener = onTabSelectedListener;
-    }
-
-    @Override
-    public void onReceive(Context context, Intent intent) {
-        onTabSelectedListener.onScreenChangeListener();
-    }
-
-    public void register() {
-        IntentFilter intentFilter = new IntentFilter();
-        intentFilter.addAction(ScreenChangeBroadcast.ACTION);
-        getBroadcastManager().registerReceiver(this, intentFilter);
-    }
-
-    public void unregister() {
-        getBroadcastManager().unregisterReceiver(this);
-    }
-
-    private LocalBroadcastManager getBroadcastManager() {
-        return LocalBroadcastManager.getInstance(NavigationApplication.instance);
-    }
-}

android/app/src/main/java/com/reactnativenavigation/events/ScreenChangedEvent.java

@@ -0,0 +1,10 @@
+package com.reactnativenavigation.events;
+
+public class ScreenChangedEvent implements Event {
+    public static final String TYPE = "ScreenChangedEvent";
+
+    @Override
+    public String getType() {
+        return TYPE;
+    }
+}

android/app/src/main/java/com/reactnativenavigation/events/Subscriber.java

@@ -0,0 +1,5 @@
+package com.reactnativenavigation.events;
+
+public interface Subscriber {
+    void onEvent(Event event);
+}

android/app/src/main/java/com/reactnativenavigation/screens/ContentViewPagerAdapter.java

@@ -7,7 +7,8 @@ import android.view.ViewGroup;
 import com.facebook.react.bridge.Arguments;
 import com.facebook.react.bridge.WritableMap;
 import com.reactnativenavigation.NavigationApplication;
-import com.reactnativenavigation.events.ScreenChangeBroadcast;
+import com.reactnativenavigation.events.EventBus;
+import com.reactnativenavigation.events.ScreenChangedEvent;
 import com.reactnativenavigation.params.TopTabParams;
 import com.reactnativenavigation.views.ContentView;
 
@@ -40,7 +41,7 @@ public class ContentViewPagerAdapter extends PagerAdapter {
     }
 
     private void sendScreenChangeBroadcast() {
-        new ScreenChangeBroadcast().send();
+        EventBus.instance.post(new ScreenChangedEvent());
     }
 
     private void sendTabSelectedEventToJs() {

android/app/src/main/java/com/reactnativenavigation/views/SnackbarAndFabContainer.java

@@ -4,23 +4,20 @@ import android.content.Context;
 import android.support.annotation.NonNull;
 import android.support.design.widget.CoordinatorLayout;
 
-import com.reactnativenavigation.events.ScreenChangeBroadcastReceiver;
+import com.reactnativenavigation.events.Event;
+import com.reactnativenavigation.events.EventBus;
+import com.reactnativenavigation.events.ScreenChangedEvent;
+import com.reactnativenavigation.events.Subscriber;
 import com.reactnativenavigation.params.FabParams;
 import com.reactnativenavigation.params.SnackbarParams;
 
-public class SnackbarAndFabContainer extends CoordinatorLayout implements Snakbar.OnDismissListener, ScreenChangeBroadcastReceiver.OnScreenChangeListener {
+public class SnackbarAndFabContainer extends CoordinatorLayout implements Snakbar.OnDismissListener, Subscriber{
     private Snakbar snakbar;
-    private ScreenChangeBroadcastReceiver screenChangeBroadcastReceiver;
     private FloatingActionButtonCoordinator actionButtonCoordinator;
 
     public SnackbarAndFabContainer(Context context) {
         super(context);
-        registerTabSelectedReceiver();
-    }
-
-    private void registerTabSelectedReceiver() {
-        screenChangeBroadcastReceiver = new ScreenChangeBroadcastReceiver(this);
-        screenChangeBroadcastReceiver.register();
+        EventBus.instance.register(this);
     }
 
     public void showSnackbar(final String navigatorEventId, final SnackbarParams params) {
@@ -43,15 +40,17 @@ public class SnackbarAndFabContainer extends CoordinatorLayout implements Snakba
     }
 
     public void destroy() {
-        screenChangeBroadcastReceiver.unregister();
-    }
-
-    @Override
-    public void onScreenChangeListener() {
-        onScreenChange();
+        EventBus.instance.unregister(this);
     }
 
     public void showFab(@NonNull FabParams fabParams) {
         actionButtonCoordinator = new FloatingActionButtonCoordinator(this, fabParams);
     }
+
+    @Override
+    public void onEvent(Event event) {
+        if (event.getType() == ScreenChangedEvent.TYPE) {
+            onScreenChange();
+        }
+    }
 }

src/deprecated/platformSpecificDeprecated.android.js

@@ -375,7 +375,9 @@ function getFab(screen) {
     const fab = screen.fab;
     debugger;
     fab.collapsedIcon = resolveAssetSource(fab.collapsedIcon).uri;
-    fab.expendedIcon = resolveAssetSource(fab.expendedIcon).uri;
+    if (fab.expendedIcon) {
+      fab.expendedIcon = resolveAssetSource(fab.expendedIcon).uri;
+    }
 
     if (fab.actions) {
       _.forEach(fab.actions, (action) => {