Problème pour supprimer article en PHP/MySQL

Marsh Posté le 23-04-2009 à 12:02:52

Bonjour à tous,

Je suis en train de créer un blog pour mon site web.
Dans celui-ci j'ai intégré la création d'article.
Puis sur la page d'accueil j'affiche tous les articles de l'utilisateur (c'est ce que fait la fonction while...)
Je m'occupe en ce moment de rajouter la fonctionnalité : supprimer l'article.
Et je n'arrive pas à supprimer un article en particulier.

Tout d'abord le code de la requête pour afficher les articles

Code :

$pseudo = $_SESSION['pseudo'];
$afficher_article = "Select `Titre_article`, `Article` from `blog`.`article` where pseudo='$pseudo'";
$resultat = mysql_query($afficher_article);
$i = 1;
while ($row = mysql_fetch_array($resultat, MYSQL_ASSOC))
{
printf ("
<div style='font-size: 16px'>
Article N°%d
</div>
<div style='font-size: 15px;'>
Titre : %s
</div>
Contenu : %s
"$_SESSION['Article'] = $row['Article'];"
<a style='text-decoration: none' href=supprimer_article.php>
<img src='images/supprimer.png'/>
Supprimer cet article
</a>",
$i, $row["Titre_article"], $row["Article"]);
$i++;
}

Et le code de la requête pour supprimer l'article :

Code :

<?php
$pseudo = $_SESSION['pseudo'];
$article = $_SESSION['Article']; // je récupère ce qu'il y'a dans l'article
$suprimer_article = "DELETE FROM `blog`.`article` where Article='$article'"; // Pour ensuite supprimer la ligne où il y'a l'article.
mysql_query($suprimer_article);
?>

Et j'ai une erreur comme ça :
Parse error: parse error in C:\Program Files\WampServer\www\Blog\Artisteer\Body\Home.php on line 26
ce qui correspond à la ligne où il y'a : "$_SESSION['Article'] = $row['Article'];"

