Skip to content

Commit 1341065

Browse files
Aaron MandleGerrit Code Review
authored andcommitted
Merge "Remove the EmailFlowController"
2 parents 3625168 + 350e756 commit 1341065

15 files changed

+350
-362
lines changed

auth/src/main/java/com/firebase/ui/auth/choreographer/email/EmailFlowController.java

Lines changed: 0 additions & 230 deletions
This file was deleted.

auth/src/main/java/com/firebase/ui/auth/choreographer/email/EmailFlowControllerTransitions.java

Lines changed: 0 additions & 22 deletions
This file was deleted.

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

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import android.app.ProgressDialog;
44
import android.content.Intent;
55
import android.os.Bundle;
6+
import android.support.annotation.StringRes;
67
import android.util.Log;
78

89
import com.firebase.ui.auth.AuthFlowFactory;
@@ -24,7 +25,7 @@ public abstract class NoControllerBaseActivity extends android.support.v7.app.Ap
2425

2526
// TODO once the controller-centric BaseActivity is gone this will no longer be duplicate code
2627
private ProgressDialog mProgressDialog;
27-
private void dismissDialog() {
28+
protected void dismissDialog() {
2829
if (mProgressDialog != null && mProgressDialog.isShowing()) {
2930
mProgressDialog.dismiss();
3031
mProgressDialog = null;
@@ -36,6 +37,10 @@ protected void showLoadingDialog(String message) {
3637
mProgressDialog = ProgressDialog.show(this, "", message, true);
3738
}
3839

40+
protected void showLoadingDialog(@StringRes int stringResource) {
41+
showLoadingDialog(getString(stringResource));
42+
}
43+
3944
@Override
4045
protected void onCreate(Bundle savedInstance) {
4146
super.onCreate(savedInstance);

auth/src/main/java/com/firebase/ui/auth/ui/account_link/AccountLinkInitActivity.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public class AccountLinkInitActivity extends NoControllerBaseActivity {
4646
@Override
4747
protected void onCreate(Bundle savedInstanceState) {
4848
super.onCreate(savedInstanceState);
49-
showLoadingDialog(getResources().getString(R.string.progress_dialog_loading));
49+
showLoadingDialog(R.string.progress_dialog_loading);
5050
mApiWrapper = FirebaseAuthWrapperFactory.getFirebaseAuthWrapper(mAppName);
5151
String email = getIntent().getStringExtra(ControllerConstants.EXTRA_EMAIL);
5252
String password = getIntent().getStringExtra(ControllerConstants.EXTRA_PASSWORD);
@@ -80,6 +80,7 @@ void next(final String email, final String password, final String provider) {
8080
new OnCompleteListener<ProviderQueryResult>() {
8181
@Override
8282
public void onComplete(@NonNull Task<ProviderQueryResult> task) {
83+
dismissDialog();
8384
List<String> providers = task.getResult().getProviders();
8485
if (providers.size() == 0) {
8586
// new account for this email
@@ -116,6 +117,7 @@ public void onComplete(@NonNull Task<ProviderQueryResult> task) {
116117
WelcomeBackIDPPrompt.createIntent(
117118
getApplicationContext(),
118119
provider,
120+
mProviderParcels,
119121
mAppName,
120122
email
121123
),
@@ -129,11 +131,11 @@ public void onComplete(@NonNull Task<ProviderQueryResult> task) {
129131
WelcomeBackIDPPrompt.createIntent(
130132
getApplicationContext(),
131133
provider,
134+
mProviderParcels,
132135
mAppName,
133136
email
134137
),
135138
RC_WELCOME_BACK_IDP_PROMPT);
136-
finish();
137139
}
138140
}
139141
});

auth/src/main/java/com/firebase/ui/auth/ui/account_link/WelcomeBackIDPPrompt.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,8 @@
4141
import com.google.firebase.auth.FirebaseUser;
4242
import com.google.firebase.auth.GoogleAuthProvider;
4343

44+
import java.util.ArrayList;
45+
4446
public class WelcomeBackIDPPrompt extends NoControllerBaseActivity
4547
implements View.OnClickListener, IDPProvider.IDPCallback {
4648

@@ -140,11 +142,13 @@ private String getEmailFromIntent() {
140142
public static Intent createIntent(
141143
Context context,
142144
String providerId,
145+
ArrayList<IDPProviderParcel> providers,
143146
String appName,
144147
String email) {
145148
return new Intent().setClass(context, WelcomeBackIDPPrompt.class)
146149
.putExtra(ControllerConstants.EXTRA_APP_NAME, appName)
147150
.putExtra(ControllerConstants.EXTRA_PROVIDER, providerId)
151+
.putExtra(ControllerConstants.EXTRA_PROVIDERS, providers)
148152
.putExtra(ControllerConstants.EXTRA_EMAIL, email);
149153
}
150154

0 commit comments

Comments
 (0)