Discussion :

[Flushovsky]

Bonjour,

Voici mon problème.

Sur une page, j'affiche un certain nombre d'informations issues d'une base de donnée.

Certaines des données sont modifiables grâce à un formulaire dont vous trouverez l'extrait qui me pose un problème ci-dessous.

On dois d'abord cocher la checkbox correspondant à la ligne ou aux lignes que l'on souhaite modifier puis faire son choix dans la liste <select>.

Sachant que l'on a la possibilité de sélectionner une ou plusieures lignes à modifier, je stock les valeurs de mes checkbox et des <select> dans des tableaux (check[] et alert[]).

Ce qui se passe c'est qu'à l'envoi des données, le tableau alert[] enregistre non seulement les données des lignes cochées mais aussi celles des lignes non cochées.

Ce qui me donne ca :

Array check[] : Array ( [0] => 26 )
Array Alert[] : Array ( [0] => vert [1] => orange [2] => vert )
Alert : orange
Requete : UPDATE wst_install SET site_alert = 'vert' WHERE id = '26'

Alors que je devrais avoir ca :

Array check[] : Array ( [0] => 26 )
Array Alert[] : Array ( [0] => orange)
Alert : orange
Requete : UPDATE wst_install SET site_alert = 'orange' WHERE id = '26'

Auriez vous une idée de la raison pour laquelle le tableau alert[] enregistre toutes les valeurs de chaque lignes au lieu de m'enregistrer uniquement les valeurs de mes lignes sélectionnées ?

J'espère avoir été clair...

Ci-dessous mon formulaire et son traitement.

Merci

Formulaire :

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
 
<form method="POST" action="'.$_SERVER['PHP_SELF'].'">';
 
while ($voir = mysql_fetch_array($result)) {
echo '
	<tr bgcolor="#FFFFFF" class="tab_donnees" height="20px">
 
		<td><input type="checkbox" name="check[]" value="'.$voir['id'].'"></td>
		<td>	
			<select class="select" name="alert[]" size="1">
				<option value="vert">Vert
				<option value="orange">Orange
				<option value="rouge">Rouge
			</select>
		</td>	
	</tr>';
}
echo '
</table>
<br>
<div class="div_rec"><input type="submit" class="rec_mods" value="Enregistrer les changements"></form>

Traitement du formulaire :

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
 
if ((isset($HTTP_POST_VARS['check']))&&(isset($HTTP_POST_VARS['alert']))) {
 
		$array_count = count($check);
		/*echo '<br>Nbre :';
		echo $array_count;
		echo '<br>Nbre 2:';
		echo $array_count2;
		echo '<br>';*/
 
    	for($y=0; $y<$array_count; $y++) {
		//echo 'OK2';
	 	$db_conn = db_connect();
        $requete = "UPDATE wst_install
					SET site_alert = '$alert[$y]'
					WHERE id = '$check[$y]'";
 
		$result = mysql_query($requete);
    }
	if ($result == true) {
		echo '<div class="div-alert-b"><br>Changements effectués avec succès !<br><a href="javascript:history.go(-1)">Continuer</a><br><br>';
		echo 'Tab Check : ';
		print_r($check);
		echo '<br>Tab Alert : ';
		print_r($alert);
		echo '<br>Alert : ';
		echo $alert[1];
		echo '<br>Requete : ';
		echo $requete;
		echo '
		<br><br></div>';
	}
	if ($result == false) {
		echo '<div class="div-alert-b"><br>Changements impossible !<br><a href="javascript:history.go(-1)">Continuer</a><br><br></div>';
	}
}

[macbook]

Renommes ton <select> en "alert" plutôt que "alert[]" pour voir ?

[Flushovsky]

J'ai déjà essayé ca et malheureusement ca ne fonctionne pas mieux.

J'ai également essayé de vider le tableau $alert à la fin du traitement des données mais ca ne fonctionne pas non plus.

[zulot]

j'ai une question a quoi correspond cette ligne :

Code :

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

$array_count = count($check);

et bien sur les valeur surtout $check

et pourquoi au lieu de ta boucle que je comprend pas trop ou tu veux en venir tu fais pas un truc du style

Code :

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

UPDATE wst_install SET site_alert = '$_POST['allerte']' WHERE id = '$_POST['check']'

Et si tu as plusieur ligne possible a cocher ou checker tu leur balance un input hidden pour cacher le numero au quel tu est et tu remplace le nom de tes input par check.$i et alert.$i ou i est la ligne a UPDATE ou tu en est.

[Flushovsky]

Code :

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

$array_count = count($check);

Cette ligne compte et renvoie le nombre de checkbox cochée.

Ce qui me saoule c'est que $check renvoie bien le bon nombre de cases cochée mais que $alert lui, ne renvoi pas le bon nombre.

En ce qui concerne ta solution , zulot, tu récupère comment les valeurs de $alert pour chaques lignes si tu ne les stocke pas dans un tableau ?

[Yogui]

Salut

