Small improvement

app/src/main/java/eu/faircode/email/FragmentSetup.java

@@ -475,15 +475,11 @@ public class FragmentSetup extends FragmentBase {
     }
 
     private void onAccountSelected(Intent data) {
-        Log.i("Selected " + data);
-        Log.logExtras(data);
-
         String name = data.getStringExtra(AccountManager.KEY_ACCOUNT_NAME);
         String type = data.getStringExtra(AccountManager.KEY_ACCOUNT_TYPE);
 
         AccountManager am = AccountManager.get(getContext());
         Account[] accounts = am.getAccountsByType(type);
-        Log.i("Accounts=" + accounts.length);
         for (final Account account : accounts)
             if (name.equals(account.name)) {
                 Snackbar.make(view, R.string.title_authorizing, Snackbar.LENGTH_LONG).show();
@@ -499,11 +495,12 @@ public class FragmentSetup extends FragmentBase {
                                 try {
                                     Bundle bundle = future.getResult();
                                     String token = bundle.getString(AccountManager.KEY_AUTHTOKEN);
-                                    Log.i("Got token=" + token);
+                                    Log.i("Got token");
                                     onAuthorized(name, token);
                                 } catch (Throwable ex) {
                                     if (ex instanceof AccountsException || ex instanceof IOException) {
-                                        if (getLifecycle().getCurrentState().isAtLeast(Lifecycle.State.RESUMED))
+                                        Log.w(ex);
+                                        if (getLifecycle().getCurrentState().isAtLeast(Lifecycle.State.STARTED))
                                             Snackbar.make(view, Helper.formatThrowable(ex), Snackbar.LENGTH_LONG).show();
                                     } else {
                                         Log.e(ex);

app/src/main/java/eu/faircode/email/MailService.java

@@ -165,21 +165,21 @@ public class MailService implements AutoCloseable {
                         if (user.equals(account.name)) {
                             Log.i("Refreshing token user=" + user);
                             am.invalidateAuthToken(type, password);
-                            String refreshed = am.blockingGetAuthToken(account, getAuthTokenType(type), true);
+                            String token = am.blockingGetAuthToken(account, getAuthTokenType(type), true);
-                            if (refreshed == null)
+                            if (token == null)
-                                throw new IllegalStateException("no token");
+                                throw new IllegalArgumentException("no token");
 
                             int count = 0;
                             DB db = DB.getInstance(context);
                             if ("imap".equals(protocol) || "imaps".equals(protocol))
-                                count = db.account().updateAccountPassword(password, refreshed);
+                                count = db.account().updateAccountPassword(password, token);
                             else if ("smtp".equals(protocol) || "smtps".equals(protocol))
-                                count = db.identity().updateIdentityPassword(password, refreshed);
+                                count = db.identity().updateIdentityPassword(password, token);
 
                             if (count != 1)
-                                throw new IllegalStateException(protocol + "=" + count);
+                                throw new IllegalArgumentException(protocol + "=" + count);
 
-                            _connect(context, host, port, user, refreshed);
+                            _connect(context, host, port, user, token);
                             break;
                         }
                 } catch (Throwable ex1) {