Discussion :

[Invité]

Bonjour, je suis bloqué depuis de longues heures sur le même problème qui pourtant devrait pas être si compliqué...
L'utilisateur peut cocher plusieurs cases de type checkbox, mais quand je regarde dans ma bdd je constate qu'à chaque fois une seule des cases a été enregistré, merci à l'avance pour votre temps, voici mon 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
15
16
17
18
19
20
21
22
23
if (isset($_POST['go']) && $_POST['go']=='Valider le vote') {
	if (!isset($_POST['ckb']) || !isset($_POST['sondage_en_cours'])) {
	$erreur = 'Aucune réponse n\'a été choisie.';
	}
 
if (empty($_POST['ckb']) || empty($_POST['sondage_en_cours'])) {
	$erreur = 'Au moins un des champs est vide.';
	}
	else {
 
	$base = mysql_connect ('localhost','root','');
	mysql_select_db ('test',$base);
 
      		$sql ='UPDATE sondage_reponses SET nb_reponses = nb_reponses + 1 WHERE id_sondage="'.$_POST['sondage_en_cours'].'" AND id="'.$_POST['ckb'].'"';
 
	mysql_query ($sql) or die ('Erreur SQL !'.$sql2.'<br />'.mysql_error());
 
	mysql_close ();
 
	$erreur = 'Merci d\'avoir voté ';
	}
}
?>

[sabotage]

Etant donné que tu fais une seul requête UPDATE sur un seul ID, tu n'as évidemment qu'une seul mise à jour.

Je soupconne que toutes tes checkbox s'appellent "ckb".
Il faut que tu utilises une notation en tableau

Code html :

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

<input type="checkbox" name="ckb[]" />

A l'arrivée $_POST['ckb'] sera un tableau contenant tous les ids à mettre à jour.

Code :

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

1
2
$ckb = array_map('intval',$_POST['ckb']);
$sql ='UPDATE sondage_reponses SET nb_reponses = nb_reponses + 1 WHERE id_sondage="'.mysql_real_escape_string($_POST['sondage_en_cours']).'" AND id IN (' . implode(',' , $ckb) .')';

Dans ton code tu omettais toutes les protections sur la requête, fais attention à ça.

[Invité]

Je te remercie encore une fois t'a compris exactement ce qui me manquait, j'ai hâte d'être rendu à ton niveau ça l'air d'un jeu d'enfant pour toi

et j'ignorais que je devais répéter la protection

Code :

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

mysql_real_escape_string

à chaque requête, je l'avais seulement utilisé 1 fois plus loin dans mon code, un gros merci!