From 176ede10c37b6c60a6045dcbbbd9d4c3d7e238ff Mon Sep 17 00:00:00 2001
From: M66B <M66B@users.noreply.github.com>
Date: Thu, 19 Sep 2019 11:18:34 +0200
Subject: [PATCH] Explain why permissions are needed

---
 .../main/java/eu/faircode/email/FragmentSetup.java    | 11 ++++++++++-
 app/src/main/res/values/strings.xml                   |  3 ++-
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/app/src/main/java/eu/faircode/email/FragmentSetup.java b/app/src/main/java/eu/faircode/email/FragmentSetup.java
index 40b5ee83c3..7637c326c1 100644
--- a/app/src/main/java/eu/faircode/email/FragmentSetup.java
+++ b/app/src/main/java/eu/faircode/email/FragmentSetup.java
@@ -464,7 +464,16 @@ public class FragmentSetup extends FragmentBase {
         if (granted)
             selectAccount();
         else
-            requestPermissions(permissions.toArray(new String[0]), ActivitySetup.REQUEST_CHOOSE_ACCOUNT);
+            new AlertDialog.Builder(getContext())
+                    .setMessage(R.string.title_setup_gmail_rationale)
+                    .setPositiveButton(android.R.string.ok, new DialogInterface.OnClickListener() {
+                        @Override
+                        public void onClick(DialogInterface dialog, int which) {
+                            requestPermissions(permissions.toArray(new String[0]), ActivitySetup.REQUEST_CHOOSE_ACCOUNT);
+                        }
+                    })
+                    .setNegativeButton(android.R.string.cancel, null)
+                    .show();
     }
 
     private void selectAccount() {
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index ebda32c53e..c14608ba13 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -137,7 +137,7 @@
     <string name="title_setup_wizard">Wizard</string>
     <string name="title_setup_gmail" translatable="false">Gmail</string>
     <string name="title_setup_other">Other provider</string>
-    <string name="title_authorizing">Authorizing &#8230;</string>
+    <string name="title_setup_gmail_rationale">Please grant permissions to select an account and read your name</string>
     <string name="title_setup_instructions">Setup instructions</string>
     <string name="title_setup_no_settings">No settings found for domain \'%1$s\'</string>
     <string name="title_setup_quick_success">An account and an identity have successfully been added</string>
@@ -419,6 +419,7 @@
     <string name="title_activesync_support">ActiveSync is not supported</string>
     <string name="title_oauth_support">OAuth is not supported</string>
     <string name="title_authorize">Authorize</string>
+    <string name="title_authorizing">Authorizing &#8230;</string>
 
     <string name="title_synchronize_now">Synchronize now</string>
     <string name="title_synchronize_all">Synchronize all messages</string>