From 12051bbc6ac059785fe1144a1054215c07b2e05a Mon Sep 17 00:00:00 2001
From: Sam Stern <samstern@google.com>
Date: Wed, 2 Jan 2019 09:45:46 -0800
Subject: [PATCH 1/4] Begin development on 4.3.1

Change-Id: I6c7fd60a6af675325049ddca4ec4f735c4d157a7
---
 buildSrc/src/main/kotlin/Config.kt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/buildSrc/src/main/kotlin/Config.kt b/buildSrc/src/main/kotlin/Config.kt
index 62372d517..b8d36cd3d 100644
--- a/buildSrc/src/main/kotlin/Config.kt
+++ b/buildSrc/src/main/kotlin/Config.kt
@@ -1,5 +1,5 @@
 object Config {
-    const val version = "4.3.0"
+    const val version = "4.3.1-SNAPSHOT"
     val submodules = listOf("auth", "auth-github", "common", "firestore", "database", "storage")
 
     private const val kotlinVersion = "1.2.61"

From 1fac68d46a766ed4922ce00cca537ef7b369c1a7 Mon Sep 17 00:00:00 2001
From: Sam Stern <samstern@google.com>
Date: Wed, 2 Jan 2019 10:45:30 -0800
Subject: [PATCH 2/4] Revert baseline textview change

Change-Id: If3693ec079a1f69f7338293910ff1f77562a479e
---
 .../auth/util/ui/BaselineTextInputLayout.java | 28 +++++++++++++++++++
 auth/src/main/res/layout/fui_phone_layout.xml |  4 +--
 2 files changed, 30 insertions(+), 2 deletions(-)
 create mode 100644 auth/src/main/java/com/firebase/ui/auth/util/ui/BaselineTextInputLayout.java

diff --git a/auth/src/main/java/com/firebase/ui/auth/util/ui/BaselineTextInputLayout.java b/auth/src/main/java/com/firebase/ui/auth/util/ui/BaselineTextInputLayout.java
new file mode 100644
index 000000000..a1c9fcf0d
--- /dev/null
+++ b/auth/src/main/java/com/firebase/ui/auth/util/ui/BaselineTextInputLayout.java
@@ -0,0 +1,28 @@
+package com.firebase.ui.auth.util.ui;
+
+import android.content.Context;
+import android.support.annotation.RestrictTo;
+import android.support.design.widget.TextInputLayout;
+import android.util.AttributeSet;
+import android.widget.EditText;
+
+@RestrictTo(RestrictTo.Scope.LIBRARY_GROUP)
+public class BaselineTextInputLayout extends TextInputLayout {
+    public BaselineTextInputLayout(Context context) {
+        super(context);
+    }
+
+    public BaselineTextInputLayout(Context context, AttributeSet attrs) {
+        super(context, attrs);
+    }
+
+    public BaselineTextInputLayout(Context context, AttributeSet attrs, int defStyleAttr) {
+        super(context, attrs, defStyleAttr);
+    }
+
+    @Override
+    public int getBaseline() {
+        EditText text = getEditText();
+        return text == null ? super.getBaseline() : text.getPaddingTop() + text.getBaseline();
+    }
+}
diff --git a/auth/src/main/res/layout/fui_phone_layout.xml b/auth/src/main/res/layout/fui_phone_layout.xml
index 1a2dece45..28cc7f7d2 100644
--- a/auth/src/main/res/layout/fui_phone_layout.xml
+++ b/auth/src/main/res/layout/fui_phone_layout.xml
@@ -25,7 +25,7 @@
                 app:layout_constraintStart_toStartOf="parent"
                 app:layout_constraintBaseline_toBaselineOf="@+id/phone_layout" />
 
-            <android.support.design.widget.TextInputLayout
+            <com.firebase.ui.auth.util.ui.BaselineTextInputLayout
                 android:id="@+id/phone_layout"
                 style="@style/FirebaseUI.TextInputLayout.PhoneField"
                 android:layout_width="0dp"
@@ -39,7 +39,7 @@
                     style="@style/FirebaseUI.TextInputEditText.PhoneField"
                     android:imeOptions="actionDone" />
 
-            </android.support.design.widget.TextInputLayout>
+            </com.firebase.ui.auth.util.ui.BaselineTextInputLayout>
 
             <Button
                 android:id="@+id/send_code"

From aaa56ccd773cca056f6821eac6d9b37f8758c12f Mon Sep 17 00:00:00 2001
From: Sam Stern <samstern@google.com>
Date: Wed, 2 Jan 2019 11:19:25 -0800
Subject: [PATCH 3/4] Show the SMS rates warning even without ToS/PP

Change-Id: Ifad1225b0a062555742536d05f54300d968a6d8f
---
 .../ui/auth/ui/phone/CheckPhoneNumberFragment.java  | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/auth/src/main/java/com/firebase/ui/auth/ui/phone/CheckPhoneNumberFragment.java b/auth/src/main/java/com/firebase/ui/auth/ui/phone/CheckPhoneNumberFragment.java
index 6bead64b8..638a0fa6e 100644
--- a/auth/src/main/java/com/firebase/ui/auth/ui/phone/CheckPhoneNumberFragment.java
+++ b/auth/src/main/java/com/firebase/ui/auth/ui/phone/CheckPhoneNumberFragment.java
@@ -46,6 +46,7 @@ public class CheckPhoneNumberFragment extends FragmentBase implements View.OnCli
     private TextInputLayout mPhoneInputLayout;
     private EditText mPhoneEditText;
     private TextView mSmsTermsText;
+    private TextView mFooterText;
 
 
     public static CheckPhoneNumberFragment newInstance(Bundle params) {
@@ -81,6 +82,7 @@ public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceStat
         mPhoneInputLayout = view.findViewById(R.id.phone_layout);
         mPhoneEditText = view.findViewById(R.id.phone_number);
         mSmsTermsText = view.findViewById(R.id.send_sms_tos);
+        mFooterText = view.findViewById(R.id.email_footer_tos_and_pp_text);
 
         mSmsTermsText.setText(getString(R.string.fui_sms_terms_of_service,
                 getString(R.string.fui_verify_phone_number)));
@@ -97,7 +99,7 @@ public void onDonePressed() {
         });
         mSubmitButton.setOnClickListener(this);
 
-        setupPrivacyDisclosures(view.<TextView>findViewById(R.id.email_footer_tos_and_pp_text));
+        setupPrivacyDisclosures();
         setupCountrySpinner();
     }
 
