From f7b1f3348a99ad7d40ca1d0ea718f5caf404d8b3 Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Mon, 9 Sep 2019 14:40:33 +0200
Subject: [PATCH] Always exclude from recents

---
 .../java/eu/faircode/email/ActivityBase.java  |  9 +----
 .../eu/faircode/email/AdapterAccount.java     |  2 +-
 .../java/eu/faircode/email/AdapterAnswer.java |  7 ++--
 .../eu/faircode/email/AdapterAttachment.java  |  2 +-
 .../java/eu/faircode/email/AdapterFolder.java |  4 +-
 .../java/eu/faircode/email/AdapterImage.java  |  2 +-
 .../eu/faircode/email/AdapterMessage.java     | 37 ++++++++++---------
 .../java/eu/faircode/email/FragmentBase.java  |  9 +----
 .../eu/faircode/email/FragmentCompose.java    |  4 +-
 .../eu/faircode/email/FragmentDialogEx.java   |  4 +-
 .../eu/faircode/email/FragmentFolders.java    |  2 +-
 .../eu/faircode/email/FragmentMessages.java   | 16 ++++----
 .../java/eu/faircode/email/FragmentRule.java  |  4 +-
 .../main/java/eu/faircode/email/Helper.java   | 24 +++++++++---
 14 files changed, 62 insertions(+), 64 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/ActivityBase.java b/app/src/main/java/eu/faircode/email/ActivityBase.java
index f24c735e95..54aec891f9 100644
--- a/app/src/main/java/eu/faircode/email/ActivityBase.java
+++ b/app/src/main/java/eu/faircode/email/ActivityBase.java
@@ -199,14 +199,7 @@ abstract class ActivityBase extends AppCompatActivity implements SharedPreferenc
 
     @Override
     public void startActivity(Intent intent) {
-        try {
-            if (Helper.hasAuthentication(this))
-                intent.addFlags(Intent.FLAG_ACTIVITY_EXCLUDE_FROM_RECENTS);
-            super.startActivity(intent);
-        } catch (ActivityNotFoundException ex) {
-            Log.e(ex);
-            ToastEx.makeText(this, getString(R.string.title_no_viewer, intent.getAction()), Toast.LENGTH_LONG).show();
-        }
+        Helper.startActivity(this, intent);
     }
 
     @Override
diff --git a/app/src/main/java/eu/faircode/email/AdapterAccount.java b/app/src/main/java/eu/faircode/email/AdapterAccount.java
index 121399a860..77216b3488 100644
--- a/app/src/main/java/eu/faircode/email/AdapterAccount.java
+++ b/app/src/main/java/eu/faircode/email/AdapterAccount.java
@@ -286,7 +286,7 @@ public class AdapterAccount extends RecyclerView.Adapter<AdapterAccount.ViewHold
                     Intent intent = new Intent(Settings.ACTION_CHANNEL_NOTIFICATION_SETTINGS)
                             .putExtra(Settings.EXTRA_APP_PACKAGE, context.getPackageName())
                             .putExtra(Settings.EXTRA_CHANNEL_ID, EntityAccount.getNotificationChannelId(account.id));
-                    context.startActivity(intent);
+                    Helper.startActivity(context, intent);
                 }
 
                 private void onActionCopy() {
diff --git a/app/src/main/java/eu/faircode/email/AdapterAnswer.java b/app/src/main/java/eu/faircode/email/AdapterAnswer.java
index b48e4d19f0..eebe2a6a12 100644
--- a/app/src/main/java/eu/faircode/email/AdapterAnswer.java
+++ b/app/src/main/java/eu/faircode/email/AdapterAnswer.java
@@ -137,9 +137,10 @@ public class AdapterAnswer extends RecyclerView.Adapter<AdapterAnswer.ViewHolder
                 }
 
                 private void onActionCompose() {
-                    context.startActivity(new Intent(context, ActivityCompose.class)
-                            .putExtra("action", "new")
-                            .putExtra("answer", answer.id));
+                    Helper.startActivity(context,
+                            new Intent(context, ActivityCompose.class)
+                                    .putExtra("action", "new")
+                                    .putExtra("answer", answer.id));
                 }
 
                 private void onActionHide(boolean hide) {
diff --git a/app/src/main/java/eu/faircode/email/AdapterAttachment.java b/app/src/main/java/eu/faircode/email/AdapterAttachment.java
index ce9c2a16d4..cdd7acaeb0 100644
--- a/app/src/main/java/eu/faircode/email/AdapterAttachment.java
+++ b/app/src/main/java/eu/faircode/email/AdapterAttachment.java
@@ -244,7 +244,7 @@ public class AdapterAttachment extends RecyclerView.Adapter<AdapterAttachment.Vi
                         context.getString(R.string.title_no_viewer, attachment.type),
                         Snackbar.LENGTH_LONG).show();
             else
