Discussion :

[popy67]

Bonjour,

Je rencontre un petit problème avec mon script JS / AJAX.
Les membres peuvent s'abonner ou se désabonner en cochant une checkbox et immédiatement via un script JS / AJAX le paramètre est modifié en base de données.

Script permettant de vérifier si l'internaute coche ou décoche la case :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
if(param == "lettre_informations")
		{
		if(document.getElementsByName("lettre_informations").checked == false)
      		{
       		val = 0;
        	}
		else
			{
			val = 1;	
			}	
		}

Checkbox :

Code :

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

1
2
3
4
5
 <table>
      <tr>
      <td><input type="checkbox" name="lettre_informations" <?php if($lettre_informations == "oui") print("checked=\"checked\"");?> onclick="modifParametres('lettre_informations','1');"/></td>
      <td>Reçevoir des lettres d'informations ainsi que des alertes relatives à mon compte par e-mail.</td>
      </tr>

Seulement voila lorsque la cas est cochée et l'internaute la décoche la valeur renvoyée est 1 alors qu'elle devrait etre de 0.
Lorsque l'internaute coche la case décochée la valeur renvoyée est bien 1 (exact).

Comment faire pour que la valeur retournée soit 0 quand l'internaute décoche la case ?

Merci d'avance pour vos réponses.

[RomainVALERI]

L'extrait JS que tu nous montres est trop court pour que l'on puisse comprendre le problème correctement. (au moins l'appel ajax dans son intégralité parce que là... on est où ? avant l'appel ? après l'appel ? dans un gestionnaire d'événement de la checkbox ? dans la callback de l'appel ajax ? mystère...)

Cependant, il y a déjà un problème apparent. Comme son nom l'indique, la méthode getElementsByName ne renvoie pas "un" élément mais une collection d'éléments. La collection renvoyée ne possédant pas de propriété checked, la condition ne peut pas être vérifiée...
Cela dit, accéder aux éléments par le name est presque toujours plus contraignant que par l'id... enfin ^^ si tu veux utiliser le name, et si ce name est unique, ajoute [0] après ton appel de méthode pour pointer sur l'unique élément présent dans la collection, ça corrigera au moins cette partie.

[popy67]

Salut,

Merci pour ta réponse.
Dans mon script nous sommes avant l'appel ajax, il est appelé par le gestionnaire d'évènements onClick.

Par contre je viens d'ajouter [0] comme tu me l'as indiqué dans ton message et le script fonctionne parfaitement bien et renvoie bien les bonnes valeurs

Merci ^^