Suppression d'enregistrement mySQL [Résolu]

[Cocalight]

Bonjour, débutant PHP je me retrouve confronter a un probleme.
J'ai developpé un script permetant l'ajout d'un enregistrement dans une base mySQL qui fonctionne a merveille.

Je me suis alors mis au travail pour creer un script qui cette fois-ci supprime un enregistrement de la bdd.

VOici le code

Code :

1
2
3
4
5
<form action="validedelete.php" method="post">
      <span class="Style14">Effacer le ban numéro :</span>
        <input type="text" name="ban_id:"> 
      <input name="submit" type="submit" class="Style14" value="Effacer le ban">
</form>

Voici le traitement dans ma page validedelete.php

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
<?
/*
On se connecte et on choisi la base de données
*/
mysql_pconnect('localhost','user', '*****') OR die ("Impossible de se connecter à la base de donnée");
mysql_select_db('lfs-teamcom') OR die("Impossible de sélectionner la base de données");
 
 
IF (!isset($ban_id)) {
  echo "<p>Aucun enregistrement à supprimer.</p>";
} else {
  $query_string = "DELETE FROM banlist WHERE ban_id=$id";
  $query=mysql_query($query_string);
  IF ($query) {
    echo "<p>L'enregistrement a été effacé</p>";
  } else {
    echo "<p>Problème lors de la suppression</p>";
    echo "<p>" . mysql_error() . "</p>";
  }
}
?>

Les champs de la base de données sont

Code :

1
2
3
4
5
ban_id (clé primaire, AUTO_INCREMENT)
pseudo
steam_id
raison
admin

Je debute depuis peu en PHP. Je lis des tonnes de tutos mais tout n'est pas forcement clair ^^

Merci de détaillé la ou je me suis trompé

Cordialement

[guitou12]

Tu as des erreurs ou ça fonctionne ?

[Cocalight]

Non le formulaire de suppression ne fonctionne pas.

Il me dit

Code :

Aucun enregistrement à supprimer.

Je precise bien l'id de l'enregistrement a supprimer pourtant et lors de la soumission aucun message d'erreur .. pourtant l'enregistrement est toujours la.

[SnakemaN]

Bonjour,

Deja premier conseils,
- recupere proprement la variable postée.sans utiliser de variable super globale
- Tu ne concatenne pas bien les variable dans ta requete

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
<?
/*
On se connecte et on choisi la base de données
*/
mysql_pconnect('localhost','user', '') or die ("Impossible de se connecter à la base de donnée");
mysql_select_db('lfs-teamcom') or die("Impossible de sélectionner la base de données");

[B]if(isset($_POST['ban_id']))
$banId = $_POST['ban_id'];
else
$banId = NULL[/B]

 
if (banId == NULL) {
echo "<p>Aucun enregistrement à supprimer.</p>";
} else {
$query_string = "DELETE FROM banlist WHERE ban_id=[B]'".$id."" [/B]";
$query=mysql_query($query_string)or die(mysql_error());
if ($query) {
echo "<p >L'enregistrement a été effacé</p>";
} else {
echo "<p>Problème lors de la suppression</p>";
echo "<p>" . mysql_error() . "</p>";
}
}
?>

Colorez votre code PHP sur les forums grâce à Developpez.com

Voila essayes deja ca, ca devrais marcher comme cela...

[Kerod]

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
IF (!isset($_POST['ban_id'])) {
  echo "<p>Aucun enregistrement à supprimer.</p>";
} else {
  $query_string = "DELETE FROM banlist WHERE ban_id=".$_POST['ban_id']
;
  $query=mysql_query($query_string);
  IF ($query) {
    echo "<p>L'enregistrement a été effacé</p>";
  } else {
    echo "<p>Problème lors de la suppression</p>";
    echo "<p>" . mysql_error() . "</p>";
  }
}

Comme ceci ?

[SnakemaN]

Le BBcode c'est bien mais des fois...

[Cocalight]

@SnakemaN

Je ne comprend pas ton code (newbie quand tu nous tiens) (je colorise mon code la prochaine fois)

@Kerod

TOujours le même résultat, a savoir

Aucun enregistrement a supprimer.
Tu peux voir le script a cette adresse => http://lfs-team.com/e107_files/banlist.php

[guitou12]

Code :

<input name="ban_id:">

[Cocalight]

Oui effectivement pas malin de ma part.

Maintenant j'ai cette erreur

Code :

1
2
3
Problème lors de la suppression
 
You have an error IN your SQL syntax; CHECK the manual that corresponds TO your MySQL server version FOR the RIGHT syntax TO USE near '' at line 1

