diff --git a/main/admin/access_url_add_courses_to_url.php b/main/admin/access_url_add_courses_to_url.php index 044b5abc8d..0234b68301 100644 --- a/main/admin/access_url_add_courses_to_url.php +++ b/main/admin/access_url_add_courses_to_url.php @@ -12,19 +12,16 @@ $language_file = 'admin'; $cidReset = true; require_once '../inc/global.inc.php'; +require_once api_get_path(LIBRARY_PATH).'urlmanager.lib.php'; + $this_section=SECTION_PLATFORM_ADMIN; -require_once (api_get_path(LIBRARY_PATH).'urlmanager.lib.php'); api_protect_global_admin_script(); if (!api_get_multiple_access_url()) { - header('Location: index.php'); - exit; + header('Location: index.php'); + exit; } -/* - Global constants and variables -*/ - $form_sent = 0; $first_letter_course = ''; $courses = array (); @@ -41,50 +38,47 @@ $tool_name = get_lang('AddCoursesToURL'); $interbreadcrumb[] = array ('url' => 'index.php', 'name' => get_lang('PlatformAdmin')); $interbreadcrumb[] = array ('url' => 'access_urls.php', 'name' => get_lang('MultipleAccessURLs')); -/* MAIN CODE */ - Display :: display_header($tool_name); echo '
'; -echo Display::url(Display::return_icon('edit.gif',get_lang('EditCoursesToURL'),''), api_get_path(WEB_CODE_PATH).'admin/access_url_edit_courses_to_url.php"'); +echo Display::url( + Display::return_icon('edit.png', get_lang('EditCoursesToURL'), ''), + api_get_path(WEB_CODE_PATH).'admin/access_url_edit_courses_to_url.php' +); echo '
'; api_display_tool_title($tool_name); -if ($_POST['form_sent']) { - $form_sent = $_POST['form_sent']; - $courses = is_array($_POST['course_list']) ? $_POST['course_list'] : array() ; - $url_list = is_array($_POST['url_list']) ? $_POST['url_list'] : array() ; - $first_letter_course = $_POST['first_letter_course']; - - foreach($users as $key => $value) { - $users[$key] = intval($value); - } - - if ($form_sent == 1) { - if ( count($courses) == 0 || count($url_list) == 0) { - Display :: display_error_message(get_lang('AtLeastOneCourseAndOneURL')); - } else { - UrlManager::add_courses_to_urls($courses,$url_list); - Display :: display_confirmation_message(get_lang('CourseBelongURL')); - } - } +if (isset($_POST['form_sent']) && $_POST['form_sent']) { + $form_sent = $_POST['form_sent']; + $courses = is_array($_POST['course_list']) ? $_POST['course_list'] : array() ; + $url_list = is_array($_POST['url_list']) ? $_POST['url_list'] : array() ; + $first_letter_course = $_POST['first_letter_course']; + + foreach ($users as $key => $value) { + $users[$key] = intval($value); + } + + if ($form_sent == 1) { + if (count($courses) == 0 || count($url_list) == 0) { + Display :: display_error_message(get_lang('AtLeastOneCourseAndOneURL')); + } else { + UrlManager::add_courses_to_urls($courses, $url_list); + Display :: display_confirmation_message(get_lang('CourseBelongURL')); + } + } } - - -/* Display GUI */ - -if(empty($first_letter_user)) { - $sql = "SELECT count(*) as num_courses FROM $tbl_course"; - $result = Database::query($sql); - $num_row = Database::fetch_array($result); - if($num_row['num_courses']>1000) - {//if there are too much num_courses to gracefully handle with the HTML select list, - // assign a default filter on users names - $first_letter_user = 'A'; - } - unset($result); +if (empty($first_letter_user)) { + $sql = "SELECT count(*) as num_courses FROM $tbl_course"; + $result = Database::query($sql); + $num_row = Database::fetch_array($result); + if ($num_row['num_courses']>1000) { + //if there are too much num_courses to gracefully handle with the HTML select list, + // assign a default filter on users names + $first_letter_user = 'A'; + } + unset($result); } $first_letter_course = Database::escape_string($first_letter_course); @@ -126,35 +120,26 @@ unset($result); - + \ No newline at end of file