Null check when restoring saved fragment tags.

Fixes NPE in destroyItem after restoring state saved where mSavedState.length was 0.
adamsp · Apr 25, 2014 · 90afeed · 90afeed
1 parent 97c6d1f
commit 90afeed
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/...n/java/com/example/fragmentstatepagerissueexample/app/FixedFragmentStatePagerAdapter.java b/...n/java/com/example/fragmentstatepagerissueexample/app/FixedFragmentStatePagerAdapter.java
@@ -218,7 +218,13 @@ public void restoreState(Parcelable state, ClassLoader loader) {
             Parcelable[] fss = bundle.getParcelableArray("states");
             mSavedState.clear();
             mFragments.clear();
-            mSavedFragmentTags = bundle.getStringArrayList("tags");
+
+            ArrayList<String> tags = bundle.getStringArrayList("tags");
+            if (tags != null) {
+                mSavedFragmentTags = tags;
+            } else {
+                mSavedFragmentTags.clear();
+            }
             if (fss != null) {
                 for (int i=0; i<fss.length; i++) {
                     mSavedState.add((Fragment.SavedState)fss[i]);