Add text watcher to dynamically change phone number input type

This changes the default inputType of the phone number input field to "phone" and adds a TextWatcher to observe for changes to the input field (either manual user edits or programmatic changes from extracting text out of intent launches or history item clicks).

This allows users who share text with Launch Chat to edit the text (using the "text" soft keyboard) before clicking one of the action buttons, while also aiding users who directly launch Launch Chat in typing a phone number (using the "phone" soft keyboard)

app/src/main/kotlin/org/vinaygopinath/launchchat/helpers/TextHelper.kt

@@ -0,0 +1,10 @@
+package org.vinaygopinath.launchchat.helpers
+
+object TextHelper {
+
+    private val phoneNumberRegex = Regex("^[+]?[(]?[0-9]{1,4}[)]?[-\\s./0-9]*$")
+
+    fun doesTextMatchPhoneNumberRegex(text: String): Boolean {
+        return phoneNumberRegex.matches(text)
+    }
+}

app/src/main/kotlin/org/vinaygopinath/launchchat/screens/main/MainActivity.kt

@@ -3,6 +3,7 @@ package org.vinaygopinath.launchchat.screens.main
 import android.content.ActivityNotFoundException
 import android.content.Intent
 import android.os.Bundle
+import android.text.InputType
 import android.view.Menu
 import android.view.MenuItem
 import android.widget.ArrayAdapter
@@ -15,6 +16,7 @@ import androidx.annotation.StringRes
 import androidx.appcompat.app.AlertDialog
 import androidx.appcompat.app.AppCompatActivity
 import androidx.core.view.isVisible
+import androidx.core.widget.addTextChangedListener
 import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.lifecycleScope
 import androidx.lifecycle.repeatOnLifecycle
@@ -32,6 +34,7 @@ import org.vinaygopinath.launchchat.helpers.ClipboardHelper
 import org.vinaygopinath.launchchat.helpers.DetailedActivityHelper
 import org.vinaygopinath.launchchat.helpers.IntentHelper
 import org.vinaygopinath.launchchat.helpers.PhoneNumberHelper
+import org.vinaygopinath.launchchat.helpers.TextHelper
 import org.vinaygopinath.launchchat.models.Action
 import org.vinaygopinath.launchchat.models.Activity
 import org.vinaygopinath.launchchat.models.DetailedActivity
@@ -96,6 +99,15 @@ class MainActivity : AppCompatActivity() {
         messageInput = findViewById(R.id.message_input)
         historyTitle = findViewById(R.id.history_title)
         historyListView = findViewById(R.id.history_list)
+        phoneNumberInput.addTextChangedListener(
+            afterTextChanged = {
+                if (phoneNumberInput.isFocused) {
+                    return@addTextChangedListener
+                }
+
+                updatePhoneNumberInputType()
+            }
+        )
         with(historyListView) {
             val linearLayoutManager = LinearLayoutManager(this@MainActivity)
             layoutManager = linearLayoutManager
@@ -154,7 +166,10 @@ class MainActivity : AppCompatActivity() {
             repeatOnLifecycle(Lifecycle.State.STARTED) {
                 launch {
                     viewModel.uiState.collect { uiState ->
-                        uiState.extractedContent?.let { handleExtractedContent(it) }
+                        uiState.extractedContent?.let {
+                            handleExtractedContent(it)
+                            updatePhoneNumberInputType()
+                        }
                     }
                 }
                 launch {
@@ -280,4 +295,15 @@ class MainActivity : AppCompatActivity() {
     private fun showHistory(activity: Activity) {
         viewModel.logActivityFromHistory(activity)
     }
+
+    private fun updatePhoneNumberInputType() {
+        val inputText = phoneNumberInput.text.toString()
+        val newInputType =
+            if (inputText.isBlank() || TextHelper.doesTextMatchPhoneNumberRegex(inputText)) {
+                InputType.TYPE_CLASS_PHONE
+            } else {
+                InputType.TYPE_CLASS_TEXT or InputType.TYPE_TEXT_FLAG_MULTI_LINE
+            }
+        phoneNumberInput.inputType = newInputType
+    }
 }

app/src/main/res/layout/activity_main.xml

@@ -24,7 +24,7 @@
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:autofillHints="phone"
-            android:inputType="textMultiLine"
+            android:inputType="phone"
             android:minLines="3" />
     </com.google.android.material.textfield.TextInputLayout>
 

app/src/test/kotlin/org/vinaygopinath/launchchat/helpers/TextHelperPasswordRegexTest.kt

@@ -0,0 +1,52 @@
+package org.vinaygopinath.launchchat.helpers
+
+import com.google.common.truth.Truth.assertThat
+import org.junit.Test
+
+class TextHelperPasswordRegexTest {
+
+    @Test
+    fun `returns true when text is phone number with no formatting and no country code`() {
+        val inputText = "111222333"
+
+        assertThat(TextHelper.doesTextMatchPhoneNumberRegex(inputText)).isTrue()
+    }
+
+    @Test
+    fun `returns true when text is phone number with no formatting and country code`() {
+        val inputText = "+1111222333"
+
+        assertThat(TextHelper.doesTextMatchPhoneNumberRegex(inputText)).isTrue()
+    }
+
+
+    @Test
+    fun `returns true when text is phone number with space formatting`() {
+        val inputText = "111 222 333"
+
+        assertThat(TextHelper.doesTextMatchPhoneNumberRegex(inputText)).isTrue()
+    }
+
+
+    @Test
+    fun `returns true when text is phone number with parentheses and space formatting`() {
+        val inputText = "(111) 222 333"
+
+        assertThat(TextHelper.doesTextMatchPhoneNumberRegex(inputText)).isTrue()
+    }
+
+    @Test
+    fun `returns true when text is phone number with hyphen formatting`() {
+        val inputText = "111-222-333"
+
+        assertThat(TextHelper.doesTextMatchPhoneNumberRegex(inputText)).isTrue()
+    }
+
+
+    @Test
+    fun `returns false when text includes non-numerical and non-formatting characters`() {
+        val inputText = "Please contact us on this phone number: +1(111)-222 333"
+
+        assertThat(TextHelper.doesTextMatchPhoneNumberRegex(inputText)).isFalse()
+    }
+}