cleanup

1 files changed, 51 insertions, 31 deletions

diff --git a/mod/group.php b/mod/group.php
index 8733614dd..75fd0f995 100644
--- a/mod/group.php
+++ b/mod/group.php
@@ -55,28 +55,7 @@ function group_post(&$a) {
 			if($r)
 				notice( t('Group name changed.') . EOL );
 		}
-		$members = $_POST['group_members_select'];
-		if(is_array($members))
-			array_walk($members,'validate_members');
-		$r = q("DELETE FROM `group_member` WHERE `gid` = %d AND `uid` = %d",
-			intval($a->argv[1]),
-			intval(local_user())
-		);
-		$result = true;
-		if(is_array($members) && count($members)) {
-			foreach($members as $member) {
-				$r = q("INSERT INTO `group_member` ( `uid`, `gid`, `contact-id`)
-					VALUES ( %d, %d, %d )",
-					intval(local_user()),
-					intval($group['id']),
-					intval($member)
-				);
-				if(! $r)
-					$result = false;
-			}
-		}
-		if($result)
-			notice( t('Membership list updated.') . EOL);
+
 		$a->page['aside'] = group_side();
 	}
 	return;	
@@ -116,8 +95,17 @@ function group_content(&$a) {
 		return; // NOTREACHED
 	}
 
+	if(($a->argc > 2) && intval($a->argv[1]) && intval($a->argv[2])) {
+		$r = q("SELECT `id` FROM `contact` WHERE `id` = %d AND `uid` = %d and `self` = 0 and `blocked` = 0 AND `pending` = 0 LIMIT 1",
+			intval($a->argv[2]),
+			intval(local_user())
+		);
+		if(count($r))
+			$change = intval($a->argv[2]);
+	}
+
+	if(($a->argc > 1) && (intval($a->argv[1]))) {
 
-	if(($a->argc == 2) && (intval($a->argv[1]))) {
 		require_once('include/acl_selectors.php');
 		$r = q("SELECT * FROM `group` WHERE `id` = %d AND `uid` = %d LIMIT 1",
 			intval($a->argv[1]),
@@ -135,6 +123,23 @@ function group_content(&$a) {
 				$preselected[] = $member['id'];
 		}
 
+		if($change) {
+			if(in_array($change,$preselected)) {
+				group_rmv_member(local_user(),$group['name'],$change);
+			}
+			else {
+				group_add_member(local_user(),$group['name'],$change);
+			}
+
+			$members = group_get_members($group['id']);
+			$preselected = array();
+			if(count($members))	{
+				foreach($members as $member)
+					$preselected[] = $member['id'];
+			}
+		}
+
+
 		$drop_tpl = load_view_file('view/group_drop.tpl');
 		$drop_txt = replace_macros($drop_tpl, array(
 			'$id' => $group['id'],
@@ -148,20 +153,27 @@ function group_content(&$a) {
 			'$gid' => $group['id'],
 			'$name' => $group['name'],
 			'$drop' => $drop_txt,
-			'$selector' => contact_select('group_members_select','group_members_select',$preselected,25,false,$celeb),
+			'$desc' => t('Click on a contact to add or remove.'),
 			'$title' => t('Group Editor'),
 			'$gname' => t('Group Name: '),
-			'$members' => t('Members:'),
 			'$submit' => t('Submit')
 		));
 
 	}
 
+	$o .= '<div id="group-update-wrapper">';
+	if($change) 
+		$o = '';
+
 	$o .= '<div id="group-members">';
-	$o .= '<h3>' . t('In Group') . '</h3>';
+	$o .= '<h3>' . t('Members') . '</h3>';
 	foreach($members as $member) {
-		$member['click'] = 'groupMember(' . $member['id'] . '); return true;';
-		$o .= micropro($member,true,'mpgroup');
+		if($member['url']) {
+			$member['click'] = 'groupChangeMember(' . $group['id'] . ',' . $member['id'] . '); return true;';
+			$o .= micropro($member,true,'mpgroup');
+		}
+		else
+			group_rmv_member(local_user(),$group['name'],$member['id']);
 	}
 
 	$o .= '</div><div id="group-members-end"></div>';
@@ -175,13 +187,21 @@ function group_content(&$a) {
 
 		if(count($r)) {
 			foreach($r as $member) {
-				$member['click'] = 'groupMember(' . $member['id'] . '); return true;';
-				$o .= micropro($member,true,'mpall');
+				if(! in_array($member['id'],$preselected)) {
+					$member['click'] = 'groupChangeMember(' . $group['id'] . ',' . $member['id'] . '); return true;';
+					$o .= micropro($member,true,'mpall');
+				}
 			}
 		}
 
 		$o .= '</div><div id="group-all-contacts-end"></div>';
 
+	if($change) {
+		echo $o;
+		killme();
+	}
+	$o .= '</div>';
 	return $o;
 
-}
\ No newline at end of file
+}
+