From 045958ccd73ec673b4e337b33d919e13af2aa25f Mon Sep 17 00:00:00 2001 From: Antoine Musso Date: Fri, 1 Oct 2004 16:12:10 +0000 Subject: [PATCH] * Remove debug statements * Fix bug when $removegroup is empty (cleared everything) --- includes/SpecialUserlevels.php | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/includes/SpecialUserlevels.php b/includes/SpecialUserlevels.php index 071acbced5..f7f68b3f0b 100644 --- a/includes/SpecialUserlevels.php +++ b/includes/SpecialUserlevels.php @@ -11,7 +11,7 @@ require_once('Group.php'); /** Entry point */ function wfSpecialUserlevels($par=null) { global $wgRequest; - // print_r($_POST); +// print_r($_POST); $form = new UserlevelsForm($wgRequest); $form->execute(); } @@ -99,12 +99,7 @@ class UserlevelsForm extends HTMLForm { */ function saveUserGroups($username,$removegroup,$addgroup) { $u = User::NewFromName($username); - - print "ADD:\n"; - print_r($addgroup); - print "DEL:\n"; - print_r($removegroup); - + if(is_null($u)) { $wgOut->addHTML('

'.wfMsg('nosuchusershort',$username).'

'); return; @@ -118,8 +113,8 @@ class UserlevelsForm extends HTMLForm { $groups = $u->getGroups(); // remove then add groups - $groups = array_diff($groups, $removegroup); - $groups = array_merge($groups, $addgroup); + if(isset($removegroup)) { $groups = array_diff($groups, $removegroup); } + if(isset($addgroup)) { $groups = array_merge($groups, $addgroup); } // save groups in user object and database $u->setGroups($groups); $u->saveSettings(); -- 2.20.1