Ticket #297: filter-tests-patch.diff

File filter-tests-patch.diff, 2.6 kB (added by arnaudj, 1 year ago)

regex correcte + tests unitaires

  • testapp/modules/jelix_tests/tests/utils.jfilter.html_cli.php

    old new  
    116116                                  "isUrl('')");*/ 
    117117    } 
    118118 
     119    public function testIsEmail() { 
     120        $this->assertTrue(jFilter::isEmail('name@provider.com'), 
     121                                  "isEmail('name@provider.com')"); 
     122        $this->assertTrue(jFilter::isEmail('firstname-initial.lastname@etudiant.univ-lille1.fr'), 
     123                                  "isEmail('firstname-initial.lastname@etudiant.univ-lille1.fr'"); 
     124        $this->assertFalse(jFilter::isEmail('name@etudiant.univ-lille1'), 
     125                                   "isEmail('name@etudiant.univ-lille1')"); 
     126        $this->assertFalse(jFilter::isEmail('name@provider.x_x'), 
     127                                   "isEmail('name@provider.x_x')"); 
     128    } 
     129 
    119130    public function testIsIpv4(){ 
    120131        $this->assertTrue(jFilter::isIPv4('0.0.0.0'), "isIPv4('0.0.0.0')"); 
    121132        $this->assertTrue(jFilter::isIPv4('127.0.0.1'), "isIPv4('127.0.0.1')"); 
  • lib/jelix/utils/jFilter.class.php

    old new  
    174174#if ENABLE_PHP_FILTER 
    175175        return filter_var($val, FILTER_VALIDATE_EMAIL); 
    176176#else 
    177         return preg_match('/^[A-Z0-9][A-Z0-9_\-]*(\.[A-Z0-9][A-Z0-9_\-]*)*@[A-Z0-9][A-Z0-9_\-]*(\.[A-Z0-9][A-Z0-9_\-]+)+$/i',$val); 
     177        return preg_match('/^[A-Z0-9][A-Z0-9_\-]*(\.[A-Z0-9][A-Z0-9_\-]*)*@[A-Z0-9][A-Z0-9_\-]*(\.[A-Z0-9][A-Z0-9_\-]*)*(\.[A-Z][A-Z]+)$/i',$val); 
    178178#endif 
    179179    } 
    180180 
  • lib/jelix-www/js/jforms.js

    old new  
    184184                        ok = (val.search(/^$/) != -1) 
    185185                        break;*/ 
    186186                    case 'email' : 
    187                         ok = (val.search(/^[A-Z0-9][A-Z0-9_\-]*(\.[A-Z0-9][A-Z0-9_\-]*)*@[A-Z0-9][A-Z0-9_\-]*(\.[A-Z0-9][A-Z0-9_\-]+)+$/i) != -1) 
     187                        ok = (val.search(/^[A-Z0-9][A-Z0-9_\-]*(\.[A-Z0-9][A-Z0-9_\-]*)*@[A-Z0-9][A-Z0-9_\-]*(\.[A-Z0-9][A-Z0-9_\-]*)*(\.[A-Z][A-Z]+)$/i) != -1) 
    188188                        break; 
    189189                    case 'ipv4' : 
    190190                         var t = val.match(/^(\d{1,3})\.(\d{1,3})\.(\d{1,3})\.(\d{1,3})$/); 
Download in other formats: Original Format