From 7516d413e088d8a85e6313599c3f8a5192349654 Mon Sep 17 00:00:00 2001
From: nbm <nbm@c91229c3-7414-0410-bfa2-8a42b809f60b>
Date: Thu, 21 Jul 2005 15:26:38 +0000
Subject: [PATCH] Use Group addMember and removeMember rather than mess directly with link objects.

---
 presentation/lookAndFeel/knowledgeTree/administration/usermanagement/editUserGroupsBL.php | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/editUserGroupsBL.php b/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/editUserGroupsBL.php
index 62fe917..1177ab6 100644
--- a/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/editUserGroupsBL.php
+++ b/presentation/lookAndFeel/knowledgeTree/administration/usermanagement/editUserGroupsBL.php
@@ -36,24 +36,20 @@ KTUtil::extractGPC('fAssign', 'fUserID', 'fUserSet', 'groupAddedLeft', 'groupAdd
  */
 function updateGroups($iUserID, $aToAddIDs, $aToRemoveIDs) {
 
-	// Add groups
+    $oUser = User::get($iUserID);
+
 	foreach ($aToAddIDs as $iGroupID ) {
 		if ($iGroupID > 0) {
-			$oUserGroup = new GroupUserLink($iGroupID, $iUserID);
-			if (!$oUserGroup->create()) {
-           		return false;
-       		}
+            $oGroup = Group::get($iGroupID);
+            $oGroup->addMember($oUser);
 		}
 	}
 
 	// Remove groups
 	foreach ($aToRemoveIDs as $iGroupID ) {
 		if ($iGroupID > 0) {
-			$oUserGroup = new GroupUserLink($iGroupID, $iUserID);
-			$oUserGroup->setUserGroupID($iGroupID,$iUserID);
-        	if(!$oUserGroup->delete()) {
-	            return false;
-   		    }
+            $oGroup = Group::get($iGroupID);
+            $oGroup->removeMember($oUser);
 		}
 	}
 
--
libgit2 0.21.4