[Conception] Suppression dans une base

Version imprimable

Bonjour,

J'ai un léger problème de suppression dans une base. Cela ne supprime rien du tout...

Quelqu'un aurait une idée car là je nage un peu... :D

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
 
 
<?php
session_start();
require("protect/conf.php");
 
if (!isset($_SESSION['pseudo']))
{
	echo("<script language=\"javascript\">location=\"index.php?page=acces_interdit\";</script>");
}
 
$sql = connect_sql();
$pseudo = $_SESSION['pseudo'];
$reponse = mysql_query ("SELECT * FROM membres WHERE pseudo='$pseudo'"); 
$donnees = mysql_fetch_array ($reponse);
 
if (isset($_GET['confirm_suppr']))
{
	$sql = connect_sql();
    mysql_query('DELETE * FROM membres WHERE id=' . $_GET['confirm_suppr']);
	mysql_close($sql);
 
if($sql) 
    {
	print "<script>window.location='./index.php?page=suppr_succes';</script>";
	}
  else
  {
    print "<script>window.location='./index.php?page=erreur';</script>";
  }}
?>

Voici le lien qui me permet de supprimer :

Code:

<a href="index.php?page=confirm_suppr&id=<?php echo $id; ?>">Oui</a>

04/04/2006, 12h15
Mr N.

$_GET ne contient pas 'confirm_suppr', mais 'page'
04/04/2006, 12h15
ska_root

:idea: ça ne serait pas plutôt :

Code:

mysql_query('DELETE * FROM membres WHERE id=' . $_GET['id']);

bonjour,

Essais peut etre ca, car je pense que tu te trompe dans le test de tes variables
Code:

1 2 3 4 5 6 7 if (isset($_GET['page']) && $_GET['page'] == 'confirm_suppr') { $sql = connect_sql(); mysql_query('DELETE * FROM membres WHERE id=' . $_GET['id']); mysql_close($sql); ...

04/04/2006, 12h55
Nesmontou

Salut, pour un DELETE, il me semble que la syntaxe serait plutôt

Code:

mysql_query('DELETE FROM membres WHERE id='.$_GET['id']);

sans *

Bon développement ;)

PS : n'oublie de faire les vérifications qui vont bien pour pouvoir utiliser $_GET['id']
04/04/2006, 14h40
djinko

J'ai modifié cela :

Code:

mysql_query('DELETE * FROM membres WHERE id=' . $_GET['confirm_suppr']);

par :

Code:

mysql_query('DELETE FROM membres WHERE id='.$_GET['id']);

Et par les autres propositions mais cela ne change rien...

Je ne comprends pas, je precise bien que si l'on clique sur le lien, on supprime les champs de la base mais rien y fait...
04/04/2006, 14h45
Mr N.

TucSale et Mr N. parleraient-ils aux murs ?
04/04/2006, 14h49
djinko

Non du tout mais j'ai bien précisé que les autres propositions ne fonctionnent pas non-plus.

Pour info, vous avez oublié "ska_root" dans votre liste...
04/04/2006, 14h54
Mr N.

Fais voir ton code corrigé.

Voici le code de la page :

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
 
<?php
session_start();
require("protect/conf.php");
 
if (!isset($_SESSION['pseudo']))
{
	echo("<script language=\"javascript\">location=\"index.php?page=acces_interdit\";</script>");
}
 
