Re: [Phpgroupware-users] Autocreation of account, some problem. Seems solved. ;)

[Chris Weiss]

>I've looked at that file, that line, and tried:
>
>       $this->db->query("SELECT account_type FROM phpgw_accounts WHERE
>               account_id='".$account_id."'",__LINE__,__FILE__);
>        //$this->db->query("SELECT account_type FROM
>               phpgw_accounts WHERE 
> account_id=".$account_id,__LINE__,__FILE__);
>
>and works. ;-)

account_id is an int, why would it need quoted in the SQL?  The real problem is
that $account_id is empty and not numeric.  So the real fix should be something
more like:

        if (!is_numeric($account_id))
        {
                $account_id = 0;
        }
        $this->db->query("SELECT account_type FROM
                phpgw_accounts WHERE 
account_id=".$account_id,__LINE__,__FILE__);