-                context.startActivity(intent);
+                Helper.startActivity(context, intent);
         }
 
         private void onDownload(EntityAttachment attachment) {
diff --git a/app/src/main/java/eu/faircode/email/AdapterFolder.java b/app/src/main/java/eu/faircode/email/AdapterFolder.java
index 1f7366bba7..a341bcd5b2 100644
--- a/app/src/main/java/eu/faircode/email/AdapterFolder.java
+++ b/app/src/main/java/eu/faircode/email/AdapterFolder.java
@@ -619,7 +619,7 @@ public class AdapterFolder extends RecyclerView.Adapter<AdapterFolder.ViewHolder
                 @RequiresApi(api = Build.VERSION_CODES.O)
                 private void onActionCreateChannel() {
                     if (!ActivityBilling.isPro(context)) {
-                        context.startActivity(new Intent(context, ActivityBilling.class));
+                        Helper.startActivity(context, new Intent(context, ActivityBilling.class));
                         return;
                     }
 
@@ -632,7 +632,7 @@ public class AdapterFolder extends RecyclerView.Adapter<AdapterFolder.ViewHolder
                     Intent intent = new Intent(Settings.ACTION_CHANNEL_NOTIFICATION_SETTINGS)
                             .putExtra(Settings.EXTRA_APP_PACKAGE, context.getPackageName())
                             .putExtra(Settings.EXTRA_CHANNEL_ID, EntityFolder.getNotificationChannelId(folder.id));
-                    context.startActivity(intent);
+                    Helper.startActivity(context, intent);
                 }
 
                 @RequiresApi(api = Build.VERSION_CODES.O)
diff --git a/app/src/main/java/eu/faircode/email/AdapterImage.java b/app/src/main/java/eu/faircode/email/AdapterImage.java
index 5bc138f034..29a105b7fc 100644
--- a/app/src/main/java/eu/faircode/email/AdapterImage.java
+++ b/app/src/main/java/eu/faircode/email/AdapterImage.java
@@ -142,7 +142,7 @@ public class AdapterImage extends RecyclerView.Adapter<AdapterImage.ViewHolder>
                     return;
                 }
 
