Discussion :

[Sylwek]

Bonjours,
Dans mon Controller j'ai

Code :

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

1
2
3
4
5
6
7
case "deleteIgr":
    require("Class/Ingredient.php");
    require("Class/IngredientsManager.php");
    $data = array ('code'=> $_POST['code']);
    $deleterIngredient = new IngredientsManager;
    $deletedIngredient=$deleterIngredient->deleted(new Ingredient($data));
    break;

Dans la classe IngredientsManager ma methode deleted est :

Code :

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

1
2
3
4
5
6
7
public function deleted(Ingredient $ingredient)
	{
		$db = $this->dbConnect();
		$q=$db->prepare('DELETE FROM Products WHERE Code = :code ');
		$q->bindValue(':code', $ingredient->code());
		$q->execute();
	}

Le POST dans mon controller est recu par AJAX avec JQuerry :

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
$('.ButtonDelete').on('click', function() {
		let id= $(this).val();
		let code=$('#code'+id).val();
		req = $.ajax({
			url: "menu.php?Option=deleteIgr",
			type : 'POST',
			data : { code : code }
		});
		req.done(function (response, status, jqXHR){
			console.log("Deleted with success");
		});
		req.fail(function (jqXHR, status, err){
			console.error("An error occurred: " + status, err);
		});
	});

La requete AJAX marche je suppose car la console m'affiche Deleted with success
Mais la ligne n'est pas supprimeee et bien presente dans la base de donnees (SQL Server)
Je ne sais pas ce que j'ai manque

[armel18]

bonjour,

vérifies que tu récupères bien la propriété membre de la classe Ingredient:

Code :

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

1
2
3
4
5
6
7
8
9
 
public function deleted(Ingredient $ingredient)
{
        var_dump($ingredient->code()); // affiche-t-il le code?
	$db = $this->dbConnect();
	$q=$db->prepare('DELETE FROM Products WHERE Code = :code ');
	$q->bindValue(':code', $ingredient->code());
 return $q->execute(); // renvoi true si la suppression a eu lieu sinon false
}

[Sylwek]

Merci,
Je vien de tester, en fait je viens de me rendre compte grace a ta technique que ca marche mais pas sur tout les enregistrement, il y a toujours ingredient code mais pas toujours la suppression, la DB n'est pas de moi, est-ce possible que certain enregistrement soit tout simplement impossible a supprimer ? car ca marche normalement maintenant sauf sur quelque recalcitrants

[andry.aime]

Bonjour,

Par défaut, PDO ne va pas afficher les messages d'erreurs.

1- Soit, pendant l’instanciation de ton PDO, fait en sorte pour que les messages d'erreurs s'affichent

Code :

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

new PDO($host, $user, $password,  array( PDO::ATTR_ERRMODE =>PDO::ERRMODE_EXCEPTION ) ); // Tu peux simplement faire un warning (ERRMODE_WARNING) au lieu de lancer une exception

Tu peux aussi utiliser setAttribute pour affecter le paramètre.

2- Soit de récupérer l'erreur si le dernier exécution a échoué: http://php.net/manual/fr/pdostatement.errorinfo.php

Pour la gestion des erreurs PDO, regarde ici: http://php.net/manual/fr/pdo.error-handling.php

Dans ton cas, je soupçonne l'échec de suppression dû à une contrainte de clé étrangère dans d'autre table qui est liée à celle ci.

A+.

[Sylwek]

Bonjour,

Par défaut, PDO ne va pas afficher les messages d'erreurs.

1- Soit, pendant l’instanciation de ton PDO, fait en sorte pour que les messages d'erreurs s'affichent

Code :

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

new PDO($host, $user, $password,  array( PDO::ATTR_ERRMODE =>PDO::ERRMODE_EXCEPTION ) ); // Tu peux simplement faire un warning (ERRMODE_WARNING) au lieu de lancer une exception

Tu peux aussi utiliser setAttribute pour affecter le paramètre.

2- Soit de récupérer l'erreur si le dernier exécution a échoué: http://php.net/manual/fr/pdostatement.errorinfo.php

Pour la gestion des erreurs PDO, regarde ici: http://php.net/manual/fr/pdo.error-handling.php

Dans ton cas, je soupçonne l'échec de suppression dû à une contrainte de clé étrangère dans d'autre table qui est liée à celle ci.

A+.

Merci en effet j'avais zapper la table de jointure :facepalm: