ishan/fenix
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Close #21702: Update host name check in add login

Browse Source 
Co-Authored-By: joaopmatos <89925580+joaopmatos@users.noreply.github.com>
This commit is contained in:
Roger Yang 2022-08-25 12:37:56 -04:00 committed by mergify[bot]
parent 75a148b882
commit f8f8b4f813
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/src/main/java/org/mozilla/fenix/settings/logins/fragment/AddLoginFragment.kt
View File

@ -9,11 +9,11 @@ import android.os.Bundle
import android.text.Editable
import android.text.InputType
import android.text.TextWatcher
import android.util.Patterns
import android.view.Menu
import android.view.MenuInflater
import android.view.MenuItem
import android.view.View
import android.webkit.URLUtil
import androidx.core.content.ContextCompat
import androidx.core.view.isVisible
import androidx.fragment.app.Fragment
@ -47,7 +47,7 @@ class AddLoginFragment : Fragment(R.layout.fragment_add_login) {
private var duplicateLogin: SavedLogin? = null
private var validPassword = true
private var validPassword = false
private var validUsername = true
private var validHostname = false
private var usernameChanged = false
@ -158,7 +158,7 @@ class AddLoginFragment : Fragment(R.layout.fragment_add_login) {
setHostnameError()
binding.clearHostnameTextButton.isEnabled = false
}
!URLUtil.isHttpUrl(hostnameText) && !URLUtil.isHttpsUrl(hostnameText) -> {
!Patterns.WEB_URL.matcher(hostnameText).matches() -> {
setHostnameError()
binding.clearHostnameTextButton.isEnabled = true
}