Forum d'entraide PHPFrance

Bonjour,

J' ai créé une page de création de nouveau mot passe en cas d'oubli en fonction de l'email.

Mais elle me met un message d'erreur car ne reconnait pas l'email (Fonction valide_form)

Pouvez vous me dire d'où cela vient.

Merci d'avance

Ma page :

<?

include("../connection.php");

include("../configuration.inc.php");

/* Le formulaire a été soumis, vérif si l'identification est ok */
if (sizeof($HTTP_POST_VARS) > 0) 
	{
	$frm = $HTTP_POST_VARS;
	$message_erreur = valide_form($frm, $erreurs);

	if (empty($message_erreur))
		{
		$nom_utilisateur = recupere_nom_utilisateur($HTTP_POST_VARS["email"]);
		
		initialise_mot_passe($nom_utilisateur);

		$DOC_TITLE = "[CQFD Production]";
		include("$repertoire_modele/haut.php");
		include("modeles/oubli_mot_passe_succes.php");
		include("$repertoire_modele/bas.php");
		die;
		}
	}

$DOC_TITLE = "[CQFD Production]";
include("$repertoire_modele/haut.php");
include("modeles/oubli_mot_passe_form.php");
include("$repertoire_modele/bas.php");

/******************************************************************************
 * FONCTIONS
 *****************************************************************************/

function valide_form(&$frm, &$erreurs)
{
/* Valide le formulaire de mot de passe, et renvoie un message d'erreur
 * Si le message est vide, alors il n'y a pas d'erreur */
	$test=$frm["email"];
	$erreurs = array();
	$msg = "";

	if (empty($frm["email"])) {
		$erreurs["email"] = true;
		$msg .= "Vous n'avez pas saisi d'adresse mail";
	}
	elseif  ((mysql_num_rows(mysql_query("SELECT * FROM tblutilisateur WHERE utilisateurEmail ='$test'")or exit ('Erreur SQL !'.$qid.'<br>'.mysql_error())) == 0)) {
		$erreurs["email"] = true;
		
		$msg .= " Cette adresse électronique n'existe pas ";
		
	}

	return $msg;
}

function recupere_nom_utilisateur($email)
{
/* Obtient le nom d'utilisateur basé sur une adresse e-mail */

	$qid = mysql_query("SELECT utilisateurEmail FROM tblutilisateur WHERE utilisateurEmail = '$email'")or exit ('Erreur SQL !'.$qid.'<br>'.mysql_error());
	if ($qid)
		{
		if (mysql_num_rows($qid) > 0)
			{
			$utilisateur = mysql_fetch_array($qid);
			}
		}

	return $utilisateur["email"];
}

?>

Mon Formulaire : oubli_mot_passe_form

<table cellpadding=0 cellspacing=0 border=0 >
			<tr><td class=tetiere align=center >Récupérer un mot de passe</td></tr>
		</table><p></p>
<table cellpadding=20>
<tr valign=top>
<td width=300 class=normal>
	<p>Entrez votre adresse e-mail pour récupérer votre mot de passe. En validant ce formulaire, 
	votre mot de passe sera réinitialisé et le nouveau mot de passe vous sera envoyé automatiquement par e-mail.
</td>

<td bgcolor="#ffffcc">
<?php
if (! empty($message_erreur))
	{
?>
	<div class=warning align=center><?=$message_erreur ?></div>
<?php
	 }
 ?>

	<form name="entryform" method="post" action="<?=$_SERVER['PHP_SELF']?>">
	<table>
		<tr>
			<td class="label">Adresse email:</td>
			<td><input type="text" name="email" size=25 value="<?=$frm['email'] ?>"></td>
		</tr>
		<tr>
			<td colspan=2 align=center><input type="submit" value="Envoyer" class="bouton"><input type="button" value="Annuler" class="bouton" onClick="javascript: history.go(-1)">
			<p class=normal>
			  <a class="normal" href="../membre.php">S'identifier</a>
			| <a class="normal" href="../index.php">Page d'accueil</a>
			</td>
	</table>
	</form>
</td>
</tr>
</table>

Salut,

Tout d'abord, je te conseille d'utiliser la méthode

$_POST[]

A la place de

$HTTP_POST_VARS

Qui est obsolète.

Ensuite, peux-tu nous en dire un peu plus sur l'erreur que tu reçois?
Quel genre, et à quelle ligne?

Merci.

Y a pas un petit soucis de logique là :

elseif  ((mysql_num_rows(mysql_query("SELECT * FROM tblutilisateur WHERE utilisateurEmail ='$test'")or exit ('Erreur SQL !'.$qid.'<br>'.mysql_error())) == 0)) { 
        $erreurs["email"] = true; 
         
        $msg .= " Cette adresse électronique n'existe pas "; 
         
    } 

S'il y a des lignes dans ton SELECT tu informes que l'adresse n'existe pas

Décompose d'avantage ton code, ca sera plus lisible et plus facile à maintenir ou a debugger puisque tu pourras tester les différentes étapes :

else {
  $sql = "SELECT * FROM tblutilisateur WHERE utilisateurEmail ='$test'";
  $rs = mysql_query($sql) or die ('Erreur SQL !'.$qid.'<br>'.mysql_error());
  if(mysql_num_rows($rs)) { // tu devrais avoir un succes ici et non une erreur :)
    $erreurs["email"] = true; 
    $msg .= " Cette adresse électronique n'existe pas "; 
  }
}

De plus, des lignes comme celle là

elseif  ((mysql_num_rows(mysql_query("SELECT * FROM tblutilisateur WHERE utilisateurEmail ='$test'")or exit ('Erreur SQL !'.$qid.'<br>'.mysql_error())) == 0)) {

C'est un nid à bug.

Je suis pour la lisibilité du code et je ne pense pas, avec les machines actuelles, que d'écrire 5 lignes de plus soient vraiment pénalisant et je trouve que c'est plus lisible comme ça ;

else
{
    $str_requete = "SELECT * FROM tblutilisateur WHERE utilisateurEmail ='$test'";
    $o_result = mysql_query($stre_requete) or exit ('Erreur SQL !'.$qid.'<br>'.mysql_error());

    $nb_result = mysql_num_rows($o_result);
    if  ($nb_resut == 0)) {
        ...
    }
}

Merci j'ai remis le code plus propre ce qui m'a permis de résoudre le pb