[ros-diffs] [dgoette] 37563: only add a user membership once to a group
dgoette at svn.reactos.org
dgoette at svn.reactos.org
Sat Nov 22 19:56:49 CET 2008
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/roscms/lib/Export_User.class.php?rev=37563&r1=37562&r2=37563&view=diff
==============================================================================
--- 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;
}
More information about the Ros-diffs
mailing list