Success! Looks like we've fixed this one. According to our records the fix was applied for TOUCH-2858 in a recent build.
  1. #1
    Sencha User
    Join Date
    Nov 2011
    Posts
    61
    Vote Rating
    3
    FrankK is on a distinguished road

      0  

    Default Ext.data.Validations e-mail regex doesn't allow all possible characters

    Ext.data.Validations e-mail regex doesn't allow all possible characters


    I do a lot of testing with e-mail addresses that have plus (+) signs in them, but they fail every time. I checked the e-mail validation regex and it doesn't look like the plus sign as well as a bunch of other valid characters are allowed.

  2. #2
    Sencha - Senior Forum Manager mitchellsimoens's Avatar
    Join Date
    Mar 2007
    Location
    Gainesville, FL
    Posts
    37,330
    Vote Rating
    847
    mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute mitchellsimoens has a reputation beyond repute

      0  

    Default


    Thank you for the report

    Code:
    Ext.define('MyModel', {
        extend : 'Ext.data.Model',
    
        config : {
            fields      : ['email'],
            validations : [
                {
                    type  : 'email',
                    field : 'email'
                }
            ]
        }
    });
    
    var model = new MyModel({
        email : 'mitchell+simoens@gmail.com'
    });
    
    var validate = model.validate();
    
    console.log(validate);
    console.log(validate.getCount()); //should be zero
    Mitchell Simoens @SenchaMitch
    Sencha Inc, Senior Forum Manager
    ________________
    Check out my GitHub, lots of nice things for Ext JS 4 and Sencha Touch 2
    https://github.com/mitchellsimoens

    Think my support is good? Get more personalized support via a support subscription. https://www.sencha.com/store/

    Need more help with your app? Hire Sencha Services services@sencha.com

    Want to learn Sencha Touch 2? Check out Sencha Touch in Action that is in print!

    When posting code, please use BBCode's CODE tags.

  3. #3
    Sencha User
    Join Date
    Mar 2007
    Location
    Haarlem, Netherlands
    Posts
    1,243
    Vote Rating
    10
    TommyMaintz will become famous soon enough TommyMaintz will become famous soon enough

      0  

    Default


    Thanks for the report. This will be fixed for the next release. In the meantime you can use the following override:

    Code:
    Ext.require('Ext.data.Validations', function() {
        Ext.data.Validations.emailRe = /^\s*[\w\-\+_]+(\.[\w\-\+_]+)*\@[\w\-\+_]+\.[\w\-\+_]+(\.[\w\-\+_]+)*\s*$/;
    });

Thread Participants: 2