From 919d4475aa62cea12f53cad4711ed72f6621d7e6 Mon Sep 17 00:00:00 2001
From: nbm <nbm@c91229c3-7414-0410-bfa2-8a42b809f60b>
Date: Thu, 21 Jul 2005 15:04:22 +0000
Subject: [PATCH] Use Group->addMember instead of creating GroupUserLink object.

---
 presentation/lookAndFeel/knowledgeTree/administration/usermanagement/addUserBL.php | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/addUserBL.php b/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/addUserBL.php
index b9a69e5..712dbe0 100644
--- a/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/addUserBL.php
+++ b/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/addUserBL.php
@@ -125,14 +125,18 @@ if (checkSession()) {
         if($oUser->create()) {
         	// now add the user to the initial group
         	$default->log->info("adding user id " . $oUser->getID() . " to group id $fGroupID"); 
-        	$oUserGroup = new GroupUserLink($fGroupID,$oUser->getID());
-        	if ($oUserGroup->create()) {
-        		// FIXME: update search permissions for the group
-        		// redirect to list page
-        		controllerRedirect("listUsers");
-        	} else {
+            $oGroup = Group::get($fGroupID);
+            if (!PEAR::isError($oGroup)) {
+                if ($oGroup->addMember($oUser)) {
+                    // FIXME: update search permissions for the group
+                    // redirect to list page
+                    exit(controllerRedirect("listUsers"));
+                } else {
+                    $oPatternCustom->setHtml(getPageGroupFail());
+                }
+            } else {
         		$oPatternCustom->setHtml(getPageGroupFail());
-        	}
+            }
         } else {
             $oPatternCustom->setHtml(getPageFail());
         }
--
libgit2 0.21.4