Rewrite phone auth with new architecture and finalize progress revamp (#1253)

Author: SUPERCILEX

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

@@ -594,6 +594,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)) {
@@ -616,7 +617,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,34 +4,24 @@
 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;
 import com.google.firebase.auth.FirebaseUser;
 
 @RestrictTo(RestrictTo.Scope.LIBRARY_GROUP)
-public class FragmentBase extends Fragment implements ProgressView {
+public abstract 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() {
@@ -44,14 +34,4 @@ public void startSaveCredentials(
             @Nullable String password) {
         mActivity.startSaveCredentials(firebaseUser, response, password);
     }
-
-    @Override
-    public void showProgress(int message) {
-        mActivity.showProgress(message);
-    }
-
-    @Override
-    public void hideProgress() {
-        mActivity.hideProgress();
-    }
 }