-                context.startActivity(intent);
+                Helper.startActivity(context, intent);
             } else {
                 if (attachment.progress == null) {
                     Bundle args = new Bundle();
diff --git a/app/src/main/java/eu/faircode/email/AdapterMessage.java b/app/src/main/java/eu/faircode/email/AdapterMessage.java
index e87095186f..86a086b549 100644
--- a/app/src/main/java/eu/faircode/email/AdapterMessage.java
+++ b/app/src/main/java/eu/faircode/email/AdapterMessage.java
@@ -1368,7 +1368,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
 
         private void onActionCalendar(TupleMessageEx message, int action) {
             if (!ActivityBilling.isPro(context)) {
-                context.startActivity(new Intent(context, ActivityBilling.class));
+                Helper.startActivity(context, new Intent(context, ActivityBilling.class));
                 return;
             }
 
@@ -1457,7 +1457,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                             .putExtra("reference", args.getLong("id"))
                             .putExtra("ics", ics)
                             .putExtra("status", status);
-                    context.startActivity(reply);
+                    Helper.startActivity(context, reply);
                 }
 
                 @Override
@@ -1563,7 +1563,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                 }
 
                 if (EntityFolder.DRAFTS.equals(message.folderType) && message.visible == 1)
-                    context.startActivity(
+                    Helper.startActivity(context,
                             new Intent(context, ActivityCompose.class)
                                     .putExtra("action", "edit")
                                     .putExtra("id", message.id));
@@ -1862,7 +1862,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                 @TargetApi(Build.VERSION_CODES.O)
                 private void onActionCreateChannel() {
                     if (!ActivityBilling.isPro(context)) {
-                        context.startActivity(new Intent(context, ActivityBilling.class));
+                        Helper.startActivity(context, new Intent(context, ActivityBilling.class));
                         return;
                     }
 
@@ -1881,7 +1881,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                     Intent intent = new Intent(Settings.ACTION_CHANNEL_NOTIFICATION_SETTINGS)
                             .putExtra(Settings.EXTRA_APP_PACKAGE, context.getPackageName())
                             .putExtra(Settings.EXTRA_CHANNEL_ID, channelId);
-                    context.startActivity(intent);
+                    Helper.startActivity(context, intent);
                 }
 
                 private void onActionDeleteChannel() {
@@ -1936,13 +1936,13 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                     Snackbar.make(parentFragment.getView(),
                             R.string.title_no_contacts, Snackbar.LENGTH_LONG).show();
                 else
-                    context.startActivity(edit);
+                    Helper.startActivity(context, edit);
             }
         }
 
         private void onToggleMessage(TupleMessageEx message) {
             if (EntityFolder.DRAFTS.equals(message.folderType))
-                context.startActivity(
+                Helper.startActivity(context,
                         new Intent(context, ActivityCompose.class)
                                 .putExtra("action", "edit")
                                 .putExtra("id", message.id));
@@ -2207,7 +2207,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
             Intent reply = new Intent(context, ActivityCompose.class)
                     .putExtra("action", action)
                     .putExtra("reference", message.id);
-            context.startActivity(reply);
+            Helper.startActivity(context, reply);
         }
 
         private void onMenuAnswer(TupleMessageEx message) {
@@ -2244,14 +2244,15 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                             @Override
                             public boolean onMenuItemClick(MenuItem target) {
                                 if (!ActivityBilling.isPro(context)) {
-                                    context.startActivity(new Intent(context, ActivityBilling.class));
+                                    Helper.startActivity(context, new Intent(context, ActivityBilling.class));
                                     return true;
                                 }
 
-                                context.startActivity(new Intent(context, ActivityCompose.class)
-                                        .putExtra("action", "reply")
-                                        .putExtra("reference", message.id)
-                                        .putExtra("answer", (long) target.getItemId()));
+                                Helper.startActivity(context,
+                                        new Intent(context, ActivityCompose.class)
+                                                .putExtra("action", "reply")
+                                                .putExtra("reference", message.id)
+                                                .putExtra("answer", (long) target.getItemId()));
                                 return true;
                             }
                         });
@@ -2747,7 +2748,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
             Intent asnew = new Intent(context, ActivityCompose.class)
                     .putExtra("action", "editasnew")
                     .putExtra("reference", message.id);
-            context.startActivity(asnew);
+            Helper.startActivity(context, asnew);
         }
 
         private void onMenuUnseen(final TupleMessageEx message) {
@@ -2974,7 +2975,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                         Snackbar.make(parentFragment.getView(),
                                 R.string.title_no_viewer, Snackbar.LENGTH_LONG).show();
                     else
-                        context.startActivity(share);
+                        Helper.startActivity(context, share);
                 }
 
                 @Override
@@ -3099,7 +3100,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
             Intent send = new Intent(Intent.ACTION_SEND);
             send.putExtra(Intent.EXTRA_STREAM, uri);
             send.setType("message/rfc822");
