Skip to content

Commit 4ac0b45

Browse files
author
Iain McGinniss
committed
Checkstyle fixes for WelcomeBackIDPPrompt
Class must still be renamed to WelcomeBackIdpPrompt to conform to style guide, but will do that in a later change to avoid disruption. Change-Id: Iceca14d624c590dfae6572462ea1b032932464b2
1 parent 9ccc7ef commit 4ac0b45

File tree

1 file changed

+22
-18
lines changed

1 file changed

+22
-18
lines changed

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

Lines changed: 22 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,9 @@
3535
import com.google.firebase.auth.FacebookAuthProvider;
3636
import com.google.firebase.auth.GoogleAuthProvider;
3737

38-
public class WelcomeBackIDPPrompt extends BaseActivity implements View.OnClickListener, IDPProvider.IDPCallback {
38+
public class WelcomeBackIDPPrompt
39+
extends BaseActivity
40+
implements View.OnClickListener, IDPProvider.IDPCallback {
3941

4042
private static final String TAG = "WelcomeBackIDPPrompt";
4143

@@ -51,25 +53,27 @@ protected void onCreate(Bundle savedInstanceState) {
5153

5254
mIDPProvider = null;
5355
for (IDPProviderParcel providerParcel: mProviderParcels) {
54-
if (mProviderId.equals(providerParcel.getProviderType())) {
55-
switch (mProviderId) {
56-
case GoogleAuthProvider.PROVIDER_ID:
57-
mIDPProvider = new GoogleProvider(this, providerParcel);
58-
break;
59-
case FacebookAuthProvider.PROVIDER_ID:
60-
mIDPProvider = new FacebookProvider(this, providerParcel);
61-
break;
62-
default:
63-
Log.w(TAG, "Unknown provider: " + mProviderId);
64-
finish(RESULT_CANCELED, getIntent());
65-
return;
66-
}
67-
}
56+
if (mProviderId.equals(providerParcel.getProviderType())) {
57+
switch (mProviderId) {
58+
case GoogleAuthProvider.PROVIDER_ID:
59+
mIDPProvider = new GoogleProvider(this, providerParcel);
60+
break;
61+
case FacebookAuthProvider.PROVIDER_ID:
62+
mIDPProvider = new FacebookProvider(this, providerParcel);
63+
break;
64+
default:
65+
Log.w(TAG, "Unknown provider: " + mProviderId);
66+
finish(RESULT_CANCELED, getIntent());
67+
return;
68+
}
69+
}
6870
}
6971

7072
if (mIDPProvider == null) {
71-
getIntent().putExtra(ControllerConstants.EXTRA_ERROR_MESSAGE,
72-
"Firebase login successful. Account linking failed due to provider not enabled by application");
73+
getIntent().putExtra(
74+
ControllerConstants.EXTRA_ERROR_MESSAGE,
75+
"Firebase login successful. Account linking failed due to provider not "
76+
+ "enabled by application");
7377
finish(RESULT_CANCELED, getIntent());
7478
}
7579

@@ -79,7 +83,7 @@ protected void onCreate(Bundle savedInstanceState) {
7983
mIDPProvider.setAuthenticationCallback(this);
8084
findViewById(R.id.welcome_back_idp_button).setOnClickListener(new View.OnClickListener() {
8185
@Override
82-
public void onClick(View v) {
86+
public void onClick(View view) {
8387
mIDPProvider.startLogin(WelcomeBackIDPPrompt.this, getEmailFromIntent());
8488
}
8589
});

0 commit comments

Comments
 (0)