From 2a96f2531b6ce8388ee0c63a335d6c48c5ed5a66 Mon Sep 17 00:00:00 2001
From: marcus <marcus@36083f99-b078-4883-b0ff-0f9b5a30f544>
Date: Thu, 26 Feb 2009 15:29:51 +0000
Subject: Closes #801, Refs #277: Group join/request logic altered.

git-svn-id: https://code.elgg.org/elgg/trunk@2965 36083f99-b078-4883-b0ff-0f9b5a30f544
---
 mod/groups/actions/addtogroup.php | 118 ++++++++++++++++----------------------
 1 file changed, 49 insertions(+), 69 deletions(-)

(limited to 'mod/groups/actions/addtogroup.php')

diff --git a/mod/groups/actions/addtogroup.php b/mod/groups/actions/addtogroup.php
index 0928c80c9..a34f5d48c 100644
--- a/mod/groups/actions/addtogroup.php
+++ b/mod/groups/actions/addtogroup.php
@@ -15,95 +15,75 @@
 	
 	gatekeeper();
 	
-	$forward_url = get_input('forward_url', $_SERVER['HTTP_REFERER']);
 	$user_guid = get_input('user_guid');
 	if (!is_array($user_guid))
 		$user_guid = array($user_guid);
 	$group_guid = get_input('group_guid');
 	
 	if (sizeof($user_guid))
-	foreach ($user_guid as $u_id)
 	{
-		$requested = false;
-		
-		$user = get_entity($u_id);
-		$group = get_entity($group_guid);
-		
-		if ( $user && $group) {
+		foreach ($user_guid as $u_id)
+		{
+			$user = get_entity($u_id);
+			$group = get_entity($group_guid);
 			
-			if ($_SESSION['user']->getGUID() == $group->owner_guid)
-			{
-				$requests = $user->group_join_request;
+			if ( $user && $group) {
 				
-				if ($requests)
+				if (get_loggedin_userid() == $group->owner_guid)
 				{
-					foreach ($requests as $request) 
+					
+					// If the group is open or the user has requested membership
+					if (
+						(check_entity_relationship($user->guid, 'membership_request', $group->guid)) ||
+						($group->isPublicMembership())
+						)
 					{
-						if ($request == $group->getGUID())
+						
+						if (!$group->isMember($user))
 						{
-							
-							// User has requested to join this group previously, so we can safely add them
-		
-							// add them
-							if ((!$group->isMember($user)) && ($group->join($user)))
-							{
-								
-								// send welcome email
-								notify_user($user->getGUID(), $group->owner_guid, 
-									sprintf(elgg_echo('groups:welcome:subject'), $group->name), 
-									sprintf(elgg_echo('groups:welcome:body'), $user->name, $group->name, $group->getURL()),
-									NULL, "email");
-									
-								system_message(elgg_echo('groups:addedtogroup'));
+							// Remove relationships
+							remove_entity_relationship($group->guid, 'invited', $user->guid);
+							remove_entity_relationship($user->guid, 'membership_request', $group->guid);
+						
+							// send welcome email
+							notify_user($user->getGUID(), $group->owner_guid, 
+								sprintf(elgg_echo('groups:welcome:subject'), $group->name), 
+								sprintf(elgg_echo('groups:welcome:body'), $user->name, $group->name, $group->getURL()),
+								NULL, "email");
 								
-							}
-							else
-								register_error(elgg_echo("groups:cantjoin"));
-							
-							$requested = true;
+							system_message(elgg_echo('groups:addedtogroup'));
 						}
-					}
-				}
-					
-				if (!$requested) 
-				{
-					// Not found in request array, so send an invite and set invite flag
-					$methods = $user->group_invite;
-					if (($methods) && (!is_array($methods)))
-						$methods = array($methods);
-					if (!$methods) $methods=array();
-					$methods[] = $group->getGUID();
-					$methods = array_unique($methods);
-					
-					$logged_in_user = get_loggedin_user();
-					
-					// Set invite flag
-					//if (!$user->setMetaData('group_invite', $group->getGUID(), "", true))
-					if (
-						(!$user->setMetaData('group_invite', $methods)) ||
-						(!$user->isFriend())
-					) 
-					{
-					//if (!$user->group_invite = $methods) { 
-						register_error(elgg_echo("groups:usernotinvited"));
+						else
+							register_error(elgg_echo("groups:cantjoin"));
 					}
 					else
 					{
-						// Send email
-						if (notify_user($user->getGUID(), $group->owner_guid, 
-								sprintf(elgg_echo('groups:invite:subject'), $user->name, $group->name), 
-								sprintf(elgg_echo('groups:invite:body'), $user->name, $group->name, "{$CONFIG->url}action/groups/join?user_guid={$user->guid}&group_guid={$group->guid}"),
-								NULL, "email"))
-							system_message(elgg_echo("groups:userinvited"));
+						if ($user->isFriend())
+						{
+							
+							// Create relationship
+							add_entity_relationship($group->guid, 'invited', $user->guid);
+							
+							// Send email
+							if (notify_user($user->getGUID(), $group->owner_guid, 
+									sprintf(elgg_echo('groups:invite:subject'), $user->name, $group->name), 
+									sprintf(elgg_echo('groups:invite:body'), $user->name, $group->name, "{$CONFIG->url}action/groups/join?user_guid={$user->guid}&group_guid={$group->guid}"),
+									NULL, "email"))
+								system_message(elgg_echo("groups:userinvited"));
+							else
+								register_error(elgg_echo("groups:usernotinvited"));
+							
+						}
 						else
 							register_error(elgg_echo("groups:usernotinvited"));
 					}
-				}
-			}
-			else
-				register_error(elgg_echo("groups:notowner"));
+				}
+				else
+					register_error(elgg_echo("groups:notowner"));
+			}
 		}
 	}
-			
-	forward($forward_url);
+	
+	forward($_SERVER['HTTP_REFERER']);
+	
 ?>
\ No newline at end of file
-- 
cgit v1.2.3