Et si vous avez une idée du moyen par lequel je dois passer pour modifier cet article (j'utilise fckeditor)

Merci d'avance

Message édité par ozone--24 le 23-04-2009 à 12:06:14

Reply

Marsh Posté le 23-04-2009 à 12:02:52

Reply

Marsh Posté le 23-04-2009 à 12:10:14

Déjà on n'utilise surtout pas la méthode HTTP GET pour quelque chose avec de tels effets de bords.

Donc jamais jamais de liens, il faut un mini-formulaire avec méthode POST et tout.

Ensuite l'autre problème est qu'apparemment t'as pas trop pigé comment fonctionnent les sessions. Il faut passer en paramètre l'id de l'article. Donc il faut un truc du genre :

Code :

<form action="supprimer_article.php" method="post">
  <div>
    <input type="hidden" name="id_article" value="<?php echo htmlspecialchars($row['id']) ?>" />
    <input type="submit" value="Supprimer" />
  </div>
</form>

Ensuite dans supprimer_article.php faut récupérer l'id (via $_POST('id_article')), vérifier que l'article existe bien, le virer, puis rediriger vers le formulaire.

Message édité par FlorentG le 23-04-2009 à 12:11:00

Reply

Marsh Posté le 23-04-2009 à 13:22:45

pourquoi tu met des guillements autour du nom de tes variables
#
# "$_SESSION['Article'] = $row['Article'];"
apres ton algo est pas correct tu utilise le meme nom de session pour tous tes articles, donc la l'id sauvegarde sera le dernier

ensuite revoit ton printf, ton $_SESSION['Article'] = $row['Article'] n'a rien a y faire dedans

dans le meme style que FLO
tu peut utiliser un lien

Code :

$pseudo = $_SESSION['pseudo'];
$afficher_article = "Select `Titre_article`, `Article` from `blog`.`article` where pseudo='$pseudo'";
$resultat = mysql_query($afficher_article);
$i = 1;
while ($row = mysql_fetch_array($resultat, MYSQL_ASSOC))
{
printf ("
<div style='font-size: 16px'>
Article N°%d
</div>
<div style='font-size: 15px;'>
Titre : %s
</div>
Contenu : %s
",
$i, $row["Titre_article"], $row["Article"]
);
echo "<a href=\"./suprimerarticle.php?article=".$row['Article']."\" ><img src='images/supprimer.png'/>Supprimer cet article</a>";
$i++;
}
<?php
if(isset($_GET['Article'])){
$pseudo = $_SESSION['pseudo'];
$article = $_GET['Article']; // je récupère ce qu'il y'a dans l'article
$suprimer_article = "DELETE FROM `blog`.`article` where Article='$article'"; // Pour ensuite supprimer la ligne où il y'a l'article.
mysql_query($suprimer_article);
}
?>

tu met un lien pour chaque article et tu passe l'id de l'article dans le lien
et tu fais appel a ta page supprimer qui recupere l'id par le GET

bon voila tu peut optimiser pas mal, déja les double quotes avec le echo(c'est un peu moche) , puis arranger un peu le code

Message édité par infoman64 le 23-04-2009 à 13:24:21

Reply

Marsh Posté le 23-04-2009 à 14:29:50

Nan pas de lien

Message cité 2 fois

Reply

Marsh Posté le 23-04-2009 à 14:44:23

J'aurais dû annoncer que j'étais débutant dans le php.
Donc j'ai modifié mon code selon vos propositions et je vous montre un screenshot de ma page d'accueil (la partie intéressante) :

Et ce que produit l'exécution quand je clic sur Supprimer cet article (au milieu de l'image), ça me donne bien une page blanche :'( :

Et maintenant le code (que j'ai modifié )

Code :

<h3> Vos Articles déjà écrit : </h3>
<?php
require("../connexion.php" );
connec();
$pseudo = $_SESSION['pseudo'];
$afficher_article = "Select `Titre_article`, `Article` from `blog`.`article` where pseudo='$pseudo'";
$resultat = mysql_query($afficher_article);
$i = 1;
?>
<form action="supprimer_article.php" method="post">
<div>
<input type="hidden" name="id_article" value=" <?php echo htmlspecialchars($row['id']) ?>" />
<input type="submit" value="Supprimer" />
</div>
</form>
<?php
while ($row = mysql_fetch_array($resultat, MYSQL_ASSOC))
{
printf ("
<div style='font-size: 16px; font-weight: bold; '>
Article N°%d
</div>
<div style='font-size: 15px;'>
Titre : %s
</div>
Contenu : %s
",
$i, $row["Titre_article"], $row["Article"]);
echo "<a href=\"./supprimer_article.php?article=".$row['Article']." // j'ai enlevé le \"> parce que sinon ça l'affichait à côté de supprimer article (juste avant l'icône)
<img src='images/supprimer.png'/> Supprimer cet article</a>";
echo '<br/><br/><br/><br/><br/><br/>';
$i++;
}
?>

Code :

<?php
if(isset($_GET['Article']))
{
$pseudo = $_SESSION['pseudo'];
$article = $_GET['Article'];
$id_article = $_POST('id_article');
$supprimer_article = "DELETE FROM `blog`.`article` WHERE Article='$article'";
echo $supprimer_article;
mysql_query($supprimer_article);
echo mysql_error();
header("Location:index.php" );
}
?>

Message édité par ozone--24 le 23-04-2009 à 14:46:45

Reply

Marsh Posté le 23-04-2009 à 15:04:13

FlorentG a écrit :

Nan pas de lien

Oui mais j'ai pas bien saisi comment savoir quel article supprimer alors ?
Parce que si je mets pas de lien j'obtiens ça :

Et du coup je peux pas cliquer sur Supprimer Article. Je comprends pas grand chose :'(
J'ai mis le <form> et le <div> qui englobe le <?php ?> c'est bien ça qui faut faire ?

Message édité par ozone--24 le 23-04-2009 à 15:07:30

Reply

Marsh Posté le 23-04-2009 à 15:35:18

Finalement j'ai trouvé avec l'aide d'un collègue, je vous mets la réponse au cas où des personnes auraient le même problème que moi :

Code :

<?php
while ($row = mysql_fetch_array($resultat, MYSQL_ASSOC))
{
printf ("
<div style='font-size: 16px; font-weight: bold; '>
Article N°%d
</div>
<div style='font-size: 15px;'>
Titre : %s
</div>
Contenu : %s
<br/>
Id : %s
",
$i, $row["Titre_article"], $row["Article"], $row["Id"]);
?>
<form action="supprimer_article.php?id_article=<?php echo $row['Id']; ?>" method="post">
<div>
<input type="submit" value="Supprimer" />
</div>
</form>
<?php
// "<a href=\"./supprimer_article.php?article=".$row['Article'].</a>"
echo "<img src='images/supprimer.png'/> Supprimer cet article";
echo '<br/><br/><br/><br/><br/><br/>';
$i++;
}
?>

Code :

<?php
require("../connexion.php" );
connec();
if( isset($id_article) )
{
echo $id_article;
$supprimer_article = "DELETE FROM `blog`.`article` WHERE Id='$id_article'";
echo $supprimer_article;
mysql_query($supprimer_article);
echo mysql_error();
}
?>

Reply

Marsh Posté le 23-04-2009 à 15:58:34

FlorentG a écrit :

Nan pas de lien

ouais mais bon c'etait pour lui donner une réponse rapide

Reply

Marsh Posté le 23-04-2009 à 16:01:23

/me tape sur les doigts de infoman64 avec une bonne règle en fer

Message édité par ozone--24 le 23-04-2009 à 16:01:47

Reply

Problème pour supprimer article en PHP/MySQL

Sujets relatifs:

Leave a Replay