developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#76 closed bug (fixed)

jFilter::isEmail n'est pas correct

Reported by: bballizlife Owned by: bballizlife
Priority: high Milestone: Jelix 1.0beta1
Component: jelix:utils Version: trunk
Severity: critical Keywords: jFilter
Cc: Blocked By:
Blocking: Documentation needed:
Hosting Provider: Php version: 5.1.6

Description

var_dump(jFilter::isEmail('loic@…'));

affiche :

int(0)

L'email est donc considéré comme invalide alors qu'il ne l'est pas.

Change History (7)

comment:1 Changed 13 years ago by laurentj

  • Owner set to laurentj
  • Priority changed from normal to high
  • Status changed from new to assigned

comment:2 Changed 13 years ago by bballizlife

Erreur trouvée dans l'expression régulière :

  • le + final devait se trouver à l'intérieur de la parenthèse car sinon la regex n'acceptait que les tld de 2 caractères uniquement
  • modification supplémentaire (mais n'intervenant pas dans le bug) : échappement des tirets dans les classes lorsqu'ils doivent être utilisés en tant que tels pour être certain qu'ils le soient

Diff

Index: lib/jelix/utils/jFilter.class.php =================================================================== --- lib/jelix/utils/jFilter.class.php (révision 343) +++ lib/jelix/utils/jFilter.class.php (copie de travail) @@ -172,11 +172,11 @@

#ifdef ENABLE_PHP_FILTER

return filter_var($val, FILTER_VALIDATE_EMAIL);

#else

  • 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);

+ 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);

#endif

}

}

comment:3 Changed 13 years ago by laurentj

ok je te laisse commiter

comment:4 Changed 13 years ago by bballizlife

  • Owner changed from laurentj to bballizlife
  • Status changed from assigned to new

comment:5 Changed 13 years ago by bballizlife

  • Status changed from new to assigned

comment:6 Changed 13 years ago by bballizlife

  • Resolution set to fixed
  • Status changed from assigned to closed

comment:7 in reply to: ↑ description Changed 13 years ago by anonymous

Replying to bballizlife:

var_dump(jFilter::isEmail('loic@…'));

affiche :

int(0)

L'email est donc considéré comme invalide alors qu'il ne l'est pas.

Note: See TracTickets for help on using tickets.