aide pour suppression

Version imprimable

Qu'est ce qu'il ne va pas dans mon code ?

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
<html>
    <head>
        <title>Suppresion du lien</title>
    </head>
    <body>
    Voulez vous vraiment supprimer le lien ?
    <br>
        <center>
            <form action="deletelien.php" method="post">
                <input type="submit" value="Oui" name="button" /></input>
                <input type="submit" value="Non" name="button" /></input>
                    <?
                    mysql_connect('localhost','login','pass');
                    mysql_select_db('base');
                    $numero_id = $_GET['ID'];
                    $type_de_reponse=$_POST['button'];
 
                        if ($type_de_reponse=="Oui"){
                            mysql_query("DELETE FROM liens_utiles WHERE ID='$numero_id'") .mysql_error();
                            echo 'liens supprimer avec succés !';
                        }elseif ($type_de_reponse=="Non"){
                            echo 'retour à l\'accueil';
                        }
                    mysql_close();
                    ?>
            </form>
        </center>
    </body>
</html>

18/08/2009, 11h39
sabotage

Ton ID n'est pas défini dans le formulaire.
18/08/2009, 11h41
Zwiter

Salut,
Il manque ID en parametre d'URL (GET).

Z.

[edit] Le temps de trouver l'erreur, on a été plus rapide :p
18/08/2009, 11h42
metagoto

Tu es sûr que tu as un $_GET['ID'] ?

Aussi, pense à faire un mysql_real_escape_string() quand tu construits ta query avec WHERE ID='$numero_id'" parceque sinon, c'est une sérieuse trappe à l'sql injection

edit: grilled

N'ouvre la connexion que si tu en as besoin (question de sécurité)
J'ai repris ton code:

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
33
34
35
36
 
<html>
    <head>
        <title>Suppresion du lien</title>
    </head>
    <body>
    Voulez vous vraiment supprimer le lien ?
    <br>
	<?php
	if($_POST['radio']){
		$numero_id = $_GET['ID'];
        $type_de_reponse = $_POST['radio'];
        if ($type_de_reponse == "oui"){
            mysql_connect('localhost','apicryptfaq','mortawindows');
			mysql_select_db('apicryptfaq');
			mysql_query("DELETE FROM liens_utiles WHERE ID='".$numero_id."'")or die(mysql_error());
			mysql_close();
            echo 'liens supprimer avec succés !';
        }else{
            echo 'retour à l\'accueil';
        }
	}
	else{
		?>
		<center>
            <form action="deletelien.php" method="post">
				<input type="radio" name="reponse" value="oui" />
				<input type="radio" name="reponse" value="non" checked />
                <input type="submit" value="Envoyer" />
            </form>
        </center>
		<?php
	}
	?>
    </body>
</html>

tu as aussi d'autres choses à mettre en place comme sécurisé des données

Citation:

Envoyé par Tsoko

N'ouvre la connexion que si tu en as besoin (question de sécurité)
J'ai repris ton code:

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
33
34
35
36
 
<html>
    <head>
        <title>Suppresion du lien</title>
    </head>
    <body>
    Voulez vous vraiment supprimer le lien ?
    <br>
    <?php
    if($_POST['radio']){
        $numero_id = $_GET['ID'];
        $type_de_reponse = $_POST['radio'];
        if ($type_de_reponse == "oui"){
            mysql_connect('localhost','apicryptfaq','mortawindows');
            mysql_select_db('apicryptfaq');
            mysql_query("DELETE FROM liens_utiles WHERE ID='".$numero_id."'")or die(mysql_error());
            mysql_close();
            echo 'liens supprimer avec succés !';
        }else{
            echo 'retour à l\'accueil';
        }
    }
    else{
        ?>
        <center>
            <form action="deletelien.php" method="post">
                <input type="radio" name="reponse" value="oui" />
                <input type="radio" name="reponse" value="non" checked />
                <input type="submit" value="Envoyer" />
            </form>
        </center>
        <?php
    }
    ?>
    </body>
</html>

tu as aussi d'autres choses à mettre en place comme sécurisé des données

Merci pour tous ces conseils, mais ton code ne marche pas encore même. :(

18/08/2009, 12h42
sabotage

Je me repete mais ID n'est toujours pas défini.
18/08/2009, 13h49
Tsoko

Oui, je pars du principe que la valeur Id est déjà renseignée !!
Maintenant en partant du principe que ta table comporte une entrée ID renseignée à 1, tu peux contrôler comme suit:

Code:

<form action="deletelien.php?ID=1" method="post">

Là ça doit marcher ;)

Mais je suppose bien évidemment que ta valeur ID est une reception de données d'une formulaire précédant celui-ci non??

Sinon effectivement, cela ne peut pas marcher car dans ce cas :
ID = ""; (vide) donc ne peut pas supprimer dans la table !!

Courage
18/08/2009, 14h25
kOrt3x

bon bah je ne vois toujours pas, je dois oublier un truc, je sèche. :calim2:
18/08/2009, 14h33
Tsoko

tu a tenu compte de nos observations concernant ID?
18/08/2009, 14h37
kOrt3x

oui, je fais plusieurs tests, mais je n'arriva pas.
(désolé):calim2:

Tsoko : ton code résoud le problème de l'ID, mais en crée un nouveau...
Il faut pas récupérer $_POST['radio'] mais bien $_POST['response'] pour récupérer la valeur du champ radio... après adaptation et test sur ma BDD, le script fonctionne...

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
33
34
35
36
37
 
<html>
    <head>
        <title>Suppresion du lien</title>
    </head>
    <body>
    Voulez vous vraiment supprimer le lien ?
    <br>
    <?php
    if(isset($_POST['response'])){
        $numero_id = $_GET['ID'];
        $type_de_reponse = $_POST['response'];
        if ($type_de_reponse == "oui"){
            mysql_connect('localhost','user','pass');
            mysql_select_db('bla');
            mysql_query("DELETE FROM news WHERE id ='" . $numero_id . "'") or die(mysql_error());
            mysql_close();
            echo 'liens supprimer avec succés !';
        }
		else{
            echo 'retour à l\'accueil';
        }
    }
    else{
        ?>
        <center>
            <form action="test.php?ID=5" method="post">
                <input type="radio" name="response" value="oui">Oui</input>
                <input type="radio" name="response" value="non" checked>Non</input>
                <input type="submit" value="Envoyer" />
            </form>
        </center>
        <?php
    }
    ?>
    </body>
</html>

18/08/2009, 15h22
Tsoko

Oups merci mistertbo :oops:

J'ai écris trop vite ;)