Correction d'erreurs et code plus propre.

lang/verifier_fr.php

@@ -7,8 +7,8 @@ $GLOBALS[$GLOBALS['idx_lang']] = array(
 
 	// E
 	'erreur_date' => 'Le format de la date n\'est pas accepté.',
-	'erreur_email' => 'L\'adresse de courriel n\'a pas un format valide.',
-	'erreur_email_nondispo' => 'L\'adresse de courriel est déjà enregistré.',
+	'erreur_email' => 'L\'adresse de courriel <em>@email@</em> n\'a pas un format valide.',
+	'erreur_email_nondispo' => 'L\'adresse de courriel <em>@email@</em> est d&eacute;j&agrave; utilis&eacute;e.',
 	'erreur_entier' => 'La valeur doit &ecirc;tre un entier.',
 	'erreur_entier_entre' => 'La valeur doit &ecirc;tre comprise entre @min@ et @max@.',
 	'erreur_entier_max' => 'La valeur doit &ecirc;tre inf&eacute;rieure &agrave; @max@.',

verifier/email.php

@@ -10,11 +10,12 @@ function verifier_email_dist($valeur, $options=array()){
 	include_spip('inc/filtres');
 	
 	// Disponibilite des courriels en base AUTEURS
-	if ($options['disponible']){
-		return verifier_disponibilite_email($valeur);
+	// Si l'adresse n'est pas disponible, on stoppe tout sinon on continue
+	if ($options['disponible'] and $erreur = verifier_disponibilite_email($valeur)){
+		return $erreur;
 	}
 	
-	// Choix du de verification de la syntaxe des courriels
+	// Choix du mode de verification de la syntaxe des courriels
 	if (!$options['mode'] or !in_array($options['mode'], array('strict'))){
 		$mode = 'normal';
 	}
@@ -22,17 +23,15 @@ function verifier_email_dist($valeur, $options=array()){
 		$mode = $options['mode'];
 	}
 		
-	if ($mode == 'normal'){	
-		if (email_valide($valeur))
-			return '';
-		else
-			return _T('verifier:erreur_email');
-	}
-	// Validation Stricte
-	else{
-		return verifier_email_de_maniere_stricte($valeur);
-	}
-	return '';
+	if ($mode == 'normal')
+		$fonction_verif = 'email_valide';
+	else
+		$fonction_verif = 'verifier_email_de_maniere_stricte';
+	
+	if (!$fonction_verif($valeur))
+		return _T('verifier:erreur_email', array('email' => $valeur));
+	else
+		return '';
 }
 
 /**
@@ -43,13 +42,10 @@ function verifier_email_dist($valeur, $options=array()){
  * (même si les autres caractères sont autorisés, tant pis, ils sont trop rares)
  */
 function verifier_email_de_maniere_stricte($valeur){
-	$erreur = _T('verifier:erreur_email');
-	$ok = '';
-
 	// Si c'est un spammeur autant arreter tout de suite
 	if (preg_match(",[\n\r].*(MIME|multipart|Content-),i", $valeur)) {
 		spip_log("Tentative d'injection de mail : $valeur");
-		return $erreur;
+		return false;
 	}
 	foreach (explode(',', $valeur) as $v) {
 		// nettoyer certains formats
@@ -57,9 +53,9 @@ function verifier_email_de_maniere_stricte($valeur){
 		$adresse = trim(preg_replace(",^[^<>\"]*<([^<>\"]+)>$,i", "\\1", $v));
 		// NOUVELLE REGEXP NE RESPECTANT PLUS RFC 822 MAIS MOINS TOLERANTE
 		if (!preg_match('/^([A-Za-z0-9]){1}([A-Za-z0-9]|-|_|\.)*@[A-Za-z0-9]([A-Za-z0-9]|-|\.){1,}\.[A-Za-z]{2,4}$/', $adresse))
-			return $erreur;
+			return false;
 	}
-	return $ok;
+	return true;
 }
 
 /**
@@ -68,11 +64,9 @@ function verifier_email_de_maniere_stricte($valeur){
  */
 function verifier_disponibilite_email($valeur){
 	include_spip('base/abstract_sql');
-	$erreur = _T('verifier:erreur_email_nondispo');
-	$ok = '';
 
-	$emailDejaUtilise = sql_getfetsel("id_auteur", "spip_auteurs", "email='".$valeur."'");
-	if($emailDejaUtilise) return $erreur;
-
-	return $ok;
-}
+	if(sql_getfetsel('id_auteur', 'spip_auteurs', 'email='.sql_quote($valeur)))
+		return _T('verifier:erreur_email_nondispo', array('email' => $valeur));
+	else
+		return '';
+}
\ No newline at end of file