greater than error

[jowatkins]

Hi,

I have the following bit of code that should give the row a backround colour of red if the target date has passed.

$today = gmdate ("Y/m/d", mktime (1,1,1998));
$target = $row->targetdate;
if ((!empty($target)) || ($target == "0000-00-00") || ($target == "NULL")) 
{
    if ($today > $target) 
    {
        $col = "#ff0000";
    }
}

Except it makes the line red even if the due date hasn't passed??

I don't understand it, the > is definately the right way round, which is my usual mistake?

Any ideas?

[Smifffy]

i would have something like this

$today = gmdate ("Y/m/d", mktime (1,1,1998));
$target = $row->targetdate;
if($today > $target)
{
echo "$target";
}
else
{
echo "<font color=\"#ff0000\">$target";
}

unless there some more code but thats wot ill do
im also assuming that the data is being gotten correctly

[jowatkins]

Thanks, but this doesn't fix the problem. $col should be the background colour of the row. If the target date has passed, I want to make the backround color red.

The target date for a task is brought from the database. If this date has passed, the background color should be set to red. But for some reason, the server isn't interpreting the code correctly (or I've completely misunderstood it) and it's returning all dates as overdue, even if they are in the future.

if($today > $target)

[jowatkins]

Found the problem,

$today has format 2002/10/09 while $target has format 2002-10-09. You can't compare slashes and dashes apparently. Doh!