From 3349945f368ace28ddc891c33d9275fdaf463b8b Mon Sep 17 00:00:00 2001
From: Florens Douwes <Floens@gmail.com>
Date: Mon, 26 May 2014 20:31:49 +0200
Subject: [PATCH] Fix for Android bug that causes duplicate baseactivity
 launches.

---
 .../floens/chan/ui/activity/BaseActivity.java | 19 +++++++++++++++++++
 .../chan/ui/activity/BoardActivity.java       |  2 ++
 2 files changed, 21 insertions(+)

diff --git a/Clover/app/src/main/java/org/floens/chan/ui/activity/BaseActivity.java b/Clover/app/src/main/java/org/floens/chan/ui/activity/BaseActivity.java
index c173b196..6af1ce63 100644
--- a/Clover/app/src/main/java/org/floens/chan/ui/activity/BaseActivity.java
+++ b/Clover/app/src/main/java/org/floens/chan/ui/activity/BaseActivity.java
@@ -34,6 +34,7 @@ import android.support.v4.widget.DrawerLayout;
 import android.support.v4.widget.SlidingPaneLayout;
 import android.support.v4.widget.SlidingPaneLayout.PanelSlideListener;
 import android.text.TextUtils;
+import android.util.Log;
 import android.view.Menu;
 import android.view.MenuItem;
 import android.view.View;
@@ -59,8 +60,11 @@ import org.floens.chan.utils.Utils;
 import java.util.List;
 
 public abstract class BaseActivity extends Activity implements PanelSlideListener, PinnedManager.PinListener {
+    private final static String TAG = "BaseActivity";
     private final static int ACTION_OPEN_URL = 1;
 
+    protected boolean wasFinished = false;
+
     protected PinnedAdapter pinnedAdapter;
     protected DrawerLayout pinDrawer;
     protected ListView pinDrawerView;
@@ -89,6 +93,21 @@ public abstract class BaseActivity extends Activity implements PanelSlideListene
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
 
+        // See http://stackoverflow.com/a/7748416/1001608
+        // Possible work around for market launches. See http://code.google.com/p/android/issues/detail?id=2373
+        // for more details. Essentially, the market launches the main activity on top of other activities.
+        // we never want this to happen. Instead, we check if we are the root and if not, we finish.
+        if (!isTaskRoot()) {
+            final Intent intent = getIntent();
+            final String intentAction = intent.getAction();
+            if (intent.hasCategory(Intent.CATEGORY_LAUNCHER) && intentAction != null && intentAction.equals(Intent.ACTION_MAIN)) {
+                Log.w(TAG, "Activity is not the root. Finishing activity instead of launching.");
+                finish();
+                wasFinished = true;
+                return;
+            }
+        }
+
         ThemeHelper.setTheme(this);
         ThemeHelper.getInstance().reloadPostViewColors(this);
 
diff --git a/Clover/app/src/main/java/org/floens/chan/ui/activity/BoardActivity.java b/Clover/app/src/main/java/org/floens/chan/ui/activity/BoardActivity.java
index 00967525..672e9e6c 100644
--- a/Clover/app/src/main/java/org/floens/chan/ui/activity/BoardActivity.java
+++ b/Clover/app/src/main/java/org/floens/chan/ui/activity/BoardActivity.java
@@ -59,6 +59,8 @@ public class BoardActivity extends BaseActivity implements ActionBar.OnNavigatio
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
 
+        if (wasFinished) return;
+
         actionBarSetToListNavigation = false;
         boardLoadable = new Loadable();
         threadLoadable = new Loadable();