On Tue, Jun 3, 2008 at 10:30 AM, Maât <[EMAIL PROTECTED]> wrote: > Chris Weiss a écrit : >> >> um, this does exactly nothing. if you added ";" I might see it, but >> this is effectively pointless. >> >> > > ||CW : 1 - 0 Maât > > :) > > as there is an (int) before $this->account_id the single quotes is not > needed for security reasons. > > though, if i'm not mistaken, acl_location is a string from the db point of > view... perhaps required by some db (pg ?) >
if it is a char it should have quotes, but if it is always a number then it should not be a char. if it is not always a number then it should not have the (int) cast. if it is not a char then it should not have the quotes, the quotes will cause an unessicary cast on the db side and some db's will also complain. _______________________________________________ phpGroupWare-developers mailing list phpGroupWare-developers@gnu.org http://lists.gnu.org/mailman/listinfo/phpgroupware-developers
