From 2890b54523bc1a36a5f562b3dd0ce60912d81936 Mon Sep 17 00:00:00 2001
From: Phillip Hsu <philliphsu@outlook.com>
Date: Mon, 19 Sep 2016 02:09:31 -0700
Subject: [PATCH] Fix bug where snackbar anchor was not being initialized after
 rotation

---
 .../clock2/alarms/AlarmsFragment.java         | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/app/src/main/java/com/philliphsu/clock2/alarms/AlarmsFragment.java b/app/src/main/java/com/philliphsu/clock2/alarms/AlarmsFragment.java
index 1ace642..d5ecd09 100644
--- a/app/src/main/java/com/philliphsu/clock2/alarms/AlarmsFragment.java
+++ b/app/src/main/java/com/philliphsu/clock2/alarms/AlarmsFragment.java
@@ -6,6 +6,7 @@ import android.media.RingtoneManager;
 import android.net.Uri;
 import android.os.Bundle;
 import android.os.Handler;
+import android.support.annotation.Nullable;
 import android.support.v4.content.Loader;
 import android.support.v7.widget.RecyclerView;
 import android.util.Log;
@@ -82,13 +83,6 @@ public class AlarmsFragment extends RecyclerViewFragment<Alarm, BaseAlarmViewHol
             // is called.
             mExpandedPosition = savedInstanceState.getInt(KEY_EXPANDED_POSITION, RecyclerView.NO_POSITION);
         }
-
-        // Will succeed because the activity is created at this point.
-        // See the Fragment lifecycle.
-        mSnackbarAnchor = getActivity().findViewById(R.id.main_content);
-        mAlarmController = new AlarmController(getActivity(), mSnackbarAnchor);
-        mAsyncUpdateHandler = new AsyncAlarmsTableUpdateHandler(getActivity(),
-                mSnackbarAnchor, this, mAlarmController);
         mTimePickerDialogController = new TimePickerDialogController(
                 getFragmentManager(), getActivity(), this);
         mTimePickerDialogController.tryRestoreCallback(makeTimePickerDialogTag());
@@ -99,6 +93,15 @@ public class AlarmsFragment extends RecyclerViewFragment<Alarm, BaseAlarmViewHol
         }
     }
 
+    @Override
+    public void onActivityCreated(@Nullable Bundle savedInstanceState) {
+        super.onActivityCreated(savedInstanceState);
+        mSnackbarAnchor = getActivity().findViewById(R.id.main_content);
+        mAlarmController = new AlarmController(getActivity(), mSnackbarAnchor);
+        mAsyncUpdateHandler = new AsyncAlarmsTableUpdateHandler(getActivity(),
+                mSnackbarAnchor, this, mAlarmController);
+    }
+
     @Override
     public void onResume() {
         super.onResume();
@@ -132,7 +135,7 @@ public class AlarmsFragment extends RecyclerViewFragment<Alarm, BaseAlarmViewHol
     @Override
     protected AlarmsCursorAdapter onCreateAdapter() {
         // Create a new adapter. This is called before we can initialize mAlarmController,
-        // so right now it is null. However, after super.onCreate() returns, it is initialized, and
+        // so right now it is null. However, after onActivityCreated() returns, it is initialized, and
         // the reference variable will be pointing to an actual object. This assignment "propagates"
         // to all references to mAlarmController.
         return new AlarmsCursorAdapter(this, mAlarmController);