[Vtigercrm-commits] [vtiger-commits] r9951 - /vtigercrm/branches/5.0.3/include/utils/CommonUtils.php

vtigercrm-commits at vtiger.fosslabs.com vtigercrm-commits at vtiger.fosslabs.com
Fri Dec 8 14:19:13 EST 2006


Author: richie
Date: Fri Dec  8 12:19:09 2006
New Revision: 9951

Log:
Security check for changowner added

Modified:
    vtigercrm/branches/5.0.3/include/utils/CommonUtils.php

Modified: vtigercrm/branches/5.0.3/include/utils/CommonUtils.php
==============================================================================
--- vtigercrm/branches/5.0.3/include/utils/CommonUtils.php (original)
+++ vtigercrm/branches/5.0.3/include/utils/CommonUtils.php Fri Dec  8 12:19:09 2006
@@ -2074,16 +2074,28 @@
 }
 function getUserslist()
 {
-	global $log,$current_user;
+	global $log,$current_user,$module,$adb,$assigned_user_id;
 	$log->debug("Entering getUserslist() method ...");
-	global $adb;
-	$result=$adb->query("select * from vtiger_users");
-	for($i=0;$i<$adb->num_rows($result);$i++)
-	{
-	       $useridlist[$i]=$adb->query_result($result,$i,'id');
-	       $usernamelist[$useridlist[$i]]=$adb->query_result($result,$i,'user_name');
-	}
-	$change_owner = get_select_options_with_id($usernamelist,$current_user->user_name);
+	require('user_privileges/user_privileges_'.$current_user->id.'.php');
+	require('user_privileges/sharing_privileges_'.$current_user->id.'.php');
+	
+	if($is_admin==false && $profileGlobalPermission[2] == 1 && ($defaultOrgSharingPermission[getTabid($module)] == 3 or $defaultOrgSharingPermission[getTabid($module)] == 0))
+	{
+		$users_combo = get_select_options_array(get_user_array(FALSE, "Active", $current_user->id,'private'), $current_user->id);
+	}
+	else
+	{
+		$users_combo = get_select_options_array(get_user_array(FALSE, "Active", $current_user->id),$current_user->id);
+	}
+	foreach($users_combo as $userid=>$value)	
+	{
+
+		foreach($value as $username=>$selected)
+		{
+			$change_owner .= "<option value=$userid $selected>".$username."</option>";
+		}
+	}
+	
 	$log->debug("Exiting getUserslist method ...");
 	return $change_owner;
 }
@@ -2091,20 +2103,40 @@
 
 function getGroupslist()
 {
-	global $log;
+	global $log,$adb,$module,$current_user;
 	$log->debug("Entering getGroupslist() method ...");
-	global $adb;
-	$result=$adb->query("select * from vtiger_groups");
-	
-	for($i=0;$i<$adb->num_rows($result);$i++)
-	{
-	       $groupidlist[$i]=$adb->query_result($result,$i,'groupid');
-	       $groupnamelist[$groupidlist[$i]]=$adb->query_result($result,$i,'groupname');
-	       
-	}
-	$change_groups_owner = get_select_options_with_id($groupnamelist,'');
+	require('user_privileges/user_privileges_'.$current_user->id.'.php');
+	require('user_privileges/sharing_privileges_'.$current_user->id.'.php');
+	if($is_admin==false && $profileGlobalPermission[2] == 1 && ($defaultOrgSharingPermission[getTabid($module)] == 3 or $defaultOrgSharingPermission[getTabid($module)] == 0))
+	{
+		$result=get_current_user_access_groups($module);
+	}
+	else
+	{
+		$result = get_group_options();
+	}
+	$groupArray = $adb->fetch_array($result);
+
+	do{
+		$groupname=$groupArray["groupname"];
+		$group_id=$groupArray["groupid"];
+		$selected = '';
+		if($groupname == $selected_groupname[0])
+		{
+			$selected = "selected";
+		}
+		if($groupname != '')
+			$group_option[$group_id] = array($groupname=>$selected);
+          }while($groupArray = $adb->fetch_array($result));
+
+	foreach($group_option as $groupid=>$value)  
+	{ 
+		foreach($value as $groupname=>$selected) 
+		{
+			$change_groups_owner .= "<option value=$groupid $selected >".$groupname."</option>";  
+		} 
+	}
 	$log->debug("Exiting getGroupslist method ...");
-
 	return $change_groups_owner;
 }
 





More information about the vtigercrm-commits mailing list