Rewrite phone auth with new architecture and finalize progress revamp

auth/src/main/java/com/firebase/ui/auth/AuthUI.java

@@ -593,6 +593,7 @@ public PhoneBuilder() {
             public PhoneBuilder setDefaultNumber(@NonNull String number) {
                 Preconditions.checkUnset(getParams(),
                         "Cannot overwrite previously set phone number",
+                        ExtraConstants.PHONE,
                         ExtraConstants.COUNTRY_ISO,
                         ExtraConstants.NATIONAL_NUMBER);
                 if (!PhoneNumberUtils.isValid(number)) {
@@ -615,7 +616,9 @@ public PhoneBuilder setDefaultNumber(@NonNull String number) {
             public PhoneBuilder setDefaultNumber(@NonNull String iso, @NonNull String number) {
                 Preconditions.checkUnset(getParams(),
                         "Cannot overwrite previously set phone number",
-                        ExtraConstants.PHONE);
+                        ExtraConstants.PHONE,
+                        ExtraConstants.COUNTRY_ISO,
+                        ExtraConstants.NATIONAL_NUMBER);
                 if (!PhoneNumberUtils.isValidIso(iso)) {
                     throw new IllegalStateException("Invalid country iso: " + iso);
                 }

auth/src/main/java/com/firebase/ui/auth/data/model/PhoneNumberVerificationRequiredException.java

@@ -0,0 +1,32 @@
+package com.firebase.ui.auth.data.model;
+
+import android.support.annotation.NonNull;
+import android.support.annotation.RestrictTo;
+
+import com.firebase.ui.auth.ErrorCodes;
+import com.firebase.ui.auth.FirebaseUiException;
+
+/**
+ * Represents an error in which the phone number couldn't be automatically verified and must
+ * therefore be manually verified by the client by sending an SMS code.
+ */
+@RestrictTo(RestrictTo.Scope.LIBRARY_GROUP)
+public class PhoneNumberVerificationRequiredException extends FirebaseUiException {
+    private final String mPhoneNumber;
+
+    /**
+     * @param number the phone number requiring verification, formatted with a country code prefix
+     */
+    public PhoneNumberVerificationRequiredException(@NonNull String number) {
+        super(ErrorCodes.PROVIDER_ERROR, "Phone number requires verification.");
+        mPhoneNumber = number;
+    }
+
+    /**
+     * @return the phone number requiring verification
+     */
+    @NonNull
+    public String getPhoneNumber() {
+        return mPhoneNumber;
+    }
+}

auth/src/main/java/com/firebase/ui/auth/data/remote/EmailSignInHandler.java

@@ -11,8 +11,8 @@
 import com.firebase.ui.auth.data.model.UserCancellationException;
 import com.firebase.ui.auth.ui.HelperActivityBase;
 import com.firebase.ui.auth.ui.email.EmailActivity;
+import com.firebase.ui.auth.viewmodel.ProviderSignInBase;
 import com.firebase.ui.auth.viewmodel.RequestCodes;
-import com.firebase.ui.auth.viewmodel.idp.ProviderSignInBase;
 
 @RestrictTo(RestrictTo.Scope.LIBRARY_GROUP)
 public class EmailSignInHandler extends ProviderSignInBase<Void> {

auth/src/main/java/com/firebase/ui/auth/data/remote/FacebookSignInHandler.java

@@ -25,7 +25,7 @@
 import com.firebase.ui.auth.data.model.User;
 import com.firebase.ui.auth.ui.HelperActivityBase;
 import com.firebase.ui.auth.util.ExtraConstants;
-import com.firebase.ui.auth.viewmodel.idp.ProviderSignInBase;
+import com.firebase.ui.auth.viewmodel.ProviderSignInBase;
 import com.google.firebase.auth.FacebookAuthProvider;
 
 import org.json.JSONException;

auth/src/main/java/com/firebase/ui/auth/data/remote/GoogleSignInHandler.java

@@ -17,8 +17,8 @@
 import com.firebase.ui.auth.data.model.User;
 import com.firebase.ui.auth.ui.HelperActivityBase;
 import com.firebase.ui.auth.util.ExtraConstants;
+import com.firebase.ui.auth.viewmodel.ProviderSignInBase;
 import com.firebase.ui.auth.viewmodel.RequestCodes;
-import com.firebase.ui.auth.viewmodel.idp.ProviderSignInBase;
 import com.google.android.gms.auth.api.signin.GoogleSignIn;
 import com.google.android.gms.auth.api.signin.GoogleSignInAccount;
 import com.google.android.gms.auth.api.signin.GoogleSignInOptions;

auth/src/main/java/com/firebase/ui/auth/data/remote/PhoneSignInHandler.java

@@ -12,8 +12,8 @@
 import com.firebase.ui.auth.data.model.UserCancellationException;
 import com.firebase.ui.auth.ui.HelperActivityBase;
 import com.firebase.ui.auth.ui.phone.PhoneActivity;
+import com.firebase.ui.auth.viewmodel.ProviderSignInBase;
 import com.firebase.ui.auth.viewmodel.RequestCodes;
-import com.firebase.ui.auth.viewmodel.idp.ProviderSignInBase;
 
 @RestrictTo(RestrictTo.Scope.LIBRARY_GROUP)
 public class PhoneSignInHandler extends ProviderSignInBase<AuthUI.IdpConfig> {

auth/src/main/java/com/firebase/ui/auth/data/remote/SignInKickstarter.java

@@ -3,6 +3,7 @@
 import android.app.Activity;
 import android.app.Application;
 import android.content.Intent;
+import android.os.Bundle;
 import android.support.annotation.NonNull;
 import android.support.annotation.Nullable;
 import android.text.TextUtils;
@@ -18,6 +19,7 @@
 import com.firebase.ui.auth.ui.idp.AuthMethodPickerActivity;
 import com.firebase.ui.auth.ui.idp.SingleSignInActivity;
 import com.firebase.ui.auth.ui.phone.PhoneActivity;
+import com.firebase.ui.auth.util.ExtraConstants;
 import com.firebase.ui.auth.util.GoogleApiUtils;
 import com.firebase.ui.auth.util.data.ProviderUtils;
 import com.firebase.ui.auth.viewmodel.AuthViewModelBase;
@@ -118,21 +120,31 @@ private void startAuthMethodChoice() {
         }
     }
 
-    private void redirectSignIn(String provider, String email) {
+    private void redirectSignIn(String provider, String id) {
         switch (provider) {
             case EmailAuthProvider.PROVIDER_ID:
                 setResult(Resource.<IdpResponse>forFailure(new IntentRequiredException(
-                        EmailActivity.createIntent(getApplication(), getArguments(), email),
+                        EmailActivity.createIntent(getApplication(), getArguments(), id),
                         RequestCodes.EMAIL_FLOW)));
                 break;
+            case PhoneAuthProvider.PROVIDER_ID:
+                Bundle args = new Bundle();
+                args.putString(ExtraConstants.PHONE, id);
+                setResult(Resource.<IdpResponse>forFailure(new IntentRequiredException(
+                        PhoneActivity.createIntent(
+                                getApplication(),
+                                getArguments(),
+                                args),
+                        RequestCodes.PHONE_FLOW)));
+                break;
             case GoogleAuthProvider.PROVIDER_ID:
             case FacebookAuthProvider.PROVIDER_ID:
             case TwitterAuthProvider.PROVIDER_ID:
                 setResult(Resource.<IdpResponse>forFailure(new IntentRequiredException(
                         SingleSignInActivity.createIntent(
                                 getApplication(),
                                 getArguments(),
-                                new User.Builder(provider, email).build()),
+                                new User.Builder(provider, id).build()),
                         RequestCodes.PROVIDER_FLOW)));
                 break;
             default:

auth/src/main/java/com/firebase/ui/auth/data/remote/TwitterSignInHandler.java

@@ -17,7 +17,7 @@
 import com.firebase.ui.auth.data.model.User;
 import com.firebase.ui.auth.ui.HelperActivityBase;
 import com.firebase.ui.auth.util.data.ProviderAvailability;
-import com.firebase.ui.auth.viewmodel.idp.ProviderSignInBase;
+import com.firebase.ui.auth.viewmodel.ProviderSignInBase;
 import com.google.firebase.auth.TwitterAuthProvider;
 import com.twitter.sdk.android.core.Callback;
 import com.twitter.sdk.android.core.Result;

auth/src/main/java/com/firebase/ui/auth/ui/AppCompatBase.java

@@ -20,9 +20,8 @@
 
 import com.firebase.ui.auth.R;
 
-@SuppressWarnings("Registered")
 @RestrictTo(RestrictTo.Scope.LIBRARY_GROUP)
-public class AppCompatBase extends HelperActivityBase {
+public abstract class AppCompatBase extends HelperActivityBase {
     @Override
     protected void onCreate(@Nullable Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);

auth/src/main/java/com/firebase/ui/auth/ui/FragmentBase.java

@@ -4,7 +4,7 @@
 import android.support.annotation.Nullable;
 import android.support.annotation.RestrictTo;
 import android.support.v4.app.Fragment;
-import android.view.ContextThemeWrapper;
+import android.support.v4.app.FragmentActivity;
 
 import com.firebase.ui.auth.IdpResponse;
 import com.firebase.ui.auth.data.model.FlowParameters;
@@ -13,25 +13,15 @@
 @RestrictTo(RestrictTo.Scope.LIBRARY_GROUP)
 public class FragmentBase extends Fragment implements ProgressView {
     private HelperActivityBase mActivity;
-    private ProgressDialogHolder mProgressDialogHolder;
 
     @Override
     public void onCreate(@Nullable Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
-
-        if (!(getActivity() instanceof HelperActivityBase)) {
+        FragmentActivity activity = getActivity();
+        if (!(activity instanceof HelperActivityBase)) {
             throw new IllegalStateException("Cannot use this fragment without the helper activity");
         }
-        mActivity = (HelperActivityBase) getActivity();
-
-        mProgressDialogHolder = new ProgressDialogHolder(new ContextThemeWrapper(
-                getContext(), getFlowParams().themeId));
-    }
-
-    @Override
-    public void onDestroy() {
-        super.onDestroy();
-        mProgressDialogHolder.dismissDialog();
+        mActivity = (HelperActivityBase) activity;
     }
 
     public FlowParameters getFlowParams() {

auth/src/main/java/com/firebase/ui/auth/ui/HelperActivityBase.java

@@ -3,7 +3,6 @@
 import android.app.Activity;
 import android.content.Context;
 import android.content.Intent;
-import android.os.Bundle;
 import android.support.annotation.NonNull;
 import android.support.annotation.Nullable;
 import android.support.annotation.RestrictTo;
@@ -13,7 +12,6 @@
 import com.firebase.ui.auth.IdpResponse;
 import com.firebase.ui.auth.data.model.FlowParameters;
 import com.firebase.ui.auth.ui.credentials.CredentialSaveActivity;
-import com.firebase.ui.auth.util.AuthHelper;
 import com.firebase.ui.auth.util.CredentialUtils;
 import com.firebase.ui.auth.util.ExtraConstants;
 import com.firebase.ui.auth.util.data.ProviderUtils;
@@ -23,14 +21,10 @@
 
 import static com.firebase.ui.auth.util.Preconditions.checkNotNull;
 
-@SuppressWarnings("Registered")
 @RestrictTo(RestrictTo.Scope.LIBRARY_GROUP)
-public class HelperActivityBase extends AppCompatActivity implements ProgressView {
+public abstract class HelperActivityBase extends AppCompatActivity implements ProgressView {
     private FlowParameters mParams;
 
-    private AuthHelper mAuthHelper;
-    private ProgressDialogHolder mProgressDialogHolder;
-
     protected static Intent createBaseIntent(
             @NonNull Context context,
             @NonNull Class<? extends Activity> target,
@@ -42,19 +36,6 @@ protected static Intent createBaseIntent(
                         checkNotNull(flowParams, "flowParams cannot be null"));
     }
 
-    @Override
-    protected void onCreate(@Nullable Bundle savedInstanceState) {
-        super.onCreate(savedInstanceState);
-        mAuthHelper = new AuthHelper(getFlowParams());
-        mProgressDialogHolder = new ProgressDialogHolder(this);
-    }
-
-    @Override
-    protected void onDestroy() {
-        super.onDestroy();
-        mProgressDialogHolder.dismissDialog();
-    }
-
     @Override
     protected void onActivityResult(int requestCode, int resultCode, Intent data) {
         super.onActivityResult(requestCode, resultCode, data);
@@ -71,10 +52,6 @@ public FlowParameters getFlowParams() {
         return mParams;
     }
 
-    public AuthHelper getAuthHelper() {
-        return mAuthHelper;
-    }
-
     public void finish(int resultCode, @Nullable Intent intent) {
         setResult(resultCode, intent);
         finish();
@@ -97,15 +74,13 @@ public void startSaveCredentials(
 
     @Override
     public void showProgress(@StringRes int message) {
-        getDialogHolder().showLoadingDialog(message);
+        throw new UnsupportedOperationException(
+                "Either a fragment or activity must handle progress updates.");
     }
 
     @Override
     public void hideProgress() {
-        getDialogHolder().dismissDialog();
-    }
-
-    protected ProgressDialogHolder getDialogHolder() {
-        return mProgressDialogHolder;
+        throw new UnsupportedOperationException(
+                "Either a fragment or activity must handle progress updates.");
     }
 }

auth/src/main/java/com/firebase/ui/auth/ui/email/CheckEmailFragment.java

@@ -8,6 +8,7 @@
 import android.support.annotation.Nullable;
 import android.support.annotation.RestrictTo;
 import android.support.design.widget.TextInputLayout;
+import android.support.v4.app.FragmentActivity;
 import android.text.TextUtils;
 import android.view.LayoutInflater;
 import android.view.View;
@@ -113,12 +114,12 @@ public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceStat
         FlowParameters flowParameters = getFlowParams();
 
         if (flowParameters.isSingleProviderFlow()) {
-            PrivacyDisclosureUtils.setupTermsOfServiceAndPrivacyPolicyText(getContext(),
+            PrivacyDisclosureUtils.setupTermsOfServiceAndPrivacyPolicyText(requireContext(),
                     flowParameters,
                     termsText);
         } else {
             termsText.setVisibility(View.GONE);
-            PrivacyDisclosureUtils.setupTermsOfServiceFooter(getContext(),
+            PrivacyDisclosureUtils.setupTermsOfServiceFooter(requireContext(),
                     flowParameters,
                     footerText);
         }
@@ -130,10 +131,11 @@ public void onActivityCreated(@Nullable Bundle savedInstanceState) {
         mHandler = ViewModelProviders.of(this).get(CheckEmailHandler.class);
         mHandler.init(getFlowParams());
 
-        if (!(getActivity() instanceof CheckEmailListener)) {
+        FragmentActivity activity = getActivity();
+        if (!(activity instanceof CheckEmailListener)) {
             throw new IllegalStateException("Activity must implement CheckEmailListener");
         }
-        mListener = (CheckEmailListener) getActivity();
+        mListener = (CheckEmailListener) activity;
 
         mHandler.getOperation().observe(this, new ResourceObserver<User>(
                 this, R.string.fui_progress_dialog_checking_accounts) {