Minor - Format code - refs BT#14170

main/mySpace/lp_tracking.php

@@ -18,7 +18,9 @@ if (isset($_GET['from']) && $_GET['from'] == 'myspace') {
     $this_section = SECTION_COURSES;
 }
 
-$session_id = isset($_REQUEST['id_session']) && !empty($_REQUEST['id_session']) ? intval($_REQUEST['id_session']) : api_get_session_id();
+$session_id = isset($_REQUEST['id_session']) && !empty($_REQUEST['id_session'])
+    ? intval($_REQUEST['id_session'])
+    : api_get_session_id();
 $export_csv = isset($_GET['export']) && $_GET['export'] == 'csv' ? true : false;
 $user_id = isset($_GET['student_id']) ? intval($_GET['student_id']) : api_get_user_id();
 $courseCode = isset($_GET['course']) ? Security::remove_XSS($_GET['course']) : api_get_course_id();
@@ -35,9 +37,11 @@ $name = $userInfo['complete_name'];
 $isBoss = UserManager::userIsBossOfStudent(api_get_user_id(), $user_id);
 
 if (!api_is_platform_admin(true) &&
-    !CourseManager :: is_course_teacher(api_get_user_id(), $courseCode) &&
+    !CourseManager::is_course_teacher(api_get_user_id(), $courseCode) &&
     !$isBoss &&
-    !Tracking::is_allowed_to_coach_student(api_get_user_id(), $user_id) && !api_is_drh() && !api_is_course_tutor()
+    !Tracking::is_allowed_to_coach_student(api_get_user_id(), $user_id) &&
+    !api_is_drh() &&
+    !api_is_course_tutor()
 ) {
     api_not_allowed(
         api_get_origin() !== 'learnpath'
@@ -45,20 +49,29 @@ if (!api_is_platform_admin(true) &&
 }
 
 if ($origin == 'user_course') {
-    $interbreadcrumb[] = ["url" => api_get_path(WEB_COURSE_PATH).$course_info['directory'], 'name' => $course_info['name']];
+    $interbreadcrumb[] = [
-    $interbreadcrumb[] = ["url" => "../user/user.php?cidReq=".$courseCode, "name" => get_lang("Users")];
+        'url' => api_get_path(WEB_COURSE_PATH).$course_info['directory'],
+        'name' => $course_info['name']
+    ];
+    $interbreadcrumb[] = [
+        'url' => "../user/user.php?cidReq=$courseCode",
+        'name' => get_lang("Users")
+    ];
 } elseif ($origin == 'tracking_course') {
-    $interbreadcrumb[] = ["url" => "../tracking/courseLog.php?cidReq=".$courseCode.'&id_session='.$session_id, "name" => get_lang("Tracking")];
+    $interbreadcrumb[] = [
+        'url' => "../tracking/courseLog.php?cidReq=$courseCode&id_session=$session_id",
+        'name' => get_lang("Tracking"),
+    ];
 } else {
-    $interbreadcrumb[] = ["url" => "index.php", "name" => get_lang('MySpace')];
+    $interbreadcrumb[] = ['url' => 'index.php', 'name' => get_lang('MySpace')];
-    $interbreadcrumb[] = ["url" => "student.php", "name" => get_lang("MyStudents")];
+    $interbreadcrumb[] = ['url' => 'student.php', 'name' => get_lang("MyStudents")];
-    $interbreadcrumb[] = ["url" => "myStudents.php?student=".$user_id, "name" => get_lang("StudentDetails")];
+    $interbreadcrumb[] = ['url' => "myStudents.php?student=$user_id", 'name' => get_lang("StudentDetails")];
     $nameTools = get_lang("DetailsStudentInCourse");
 }
 
 $interbreadcrumb[] = [
-    "url" => "myStudents.php?student=".$user_id."&course=".$courseCode."&details=true&origin=".$origin,
+    'url' => "myStudents.php?student=$user_id&course=$courseCode&details=true&origin=$origin",
-    "name" => get_lang("DetailsStudentInCourse"),
+    'name' => get_lang("DetailsStudentInCourse"),
 ];
 $nameTools = get_lang('LearningPathDetails');
 $sql = 'SELECT name	FROM '.Database::get_course_table(TABLE_LP_MAIN).' 
@@ -70,29 +83,35 @@ $origin = 'tracking';
 
 $output = require_once api_get_path(SYS_CODE_PATH).'lp/lp_stats.php';
 
-Display :: display_header($nameTools);
+$actions = [];
-echo '<div class ="actions">';
+$actions[] = Display::url(
-echo '<a href="javascript:history.back();">'.
+    Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM),
-    Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
+    'javascript:history.back();'
-echo '<a href="javascript: void(0);" onclick="javascript: window.print();">
+);
-        '.Display::return_icon('printer.png', get_lang('Print'), '', ICON_SIZE_MEDIUM).'</a>';
+$actions[] = Display::url(
-echo '<a href="'.api_get_self().'?export=csv&'.Security::remove_XSS($_SERVER['QUERY_STRING']).'">
+    Display::return_icon('printer.png', get_lang('Print'), '', ICON_SIZE_MEDIUM),
-        '.Display::return_icon('export_csv.png', get_lang('ExportAsCSV'), '', ICON_SIZE_MEDIUM).'</a>';
+    'window.print();'
-echo '</div>';
+);
-echo '<div class="clear"></div>';
+$actions[] = Display::url(
-$session_name = api_get_session_name($session_id);
+    Display::return_icon('export_csv.png', get_lang('ExportAsCSV'), '', ICON_SIZE_MEDIUM),
-$table_title = ($session_name ? Display::return_icon('session.png', get_lang('Session'), [], ICON_SIZE_SMALL).' '.$session_name.' ' : ' ').
+    api_get_self().'?export=csv&'.Security::remove_XSS($_SERVER['QUERY_STRING'])
-    Display::return_icon('course.png', get_lang('Course'), [], ICON_SIZE_SMALL).' '.$course_info['name'].' '.
+);
-    Display::return_icon('user.png', get_lang('User'), [], ICON_SIZE_SMALL).' '.$name;
+
+Display::display_header($nameTools);
+echo Display::toolbarAction(
+    'actions',
+    [implode(PHP_EOL, $actions)]
+);
+
+$table_title = $session_id
+    ? Display::return_icon('session.png', get_lang('Session')).PHP_EOL.api_get_session_name($session_id).PHP_EOL
+    : PHP_EOL;
+$table_title .= Display::return_icon('course.png', get_lang('Course')).PHP_EOL.$course_info['name'].PHP_EOL
+    .Display::return_icon('user.png', get_lang('User')).' '.$name;
 
 echo Display::page_header($table_title);
 echo Display::page_subheader(
-    '<h3>'.Display::return_icon(
+    Display::return_icon('learnpath.png', get_lang('ToolLearnpath')).PHP_EOL.$lp_title
-        'learnpath.png',
-        get_lang('ToolLearnpath'),
-        [],
-        ICON_SIZE_SMALL
-    ).' '.$lp_title.'</h3>'
 );
 echo $output;
-Display :: display_footer();
+Display::display_footer();