Discussion :

[guyfoot]

Bonjour à toutes et tous,

Débutant en PHP, je suis confronté à un problème : je n'arrive pas à vérifier si un utilisateur a sélectionné quelquechose ou non dans une liste de choix de mon formulaire. J'ai sans doute fais des erreurs dans mon code, mais je piétine depuis 2 jours . Et donc je cherche une bonne âme pour me donner un coup de pouce

Voici le code dans mon formulaire (fichier TPL):

Code :

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

1
2
3
4
5
6
7
8
<input type="text" name="subject"  maxlength="70" tabindex="1" 
class="post" value="{SUBJECT}" />
 
<textarea name="message" rows="15"  wrap="virtual" tabindex="2"
class="post" onselect="storeCaret(this);" onclick="storeCaret(this);" 
onkeyup="storeCaret(this);" id="message">{MESSAGE}</textarea>
 
<td class="row1">{SELECT_CIBLE}</td>

Dans posting.php :

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
20
21
22
 
$selection_cible = '<option selected>Sélectionnez la ou les région(s) cible(s)</option>';
			$select_cible = '<select name="cible[]" multiple="multiple" size="6" tabindex="4">';
 
			$sql = "SELECT * 
					FROM " . CIBLES_TABLE . "
					WHERE cible_id <> 1
					ORDER BY cible_id ASC";
 
			$select_cible .= $selection_cible;
 
			while( $row = $db->sql_fetchrow($result) )
			{
				$select_cible .= '<option value="' . $row['cible_lettre'] . '">' . str_replace("\'", "'", $row['cible_nom']) . '</option>';
			}
 
			$select_cible .= '</select>';	
 
		$template->assign_vars(array(
			'TOPIC_TITLE' => $preview_subject,
			'POST_SUBJECT' => $preview_subject,
			'SELECT_CIBLE' => $select_cible,					);

Et voilà enfin la fonction checkForm (la vérification fonctionne parfaitement pour les champs subject et message, mais pas pour cible) :

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
function checkForm() {
    formErrors = false;    
    if (document.post.subject.value.length < 1) {
        formErrors = "Veuillez saisir un titre";
    } else if (document.post.message.value.length < 10) { 
        formErrors = "Veuillez saisir un message plus long";
    } else if (document.post.cible.value == "") { 
        formErrors = "Veuillez sélectionner une cible géographique";
    }
    if (formErrors) {
        alert(formErrors);
        return false;
    } else {
        return true;
    }
}

Merci de votre aide

[julien.63]

Salut,
j'ai pas testé mais tu peux essayer ceci :

Code :

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

else if (document.post.cible.checked == false)

[guyfoot]

Merci de ton aide Julien, mais ça ne marche pas mieux :

pas d'alerte avec (document.post.cible.checked == false), ni avec == true d'ailleurs

[julien.63]

oups j'avais mal lu, il ne s'agit pas d'une checkbox, donc ça risquait pas de marcher...

Par contre, concernant ton souci, tu déclares cible comme un tableu :

Code :

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

name="cible[]"

donc, il doit falloir faire une boucle pour tester les entrées de ton tableau.

Code :

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

document.post.cible[0].value == ""

[guyfoot]

toujours pas... je dois dois m'etre planté encore ailleurs
Finalement je crois que ça serait plus rapide de mettre un choix par défaut au lieu du libellé "Sélectionner..."
Sais tu comment je dois faire ?

[julien.63]

je crois qu'on s'approche...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<script type="text/javascript">
function checkForm(){
	alert(document.forms.f.cible.options[0].value);
}
</script>
 
<form name="f" action="" method="post">
<select name="cible" multiple="multiple" size="6" tabindex="4">
<option value="1">AAAA</option>
<option value="2">BBBB</option>
<option value="3">CCCC</option>
</select>
<input type="button" onclick="checkForm();" />
</form>

ceci fonctionne

[guyfoot]

Merci beaucoup ! Grâce à ton aide j'avance.

Maintenant ma fonction marche avec les 2 bouts de code suivants :

Code :

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

1
2
3
4
5
...
else if (document.forms.post.cible.value == "") { 
 
        formErrors = "Veuillez sélectionner une cible géographique";
...

Code :

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

1
2
3
4
5
6
 
<select name="cible" multiple="multiple" size="6" tabindex="4">
<option value="1">AAAA</option>
<option value="2">BBBB</option>
<option value="3">CCCC</option>
</select>

Mais pas avec ma variable :

Code :

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

<span class="gen"><b>{CIBLE}</b></span><br />

avec dans posting.php :

Code :

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

1
2
$selection_cible = '<option selected value="0">Sélectionnez la ou les région(s) cible(s)</option>';
...

[julien.63]

As-tu un message d'erreur?

Est-ce que ça marche en utilisant le tableaux options de cible?
un truc du genre

Code :

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

1
2
3
4
5
6
7
8
9
 
var i;
var hasOption = false;
for (i=0;i<document.forms.f.cible.options.length;i++){
       if (document.forms.f.cible.options[i].value != "")
               hasOption = true;
}
if (!hasOption)
      formErrors = "Veuillez sélectionner une cible géographique";

Fait voir le code html de ton select qui est généré par ta page php.

[guyfoot]

Voilà le code généré par posting.php

Code :

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

1
2
3
4
5
6
7
8
9
 
<select name="cible[]" multiple="multiple" size="6" tabindex="4">
<option selected value="0">Sélectionnez la ou les région(s) cible(s)</option>
<option value="a">France entière</option>
<option value="b">Ile-de-France</option>
<option value="c">Nord-Pas-de-Calais / Picardie</option>
<option value="d">Alsace / Lorraine / Champagne-Ardenne</option>
<option value="e">Bourgogne / Franche-Comté</option>
</select>

[julien.63]

bon en fait, c'est plus simple que ça.

ton test javascript :

Code :

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

1
2
if (document.forms.f.cible.value == "0")
     formErrors = "Veuillez sélectionner une cible géographique";

enlève les [] derrière cible dans ton html

Code :

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

<select name="cible" multiple="multiple" size="6" tabindex="4">

qu'est ce que ça donne?

[guyfoot]

Effectivement ça marche maintenant
Je te remercie vivement pour tes conseils !

Voici le code final de la fonction :

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
function checkForm() {
    formErrors = false;    
	if (document.post.subject.value.length < 1) {
        formErrors = "Veuillez saisir un titre";
    } else if (document.post.message.value.length < 10) { 
        formErrors = "Veuillez saisir un message plus long";
    } else if (document.post.cible.value == "" ) { 
        formErrors = "Veuillez sélectionner une cible géographique";
    }
    if (formErrors) {
        alert(formErrors);
        return false;
    } else {
        return true;
    }
}

et dans le TPL

Code :

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

1
2
3
4
5
6
7
8
9
 
<select name="cible" size="6" tabindex="4">
<option value="a">France entière</option>
<option value="b" selected="selected">Ile-de-France</option>
<option value="c">Nord-Pas-de-Calais / Picardie</option>
<option value="d">Alsace / Lorraine / Champagne-Ardenne</option>
<option value="e">Bourgogne / Franche-Comté</option>
<option value="f">Rhône-Alpes / Auvergne</option>
<option value="g">Provence-Alpes-Côte d'Azur / Languedoc-Roussillon</option></select>

J'ai finalement supprimé l'option selected, qui posait problème lors des retours en arrière dans le navigateur.

Encore merci!