diff --git a/main/lp/blank.php b/main/lp/blank.php index d5e4bedbf0..bc6bec005f 100755 --- a/main/lp/blank.php +++ b/main/lp/blank.php @@ -21,24 +21,39 @@ $message = ''; if (isset($_GET['error'])) { switch ($_GET['error']) { case 'document_deleted': - $message = Display::return_message(get_lang('DocumentHasBeenDeleted'), 'error'); + $message = Display::return_message( + get_lang('DocumentHasBeenDeleted'), + 'error' + ); break; case 'prerequisites': - $message = Display::return_message(get_lang('LearnpathPrereqNotCompleted'), 'warning'); + $message = Display::return_message( + get_lang('LearnpathPrereqNotCompleted'), + 'warning' + ); break; case 'document_not_found': - $message = Display::return_message(get_lang('FileNotFound'), 'warning'); + $message = Display::return_message( + get_lang('FileNotFound'), + 'warning' + ); break; case 'reached_one_attempt': - $message = Display::return_message(get_lang('ReachedOneAttempt'), 'warning'); + $message = Display::return_message( + get_lang('ReachedOneAttempt'), + 'warning' + ); break; case 'x_frames_options': $src = Session::read('x_frame_source'); if (!empty($src)) { - $icon = ' '; - + $icon = ''; $message = Display::return_message( - Display::url($icon.$src, $src, ['class' => 'btn generated', 'target' => '_blank']), + Display::url( + $icon.$src, + $src, + ['class' => 'btn generated', 'target' => '_blank'] + ), 'normal', false ); diff --git a/main/mySpace/myStudents.php b/main/mySpace/myStudents.php index 3dec021bb5..f6abc82ffb 100755 --- a/main/mySpace/myStudents.php +++ b/main/mySpace/myStudents.php @@ -588,18 +588,20 @@ $userGroups = $userGroupManager->getNameListByUser( if (!empty($course_code)) { ?> - $student_id, - 'course' => $course_code, - 'origin' => $origin, - 'cidReq' => $course_code, - 'id_session' => $sessionId - ]) - ); - ?> + + $student_id, + 'course' => $course_code, + 'origin' => $origin, + 'cidReq' => $course_code, + 'id_session' => $sessionId + ]) + ); + ?> + getNameListByUser( $isBoss = UserManager::userIsBossOfStudent(api_get_user_id(), $student_id); if ($isBoss || api_is_platform_admin()) { $extraFieldValue = new ExtraFieldValue('user'); - $value = $extraFieldValue->get_values_by_handler_and_field_variable($student_id, 'legal_accept'); + $value = $extraFieldValue->get_values_by_handler_and_field_variable( + $student_id, + 'legal_accept' + ); $icon = Display::return_icon('accept_na.png'); if (isset($value['value'])) { list($legalId, $legalLanguageId, $legalTime) = explode(':', $value['value']); diff --git a/main/session/add_students_to_session.php b/main/session/add_students_to_session.php index 22188af7fe..509c3546ee 100644 --- a/main/session/add_students_to_session.php +++ b/main/session/add_students_to_session.php @@ -1,5 +1,6 @@ 'session_list.php', 'name' => get_lang('SessionList')); +$interbreadcrumb[] = array( + 'url' => 'session_list.php', + 'name' => get_lang('SessionList') +); // Setting the name of the tool $tool_name = get_lang('SubscribeStudentsToSession'); @@ -24,17 +28,19 @@ if (isset($_REQUEST['add_type']) && $_REQUEST['add_type'] != '') { $form_sent = 0; $errorMsg = ''; $users = $sessions = array(); -$id = isset($_GET['id']) ? intval($_GET['id']) : null; +$id = isset($_GET['id']) ? intval($_GET['id']) : 0; SessionManager::protectSession($id); -$htmlResult = null; - +$htmlResult = ''; if (isset($_POST['form_sent']) && $_POST['form_sent']) { $form_sent = $_POST['form_sent']; if ($form_sent == 1) { $sessionSourceList = $_POST['sessions']; $sessionDestinationList = $_POST['sessions_destination']; - $result = SessionManager::copyStudentsFromSession($sessionSourceList, $sessionDestinationList); + $result = SessionManager::copyStudentsFromSession( + $sessionSourceList, + $sessionDestinationList + ); foreach ($result as $message) { $htmlResult .= $message; } diff --git a/main/session/add_teachers_to_session.php b/main/session/add_teachers_to_session.php index 37966947a8..c417055561 100644 --- a/main/session/add_teachers_to_session.php +++ b/main/session/add_teachers_to_session.php @@ -1,5 +1,6 @@ 'index.php','name' => get_lang('PlatformAdmin')); -$interbreadcrumb[] = array('url' => 'session_list.php', 'name' => get_lang('SessionList')); +// Setting breadcrumbs +$interbreadcrumb[] = array( + 'url' => 'session_list.php', + 'name' => get_lang('SessionList') +); // Setting the name of the tool $tool_name = get_lang('EnrollTrainersFromExistingSessions'); -$form_sent = 0; -$errorMsg = ''; +$form_sent = 0; +$errorMsg = ''; $id = intval($_GET['id']); SessionManager::protectSession($id); -$htmlResult = null; - +$htmlResult = ''; if (isset($_POST['form_sent']) && $_POST['form_sent']) { $form_sent = $_POST['form_sent']; - if ($form_sent == 1 && isset($_POST['sessions']) && isset($_POST['courses'])) { + if ($form_sent == 1 && + isset($_POST['sessions']) && isset($_POST['courses']) + ) { $sessions = $_POST['sessions']; $courses = $_POST['courses']; - - $htmlResult = SessionManager::copyCoachesFromSessionToCourse($sessions, $courses); + $htmlResult = SessionManager::copyCoachesFromSessionToCourse( + $sessions, + $courses + ); } } diff --git a/tests/scripts/fix_course_index.php b/tests/scripts/fix_course_index.php index 67e6ef3e0a..1cd21c48ab 100644 --- a/tests/scripts/fix_course_index.php +++ b/tests/scripts/fix_course_index.php @@ -1,7 +1,7 @@