Minor - fix php errors

public/main/admin/course_list.php

@@ -232,6 +232,7 @@ function get_course_data_by_session($from, $number_of_items, $column, $direction
     $course_table = Database::get_main_table(TABLE_MAIN_COURSE);
     $session_rel_course = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
     $tblCourseCategory = Database::get_main_table(TABLE_MAIN_CATEGORY);
+
     $session = Database::get_main_table(TABLE_MAIN_SESSION);
     $from = (int) $from;
     $number_of_items = (int) $number_of_items;
@@ -246,19 +247,17 @@ function get_course_data_by_session($from, $number_of_items, $column, $direction
                 c.title AS col1,
                 c.code AS col2,
                 c.course_language AS col3,
-                course_category.code AS col4,
-                c.subscribe AS col5,
-                c.unsubscribe AS col6,
-                c.code AS col7,
-                c.visibility AS col8,
-                c.directory as col9,
+                c.subscribe AS col4,
+                c.unsubscribe AS col5,
+                c.code AS col6,
+                c.visibility AS col7,
+                c.directory as col8,
                 c.visual_code
             FROM $course_table c
             INNER JOIN $session_rel_course r
             ON c.id = r.c_id
             INNER JOIN $session s
             ON r.session_id = s.id
-            LEFT JOIN $tblCourseCategory ON c.category_id = course_category.id
             ";
 
     if (isset($_GET['session_id']) && !empty($_GET['session_id'])) {

public/main/admin/skill.php

@@ -22,7 +22,7 @@ if (isset($_GET['action']) && in_array($_GET['action'], ['add', 'edit', 'delete'
     $action = $_GET['action'];
 }
 
-$id = isset($_GET['id']) ? $_GET['id'] : '';
+$id = $_GET['id'] ?? '';
 
 $item = null;
 if (!empty($id)) {

public/main/admin/statistics/index.php

@@ -12,7 +12,7 @@ api_protect_admin_script();
 
 $interbreadcrumb[] = ['url' => '../index.php', 'name' => get_lang('Administration')];
 
-$report = isset($_REQUEST['report']) ? $_REQUEST['report'] : '';
+$report = $_REQUEST['report'] ?? '';
 $sessionDuration = isset($_GET['session_duration']) ? (int) $_GET['session_duration'] : '';
 $validated = false;
 
@@ -854,7 +854,6 @@ switch ($report) {
                 $item[] = $certificate ? get_lang('Yes') : get_lang('No');
                 $item[] = $birthDate;
                 $data[] = $item;
-                $row++;
             }
 
             if (isset($_REQUEST['action_table']) && 'export' === $_REQUEST['action_table']) {

public/main/admin/system_announcements.php

@@ -37,7 +37,7 @@ $visibleList = $repo->getVisibilityList();
 
 $tool_name = null;
 if (empty($_GET['lang'])) {
-    $_GET['lang'] = isset($_SESSION['user_language_choice']) ? $_SESSION['user_language_choice'] : null;
+    $_GET['lang'] = $_SESSION['user_language_choice'] ?? null;
 }
 
 if (!empty($action)) {
@@ -420,6 +420,7 @@ if ($action_todo) {
 }
 
 if ($show_announcement_list) {
+    var_dump(api_get_url_entity()->getId());
     $criteria = ['url' => api_get_url_entity()];
     $announcements = $repo->findBy($criteria);
     $announcement_data = [];