- simplyfied code by using joinGroup() and leaveGroup()

This commit is contained in:
steinm 2011-10-10 14:10:16 +00:00
parent f055bda7a3
commit 7990a1caef

View File

@ -44,23 +44,13 @@ if($ismanager) {
$curuser = $dms->getUser($_REQUEST['userid']); $curuser = $dms->getUser($_REQUEST['userid']);
$members = $group->getUsers(); $members = $group->getUsers();
// Check if user is alread in group
$ismember = false;
foreach($members as $member) {
if($member->getId() == $curuser->getId())
$ismember = true;
}
// Add user to group // Add user to group
if ($_REQUEST['action'] == "add") { if ($_REQUEST['action'] == "add") {
if(!$ismember) { $curuser->joinGroup($group);
$group->addUser($curuser);
}
} }
// Delete user from group // Delete user from group
elseif($_REQUEST['action'] == 'del') { elseif($_REQUEST['action'] == 'del') {
if($ismember) { $curuser->leaveGroup($group);
$group->removeUser($curuser);
}
} }
} else { } else {
UI::exitError(getMLText("admin_tools"),getMLText("access_denied")); UI::exitError(getMLText("admin_tools"),getMLText("access_denied"));