[2.2.1] Regression: Email Validator validates correct mail addresses as false

[Shyru]

In Ext 2.2.1 the email-validator was changed a little bit to fix hanging browsers. (As explained in the changelog)

Unfortunatly this also broke validation for some valid emails. All emails containting a dash (-) are now not validated as correct email addresses.

You can easily check this behaviour in the form example page:
http://extjs.com/deploy/dev/examples/form/dynamic.html

Enter the valid address: test.test@example.com - all ok
Enter the valid address: test-test@example.com - not validated as correct email.

You can check in an example of Ext 2.2 the email test-test@example.com correctly passes validation.

I tried to fix the bug myself changing the regular expression and undoing one of the added slashes of the change from 2.2 to 2.2.1, but that did not fix the Bug. I'm also not a RegExp guru, and don't know which bugs in browser where corrected by the change introduced in 2.2.1.
Could anybody provide a fixed RegExp?

[72]

Confirms this bug. I don't know why is that, but we can hope that v3 will be released sooner or later

[shibubh]

try this regExp

 /^([\w\-]+)(\.[\w]+)*@([\w\-]+\.){1,5}([A-Za-z]){2,4}$/;

[Condor]

Not exactly.

The Ext 2.2 regexp was:

var email = /^([\w]+)(.[\w]+)*@([\w-]+\.){1,5}([A-Za-z]){2,4}$/;

(the . wildcard was obviously wrong)

This was fixed in Ext 2.2.1 by prefixing the . with a \:

var email = /^([\w]+)(\.[\w]+)*@([\w\-]+\.){1,5}([A-Za-z]){2,4}$/;

The only problem is that it now only allows word characters (A-Za-z0-9_) a no other characters (!#$%&'*+-/=?^`{|}~). Of these characters at least - should be supported:

var email = /^([\w\-]+)(\.[\w\-]+)*@([\w\-]+\.){1,5}([A-Za-z]){2,4}$/;

[usiw]

Here is a regular expression pulled from a larger, more RFC correct regex that should suffice for nearly all real world email addresses:

^(?!\.)(?>\.?[a-zA-Z\d!#$%&'*+\-/=?^_`{|}~]+)+@((?!-)[a-zA-Z\d\-]+(?<!-)\.)+[a-zA-Z]{2,}$

I pulled out support for

"my name" <my_email@domain.com>

type emails as well as the

email@[120.90.90.120]

format to simplify things.

[raj_zero1]

i hab a problem with email validation using regex... m fine with email validation for emails like:
abc.xyz_123-abc@something.com
but the problem is i have to allow character ' too... bt i donno what will be regex.
example of email can be:
abc_123-abc's.nepal@something.com

i need help ..

[mystix]

i need help ..

http://extjs.com/learn/Ext_Forum_Help

[raj_zero1]

i found the solution to email validation for email such as
abc.123-abc'nepal@mavendeveloper.com

the solution to this is

regex :/^([\w\-\'\-]+)(\.[\w-\'\-]+)*@([\w\-]+\.){1,5}([A-Za-z]){2,4}$/

http://www.mavendeveloper.com

[temporary]

Looks like this is still not fixed in 2.3.0 and 3.0.0

[temporary]

And we have two valid TLDs with 6 chars now - .travel and .museum