Discussion :

[cyril3d]

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 :

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

<a href="infos_commande.php?allsupprim=<?php echo $_GET['devis'];?>&amp;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 :

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

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; ?>&amp;id_commande=<?php echo $id_commande;?>">Voir ses devis</a>

maintenant voila mon script:

Code :

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

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

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

[darkstar123456]

Effectivement, ta synthaxe n'est pas bonne...

Code :

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

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

devient

Code :

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

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

[cyril3d]

je suis bete!


merci

[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?

[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

[cyril3d]

heuuuuuuuuuuu...


tu peux parler moins vite s'il te plais j'ai du mal a te suivre...

[boo64]

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

[cyril3d]

tiens:

Code SQL :

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

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 ;

et

Code SQL :

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

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 ;

[boo64]

pour assurer la cohérence de ta base et faire ce que je te dit il te manque une clé etrangére

[cyril3d]

ha...et comment j'en rajoute?

[boo64]

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 SQL :

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
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 ;

[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?

[boo64]

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

. 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

[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

[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...

[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...