Only set urlEditText.doOnTextChanged { … } listener once

KevinBoulongne · KevinBoulongne · commit f840b7e77552 · 2024-08-13T09:59:52.000+02:00
diff --git a/app/src/main/java/com/infomaniak/mail/ui/newMessage/InsertLinkDialog.kt b/app/src/main/java/com/infomaniak/mail/ui/newMessage/InsertLinkDialog.kt
@@ -57,6 +57,9 @@ class InsertLinkDialog @Inject constructor(
                     resetDialogState()
                     setConfirmButtonListener(dialog)
                 }
+                urlEditText.doOnTextChanged { _, _, _, _ ->
+                    urlLayout.setError(null)
+                }
             }
     }
 
@@ -112,10 +115,6 @@ class InsertLinkDialog @Inject constructor(
                 urlLayout.setError(activityContext.getString(R.string.snackbarInvalidUrl))
             }
         }
-
-        urlEditText.doOnTextChanged { _, _, _, _ ->
-            urlLayout.setError(null)
-        }
     }
 
     private fun addMissingHttpsProtocol(link: String): String {

Original file line number	Diff line number	Diff line change
`@@ -57,6 +57,9 @@ class InsertLinkDialog @Inject constructor(`
`57`	`57`	`resetDialogState()`
`58`	`58`	`setConfirmButtonListener(dialog)`
`59`	`59`	`}`
	`60`	`+ urlEditText.doOnTextChanged { _, _, _, _ ->`
	`61`	`+ urlLayout.setError(null)`
	`62`	`+ }`
`60`	`63`	`}`
`61`	`64`	`}`
`62`	`65`
`@@ -112,10 +115,6 @@ class InsertLinkDialog @Inject constructor(`
`112`	`115`	`urlLayout.setError(activityContext.getString(R.string.snackbarInvalidUrl))`
`113`	`116`	`}`
`114`	`117`	`}`
`115`		`-`
`116`		`- urlEditText.doOnTextChanged { _, _, _, _ ->`
`117`		`- urlLayout.setError(null)`
`118`		`- }`
`119`	`118`	`}`
`120`	`119`
`121`	`120`	`private fun addMissingHttpsProtocol(link: String): String {`