Skip to content

Commit 58b8b11

Browse files
committed
Merge branch 'version-4.3.3-dev' into version-4.4.0-dev
Change-Id: I0e404042125e45db3da96b061fa64f65c6e4d5c1
2 parents dba6df5 + 2fd768e commit 58b8b11

File tree

7 files changed

+12
-4
lines changed

7 files changed

+12
-4
lines changed

auth/build.gradle.kts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ android {
1313
disable("UnusedQuantity")
1414
disable("UnknownNullness") // TODO fix in future PR
1515
disable("TypographyQuotes") // Straight versus directional quotes
16+
disable("MissingTranslation") // TODO remove after translations for fui_missing_first_and_last_name are added
1617
}
1718

1819
testOptions {

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

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,8 @@ public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceStat
154154
mPasswordInput,
155155
getResources().getInteger(R.integer.fui_min_password_length));
156156
mNameValidator = requireName
157-
? new RequiredFieldValidator(nameInput)
157+
? new RequiredFieldValidator(nameInput,
158+
getResources().getString(R.string.fui_missing_first_and_last_name))
158159
: new NoOpValidator(nameInput);
159160
mEmailFieldValidator = new EmailFieldValidator(mEmailInput);
160161

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ private void validateAndSignIn() {
160160
private void validateAndSignIn(String password) {
161161
// Check for null or empty password
162162
if (TextUtils.isEmpty(password)) {
163-
mPasswordLayout.setError(getString(R.string.fui_required_field));
163+
mPasswordLayout.setError(getString(R.string.fui_error_invalid_password));
164164
return;
165165
} else {
166166
mPasswordLayout.setError(null);

auth/src/main/java/com/firebase/ui/auth/util/ui/fieldvalidators/RequiredFieldValidator.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,11 @@ public RequiredFieldValidator(TextInputLayout errorContainer) {
2626
mErrorMessage = mErrorContainer.getResources().getString(R.string.fui_required_field);
2727
}
2828

29+
public RequiredFieldValidator(TextInputLayout errorContainer, String errorMessage) {
30+
super(errorContainer);
31+
mErrorMessage = errorMessage;
32+
}
33+
2934
@Override
3035
protected boolean isValid(CharSequence charSequence) {
3136
return charSequence != null && charSequence.length() > 0;

auth/src/main/res/values/strings.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@
3737
<string name="fui_required_field" translation_description="Inline error for required field">You can\'t leave this empty.</string>
3838
<string name="fui_invalid_email_address" translation_description="Inline error for invalid email address">That email address isn\'t correct</string>
3939
<string name="fui_missing_email_address" translation_description="Inline error for empty email address in input field">Enter your email address to continue</string>
40+
<string name="fui_missing_first_and_last_name" translation_description="Inline error for empty email address in input field">Please enter a first and last name.</string>
4041
<string name="fui_progress_dialog_checking_accounts" translation_description="Progress dialog text while checking for existing accounts">Checking for existing accounts…</string>
4142

4243
<!-- Email sign up -->

auth/src/test/java/com/firebase/ui/auth/ui/email/EmailActivityTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ public void testSignUpButton_validatesFields() {
127127
TextInputLayout passwordLayout = emailActivity.findViewById(R.id.password_layout);
128128

129129
assertEquals(
130-
emailActivity.getString(R.string.fui_required_field),
130+
emailActivity.getString(R.string.fui_missing_first_and_last_name),
131131
nameLayout.getError().toString());
132132
assertEquals(
133133
String.format(

auth/src/test/java/com/firebase/ui/auth/ui/email/WelcomeBackPasswordPromptTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ public void testSignInButton_validatesFields() {
6969
welcomeBack.findViewById(R.id.password_layout);
7070

7171
assertEquals(
72-
welcomeBack.getString(R.string.fui_required_field),
72+
welcomeBack.getString(R.string.fui_error_invalid_password),
7373
passwordLayout.getError().toString());
7474

7575
// should block and not start a new activity

0 commit comments

Comments
 (0)