Fais attention à ne pas utiliser register_globals à "On", c'est plus sûr.
Du coup, fais attention à :

Code :

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

$array_count = count($check);

De manière générale, essaie d'être homogène dans l'accès à tes variables.

[FCYPBA]

Salut

C'est normal que check[] et alert[] n'est pas le meme nombre d'élément.

check[] n'est renseigné qu'avec les valeurs cochées alors que alert[] est renseigné avec les valeurs de tous les select puisqu'ils ne sont pas filtrés.

Il faudrait faire un traitement en js pour ne transmettre que celles dont la checkbox est cochée.

Pierre

[Flushovsky]

Salut,

C'est bien ce qui me semblait mais j'étais pas sur.

Par contre un traitement en js, là je suis impuissant...

A moins que je remplace mon select par des checkbox sachant que je n'ai que trois choix possible.

[Yogui]

L'autre solution, à mon avis plus efficace, est de faire :

Code :

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

1
2
3
4
5
6
7
8
$checks = array();
if(!empty($_POST['check']){
   foreach($_POST['check'] as $check){
      if(!empty($check)){
         $checks[] = $check;
      }
   }
}

À la fin, le tableau $checks ne contiendra que les cases cochées.

[FCYPBA]

Ce que tu peux faire, j'ai pas testé mais je pense que cela fonctionne:

Tu mets tous tes selects 'disabled' par défaut. Donc ils ne sont pas transmis.

Quand une case est coché, il faut rendre le select correspondant actif.

Pour identifier un select, il faudrait lui attribuer un 'id' ayant pour valeur la value de la checkbox correspondante.

Voilà

Pierre

[Flushovsky]

J'ai testé en mettant des chexbox à la place du select. Et ca fonctionne. Mais c'est moins beau.

Je vais essayer vous suggetions.

Merci

[Flushovsky]

Mon problème ne viens pas du tableau $check qui renvoie lui le nombre correct de cases cochée mais vient de mon champ <select>

[Flushovsky]

J'ai réussi à trouver un srcipt me permettant de désactiver le champ select. Par contre il ne m'applique la désactivation que sur la première ligne de mon tableau.

Et au chargement de la page, les champs ne sont pas désactivé, il faut que je slique deux fois sur la checkbox pour désactiver le champ.

Voici le code :

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 language="Javascript">
// ==================
//	Activations - Désactivations
// ==================
function GereControle(Controleur, Controle, Masquer) {
var objControleur = document.getElementById(Controleur);
var objControle = document.getElementById(Controle);
	if (Masquer=='1')
		objControle.style.visibility=(objControleur.checked==true)?'visible':'hidden';
	else
		objControle.disabled=(objControleur.checked==true)?false:true;
	return true;
}
</script>

Code :

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

1
2
3
4
5
6
7
<input type="checkbox" name="check[]" id="chkb_1" onClick="GereControle(\'chkb_1\', \'texte_1\', \'0\');" value="'.$voir['id'].'">
 
<select class="select" name="alert[]" id="texte_1" size="1">
				<option value="vert">Vert
				<option value="orange">Orange
				<option value="rouge">Rouge
			</select>

[Flushovsky]

J'ai réussi à trouver un srcipt me permettant de désactiver le champ select. Par contre il ne m'applique la désactivation que sur la première ligne de mon tableau.

Et au chargement de la page, les champs ne sont pas désactivé, il faut que je slique deux fois sur la checkbox pour désactiver le champ.

Voici le code :

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 language="Javascript">
// ==================
//	Activations - Désactivations
// ==================
function GereControle(Controleur, Controle, Masquer) {
var objControleur = document.getElementById(Controleur);
var objControle = document.getElementById(Controle);
	if (Masquer=='1')
		objControle.style.visibility=(objControleur.checked==true)?'visible':'hidden';
	else
		objControle.disabled=(objControleur.checked==true)?false:true;
	return true;
}
</script>

Code :

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

1
2
3
4
5
6
7
<input type="checkbox" name="check[]" id="chkb_1" onClick="GereControle(\'chkb_1\', \'texte_1\', \'0\');" value="'.$voir['id'].'">
 
<select class="select" name="alert[]" id="texte_1" size="1">
				<option value="vert">Vert
				<option value="orange">Orange
				<option value="rouge">Rouge
			</select>

[FCYPBA]

pour désactiver les champs au chargement de la page, il faut penser à mettre l'attribut disabled comme ceci :

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
 
while ($voir = mysql_fetch_array($result)) {
echo '
   <tr bgcolor="#FFFFFF" class="tab_donnees" height="20px">
 
      <td><input type="checkbox" name="check[]" value="'.$voir['id'].'"></td>
      <td>   
         <select class="select" name="alert[]" id="texte_1" size="1" disabled>
            <option value="vert">Vert
            <option value="orange">Orange
            <option value="rouge">Rouge
         </select>
      </td>   
   </tr>';
}

[Flushovsky]

Merci de votre aide, je viens de résoudre mon problème ;-)