Deleting blog entry

[DM3Dan]

I have a link going to 'deletepost.php' from my admin page. There is a link such as this:
example.com/deletepost.php?postID=8
However, when I click that link, I always get my 'Deletion Failed' message. Any ideas?

Deletepost.php:

<?php
include 'top.php';

$postID = $_GET['postID']; 
$postID = intval($_GET['postid']);
$delete = "DELETE * FROM posttbl WHERE postID=$postID";


$dbcnx = mysql_connect("localhost", "****", "****"); 
mysql_select_db("****");

    if(!isset($_SESSION['authorized']))
    {
        header('Location: login.php');
    }

$delete = mysql_query($delete);

    if($delete)
    {
        header('Location: index.php');
    }

    else
    {
        exit('Deletion failed');
    }
    echo mysql_error();

?>

[gavwvin]

after this line:

$delete = mysql_query($delete); 

put this line

echo mysql_error(); 

That should tell you the problem with your query

[klik]

Shouldn't

$postID = intval($_GET['postid']);

Be

$postID = intval($_GET['postID']);

[DM3Dan]

after this line:

$delete = mysql_query($delete); 

put this line

echo mysql_error(); 

That should tell you the problem with your query

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 '* FROM posttbl WHERE postID=0' at line 1Deletion failed

AKA, postID isn't being carried from my link. I geuss I can't just put ?postID=8 at the end of my URL and expect it to carry that over as postID

[Dylan B]

$delete = "DELETE FROM posttbl WHERE postID=$postID";

You don't need the wildcard for deletion queries, I don't think.

[DM3Dan]

Ok, now it just redirects me to index.php, without deleting the post.

Here is my admin page(Well, the part that relates to this).

<?php
$allposts = "SELECT * FROM posttbl";

$allposts = mysql_query($allposts);
echo '<table>';

while($row = mysql_fetch_array($allposts))
{
$ID = $row['postID'];
echo<<<EOV
<tr style="font-size:80%"><td style="padding:5px">
<a href="viewposts.php?postID={$ID}">{$row['postTITLE']}</a></td>
<td><a href="editpost.php?postID={$ID}"> Edit</a>/
<a href="deletepost.php?postID={$ID}"> Delete</tr>
EOV;
}
?>

[gavwvin]

Theres your problem:

$postID = intval($_GET['postid']); 

should be

$postID = intval($_GET['postID']); 

[DM3Dan]

GAH!

Nice catch, it works now .

[Young Twig]

$postID = $_GET['postID']; 
$postID = intval($_GET['postid']); 

Why, Daniel, why?

[DM3Dan]

I fixed it now. And you coded that part in advisory, retard.

[klik]

Isn't that what I suggested?

[Young Twig]

I fixed it now. And you coded that part in advisory, retard.

I did not write that.

Edit:

Oh, and Dan... let this be a lesson: capital letters suck.

[Dylan B]

Off Topic:


Mr. Hehir, where have you been lately?

[Young Twig]

MSN's off, huh? I'll get on.

[DM3Dan]

Edit:

Oh, and Dan... let this be a lesson: capital letters suck.

Not me... Dav. I use lowers, except in functions.