CGI fault

jrmg

I have a script I want to set a cron job against to act as a CGI - It works like this (this is only part of the page):

/ Check for clients at end of 14 day trial /
$query = <<END_QUERY
select first_name, last_name, username, password, email
from clients
where (TO_DAYS(now())-TO_DAYS(date_created))='$trial_days'
END_QUERY;
$result = mysql_query($query)
or die(mysql_error());

But I want to specify that it should ONLY effect clients who's 'property_num' field is less than five in the MySQL DB. So I changed it to this:

/ Check for clients at end of 14 day trial /
$query = <<END_QUERY
select first_name, last_name, username, password, email, property_num
from clients
where (TO_DAYS(now())-TO_DAYS(date_created))='$trial_days' AND property_num < 5"
END_QUERY;
$result = mysql_query($query)
or die(mysql_error());

However, now it doesn't work? Any thoughts as to what could be wrong? The full code is attached if required. Thanks.

neiljmac

Ah. Think I have just spotted the problem.

You need to set $query = <<<END_QUERY rather than $query = <<END_QUERY

Perl uses << PHP uses <<<

See if that does the trick.

Neil.

jrmg

Thanks, but <<< doesn't work, however I have discovered that if I open this code in Dreamweaver MX, as soon as I make a change it stuffs it up and I have to copy my backup version? I am now using a text editor and have found that the error I am getting with the revised code (with the property_num changes) is:

You have an error in your SQL syntax near 'from clients where (TO_DAYS(now())-TO_DAYS(date_created))='14' AND property_num' at line 2

So I must be pretty close but I still can't lock it down - AHHH! What do you think (almost there).

hand

<?
$query = <<<END_QUERY
select first_name, last_name, username, password, email, property_num 
from clients
where (TO_DAYS(now())-TO_DAYS(date_created))='$trial_days' AND property_num < 5
END_QUERY;

// or 

$query = "
select first_name, last_name, username, password, email, property_num 
from clients
where (TO_DAYS(now())-TO_DAYS(date_created))='$trial_days' AND property_num < 5
";
?>

... AND property_num < 5
instead of
... AND property_num < 5"

jrmg

Thanks to all for your help.