[ros-diffs] [dgoette] 37563: only add a user membership once to a group

Author: dgoette
Date: Sat Nov 22 12:56:48 2008
New Revision: 37563

URL: http://svn.reactos.org/svn/reactos?rev=37563&view=rev
Log:
only add a user membership once to a group

Modified:
    branches/danny-web/reactos.org/htdocs/roscms/lib/Export_User.class.php

Modified: branches/danny-web/reactos.org/htdocs/roscms/lib/Export_User.class.php
URL:
http://svn.reactos.org/svn/reactos/branches/danny-web/reactos.org/htdocs/ro…
==============================================================================
--- branches/danny-web/reactos.org/htdocs/roscms/lib/Export_User.class.php [iso-8859-1]
(original)
+++ branches/danny-web/reactos.org/htdocs/roscms/lib/Export_User.class.php [iso-8859-1]
Sat Nov 22 12:56:48 2008
@@ -84,16 +84,23 @@
           $user_lang = false;
         }
 
+        // do some actions
         switch ($flag) {
           case 'addmembership':
-            $stmt=DBConnection::getInstance()->prepare("INSERT INTO
usergroup_members ( usergroupmember_userid , usergroupmember_usergroupid ) VALUES (
:user_id, :group_id )");
-            $stmt->bindParam('user_id',$user_id,PDO::PARAM_INT);
-            $stmt->bindParam('group_id',$group_id,PDO::PARAM_INT);
-            $stmt->execute();
-            if ($user_lang !== false) {
-              Log::writeLangMedium("add user account membership:
user-id=".$user_id.", group-id=".$RosCMS_GET_d_value2." done by
".$roscms_intern_account_id." {data_user_out}", $user_lang);
-            }
-            Log::writeMedium('add user account membership:
user-id='.$user_id.', group-id='.$group_id.' done by
'.$roscms_intern_account_id.' {data_user_out}');
+            // check if user is already member, so we don't add him twice
+            if (!ROSUser::isMemberOfGroup($user_id,$group_id)) {
+            
+              // insert new membership
+              $stmt=DBConnection::getInstance()->prepare("INSERT INTO
usergroup_members ( usergroupmember_userid , usergroupmember_usergroupid ) VALUES (
:user_id, :group_id )");
+              $stmt->bindParam('user_id',$user_id,PDO::PARAM_INT);
+              $stmt->bindParam('group_id',$group_id,PDO::PARAM_INT);
+              $stmt->execute();
+              if ($user_lang !== false) {
+                Log::writeLangMedium("add user account membership:
user-id=".$user_id.", group-id=".$RosCMS_GET_d_value2." done by
".$roscms_intern_account_id." {data_user_out}", $user_lang);
+              }
+              Log::writeMedium('add user account membership:
user-id='.$user_id.', group-id='.$group_id.' done by
'.$roscms_intern_account_id.' {data_user_out}');
+            }
+            // preselect displayed content
             $flag = 'detail';
             break;
 
@@ -106,6 +113,7 @@
               Log::writeLangMedium('delete user account membership:
user-id='.$user_id.', group-id='.$group_id.' done by
'.$roscms_intern_account_id.' {data_user_out}', $user_lang);
             }
             Log::writeMedium('delete user account membership:
user-id='.$user_id.', group-id='.$group_id.' done by
'.$roscms_intern_account_id.' {data_user_out}');
+            // preselect displayed content
             $flag = 'detail';
             break;
 
@@ -118,7 +126,8 @@
               Log::writeLangMedium('change user account language:
user-id='.$user_id.', lang-id='.$group_id.' done by
'.$roscms_intern_account_id.' {data_user_out}', $user_lang);
             }
             Log::writeMedium('change user account language:
user-id='.$user_id.', lang-id='.$group_id.' done by
'.$roscms_intern_account_id.' {data_user_out}');
-            $flag = "detail";
+            // preselect displayed content
+            $flag = 'detail';
             break;
         }