Re: Corn job anomaly
| From: | Karl DeSaulniers | Date: | Tue, 27 Sep 2016 07:37:44 +0000 |
| Subject: | Re: Corn job anomaly | ||
| References: | 1 2 3 | Groups: | php.db |
| Request: | Send a blank email to php-db+get-49147@lists.php.net to get a copy of this message | ||
> On Sep 27, 2016, at 2:24 AM, Ford, Mike <M.Ford@leedsbeckett.ac.uk> wrote:
>
>> -----Original Message-----
>> From: Karl DeSaulniers [mailto:karl@designdrumm.com]
>> Sent: 25 September 2016 09:59
>> To: PHP List Database <php-db@lists.php.net>
>> Subject: Re: [PHP-DB] Corn job anomaly
>>
>> Now I am getting an error with mysql syntax.
>>
>> "SELECT otn.*, cf.* FROM ".ORDER_TABLE." otn LEFT JOIN
>> ".FIELDS_TABLE." cf
>> ON cf.Order_ID = otn.Order_ID WHERE cf.Earliest_Pickup >= DATE(NOW()) AND
>> cf.Earliest_Pickup <= DATE(NOW() + INTERVAL ".($Num_Days_Away+1)." DAY) AND
>> otn.Order_Status != 'Shipping' AND otn.Order_Status !=
>> 'Completed'"
>>
>> is giving me this error:
>> You have an error in your SQL syntax; check the manual that
>> corresponds to your MySQL server version for the right syntax to use near
>> '\"Shipping\" AND otn.Order_Status != \"Completed\"' at line
>> 1
>
> I don't think INTERVAL works like that - you probably need something like:
>
> ... cf.Earliest_Pickup <= DATE_ADD(CURDATE(), INTERVAL ".($Num_Days_Away+1)."
> DAY) AND ...
>
> Incidentally, as I understand it CURDATE() does exactly the same as DATE(NOW()), and is
> probably more readable.
>
> Cheers!
>
> Mike
>
> --
> Mike Ford,
Hi Mike,
I figured out what was wrong with my script on that error.
I had mysqli_real_escape_string() wrapping the whole select statement.
The INTERVAL part I got from PHP.net and it is working just fine. Just fyi.
Once I wrapped just ($Num_Days_Away+1) part it started working.
Thanks for responding!
Best,
Karl DeSaulniers
Design Drumm
http://designdrumm.com