[PHP+javascript] Envoi de form dans un mail :-)

aKC1a · 12 Mai 2005

Voila j'essaie d'envoyer un form de manière structurée dans un mail.

Jusque la aucun soucis.
Mais j'ai rajouter une function javascript pour qu'il controle certains de mes champs de form avant envoi. Emial valide etc ...

et la qd je fait volontairement une faute dans le form il me met bien une alerte comme demandée mais il envoie le form a mon mail quand meme ...

Est ce que kkun sait d'ou ca vient ?

code javascript :

Code:

&lt;script type="text/javascript">
function verif_champs()
{
	if(document.form1.name.value == "")
	{
		alert("Enter a name please");
		document.forml.name.focus();
		return false;
	}
	if(document.forml.firstname.value == "")
	{
		alert("Enter a firstname please");
		document.forml.firstname.focus();
		return false;
	}
	if(document.forml.society.value == "")
	{
		alert("Enter a Society name please");
		document.forml.society.focus();
		return false;
	}
	if(document.forml.func.value == "")
	{
		alert("Enter your function please");
		document.forml.func.focus();
		return false;
	}
	if(document.forml.mail.value == "")
	{
		alert("Enter an e-mail adress please");
		document.forml.mail.focus();
		return false;
	}
	if(document.forml.adresse.value.indexOf('@') == -1)
	{
		alert("Your e-mail adress is invalid!");
		document.forml.adresse.focus();
		return false;
	}
	if(document.forml.adresse.value.length &lt;7)
	{
		alert("Your e-mail adress is invalid");
		document.forml.adresse.focus();
		return false;
	}
	if(document.forml.phone.value == "")
	{
		alert("Enter a phonenumber please");
		document.forml.phone.focus();
		return false;
	}
}
&lt;/script>

code php (placé tout au dessus d ela page, avant la DTD):

Code:

&lt;?php
if($_POST['action'] == 'submitted')
{
	$adress = "blablabla@gmail.com";
	$sujet = "Contact via votre site web";
	$message = "Name: {$_POST['name']}\nFirstname: {$_POST['firstname']}\n";
	$message .= "Society: {$_POST['society']}\nFunc: {$_POST['func']}\n";
	$message .= "Email: {$_POST['mail']}\n";
	$message .= "Phone: {$_POST['phone']}\nMessage: {$_POST['msg']}\n";
	$entete = "From: Isobar_Belgium\n";
	mail($adress, $sujet, $message, $entete);
	//page de sortie
	$insertGoTo = "confirm.php";
	if (isset($HTTP_SERVER_VARS['QUERY_STRING'])) {
    	$insertGoTo .= (strpos($insertGoTo, '?')) ? "&amp;" : "?";
		$insertGoTo .= $HTTP_SERVER_VARS['QUERY_STRING'];
		}
	header(sprintf("Location: %s", $insertGoTo));
}
?>

et temps que j'y suis, le code de mon form

Code:

&lt;form name="form1" method="post" action="&lt;?php echo $_SERVER['PHP_SELF']; ?>" onSubmit="return verif_champs()">
    			      Name : &lt;input type="text" name="name">&lt;br>&lt;br>
					  Firstname : &lt;input type="text" name="firstname">&lt;br>&lt;br>
					  Society : &lt;input type="text" name="society">&lt;br>&lt;br>
					  Function : &lt;input type="text" name="func">&lt;br>&lt;br>
					  E-mail : &lt;input type="text" name="adresse">&lt;br>&lt;br>
					  Phone : &lt;input type="text" name="phone">&lt;br>&lt;br>
					  Message : &lt;br>&lt;textarea name="msg" cols="60" rows="5">&lt;/textarea>&lt;br>&lt;br>
					  &lt;input type="hidden" name="action" value="submitted">
						&lt;input type="submit" name="Submit" value="Envoyer">
  			          &lt;/form>

C'est surement une connerie mais je vois pas laquelle

Merci d'avance

dania の悪夢 · 12 Mai 2005

verif_champ() est une fonction (car il y a des return) que tu utilises comme une procedure

;
tu ne "return" jamais true;
tu renvoie un false dans les patates (pas de variable pour l'y mettre!);

mon idée : mettre un simple button qui verifie si les données sont bonnes et si elles sont bonnes, alors tu envoies

pour verif_champ() utilises plutot une structure

Code:

if (première champ incorrect)
elseif (champ suivant incorrect)
elseif (erreur suivante)
else (il n'y a pas d'erreur alors envoyer)

tu auras plus simple pour envoyer au bon moment et tu n'auras plus besoin de return car tu auras une procedure

null · 12 Mai 2005

Plus simple, tu mets ta fonction sur "onClick" de "submit" et si tout test "OK", tu fais "this.form.submit();"

aKC1a · 12 Mai 2005

c'est bon j'ai trouvé

Merci a tous.

Bizoo

Ezekiel ! · 12 Mai 2005

juste pour info on peut désactivé le java script... Le mieux serai de coupler ton script js avec une verif en php...

sebge2 · 30 Mai 2005

zywakx a dit:
verif_champ() est une fonction (car il y a des return) que tu utilises comme une procedure ;
tu ne "return" jamais true;
tu renvoie un false dans les patates (pas de variable pour l'y mettre!);

mon idée : mettre un simple button qui verifie si les données sont bonnes et si elles sont bonnes, alors tu envoies

pour verif_champ() utilises plutot une structure

Code:

if (première champ incorrect) elseif (champ suivant incorrect) elseif (erreur suivante) else (il n'y a pas d'erreur alors envoyer)

tu auras plus simple pour envoyer au bon moment et tu n'auras plus besoin de return car tu auras une procedure

oui je pensais à la meme chose

function ()
variable=true;
if(champ faux) variable=false;
....
return variable;

au fait c'était quoi la solution à ton problème ?

aKC1a · 1 Juin 2005

le truc c pas de faire le plus possible du coté utilisateur plutot ?

Xou · 1 Juin 2005

Tout dépend de ton script...

Si celui ci est lourd, autant l'allégé, sinon, autant vérifier.

Et pis, il vaut mieux être sûr que ce sera bien vérifié, le PHP est plus sûr je pense.

[PHP+javascript] Envoi de form dans un mail :-)

aKC1a

OldSchool CS Star

dania の悪夢

AFK

null

ose();

aKC1a

OldSchool CS Star

Ezekiel !

Elite

sebge2

Elite

aKC1a

OldSchool CS Star

Xou

I ♥ rien

Autres sujets récents