From 796f43291d1e872b5dddd386fdb006c042fc49d0 Mon Sep 17 00:00:00 2001 From: Chris Mathew Date: Tue, 20 Nov 2018 13:49:14 -0800 Subject: [PATCH] require screengroup subclasses to call super when overriding lifecycle events --- .../main/java/com/wealthfront/magellan/ScreenGroup.java | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/magellan-library/src/main/java/com/wealthfront/magellan/ScreenGroup.java b/magellan-library/src/main/java/com/wealthfront/magellan/ScreenGroup.java index 5f23e58b..63f27fef 100644 --- a/magellan-library/src/main/java/com/wealthfront/magellan/ScreenGroup.java +++ b/magellan-library/src/main/java/com/wealthfront/magellan/ScreenGroup.java @@ -2,6 +2,7 @@ import android.content.Context; import android.os.Bundle; +import android.support.annotation.CallSuper; import android.view.ViewGroup; import java.util.ArrayList; @@ -36,6 +37,7 @@ public void addScreens(List screens) { } @Override + @CallSuper protected void onShow(Context context) { for (Screen screen : screens) { screen.recreateView(getActivity(), getNavigator()); @@ -45,6 +47,7 @@ protected void onShow(Context context) { } @Override + @CallSuper protected void onRestore(Bundle savedInstanceState) { for (Screen screen : screens) { screen.onRestore(savedInstanceState); @@ -52,6 +55,7 @@ protected void onRestore(Bundle savedInstanceState) { } @Override + @CallSuper protected void onResume(Context context) { for (Screen screen : screens) { screen.onResume(context); @@ -59,6 +63,7 @@ protected void onResume(Context context) { } @Override + @CallSuper protected void onPause(Context context) { for (Screen screen : screens) { screen.onPause(context); @@ -66,6 +71,7 @@ protected void onPause(Context context) { } @Override + @CallSuper protected void onSave(Bundle outState) { for (Screen screen : screens) { screen.onSave(outState); @@ -73,6 +79,7 @@ protected void onSave(Bundle outState) { } @Override + @CallSuper protected void onHide(Context context) { for (Screen screen : screens) { screen.onHide(context);