?
avatar brianteeman
brianteeman
4 Apr 2020

Email address validation takes place both on input by the browser and on submit by our code.

The problem is that there has been a change in our validation code for J4 that does not match the browser validation.

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/email

A single properly-formed e-mail address. This doesn't necessarily mean the e-mail address exists, but it is at least formatted correctly. In simple terms, this means username@domain or username@domain.tld.

So if I enter into an email field in user management the browser will respond
brian This value is not valid
brian@example "no error message"
brian@example.com "no error message"

However the regex has changed (#19300 I think) and brian@example is rejected on submission

@wilsonge this is a summary of what I mentioned

avatar brianteeman brianteeman - open - 4 Apr 2020
avatar joomla-cms-bot joomla-cms-bot - change - 4 Apr 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 4 Apr 2020
avatar brianteeman
brianteeman - comment - 4 Apr 2020

For the record I dont see why we have any extra code at all doing additional validation - there is no need

avatar Quy
Quy - comment - 4 Apr 2020

Is #27948 related/duplicate?

avatar wilsonge
wilsonge - comment - 4 Apr 2020

Yes

avatar brianteeman
brianteeman - comment - 4 Apr 2020

Related. Same root cause thinking we can validate emails better than browsers

avatar brianteeman brianteeman - change - 4 Apr 2020
The description was changed
avatar brianteeman brianteeman - edited - 4 Apr 2020
avatar brianteeman
brianteeman - comment - 4 Apr 2020

The simple and correct method is to simply remove the js. It doesn't add anything at all and just causes issues

avatar brianteeman brianteeman - change - 4 Apr 2020
The description was changed
avatar brianteeman brianteeman - edited - 4 Apr 2020
avatar brianteeman
brianteeman - comment - 4 Apr 2020

I am closing this as it turns out NOT to be anything to do with the email validation code but instead is the same as #27948 and is related to phpmailer

avatar brianteeman brianteeman - close - 4 Apr 2020
avatar brianteeman brianteeman - change - 4 Apr 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-04-04 18:53:01
Closed_By brianteeman

Add a Comment

Login with GitHub to post a comment