Delete Function Not Working

[computermax2328]

Hello Again,

 

I think I need a second set of eyes on this one. I did a couple of tests and the $id is passing as a hidden input through the form. I think there is something wrong with my delete query, but I am not really sure. The if statements are working at the end of the page, but the delete query is not happening. Take a look?

 

<?php

require("../includes/connection.php");

require("../includes/sessioncheck.php");

?>

    <?php

        $yes = $_POST['delyes'];

        $no = $_POST['delno'];

        $id = $_POST['hiddenid'];

        $select = "SELECT * FROM `database` WHERE id=$id";

        if (!$select) {

            die ("Invaild Query: " . mysql_error());

        }

        $result = mysql_query($select);

        if (!$result) {

            echo ("Database Select Failed " . mysql_error());

        }

        $row = mysql_fetch_array($result);

        if (!$row) {

            die ("Invaild Fetch: " . mysql_error());

        }

            $a = $row['a'];

            $b = $row['b'];

            $c = $row['c'];

            $d = $row['d'];

            $e = $row['e'];

            $f = $row['f'];

            $g = $row['g'];

            $h = $row['h'];

            $i = $row['i'];

            $j = $row['j'];

            $k = $row['k'];

            $l = $row['l'];

            $m = $row['m'];

            $delid = $row['id'];

        ?>

 

<?php

$query = "DELETE FROM `database` WHERE `a`='$a',`b`='$b',`c`='$c',`d`='$d',`e`='$e',`f`='$f',`g`='$g',`h`='$h',`i`='$i',`j`='$j',`k`='$k',`l`='$l',`m`='$m',`id`='$delid'";

if ($yes) {

    mysql_query($query,$connection);

        header('Location: ../pages/data_DB.php');

}

else {

    echo ("Database Delete Failed " . mysql_error());

}

if (empty($yes)) {

    header('Location: ../pages/data_delete_list.php');

    exit;

}

?>

[marcus]

When dealing with multiple where clauses you need to use AND and/or OR to separate each clause.

 

$sql = "DELETE FROM `x` WHERE `y`='z' AND `z`='y'";

[ZulfadlyAshBurn]

ps computermax2328m, do remember to place your codes in the code tags. It kinds of look "ugly" without it.

 

Try out what mgallforever said

[computermax2328]

Thanks for taking a look AND thanks for the code tag tip. You see what I did there???? AND. I love code jokes.

 

Anyway!  I got something like this now.

 

$query = "DELETE FROM `database` WHERE `a`='$a' AND `b`='$b' AND `c`='$c'";

 

AND I am not getting anything. Am I executing the query incorrectly?

 

Thanks again,

[ZulfadlyAshBurn]

Oh LOL! Please repost your current code. Might be a syntax error etc...

[computermax2328]

All of my code is posted above. I just added the ANDs in between the different delete values.

[ZulfadlyAshBurn]

Turn on error reporting and if there's any error, post it here.

 

this should be placed before any other php codes.

<?php
error_reporting(E_ALL);
ini_set('display_errors', '1');
?>

[PFMaBiSmAd]

The main point of using an id (identifier) for data is so that you can reference that data by its id. You should not need to use more than WHERE id=$id in the delete query.

 

One possible reason your exiting query is not working is that some of the data values you are putting back into the WHERE clause may contain sql special characters and they are breaking the sql syntax. If this is the case, you would need to escape all that data before putting it into the where clause. It will be much simpler to just use the id in the where clause.

[insidus]

You should also write

exit;

As

exit();