$sql = connect_sql();
$pseudo = $_SESSION['pseudo'];
$reponse = mysql_query ("SELECT * FROM membres WHERE pseudo='$pseudo'"); 
while ($var = mysql_fetch_array ($reponse))	 
{
$id = $var['id'];
 
if (isset($_GET['confirm_suppr&id']))
{
$sql = connect_sql();
    mysql_query('DELETE FROM membres WHERE id=' . $_GET['confirm_suppr&id']);
mysql_close($sql);
 
if($sql) 
    {
	print "<script>window.location='./index.php?page=suppr_succes';</script>";
	}
  else
  {
    print "<script>window.location='./index.php?page=erreur';</script>";
  }}
?>

Le lien :

Code:

[ <a href="index.php?page=confirm_suppr&id=<?php echo $id; ?>">Oui</a> ]

Pour info, lorsque je clique sur le lien la page s'actualise comme si le lien n'était pas actif...

04/04/2006, 15h32
yiannis

bonjour,

euh.... y aurait pas une erreur la:

Code:

mysql_query('DELETE FROM membres WHERE id=' . $_GET['confirm_suppr&id=']);

ce ne serait pas plutot:

Code:

mysql_query('DELETE FROM membres WHERE id=' . $_GET['id=']);

:wink:
04/04/2006, 15h32
Mr N.

Code:

if (isset($_GET['confirm_suppr&id=']))

:koi: C'est quoi ça ? J'ai comme l'impression qu'il manque des notions de bases là :roll:

Relis mieux la proposition de TucSale...
04/04/2006, 15h37
djinko

erreur de ma part mais de suite corrigé...
04/04/2006, 15h44
djinko
si mes notions de bases sont absentes, vous pouvez m'expliquez pourquoi comme cela ça fonctionne alors???
Code:

1 2 3 4 5 6 7 8 if (isset($_GET['supprimer'])) { $sql = connect_sql(); mysql_query('DELETE FROM membres WHERE id=' . $_GET['supprimer']); mysql_close($sql); }
Avec ce lien :

Code:

<a href="confirm_suppr.php?supprimer=<?php echo $id; ?>">Supprimer</a>
04/04/2006, 15h51
yiannis

ca fonctionne car tu teste si tu as bien ta variable $_GET['supprimer']

Code:

if (isset($_GET['supprimer']))

puis pour ta requette de suppression tu prends la valeur de ta variable $_GET['supprimer'].

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
<?php 
session_start(); 
require("protect/conf.php"); 
 
if (!isset($_SESSION['pseudo'])) 
{ 
   echo("<script language=\"javascript\">location=\"index.php?page=acces_interdit\";</script>"); 
} 
 
$sql = connect_sql(); 
$pseudo = $_SESSION['pseudo']; 
$reponse = mysql_query ("SELECT * FROM membres WHERE pseudo='$pseudo'"); 
while ($var = mysql_fetch_array ($reponse))    
{ 
$id = $var['id']; 
 
if (isset($_GET['confirm_suppr']) && isset($_GET['id'])) 
{ 
$sql = connect_sql(); 
    mysql_query('DELETE FROM membres WHERE id=' . $_GET['id']); 
mysql_close($sql); 
 
if($sql) 
    { 
   print "<script>window.location='./index.php?page=suppr_succes';</script>"; 
   } 
  else 
  { 
    print "<script>window.location='./index.php?page=erreur';</script>"; 
  }} 
?>

devrait fonctionner... ^^

04/04/2006, 15h57
Mr N.

Toujours pas... ^^
Enfin tout dépend de l'url qui appelle ce script...
04/04/2006, 16h02
johweb

Je n'ai pas détaillé, mais j'ai modifié ces deux lignes :

Code:

if (isset($_GET['confirm_suppr&id']))

et

Code:

mysql_query('DELETE FROM membres WHERE id=' . $_GET['confirm_suppr&id']);

par

Code:

if (isset($_GET['confirm_suppr']) && isset($_GET['id']))

et

Code:

mysql_query('DELETE FROM membres WHERE id=' . $_GET['id']);

et c'est "appelable" par la page :

Code:

<a href="index.php?confirm_suppr=oui&id=<?php echo $id; ?>">Oui</a>

Sinon, si cela ne marche toujours pas, merci de préciser les erreurs renvoyées... ^^
04/04/2006, 16h04
Mr N.

On est d'accord alors 8)

Ca j'avais bien capté mais de cette maniere, cela ne m'arrange pas...

C'est pour cela que je demande si quelqu'un aurait une idée si je procede d'une autre maniere...

en ayant modifié de cette maniere...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 if (isset($_GET['page'])) { $sql = connect_sql(); mysql_query('DELETE FROM membres WHERE membres.id=' . $_GET['id']); mysql_close($sql); } if($sql) { print "<script>window.location='./index.php?page=suppr_succes';</script>"; } else { print "<script>window.location='./index.php?page=erreur';</script>"; } ?>
Il me précise bien que le compte a été supprimé avec succes mais rien ne se passe dans la bdd... 8O

04/04/2006, 16h19
Mr N.

Si rien ne se passe, c'est que soit tu n'es pas rentré dans ton if, soit l'id n'existe pas, soit il y a une erreur dans la requete envoyée à mysql...