Suppression d'une ligne dans une table sql

Version imprimable

Bonjour,
J'ai créé un formulaire checkbox. Les checkbox sélectionnées doivent être supprimée de la base de données dans laquelle elles sont enregistrées. Pour cela, j'ai écris un bout de code php mais ca ne fonctionne pas (je coche mes cases, je valide, je suis redirigée comme demandé. Aucun message d'erreur et aucune suppression dans la base).
Code:

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 <?php // On vérifie que le formulaire vient d'être envoié. if (!isset($_POST['submit'])) { } // On regarde si nos variables ne sont pas vides. elseif ((isset($_POST['submit'])) && (!empty($_POST['supstage']))) { // On supprime la sélection. $idstage = htmlspecialchars($donnees['idstage']); $bdd = new PDO('mysql:host=..........;dbname=..........;', '..........', '............'); $req = $bdd->prepare('DELETE FROM stages WHERE idstage = '.$idstage.''); $req->execute(array( 'idstage' => $idstage)); //Redirection et rafraichissement de la page $delai=1; $url='stages.php'; header("Refresh: $delai;url=$url"); $req->closeCursor(); } else { echo"<script>alert('Veuillez cocher au moins un stage');</script>"; } ?>
Merciiiiiiii!!!!!

25/03/2015, 13h07
sabotage

D'ou vient $donnees ?
25/03/2015, 13h13
Jess86

C'est l'identifiant enregistré dans ma base de données. Chaque stage se trouvant enregistré dans la base de données a son propre identifiant. C'est avec cette donnée que je reconnais le stage choisi donc le stage qui doit être supprimé de la base de données.
25/03/2015, 13h26
sabotage

Il est défini ou par rapport au script que tu nous montres et qu'est ce qu'il vient faire la puisqu'on parle d'un formulaire ?

Voici mon formulaire: $donnees sur la ligne 10.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
<form action="stage.php" method="post">
 
<?php
// Poste de l'ensemble des stages.
$bdd = new PDO('mysql:host=;dbname=', '', '');
$reponse = $bdd->query('SELECT idstage, titrestage, DATE_FORMAT(datestage, \'%d/%m/%Y\') AS datestage_fr FROM stages ORDER BY idstage DESC LIMIT 0, 20');
while ($donnees = $reponse->fetch()) {
echo '<h4 class="suppression">
      <input type="checkbox" value="' . htmlspecialchars($donnees['idstage']) . '" name="supstage[]">
      <div class="supdate">' . htmlspecialchars($donnees['datestage_fr']) . '</div>
      <div class="suptitre">' . htmlspecialchars($donnees['titrestage']) . '</div>
      </h4>';
}
$reponse->closeCursor();
?>
 
<br>
<input type="submit" value="supprimer" name="submit">
<input type="reset" value="Annuler">
</form>

25/03/2015, 14h18
sabotage

Le champs qui contient c'est les id a supprimer c'est "supstage" si tu ne l'utilises pas dans ton traitement c'est normal que ça ne fonctionne pas.

Salut ! :)

Je ne comprend pas comment tu fonctionnes dans ton traitement.

Tiens si ça peut t'aider :

formulaire php avec checkbox ->

Code:

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
 
<?php
if($dossier = opendir('../URL/'))
{
	while(false !== ($fichier = readdir($dossier)))
	{
		$rest = substr($fichier, 0, 4);
		if($rest == "Quiz" || $rest == "quiz")
		{
			if($fichier != '.' && $fichier != '..' && $fichier != 'index.php')
			{
				$nb_fichier++;
				echo '<form method="post" action="">
					<TD class="tg-qgr3" width=2%><input type="checkbox" name="checkQuiz[]" value='.$fichier.' \/></TD> // C'est cette partie qui t'intéresse
					<TD class="tg-qgr3" width=33% ><a href="../URL/'.$fichier.'">'.$fichier.'</a></TD>
					</TR><TR>
			
				';
			}
		}
	}
	closedir($dossier);
}
else echo 'Le dossier n\'a pas pu être ouvert';
?>
 
<div class="Delete">
<button type="submit" class="btn btn-danger" name="Delete" value="Delete"><span class="glyphicon glyphicon-remove-sign"></span> Delete</button>  <!-- Mon bouton de type submit -->
</div>
</form>

Et voilà le traitement de ma checkbox "checkQuiz" ->

Code:

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
37
38
39
40
 
<?php
if(isset($_POST['Delete']))
{
	if(!empty($_POST['checkQuiz']))
	{
	foreach($_POST['checkQuiz'] as $valeur)
	{
            $chemin = "../URL/$valeur"; 
$result = recursive_rmdir($chemin,$follow);
 
		try {
		    $sql = "DELETE FROM ...";
		    $bdd->exec($sql);
		}
		catch (Exception $e) {
		die('Erreur : '.$e->getMessage());
		}
 
		try {
		    $sql2 = "DELETE FROM ...";
		    $bdd->exec($sql2);
		}
		catch (Exception $e) {
			die('Erreur : '.$e->getMessage());
		}
			//Refresh la page pour supprimer de l'affichage le dossier supprimé
			echo"<meta http-equiv='refresh' content='1;URL=...'>";
		}
		}else echo'<br/><br/>
		<div class="col-lg-5">
		</div>
		<div class="alert alert-danger alert-dismissable col-lg-2">
		<button type="button" class="close" data-dismiss="alert">&times;</button>
		<strong>Error !</strong> 0 Quiz selected !
		</div>
		<div class="col-lg-5">
		</div><br/><br/><br/><br/>
		';
?>

En espérant t'avoir aidé !

Cordialement,

25/03/2015, 16h25
Jess86
J'ai transformé ma requête comme ça, mais j'ai toujours exactement le mm résultat : pas de message d'erreur mais aucune ligne de supprimé dans ma base. :(
Code:

1 2 3 4 $req = $bdd->prepare('DELETE FROM stages WHERE idstage = '.$_POST['supstage'].''); $req->execute(array( 'idstage' => $_POST['supstage']));
25/03/2015, 16h28
sabotage

$_POST['supstage'] est un tableau, il faut le parcourir.

Il te faut utiliser un foreach pour supprimer le contenu du tableau !
Code:

1 2 3 4 5 6 7 <?php foreach($_POST['supstage'] as $valeur) { $req = "DELETE FROM stages WHERE idstage = '.$valeur.' "; $bdd->exec($req); } ?>

Plutôt

Code:

1
2
3
4
5
6
<?php
$sth = $bdd->prepare("DELETE FROM stages WHERE idstage = :idstage");
foreach($_POST['supstage'] as $valeur) {
     $bdd->execute(array(':idstage'=>$valeur));
}
?>

25/03/2015, 17h04
Kyuudan

C'est vrai que ça me semble mieux.
Quel est l’intérêt de sortir la requête du foreach ? Et le prepare est-il vraiment utile ?
25/03/2015, 17h15
Jess86

Aaaaaaaah, merci, ca marche!!!:mrgreen:
25/03/2015, 20h23
sabotage

Citation:

Quel est l’intérêt de sortir la requête du foreach ? Et le prepare est-il vraiment utile ?

La requête est sortie puisqu'il s'agit d'une requête préparée.
La préparation est essentielle, sinon les injections SQL sont possibles.