-            context.startActivity(send);
+            Helper.startActivity(context, send);
         }
 
         ItemDetailsLookup.ItemDetails<Long> getItemDetails(@NonNull MotionEvent motionEvent) {
@@ -4081,7 +4082,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                         @Override
                         public void onClick(DialogInterface dialog, int which) {
                             if (!ActivityBilling.isPro(getContext())) {
-                                getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+                                startActivity(new Intent(getContext(), ActivityBilling.class));
                                 return;
                             }
 
@@ -4157,7 +4158,7 @@ public class AdapterMessage extends RecyclerView.Adapter<AdapterMessage.ViewHold
                         @Override
                         public void onClick(DialogInterface dialog, int which) {
                             if (!ActivityBilling.isPro(getContext())) {
-                                getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+                                startActivity(new Intent(getContext(), ActivityBilling.class));
                                 return;
                             }
 
diff --git a/app/src/main/java/eu/faircode/email/FragmentBase.java b/app/src/main/java/eu/faircode/email/FragmentBase.java
index 7e96997ecf..3660dc37b0 100644
--- a/app/src/main/java/eu/faircode/email/FragmentBase.java
+++ b/app/src/main/java/eu/faircode/email/FragmentBase.java
@@ -64,14 +64,7 @@ public class FragmentBase extends Fragment {
 
     @Override
     public void startActivity(Intent intent) {
-        try {
-            if (Helper.hasAuthentication(getContext()))
-                intent.addFlags(Intent.FLAG_ACTIVITY_EXCLUDE_FROM_RECENTS);
-            super.startActivity(intent);
-        } catch (ActivityNotFoundException ex) {
-            Log.e(ex);
-            ToastEx.makeText(getContext(), getString(R.string.title_no_viewer, intent.getAction()), Toast.LENGTH_LONG).show();
-        }
+        Helper.startActivity(getContext(), intent);
     }
 
     @Override
diff --git a/app/src/main/java/eu/faircode/email/FragmentCompose.java b/app/src/main/java/eu/faircode/email/FragmentCompose.java
index 6496c9b678..1afcca5163 100644
--- a/app/src/main/java/eu/faircode/email/FragmentCompose.java
+++ b/app/src/main/java/eu/faircode/email/FragmentCompose.java
@@ -940,7 +940,7 @@ public class FragmentCompose extends FragmentBase {
 
     private void onMenuAnswer() {
         if (!ActivityBilling.isPro(getContext())) {
-            getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+            startActivity(new Intent(getContext(), ActivityBilling.class));
             return;
         }
 
@@ -1586,7 +1586,7 @@ public class FragmentCompose extends FragmentBase {
 
     private void onSendAfter(long time) {
         if (!ActivityBilling.isPro(getContext())) {
-            getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+            startActivity(new Intent(getContext(), ActivityBilling.class));
             return;
         }
 
diff --git a/app/src/main/java/eu/faircode/email/FragmentDialogEx.java b/app/src/main/java/eu/faircode/email/FragmentDialogEx.java
index d5a69933fa..cec0d5aa60 100644
--- a/app/src/main/java/eu/faircode/email/FragmentDialogEx.java
+++ b/app/src/main/java/eu/faircode/email/FragmentDialogEx.java
@@ -97,9 +97,7 @@ public class FragmentDialogEx extends DialogFragment {
 
     @Override
     public void startActivity(Intent intent) {
-        if (Helper.hasAuthentication(getContext()))
-            intent.addFlags(Intent.FLAG_ACTIVITY_EXCLUDE_FROM_RECENTS);
-        super.startActivity(intent);
+        Helper.startActivity(getContext(), intent);
     }
 
     @Override
diff --git a/app/src/main/java/eu/faircode/email/FragmentFolders.java b/app/src/main/java/eu/faircode/email/FragmentFolders.java
index 7f303e2795..f49023c46e 100644
--- a/app/src/main/java/eu/faircode/email/FragmentFolders.java
+++ b/app/src/main/java/eu/faircode/email/FragmentFolders.java
@@ -356,7 +356,7 @@ public class FragmentFolders extends FragmentBase {
                     snackbar.setAction(R.string.title_fix, new View.OnClickListener() {
                         @Override
                         public void onClick(View view) {
-                            getContext().startActivity(
+                            startActivity(
                                     new Intent(getContext(), ActivitySetup.class)
                                             .putExtra("tab", "connection"));
                         }
diff --git a/app/src/main/java/eu/faircode/email/FragmentMessages.java b/app/src/main/java/eu/faircode/email/FragmentMessages.java
index af4b366d55..0cec1cdc05 100644
--- a/app/src/main/java/eu/faircode/email/FragmentMessages.java
+++ b/app/src/main/java/eu/faircode/email/FragmentMessages.java
@@ -396,7 +396,7 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
         tvSupport.setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View v) {
-                getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+                startActivity(new Intent(getContext(), ActivityBilling.class));
             }
         });
 
@@ -1169,7 +1169,7 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
                     snackbar.setAction(R.string.title_fix, new View.OnClickListener() {
                         @Override
                         public void onClick(View view) {
-                            getContext().startActivity(
+                            startActivity(
                                     new Intent(getContext(), ActivitySetup.class)
                                             .putExtra("tab", "connection"));
                         }
@@ -2900,7 +2900,7 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
                     snackbar.setAction(R.string.title_fix, new View.OnClickListener() {
                         @Override
                         public void onClick(View view) {
-                            getContext().startActivity(
+                            startActivity(
                                     new Intent(getContext(), ActivitySetup.class)
                                             .putExtra("tab", "connection"));
                         }
@@ -3609,7 +3609,7 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
                 case REQUEST_MESSAGES_COLOR:
                     if (resultCode == RESULT_OK && data != null) {
                         if (!ActivityBilling.isPro(getContext())) {
-                            getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+                            startActivity(new Intent(getContext(), ActivityBilling.class));
                             return;
                         }
 
@@ -4209,7 +4209,7 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
 
     private void onColor(long id, int color) {
         if (!ActivityBilling.isPro(getContext())) {
-            getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+            startActivity(new Intent(getContext(), ActivityBilling.class));
             return;
         }
 
@@ -4246,7 +4246,7 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
 
     private void onSnooze(Bundle args) {
         if (!ActivityBilling.isPro(getContext())) {
-            getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+            startActivity(new Intent(getContext(), ActivityBilling.class));
             return;
         }
 
@@ -4299,7 +4299,7 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
 
     private void onSnoozeSelection(Bundle args) {
         if (!ActivityBilling.isPro(getContext())) {
-            getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+            startActivity(new Intent(getContext(), ActivityBilling.class));
             return;
         }
 
@@ -4562,7 +4562,7 @@ public class FragmentMessages extends FragmentBase implements SharedPreferences.
             final Context context, final LifecycleOwner owner, final FragmentManager manager,
             long folder, boolean server, String query) {
         if (server && !ActivityBilling.isPro(context)) {
-            context.startActivity(new Intent(context, ActivityBilling.class));
+            Helper.startActivity(context, new Intent(context, ActivityBilling.class));
             return;
         }
 
diff --git a/app/src/main/java/eu/faircode/email/FragmentRule.java b/app/src/main/java/eu/faircode/email/FragmentRule.java
index 2a30e251c5..e0324d4ac5 100644
--- a/app/src/main/java/eu/faircode/email/FragmentRule.java
+++ b/app/src/main/java/eu/faircode/email/FragmentRule.java
@@ -492,7 +492,7 @@ public class FragmentRule extends FragmentBase {
                 case REQUEST_COLOR:
                     if (resultCode == RESULT_OK && data != null) {
                         if (!ActivityBilling.isPro(getContext())) {
-                            getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+                            startActivity(new Intent(getContext(), ActivityBilling.class));
                             return;
                         }
 
@@ -771,7 +771,7 @@ public class FragmentRule extends FragmentBase {
 
     private void onActionSave() {
         if (!ActivityBilling.isPro(getContext())) {
-            getContext().startActivity(new Intent(getContext(), ActivityBilling.class));
+            startActivity(new Intent(getContext(), ActivityBilling.class));
             return;
         }
 
diff --git a/app/src/main/java/eu/faircode/email/Helper.java b/app/src/main/java/eu/faircode/email/Helper.java
index 356a33b8fa..0a87cf36a4 100644
--- a/app/src/main/java/eu/faircode/email/Helper.java
+++ b/app/src/main/java/eu/faircode/email/Helper.java
@@ -189,7 +189,7 @@ public class Helper {
         if ("http".equals(uri.getScheme()) || "https".equals(uri.getScheme()))
             view(context, intent.getData(), false);
         else
-            context.startActivity(intent);
+            startActivity(context, intent);
     }
 
     static void view(Context context, Uri uri, boolean browse) {
@@ -197,7 +197,7 @@ public class Helper {
 
         if (browse || !hasCustomTabs(context, uri)) {
             Intent view = new Intent(Intent.ACTION_VIEW, uri);
-            context.startActivity(getChooser(context, view));
+            startActivity(context, getChooser(context, view));
         } else {
             // https://developer.chrome.com/multidevice/android/customtabs
             CustomTabsIntent.Builder builder = new CustomTabsIntent.Builder();
@@ -250,6 +250,17 @@ public class Helper {
             return new Intent(Intent.ACTION_VIEW, Uri.parse(XDA_URI));
     }
 
+    static void startActivity(Context context, Intent intent) {
+        try {
+            if (Helper.hasAuthentication(context))
+                intent.addFlags(Intent.FLAG_ACTIVITY_EXCLUDE_FROM_RECENTS);
+            context.startActivity(intent);
+        } catch (ActivityNotFoundException ex) {
+            Log.e(ex);
+            ToastEx.makeText(context, context.getString(R.string.title_no_viewer, intent.getAction()), Toast.LENGTH_LONG).show();
+        }
+    }
+
     // Graphics
 
     static int dp2pixels(Context context, int dp) {
@@ -495,9 +506,10 @@ public class Helper {
 
                                 @Override
                                 protected void onExecuted(Bundle args, Long id) {
-                                    context.startActivity(new Intent(context, ActivityCompose.class)
-                                            .putExtra("action", "edit")
-                                            .putExtra("id", id));
+                                    Helper.startActivity(context,
+                                            new Intent(context, ActivityCompose.class)
+                                                    .putExtra("action", "edit")
+                                                    .putExtra("id", id));
                                 }
 
                                 @Override
@@ -825,7 +837,7 @@ public class Helper {
             tv.setOnClickListener(new View.OnClickListener() {
                 @Override
                 public void onClick(View view) {
-                    tv.getContext().startActivity(new Intent(tv.getContext(), ActivityBilling.class));
+                    Helper.startActivity(tv.getContext(), new Intent(tv.getContext(), ActivityBilling.class));
                 }
             });
         }