diff options
author | Tom Hennen <tom.hennen@gmail.com> | 2015-06-06 14:22:26 -0400 |
---|---|---|
committer | Tom Hennen <tom.hennen@gmail.com> | 2015-06-06 14:22:26 -0400 |
commit | e903a13a554b6f4226120a99d1df53395386eab0 (patch) | |
tree | 9a307676a7882bcee487045449137fbbfe5cab08 /app/src/main/java/de/danoeh/antennapod | |
parent | 47e09623a9a4d19a027d635ea86f991035c4cc28 (diff) | |
download | AntennaPod-e903a13a554b6f4226120a99d1df53395386eab0.zip |
Avoid IllegalStateExceptions
commitAllowingStateLoss should do this for us. As noted in the TODO there is probably a better
way, but now is not time.
Diffstat (limited to 'app/src/main/java/de/danoeh/antennapod')
-rw-r--r-- | app/src/main/java/de/danoeh/antennapod/activity/MainActivity.java | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/app/src/main/java/de/danoeh/antennapod/activity/MainActivity.java b/app/src/main/java/de/danoeh/antennapod/activity/MainActivity.java index 0645c4620..254749cd6 100644 --- a/app/src/main/java/de/danoeh/antennapod/activity/MainActivity.java +++ b/app/src/main/java/de/danoeh/antennapod/activity/MainActivity.java @@ -332,7 +332,13 @@ public class MainActivity extends ActionBarActivity implements NavDrawerActivity FragmentTransaction t = fragmentManager.beginTransaction(); t.replace(R.id.main_view, fragment, "main"); fragmentManager.popBackStack(); - t.commit(); + // TODO: we have to allow state loss here + // since this function can get called from an AsyncTask which + // could be finishing after our app has already committed state + // and is about to get shutdown. What we *should* do is + // not commit anything in an AsyncTask, but that's a bigger + // change than we want now. + t.commitAllowingStateLoss(); if (navAdapter != null) { navAdapter.notifyDataSetChanged(); } |