Discussion :

[djinko]

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

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

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

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

[Mr N.]

$_GET ne contient pas 'confirm_suppr', mais 'page'

[ska_root]

ça ne serait pas plutôt :

Code :

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

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

[TucSale]

bonjour,

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

Code :

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

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

[Nesmontou]

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

Code :

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

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']

[djinko]

J'ai modifié cela :

Code :

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

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

par :

Code :

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

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

[Mr N.]

TucSale et Mr N. parleraient-ils aux murs ?

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

[Mr N.]

Fais voir ton code corrigé.

[djinko]

Voici le code de la page :

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

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

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

[yiannis]

bonjour,

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

Code :

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

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

ce ne serait pas plutot:

Code :

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

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

[Mr N.]

Code :

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

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

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

Relis mieux la proposition de TucSale...

[djinko]

erreur de ma part mais de suite corrigé...

[djinko]

si mes notions de bases sont absentes, vous pouvez m'expliquez pourquoi comme cela ça fonctionne alors???

Code :

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

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 :

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

<a href="confirm_suppr.php?supprimer=<?php echo $id; ?>">Supprimer</a>

[yiannis]

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

Code :

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

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

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

[johweb]

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

[Mr N.]

Toujours pas... ^^
Enfin tout dépend de l'url qui appelle ce script...

[johweb]

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

Code :

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

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

et

Code :

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

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

par

Code :

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

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

et

Code :

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

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

et c'est "appelable" par la page :

Code :

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

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

[Mr N.]

On est d'accord alors 8)

[djinko]

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 :

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

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