This DELETE code fails, any ideas?

Anon

Hi, this code just seems to do nothing.

It's proberably something really simple. Can somebody tell me what I have done wrong.

<? //DELETE OLD EVENTS
//Connect
include "variables.php";

$datetoday = date("Y-m-d");
$query = "DELETE * FROM tblEvents WHERE DateEnd<$datetoday";
mysql_query($query);
$result = mysql_query($query);
if ($result) {
echo "Worked";
}
?>

Thanks

junebug

leave out the *

$query = "delete form tblEvents where DateEnd < $datetoday";

junebug

sorry that should be from:

$query = "delete from tblEvents where DateEnd < $datetoday";

Anon

The date in DateEnd I am trying to delete is 2001-01-01 but the code just doesn't seem to do anything.

Still not working with * removed.

junebug

here try this:

$query = "DELETE FROM tblEvents WHERE DateEnd < '$datetoday'";

just add in the single quotes

Anon

Nope that hasn't impoved it.

Any other ideas.

Cheers for the help by the way.

Anon

What database are you using? Doing date comparisions is tricky sometimes. You may have to convert your $datetoday inside the SQL command to become a date before your < will work properly.

If you are using MySQL you will probably have to use the CURRDATE() or SYSDATE() function to get today's date.

Anon

Check your data type. I once had trouble when I defined a column as TEXT instead of INT.

I then wondered why I couldn't use the numeric values stored in the text column. But php didn't recognize the values as integers until I modified the column.

Richie.

http://sabyo.dyns.cx/

Anon

I get....

Fatal error: Call to undefined function: currdate()

I am new to PHP so how do I use CURRDATE() the same as date()?

kailien

$sql = "delete from tblEvents where DateEnd < now()";

also, make sure DateEnd is a date field!

Kai

Anon

Data type is date.

Anon

CURRDATE() is a MySQL function, it returns the current system date, which is what I thought your $datetoday was holding. So instead of haveing DateEnd < $datetoday use DateEnd < CURRDATE()

Anon

Oops it's CURDATE()

Anon

I still get....

Fatal error: Call to undefined function: curdate()

Now using this code

<? //DELETE OLD EVENTS
include "variables.php";
$datetoday = date("Y-m-d");
$datetoday = CURDATE();
print ($datetoday);
$query = "DELETE FROM tblEvents WHERE DateEnd < $datetoday";
// $query = "DELETE FROM tblEvents WHERE DateEnd < now()";
mysql_query($query);
$result = mysql_query($query);
if ($result) {
print ("Worked");
}
else
{
print ("Didn't Work");
}
?>

Some development code in there

Anon

Try this and check whether it works.
<? //DELETE OLD EVENTS
//Connect
include "variables.php";
$datetoday = date("Y-m-d");
$query = "DELETE * FROM tblEvents WHERE DateEnd<$datetoday";
//mysql_query($query);
$result = mysql_query($query);
if ($result) {
echo "Worked";
}
?>

Anon

<? //DELETE OLD EVENTS
include "variables.php";
$datetoday = date("Y-m-d");
// $datetoday = CURDATE();
print ($datetoday);
// $query = "DELETE FROM tblEvents WHERE DateEnd < $datetoday";
$query = "DELETE FROM tblEvents WHERE DateEnd < now()";
mysql_query($query);
$result = mysql_query($query);
?>

Anon

Nothing happened.

Anon

That's cause you aren't understanding me. CURDATE() is not a PHP function. It is a MySQL function so it has to be inside your $query variable.

<b>$query = "DELETE FROM tblEvents WHERE DateEnd < CURDATE()";</b>

Anon

Try this
<? //DELETE OLD EVENTS
//Connect
include "variables.php";
$datetoday = date("Y-m-d");
$query = "DELETE FROM tblEvents WHERE DateEnd<$datetoday";
//mysql_query($query);
$result = mysql_query($query);
if ($result) {
echo "Worked";
}
?>

toma42

//get today
$today = now();
$sql = "delete from table where unix_timestamp(somedatetime) < $today";