Trouble deleting a record using PHP. Code included

Question

I Have simple PHP functions that produce html links which should delete a record, however the delete link only manages to refresh the page.

I know the solution is simple but I am new to PHP so could somebody please be kind enough to point me in the right direction? thank you. help greatly appreciated.

fuctions.php

<?php
     include('includes/connect.php'); 
  function getPosts() {
        $query = mysql_query("SELECT * FROM posts") or die(mysql_error());
        while($post = mysql_fetch_assoc($query)) {
            echo "<tr><td>" . $post['Title'] . "</td><td>" . $post['Author'] . "</td><td><a href=\"delete.php?id=" . $post['ID'] . "\">Delete</a><br /><a href=\"edit.php>?id=" . $post['ID'] . "\">Edit</a></td></tr>";
        }
    }
    function deletePost($id) {
        $id = (int) $id;
        mysql_query("DELETE FROM posts WHERE ID = '$id'") or die(mysql_error());
        header("Location: posts.php");
  }
?>

delete.php

<?php
   include('includes/functions.php');
   deletePost($_GET['ID']);
?>

you don't pass parameter called ID to your script.

Your Common Sense
– Your Common Sense

2011-01-27 14:40:43 +00:00
Commented Jan 27, 2011 at 14:40 — Your Common Sense
– Your Common Sense, Commented Jan 27, 2011 at 14:40
deletePost($_GET['id']); case sensitive

KJYe.Name
– KJYe.Name

2011-01-27 14:46:12 +00:00
Commented Jan 27, 2011 at 14:46 — KJYe.Name
– KJYe.Name, Commented Jan 27, 2011 at 14:46

Michael Irigoyen · Accepted Answer · 2011-01-27 14:43:17Z

4

In your delete.php file, you call:

deletePost($_GET['ID']);

However, in your link you use:

delete.php?id=

It is a case issue, make them both be upper case ID or lower case id.

answered Jan 27, 2011 at 14:43

Michael Irigoyen

23.1k18 gold badges92 silver badges132 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Kristoffer Sall-Storgaard · Accepted Answer · 2011-01-27 14:49:57Z

0

Check your casing.

You are setting the GET paramets name to id while checking for ID

answered Jan 27, 2011 at 14:49

Kristoffer Sall-Storgaard

10.7k5 gold badges39 silver badges46 bronze badges

Comments

Oleg · Accepted Answer · 2011-01-27 15:27:26Z

0

Print your sql delete command before executing and it I suppose will answer your question.

answered Jan 27, 2011 at 15:27

Oleg

2,8318 gold badges40 silver badges63 bronze badges

Collectives™ on Stack Overflow

Trouble deleting a record using PHP. Code included

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related