@@ -171,17 +173,20 @@ private String getPseudoValidPhoneNumber() {
                 everythingElse, mCountryListSpinner.getSelectedCountryInfo());
     }
 
-    private void setupPrivacyDisclosures(TextView footerText) {
+    private void setupPrivacyDisclosures() {
         FlowParameters params = getFlowParams();
 
-        if (!params.shouldShowProviderChoice()) {
+        boolean termsAndPrivacyUrlsProvided = params.isTermsOfServiceUrlProvided()
+                && params.isPrivacyPolicyUrlProvided();
+
+        if (!params.shouldShowProviderChoice() && termsAndPrivacyUrlsProvided) {
             PrivacyDisclosureUtils.setupTermsOfServiceAndPrivacyPolicySmsText(requireContext(),
                     params,
                     mSmsTermsText);
         } else {
             PrivacyDisclosureUtils.setupTermsOfServiceFooter(requireContext(),
                     params,
-                    footerText);
+                    mFooterText);
 
             String verifyText = getString(R.string.fui_verify_phone_number);
             mSmsTermsText.setText(getString(R.string.fui_sms_terms_of_service, verifyText));

From d5e98d10d62d3a75932bcf2e388d84c4277e846b Mon Sep 17 00:00:00 2001
From: Sam Stern <samstern@google.com>
Date: Thu, 3 Jan 2019 15:16:42 -0800
Subject: [PATCH 4/4] Version 4.3.1

Change-Id: I68ef0b4d19c70db0d5ddece3c98845b941a920f5
---
 README.md                          | 10 +++++-----
 auth/README.md                     |  4 ++--
 buildSrc/src/main/kotlin/Config.kt |  2 +-
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/README.md b/README.md
index d370f04e3..667ef33e3 100644
--- a/README.md
+++ b/README.md
@@ -50,19 +50,19 @@ libraries.
 ```groovy
 dependencies {
     // FirebaseUI for Firebase Realtime Database
-    implementation 'com.firebaseui:firebase-ui-database:4.3.0'
+    implementation 'com.firebaseui:firebase-ui-database:4.3.1'
 
     // FirebaseUI for Cloud Firestore
-    implementation 'com.firebaseui:firebase-ui-firestore:4.3.0'
+    implementation 'com.firebaseui:firebase-ui-firestore:4.3.1'
 
     // FirebaseUI for Firebase Auth
-    implementation 'com.firebaseui:firebase-ui-auth:4.3.0'
+    implementation 'com.firebaseui:firebase-ui-auth:4.3.1'
 
     // FirebaseUI for Firebase Auth (GitHub provider)
-    implementation 'com.firebaseui:firebase-ui-auth-github:4.3.0'
+    implementation 'com.firebaseui:firebase-ui-auth-github:4.3.1'
 
     // FirebaseUI for Cloud Storage
-    implementation 'com.firebaseui:firebase-ui-storage:4.3.0'
+    implementation 'com.firebaseui:firebase-ui-storage:4.3.1'
 }
 ```
 
diff --git a/auth/README.md b/auth/README.md
index f460563a6..801baa6d2 100644
--- a/auth/README.md
+++ b/auth/README.md
@@ -66,10 +66,10 @@ Gradle, add the dependency:
 ```groovy
 dependencies {
     // ...
-    implementation 'com.firebaseui:firebase-ui-auth:4.3.0'
+    implementation 'com.firebaseui:firebase-ui-auth:4.3.1'
 
     // Required only if GitHub OAuth support is required
-    implementation 'com.firebaseui:firebase-ui-auth-github:4.3.0'
+    implementation 'com.firebaseui:firebase-ui-auth-github:4.3.1'
 
     // Required only if Facebook login support is required
     // Find the latest Facebook SDK releases here: https://goo.gl/Ce5L94
diff --git a/buildSrc/src/main/kotlin/Config.kt b/buildSrc/src/main/kotlin/Config.kt
index b8d36cd3d..98221510a 100644
--- a/buildSrc/src/main/kotlin/Config.kt
+++ b/buildSrc/src/main/kotlin/Config.kt
@@ -1,5 +1,5 @@
 object Config {
-    const val version = "4.3.1-SNAPSHOT"
+    const val version = "4.3.1"
     val submodules = listOf("auth", "auth-github", "common", "firestore", "database", "storage")
 
     private const val kotlinVersion = "1.2.61"