diff --git a/main/inc/lib/usermanager.lib.php b/main/inc/lib/usermanager.lib.php index ea8b0f06fb..e2de0cd349 100755 --- a/main/inc/lib/usermanager.lib.php +++ b/main/inc/lib/usermanager.lib.php @@ -178,6 +178,7 @@ class UserManager expiration_date = '".Database::escape_string($expiration_date)."', hr_dept_id = '".Database::escape_string($hr_dept_id)."', active = '".Database::escape_string($active)."'"; + $result = Database::query($sql); if ($result) { diff --git a/tests/datafiller/fill_users.php b/tests/datafiller/fill_users.php index 396f38e475..0b32fa97e1 100755 --- a/tests/datafiller/fill_users.php +++ b/tests/datafiller/fill_users.php @@ -13,19 +13,32 @@ * internal functions. * @return array List of user IDs for the users that have just been inserted */ -function fill_users() { - $eol = PHP_EOL; +function fill_users() +{ $users = array(); //declare only to avoid parsing notice require_once 'data_users.php'; //fill the $users array $output = array(); $output[] = array('title'=>'Users Filling Report:'); - $i = 1; foreach ($users as $i => $user) { //first check that the first item doesn't exist already $output[$i]['line-init'] = $user['firstname']; - $res = UserManager::create_user($user['firstname'],$user['lastname'],$user['status'],$user['email'],$user['username'],$user['pass'],null,null,null,null,$user['auth_source'],null,$user['active']); + $res = UserManager::create_user( + $user['firstname'], + $user['lastname'], + $user['status'], + $user['email'], + $user['username'], + $user['pass'], + null, + null, + null, + null, + $user['auth_source'], + null, + $user['active'] + ); $output[$i]['line-info'] = $res ? get_lang('Inserted') : get_lang('NotInserted'); - $i++; } + return $output; }