Discussion :

[_tony]

Bonjour,
Petit souci que je ne comprend pas avec IE (toutes versions):
J'utilise javascript pour vérifier les champs d'un formulaire de contact.
J'ai donc placé onsubmit="return verif()" dans la balise <form>.
Allez savoir pourquoi, ça ne marche pas avec IE: la page php traitant le formulaire est appelée avant même la vérification, et le message est envoyé même vide (je n'ai pas encore préparé la vérification côté serveur ^^)!

Je n'ai aucune piste...
Indications supplémentaires:
- la page en question contient du php donc porte l'extension .php
- le doctype est xhtml 1.0 transitional

[SpaceFrog]

sans code ...

[_tony]

Pardon, je pensais que ça suffirait comme explication...

la fonction javascript (extrait sur le champ pseudo, le reste est du même acabit):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
function verif() {
form=document.contact;
if (form.pseudo.value == "") {
alert ('erreur');
return false;
}
else {
return true;
}
}

et le formulaire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
<form method="post" action="../mail.php" name="contact" id="contact" onsubmit="return verif();">
<p><br />
       <label for="pseudo">Votre nom/pseudo :</label>
       <input type="text" name="pseudo" id="pseudo" size="20" maxlength="10"/><br /><br />
       <label for="email">Votre e-mail :</label>
       <input type="text" name="email" id="email" size="40" /><br /><br />
       <label for="message">Votre message :</label><br />
       <textarea name="message" id="message" rows="6" cols="50"></textarea><br /><br />
       <input type="submit" value="Envoyer"/>
       <input type="hidden" name="langue" value="fr/credits.php"/>
</p>
</form>

Testé et approuvé sous FF, Opera, Safari et Chrome. Zéro pointé sur IE 6, 7 et 8.

[Bovino]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

form=document.contact;

Répété un nombre incalculable de fois : ne jamais donner à des variables (surtout non déclarées avec var) des noms de propriétés existantes en JavaScript !

[SpaceFrog]

et une synatxe plus rigoureuse...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
function verif() {
myform=document.forms['contact'];
if (myform.pseudo.value == "") {

etc ...

[Orni-Dev]

Je pense que cet exemple issue de la documentation microsoft pourrait t'aiguiller http://msdn.microsoft.com/en-us/libr...8VS.85%29.aspx

[SpaceFrog]

en général c'est plutot la msdn microsoft qui renvoie chez nous ... ^^

[_tony]

Merci pour votre rapidité! Je savais bien qu'il s'agissait d'un problème de rigueur, mais je ne savais pas où ^^
Alors cette fois la fonction vérif() est bien lancée, et retourne false, mais il reste un problème: les autres actions (ici le message d'erreur) ne fonctionnent pas avec IE 7 et 8. Résultat: l'utilisateur clique sur "envoyer" et... il ne se passe plus rien du tout!

[SpaceFrog]

autres actions ?
y'a d'autre code ?

[_tony]

Oui il y avait d'autre code, mais maintenant tout roule! La rigueur, y'a que ça de vrai! Merci pour votre rapidité en tout cas!

[Sergejack]

On peut voir le nouveau code ?

C'est pour le voler et le revendre (ou t'aider, c'est comme tu préfères).

[_tony]

Je sais pas combien vous en tirerez...
Un autre truc tient: je répète plusieurs fois:
erreur.style.display="block";
return false;
Je l'avais d'abord placé dans une fonction pour l'appeler quand nécessaire, mais rebelote: plus de return false avec IE... Du coup comme ça faisait pas 40 lignes je l'ai laissé comme ça.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
function verif() {
var myform=document.forms['contact'];
var erreur=document.getElementById('erreur');
var regex = /^[a-zA-Z0-9._-]+@[a-z0-9._-]{2,}\.[a-z]{2,4}$/;
if (myform.pseudo.value == "") {
erreur.style.display="block";
return false;
} else if (myform.message.value == "") {
erreur.style.display="block";
return false;
} else if (myform.email.value == "" || !regex.test(myform.email.value)) {
erreur.style.display="block";
return false;
}
else {
erreur.style.display="none";
return true;
}
}

[nod__]

Bah a ce moment là tu met toutes les conditions dans le même if. ça reviens strictement au même ici.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
if ((condition1) || (condition2) || (condition3)) {
   return false;
}
else {
   return true;
}

[_tony]

C'est pas faux...