Suppression de données à partir des paramètres passés en lien

Version imprimable

bonjour a tous

je voudrais effacer de ma table tous les devis d'un client

or dans un devis il y a des articles(il y a donc 2 tables: 1 table pour les devis appelé "commandes" et une table pour les articles commandés appelée "article_commande")

je reussi a effacer mes devis mais pas mes articles! alors voici le lien permettant normalement de detruire tous mes devis + articles:

Code:

<a href="infos_commande.php?allsupprim=<?php echo $_GET['devis'];?>&allsupprim2=<? echo $_GET['id_commande'];?>"onClick="return confirm('Etes vous sûr de vouloir effectuer cette action?');">supprimer tous les devis</a>

$_GET['devis'] et $_GET['id_commande'] viennent de la page precedente:
Code:

1 2 3 4 5 6 7 8 9 10 $va=mysql_query("SELECT * FROM article_commande"); $vam=mysql_fetch_array($va); $id_commande=$vam['id_commandes']; $recherche = mysql_query ("SELECT * FROM membre ORDER BY id"); while ($var = mysql_fetch_array ($recherche)) { $id = $var['id']; } <a href="infos_commande.php?devis=<?php echo $id; ?>&id_commande=<?php echo $id_commande;?>">Voir ses devis</a>
maintenant voila mon script:
Code:

1 2 3 4 5 6 7 if(isset($_GET['allsupprim'])) { $allsuppr2=$_GET['allsupprim2']; mysql_query("DELETE FROM article_commande WHERE id_commandes=".$allsuppr2)or die(mysql_error()); mysql_query("DELETE FROM commandes WHERE id_membre='". $_GET['allsupprim']."'")or die(mysql_error()); echo'<a href="memberlist.php">retour</a>'; }
et ca m'affiche comme erreur

Citation:

Erreur de syntaxe près de 'FROM article_commande WHERE id_commandes=16' à la ligne 1

j'ai regardé et 16 correspond bien a l'id de la commande dans la bse de donnee...

ou est mon probleme?

merci

31/03/2008, 16h43
darkstar123456
Effectivement, ta synthaxe n'est pas bonne...

Code:

mysql_query("DELETE FROM article_commande WHERE id_commandes=".$allsuppr2)

devient
Code:

1 2 mysql_query("DELETE FROM article_commande WHERE id_commandes='".$allsuppr2."'")
31/03/2008, 16h51
cyril3d

je suis bete!

merci:D
31/03/2008, 17h07
cyril3d

mais non!

ca change rien!

curieusement mon code marche vu qu'il ne s'arrete pas a mon or die...

mais il n'efface que les devis pas le contenu des champs de aticle_commande...

une solution?
31/03/2008, 17h16
boo64

logiquement si tu as une clé etrangere reliant commande et article tu la met en ON DELETE CASCADE ce qui te permet de supprimer juste dans la table commande et tu n'as pas besoin de faire les autres delete ;)
31/03/2008, 17h17
cyril3d

heuuuuuuuuuuu...

tu peux parler moins vite s'il te plais j'ai du mal a te suivre...:lol:
31/03/2008, 17h21
boo64

peux tu me donner la structure de tes 2 tables que je t'explique bien

tiens:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
--
-- Structure de la table `article_commande`
-- 
 
CREATE TABLE `article_commande` (
  `id` int(255) NOT NULL auto_increment,
  `nom` text NOT NULL,
  `qte` tinyint(255) NOT NULL,
  `designation` text NOT NULL,
  `id_commandes` int(255) NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin1 AUTO_INCREMENT=4 ;

Code:

1
2
3
4
5
6
7
8
9
10
11
-- 
-- Structure de la table `commandes`
-- 
 
CREATE TABLE `commandes` (
  `id` int(255) NOT NULL auto_increment,
  `id_membre` tinyint(10) default '0',
  `date` date NOT NULL default '0000-00-00',
  `heure` time NOT NULL default '00:00:00',
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin1 AUTO_INCREMENT=31 ;

31/03/2008, 17h28
boo64

pour assurer la cohérence de ta base et faire ce que je te dit il te manque une clé etrangére
31/03/2008, 17h31
cyril3d

ha...et comment j'en rajoute?

tes tables doivent être de type innodb

http://dev.mysql.com/doc/refman/5.0/...nstraints.html

lii ceci ;) et en l'appliquant sur tes tables

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
CREATE TABLE `commandes` (
`id` int(255) NOT NULL auto_increment,
`id_membre` tinyint(10) default '0',
`date` date NOT NULL default '0000-00-00',
`heure` time NOT NULL default '00:00:00',
PRIMARY KEY (`id`)
) TYPE=INNODB DEFAULT CHARSET=latin1 AUTO_INCREMENT=31 ;
 
 
CREATE TABLE `article_commande` (
`id` int(255) NOT NULL auto_increment,
`nom` text NOT NULL,
`qte` tinyint(255) NOT NULL,
`designation` text NOT NULL,
`id_commandes` int(255) NOT NULL,
PRIMARY KEY (`id`),
INDEX (id_commandes),
FOREIGN KEY (id_commandes) REFERENCES commandes(id) ON DELETE CASCADE
) TYPE=INNODB DEFAULT CHARSET=latin1 AUTO_INCREMENT=4 ;

31/03/2008, 17h40
cyril3d

et une fois que j'ai recréé mes tables quel code je dois mettre et ou pour effacer a la fois mes devis ET le contenu de ceux ci?
31/03/2008, 17h45
boo64

vu que tu as maintenant ON DELETE CASCADE la suppression de la commande va entrainer automatiquement la suppression de l'article

Citation:

. Si ON DELETE CASCADE est spécifiée, et qu'une ligne de la table parente est effacée, alors InnoDB va automatiquement effacer toute les lignes qui sont dans la table fille et dont les valeurs de clé étrangère sont celles référencées dans la ligne parente

et comme apparamment ;) tu arrives avec ceci à supprimer les commandes:

DELETE FROM commandes WHERE id_membre='". $_GET['allsupprim']."'"

ps:si ça marche pas je regarderai un peu plus tard la boulot fini ;)
31/03/2008, 17h58
cyril3d

MERCI

je suis bloqué dessus depuis ce matin

et la : ca marche!(c'est pas la premiere fois que tu me debloques)

bon travail
31/03/2008, 18h41
cyril3d

haaaaaaaaaaa

je peux pas utiliser cette methode!

parceque l'utilisateur doit avoir le choix entre tout effacer ou effacer un devis par un devis (cette partie du code marche a merveille)

par contre si j'utilise ce code, je suis obligé d'effacer tous les articles du meme utilisateur : il ne peux plus les effacer un par un!

dommage,la solution me plaisait pourtant...
01/04/2008, 14h52
boo64

remontre moi ton code stp

pasque je vois pas le probleme si la commande est effacé on efface tous les enregistrement lié a cette commande...