diff --git a/main/admin/usergroup_user_import.php b/main/admin/usergroup_user_import.php
index 5b65615430..4864a6d124 100644
--- a/main/admin/usergroup_user_import.php
+++ b/main/admin/usergroup_user_import.php
@@ -1,69 +1,69 @@
$user_class) {
$user_class['line'] = $index + 1;
// 1. Check whether mandatory fields are set.
- $mandatory_fields = array ('UserName', 'ClassName');
-
- foreach ($mandatory_fields as $key => $field) {
- if (!isset ($user_class[$field]) || strlen($user_class[$field]) == 0) {
- $user_class['error'] = get_lang($field.'Mandatory');
+ $mandatory_fields = array('UserName', 'ClassName');
+
+ foreach ($mandatory_fields as $key => $field) {
+ if (!isset($user_class[$field]) || strlen($user_class[$field]) == 0) {
+ $user_class['error'] = get_lang($field . 'Mandatory');
$errors[] = $user_class;
}
}
-
+
// 2. Check whether classcode exists.
- if (isset ($user_class['ClassName']) && strlen($user_class['ClassName']) != 0) {
+ if (isset($user_class['ClassName']) && strlen($user_class['ClassName']) != 0) {
// 2.1 Check whether code has been allready used in this CVS-file.
- if (!isset ($classcodes[$user_class['ClassName']])) {
+ if (!isset($classcodes[$user_class['ClassName']])) {
// 2.1.1 Check whether code exists in DB
- $exists = $usergroup->usergroup_exists($user_class['ClassName']);
+ $exists = $usergroup->usergroup_exists($user_class['ClassName']);
if (!$exists) {
- $user_class['error'] = get_lang('CodeDoesNotExists').': '.$user_class['ClassName'];
+ $user_class['error'] = get_lang('CodeDoesNotExists') . ': ' . $user_class['ClassName'];
$errors[] = $user_class;
} else {
$classcodes[$user_class['CourseCode']] = 1;
}
}
}
-
+
// 3. Check username, first, check whether it is empty.
if (!UserManager::is_username_empty($user_class['UserName'])) {
// 3.1. Check whether username is too long.
if (UserManager::is_username_too_long($user_class['UserName'])) {
- $user_class['error'] = get_lang('UserNameTooLong').': '.$user_class['UserName'];
+ $user_class['error'] = get_lang('UserNameTooLong') . ': ' . $user_class['UserName'];
$errors[] = $user_class;
}
-
- $username = UserManager::purify_username($user_class['UserName'], $purification_option_for_usernames);
+
+ $username = UserManager::purify_username($user_class['UserName'], $purification_option_for_usernames);
// 3.2. Check whether username exists.
if (UserManager::is_username_available($username)) {
- $user_class['error'] = get_lang('UnknownUser').': '.$username;
+ $user_class['error'] = get_lang('UnknownUser') . ': ' . $username;
$errors[] = $user_class;
}
}
@@ -79,21 +79,21 @@ function save_data($users_classes) {
global $purification_option_for_usernames;
// Table definitions.
- $user_table = Database :: get_main_table(TABLE_MAIN_USER);
-
+ $user_table = Database :: get_main_table(TABLE_MAIN_USER);
+
$usergroup = new UserGroup();
-
+
// Data parsing: purification + conversion (UserName, ClassName) --> (user_is, class_id)
- $csv_data = array ();
+ $csv_data = array();
if (!empty($users_classes)) {
foreach ($users_classes as $user_class) {
- $sql1 = "SELECT user_id FROM $user_table WHERE username = '".Database::escape_string(UserManager::purify_username($user_class['UserName'], $purification_option_for_usernames))."'";
+ $sql1 = "SELECT user_id FROM $user_table WHERE username = '" . Database::escape_string(UserManager::purify_username($user_class['UserName'], $purification_option_for_usernames)) . "'";
$res1 = Database::query($sql1);
$obj1 = Database::fetch_object($res1);
$usergroup = new UserGroup();
- $id = $usergroup->get_id_by_name($user_class['ClassName']);
-
+ $id = $usergroup->get_id_by_name($user_class['ClassName']);
+
if ($obj1 && $id) {
$csv_data[$id]['user_list'][] = $obj1->user_id;
$csv_data[$id]['user_list_name'][] = $user_class['UserName'];
@@ -101,51 +101,22 @@ function save_data($users_classes) {
}
}
}
-
+
// Logic for processing the request (data + UI options).
-
+
if (!empty($csv_data)) {
foreach ($csv_data as $class_id => $user_data) {
$user_list = $user_data['user_list'];
-
+
$class_name = $user_data['class_name'];
$user_list_name = $user_data['user_list_name'];
-
$usergroup->subscribe_users_to_usergroup($class_id, $user_list);
+ $message = Display::return_message(get_lang('Class') . ': ' . $class_name . '
', 'normal', false);
+ $message .= Display::return_message(get_lang('Users') . ': ' . implode(', ', $user_list_name));
- /*
- $sql = "SELECT class_id FROM $class_user_table cu WHERE cu.user_id = $user_id";
- $res = Database::query($sql);
- while ($obj = Database::fetch_object($res)) {
- $db_subscriptions[$obj->class_id] = 1;
- }
- $to_subscribe = array_diff(array_keys($csv_subscriptions), array_keys($db_subscriptions));
- $to_unsubscribe = array_diff(array_keys($db_subscriptions), array_keys($csv_subscriptions));
-
- // Subscriptions for new classes.
- if ($_POST['subscribe']) {
- foreach ($to_subscribe as $class_id) {
- ClassManager::add_user($user_id, $class_id);
- }
- }
- // Unsubscription from previous classes.
- if ($_POST['unsubscribe']) {
- foreach ($to_unsubscribe as $class_id) {
- ClassManager::unsubscribe_user($user_id, $class_id);
- }
- }*/
- $message = Display::return_message(get_lang('Class').': '.$class_name.'
', 'normal', false);
- $message .= Display::return_message(get_lang('Users').': '.implode(', ', $user_list_name));
-
return $message;
-
}
}
-
-
-
-
-
}
/**
@@ -166,13 +137,13 @@ require_once '../inc/global.inc.php';
$this_section = SECTION_PLATFORM_ADMIN;
api_protect_admin_script(true);
-require_once api_get_path(LIBRARY_PATH).'fileManage.lib.php';
-require_once api_get_path(LIBRARY_PATH).'import.lib.php';
+require_once api_get_path(LIBRARY_PATH) . 'fileManage.lib.php';
+require_once api_get_path(LIBRARY_PATH) . 'import.lib.php';
-$tool_name = get_lang('AddUsersToAClass').' CSV';
+$tool_name = get_lang('AddUsersToAClass') . ' CSV';
-$interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
-$interbreadcrumb[] = array ('url' => 'usergroups.php', 'name' => get_lang('Classes'));
+$interbreadcrumb[] = array('url' => 'index.php', 'name' => get_lang('PlatformAdmin'));
+$interbreadcrumb[] = array('url' => 'usergroups.php', 'name' => get_lang('Classes'));
// Set this option to true to enforce strict purification for usenames.
$purification_option_for_usernames = false;
@@ -188,9 +159,9 @@ $form->addElement('style_submit_button', 'submit', get_lang('Import'), 'class="s
if ($form->validate()) {
$users_classes = parse_csv_data($_FILES['import_file']['tmp_name']);
- $errors = validate_data($users_classes);
- if (count($errors) == 0) {
- $return = save_data($users_classes);
+ $errors = validate_data($users_classes);
+ if (count($errors) == 0) {
+ $return = save_data($users_classes);
}
}
@@ -203,7 +174,7 @@ if (isset($return) && $return) {
if (count($errors) != 0) {
$error_message = "\n";
foreach ($errors as $index => $error_class_user) {
- $error_message .= get_lang('Line').' '.$error_class_user['line'].': '.$error_class_user['error'].'';
+ $error_message .= get_lang('Line') . ' ' . $error_class_user['line'] . ': ' . $error_class_user['error'] . '';
$error_message .= "
";
}
$error_message .= "\n";
@@ -211,7 +182,7 @@ if (count($errors) != 0) {
}
$form->display();
?>
-
:
+:
UserName;ClassName jdoe;class01