From: Antoine Musso <hashar@users.mediawiki.org>
Date: Fri, 1 Oct 2004 16:12:10 +0000 (+0000)
Subject: * Remove debug statements
X-Git-Tag: 1.5.0alpha1~1708
X-Git-Url: http://git.cyclocoop.org/%40spipnet%40?a=commitdiff_plain;h=045958ccd73ec673b4e337b33d919e13af2aa25f;p=lhc%2Fweb%2Fwiklou.git

* Remove debug statements
* Fix bug when $removegroup is empty (cleared everything)
---

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('<p>'.wfMsg('nosuchusershort',$username).'</p>');
 			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();