extern/egroupware_official
1
0
Fork 0
mirror of https://github.com/EGroupware/egroupware.git synced 2025-02-24 14:22:45 +01:00
Code Issues Packages Projects Releases Wiki Activity

bugfix: set_memberships removed the not longer set memberships from $this->account_id and not the given one (which caused the admin to loose the membership)

Browse Source
This commit is contained in:
Ralf Becker 2006-06-11 14:27:14 +00:00
parent c942df179d
commit 287726dead
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
phpgwapi/inc/class.accounts_ldap.inc.php
View File

@ -715,21 +715,21 @@ class accounts_backend
* Sets the memberships of the given account
*
* @param array $groups array with gidnumbers
* @param int $accountid uidnumber
* @param int $account_id uidnumber
*/
function set_memberships($groups,$accountid)
function set_memberships($groups,$account_id)
{
//echo "<p>accounts_ldap::set_memberships(".print_r($groups,true).",$accountid)</p>\n";
//echo "<p>accounts_ldap::set_memberships(".print_r($groups,true).",$account_id)</p>\n";
// remove not longer existing memberships
if (($old_memberships = $this->memberships($accountid)))
if (($old_memberships = $this->memberships($account_id)))
{
$old_memberships = array_keys($old_memberships);
foreach(array_diff($old_memberships,$groups) as $gid)
{
if (($members = $this->members($gid)))
{
unset($members[$this->account_id]);
unset($members[$account_id]);
$this->set_members($members,$gid);
}
}
@ -738,7 +738,7 @@ class accounts_backend
foreach($old_memberships ? array_diff($groups,$old_memberships) : $groups as $gid)
{
$members = $this->members($gid);
$members[$accountid] = $this->id2name($accountid);
$members[$account_id] = $this->id2name($account_id);
$this->set_members($members,$gid);
}
}