True, I guess it never hurts to try. Busy making orders and hadn't
tested just yet.
I just wanted a professional opinion on if the syntax was common to
use, mostly.
Didn't want to get a Jr PHP job and use it and have everyone looking
at me sideways.. :P
Thanks all,
Best,
Karl
On Jan 15, 2013, at 6:47 AM, Bastien wrote:
Try It And See
Bastien Koert
On 2013-01-15, at 7:43 AM, Karl DeSaulniers <karl@xxxxxxxxxxxxxxx>
wrote:
On Jan 15, 2013, at 5:31 AM, David Robley wrote:
Karl DeSaulniers wrote:
Hello Everyone,
Hope your 2013 is treating you well.
Quick question and apologies if it is a stupid question.
Is this a viable syntax?
$sql = "SELECT orderid
FROM ORDERS_TABLE
WHERE orderstatus = 'Cancelled' OR (orderstatus = ('New' OR
'Denied' OR 'Expired' OR 'Failed' OR 'Pending' OR 'Refunded' OR
'Reversed' OR 'Under Review' OR 'Voided') AND orderdate <
'".mysqli_real_escape_string($yesterday)."')";
Namely the orderstatus = (a whole bunch of options).
In my database `orderstatus` field is an enum() list btw.
Or is there a better way to check multiple options against an enum
inside your select statement?
Reason I am doing this is to avoid having to do...
$sql = "SELECT orderid
FROM ORDERS_TABLE
WHERE orderstatus = 'Cancelled' OR (orderstatus = 'New' AND
orderdate < '".mysqli_real_escape_string($yesterday)."') OR
(orderstatus = 'Denied' AND orderdate <
'".mysqli_real_escape_string($yesterday)."') OR (orderstatus =
'Expired' AND orderdate <
'".mysqli_real_escape_string($yesterday)."') ... etc";
TIA,
Karl DeSaulniers
Design Drumm
http://designdrumm.com
The best way to check if your SQL query is valid is to feed it to
mysql
which will return an error if the syntax is incorrect :-) So TIAS!
Although
it looks wrong to me.
But you might try the IN operator - something like
OR (orderstatus IN ('New', 'Denied', 'Expired', 'Failed', 'Pending',
'Refunded', 'Reversed', 'Under Review', 'Voided') AND orderdate <
'".mysqli_real_escape_string($yesterday)."')
Personally I would have used a lookup table for the order status.
--
Cheers
David Robley
A cynic smells flowers and looks for the casket.
--
PHP Database Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
Thanks for the suggestion. Your the second to offer the IN as a
solution.
I will try that out. I currently have enough tables and don't want
to create a separate table for the status.
BTW what is TIAS... havent seen that one.. lol
Best,
Karl DeSaulniers
Design Drumm
http://designdrumm.com
--
PHP Database Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
--
PHP Database Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
Karl DeSaulniers
Design Drumm
http://designdrumm.com
--
PHP Database Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php