extern/egroupware
1
0
Fork 1
mirror of https://github.com/EGroupware/egroupware.git synced 2024-12-27 09:09:04 +01:00
Code Issues Packages Projects Releases Wiki Activity

accounts_sql::members() use single sql query with join to acl table instead calling id2name for each account, as it is quicker and uses less memory

Browse Source
This commit is contained in:
Ralf Becker 2012-07-16 12:27:01 +00:00
parent 04f732e653
commit dea7e46a2d
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
phpgwapi/inc/class.accounts_sql.inc.php
View File

@ -274,14 +274,18 @@ class accounts_sql
*/
function members($account_id)
{
if (!($uids = $GLOBALS['egw']->acl->get_ids_for_location($account_id, 1, 'phpgw_group')))
{
return array();
}
if (!is_numeric($account_id)) $account_id = $this-name2id($account_id);
$members = array();
foreach ($uids as $uid)
foreach($this->db->select($this->table, 'account_id,account_lid',
$this->db->expression(acl::TABLE, array(
'acl_appname' => 'phpgw_group',
'acl_location' => $account_id,
)), __LINE__, __FILE__, false,
'JOIN '.acl::TABLE.' ON account_id=acl_owner'
) as $row)
{
$members[$uid] = $this->id2name($uid);
$members[$row['account_id']] = $row['account_lid'];
}
//echo "accounts::members($accountid)"; _debug_array($members);
return $members;