Code :

<?
mysql_pconnect('localhost','user', '') or die ("Impossible de se connecter à la base de donnée");
mysql_select_db('lfs-teamcom') or die("Impossible de sélectionner la base de données");

if (!isset($_POST['ban_id'])) {
  echo "<p>Aucun enregistrement à supprimer.</p>";
} else {
  $query_string = "DELETE FROM banlist WHERE ban_id=".$_POST['ban_id']
;
  $query=mysql_query($query_string);
  if ($query) {
      echo "<p>L'enregistrement a été effacé</p>";
  } else {
      echo "<p>Problème lors de la suppression</p>";
      echo "<p>" . mysql_error() . "</p>";
  }
}
?>

Colorez votre code PHP sur les forums grâce à Developpez.com

[Kerod]

Ca vient de ton champ post il doit etre vide..Vérifies bien qu'il y a un contenu

[SnakemaN]

Code :

1
2
 
$query_string = "DELETE FROM banlist WHERE ban_id='".$_POST['ban_id']."' ";

Envoyé par fanta agrum

Je ne comprend pas ton code (newbie quand tu nous tiens) (je colorise mon code la prochaine fois)

c'est normal c'est le BBcode qui fait des siennes, c'est normal ce n'est qu' un bébé (<- Ultimate Joke Inside )

[guitou12]

C'est laborieux tout ça, révise un peu ton html

Code :

<form action="validedelete.php" method="$_POST">

--> method="post"



EDIT: tu avais juste sur ta 1ère version pourquoi avoir changé ?

[Kerod]

Envoyé par SnakemaN

Code :

Ce serait mieux si tu faisais le contraire

Code :

1
2
 
$query_string = "DELETE FROM banlist WHERE ban_id='".$_POST['ban_id']."' ";

[SnakemaN]

Envoyé par Kerod

Ce serait mieux si tu faisais le contraire

Code :

Qui qui me parle ??

[Kerod]

$query_string = "DELETE FROM banlist WHERE ban_id="'.$_POST['ban_id'].'" ";

Les quotes doivent être à l'intérieur...Pas d'accord ??

[guitou12]

T'as une balise </form> de trop qui traine en fin de page !!!

[guitou12]

Ce qu'à écrit Snakeman est juste mais ce qu'à cité Kerod est faux.... qui des 2 a raison ?

Il me semble que

Code :

$query_string = "DELETE FROM banlist WHERE ban_id='".$_POST['ban_id']."' ";

est la bonne réponse, ceci dit si son ID est un int c'est un peu inutile !

[SnakemaN]

Et puis je ne sais pas le niveau de securité voulu, mais je mettrai un petit htmlentities()
Au cas ou...

[Cocalight]

@guitou : Je m'en sort comme je peut dsl
@snakeman : htmlentities() connais pas du tout (j'ai déja du mal avec un simple formulaire si tu me donne la marche a suivre

Donc je récapitule mon code formulaire et traitement :

Formulaire :

Code :

1
2
3
4
5
6
7
<form action="validedelete.php" method="post">
  <p><span class="Style14">Effacer le ban numéro :</span>
      <input type="text" name='ban_id'>   
 
    <input name="submit" type="submit" class="Style14" value="Effacer le ban">
  </p>
</form>

traitement (validedelete.php) :

<?
/* On se connecte et on choisi la base de données */
mysql_pconnect('localhost','Coyote', '*****')
or die ("Impossible de se connecter à la base de donnée");
mysql_select_db('lfs-teamcom') or die("Impossible de sélectionner la base de données");
if(isset($_POST['ban_id']))
$banId = $_POST['ban_id'];
else $banId = NULL
if (banId == NULL)
{ echo "<p>Aucun enregistrement à supprimer.</p>";
} else {
$query_string = "DELETE FROM banlist WHERE ban_id="'.$_POST['ban_id'].'" ";
$query=mysql_query($query_string)or die(mysql_error());
if ($query) { echo "<p >L'enregistrement a été effacé</p>";
} else {
echo "<p>Problème lors de la suppression</p>";
echo "<p>" . mysql_error() . "</p>";
}
}
?>

Colorez votre code PHP sur les forums grâce à Developpez.com

[guitou12]

C'est sur que si tu te relis pas....

Code :

1
2
3
4
if ($banId == NULL)
{ echo "<p>Aucun enregistrement à supprimer.</p>";
} else {
$query_string = "DELETE FROM banlist WHERE ban_id='".$_POST['ban_id']."' ";