Internal: Remove api_get_jqgrid_js function

AngelFQC · AngelFQC · commit cabe770b8221 · 2025-10-31T16:11:30.000-05:00
diff --git a/public/main/admin/career_dashboard.php b/public/main/admin/career_dashboard.php
@@ -18,9 +18,6 @@
 
 $this_section = SECTION_PLATFORM_ADMIN;
 
-//Adds the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 // setting breadcrumbs
 $interbreadcrumb[] = [
     'url' => 'index.php',
diff --git a/public/main/admin/careers.php b/public/main/admin/careers.php
@@ -12,9 +12,6 @@
 $allowCareer = ('true' === api_get_setting('session.allow_session_admin_read_careers'));
 api_protect_admin_script($allowCareer);
 
-// Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 // setting breadcrumbs
 $interbreadcrumb[] = [
     'url' => 'index.php',
diff --git a/public/main/admin/extra_field_options.php b/public/main/admin/extra_field_options.php
@@ -12,9 +12,6 @@
 
 api_protect_admin_script();
 
-//Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 // setting breadcrumbs
 $interbreadcrumb[] = ['url' => 'index.php', 'name' => get_lang('Administration')];
 
diff --git a/public/main/admin/extra_fields.php b/public/main/admin/extra_fields.php
@@ -14,9 +14,6 @@
 
 api_protect_admin_script();
 
-//Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 // setting breadcrumbs
 $interbreadcrumb[] = ['url' => 'index.php', 'name' => get_lang('Administration')];
 
diff --git a/public/main/admin/grade_models.php b/public/main/admin/grade_models.php
@@ -15,9 +15,6 @@
     api_not_allowed(true);
 }
 
-//Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 // setting breadcrumbs
 $interbreadcrumb[] = ['url' => 'index.php', 'name' => get_lang('Administration')];
 
diff --git a/public/main/admin/promotions.php b/public/main/admin/promotions.php
@@ -11,9 +11,6 @@
 
 api_protect_admin_script();
 
-//Adds the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 // setting breadcrumbs
 $interbreadcrumb[] = ['url' => 'index.php', 'name' => get_lang('Administration')];
 $interbreadcrumb[] = ['url' => 'career_dashboard.php', 'name' => get_lang('Careers and promotions')];
diff --git a/public/main/admin/statistics/index.php b/public/main/admin/statistics/index.php
@@ -321,10 +321,6 @@
     }
 }
 
-if ('user_session' === $report) {
-    $htmlHeadXtra[] = api_get_jqgrid_js();
-}
-
 if (isset($_GET['export'])) {
     ob_start();
 }
diff --git a/public/main/admin/usergroup_users.php b/public/main/admin/usergroup_users.php
@@ -25,8 +25,6 @@
     $calendarPlugin = LearningCalendarPlugin::create();
 }
 
-// Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $action = isset($_GET['action']) ? Security::remove_XSS($_GET['action']) : null;
 $userId = isset($_GET['user_id']) ? (int) $_GET['user_id'] : 0;
 $calendarId = isset($_REQUEST['calendar_id']) ? (int) $_REQUEST['calendar_id'] : 0;
diff --git a/public/main/admin/usergroups.php b/public/main/admin/usergroups.php
@@ -15,9 +15,6 @@
 $usergroup = new UserGroupModel();
 $usergroup->protectScript();
 
-// Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 // setting breadcrumbs
 $action = isset($_GET['action']) ? Security::remove_XSS($_GET['action']) : '';
 $userGroupId = isset($_GET['id']) ? (int) $_GET['id'] : 0;
diff --git a/public/main/announcements/announcements.php b/public/main/announcements/announcements.php
@@ -154,8 +154,6 @@
 
         break;
     case 'list':
-        $htmlHeadXtra[] = api_get_jqgrid_js();
-
         $searchForm = new FormValidator(
             'search_simple',
             'post',
diff --git a/public/main/exercise/category.php b/public/main/exercise/category.php
@@ -31,9 +31,6 @@
 $check = Security::check_token('request');
 $token = Security::get_token();
 
-//Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 //The order is important you need to check the the $column variable in the model.ajax.php file
 $columns = [
     get_lang('Name'),
diff --git a/public/main/exercise/exercise_report.php b/public/main/exercise/exercise_report.php
@@ -21,7 +21,6 @@
 $this_section = SECTION_COURSES;
 
 $htmlHeadXtra[] = api_get_asset('js-cookie/src/js.cookie.js');
-$htmlHeadXtra[] = api_get_jqgrid_js();
 
 $filter_user = isset($_REQUEST['filter_by_user']) ? (int) $_REQUEST['filter_by_user'] : null;
 $isBossOfStudent = false;
diff --git a/public/main/exercise/live_stats.php b/public/main/exercise/live_stats.php
@@ -30,9 +30,6 @@
     'name' => $objExercise->selectTitle(true),
 ];
 
-//Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 // The header.
 Display::display_header(get_lang('Learners who\'re taking the exercise right now'));
 
diff --git a/public/main/inc/lib/api.lib.php b/public/main/inc/lib/api.lib.php
@@ -5727,14 +5727,6 @@ function api_get_jquery_ui_css_web_path()
     return api_get_path(WEB_PUBLIC_PATH).'assets/jquery-ui/themes/smoothness/jquery-ui.min.css';
 }
 
-function api_get_jqgrid_js()
-{
-    return '<link
-        href="'.api_get_path(WEB_PUBLIC_PATH).'build/legacy_free-jqgrid.css" rel="stylesheet" media="screen" type="text/css" />'
-        .PHP_EOL
-        .api_get_build_js('legacy_free-jqgrid.js');
-}
-
 /**
  * Returns the jquery library js and css headers.
  *
diff --git a/public/main/mail_template/list.php b/public/main/mail_template/list.php
@@ -5,9 +5,6 @@
 
 api_protect_admin_script();
 
-//Adds the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 $action = !empty($_REQUEST['action']) ? $_REQUEST['action'] : 'list';
 $check = Security::check_token('request');
 $token = Security::get_token();
diff --git a/public/main/my_space/admin_view.php b/public/main/my_space/admin_view.php
@@ -20,7 +20,6 @@
     $display = Security::remove_XSS($_POST['display']);
 }
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $htmlHeadXtra[] = '<script
 src="'.api_get_path(WEB_PUBLIC_PATH).'assets/jquery.easy-pie-chart/dist/jquery.easypiechart.js"></script>
 <script>
diff --git a/public/main/my_space/company_reports.php b/public/main/my_space/company_reports.php
@@ -23,7 +23,6 @@
 
 $this_section = SECTION_TRACKING;
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $sessionId = isset($_GET['session_id']) ? (int) $_GET['session_id'] : -1;
 
 //jqgrid will use this URL to do the selects
diff --git a/public/main/my_space/company_reports_resumed.php b/public/main/my_space/company_reports_resumed.php
@@ -21,7 +21,6 @@
 
 $tool_name = get_lang('Report');
 $this_section = SECTION_TRACKING;
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $sessionId = isset($_GET['session_id']) ? intval($_GET['session_id']) : -1;
 
 // jqgrid will use this URL to do the selects
diff --git a/public/main/my_space/exercise_category_report.php b/public/main/my_space/exercise_category_report.php
@@ -27,7 +27,6 @@
 $defaults['start_date'] = isset($_GET['start_date']) ? Security::remove_XSS($_GET['start_date']) : '';
 $defaults['course_id'] = $courseId;
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $htmlHeadXtra[] = '<script>
 $(function() {
     $("#exercise_course_id").on("change", function(e) {
diff --git a/public/main/my_space/index.php b/public/main/my_space/index.php
@@ -12,7 +12,6 @@
 
 api_block_anonymous_users();
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
 //$htmlHeadXtra[] = '<script type="text/javascript" src="'.api_get_path(WEB_PUBLIC_PATH).'assets/jquery.easy-pie-chart/dist/jquery.easypiechart.js"></script>';
 $this_section = SECTION_TRACKING;
 
diff --git a/public/main/my_space/session.php b/public/main/my_space/session.php
@@ -297,7 +297,6 @@
         exit;
         break;
 }
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $interbreadcrumb[] = ['url' => 'index.php', 'name' => get_lang('Reporting')];
 Display::display_header(get_lang('Course sessions'));
 
diff --git a/public/main/search/load_search.php b/public/main/search/load_search.php
@@ -1170,8 +1170,6 @@
     }
 }';
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 $griJs = Display::grid_js(
     'sessions',
     $url,
diff --git a/public/main/session/index.php b/public/main/session/index.php
@@ -31,7 +31,6 @@
 }
 
 $this_section = SECTION_COURSES;
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $course_id = isset($_GET['course_id']) ? (int) $_GET['course_id'] : null;
 Session::write('id_session', $session_id);
 
diff --git a/public/main/session/scheduled_announcement.php b/public/main/session/scheduled_announcement.php
@@ -26,7 +26,6 @@
 
 $sessionUrl = api_get_path(WEB_CODE_PATH).'session/resume_session.php?id_session='.$sessionId;
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $interbreadcrumb[] = [
     'url' => 'session_list.php',
     'name' => get_lang('Session list'),
diff --git a/public/main/session/session_list.php b/public/main/session/session_list.php
@@ -13,9 +13,6 @@
 
 SessionManager::protectSession(null, false);
 
-// Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 $action = $_REQUEST['action'] ?? null;
 $idChecked = $_REQUEST['idChecked'] ?? null;
 $idMultiple = $_REQUEST['id'] ?? null;
diff --git a/public/main/session/session_list_custom.php b/public/main/session/session_list_custom.php
@@ -16,9 +16,6 @@
 
 SessionManager::protectSession(null, false);
 
-// Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 $action = isset($_REQUEST['action']) ? $_REQUEST['action'] : null;
 $idChecked = isset($_REQUEST['idChecked']) ? $_REQUEST['idChecked'] : null;
 $listType = isset($_REQUEST['list_type']) ? Security::remove_XSS($_REQUEST['list_type']) : SessionManager::getDefaultSessionTab();
diff --git a/public/main/skills/skills_gradebook.php b/public/main/skills/skills_gradebook.php
@@ -11,9 +11,6 @@
 api_protect_admin_script();
 SkillModel::isAllowed();
 
-//Adds the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 $action = isset($_REQUEST['action']) ? $_REQUEST['action'] : 'display';
 
 // setting breadcrumbs
diff --git a/public/main/social/skills_ranking.php b/public/main/social/skills_ranking.php
@@ -11,9 +11,6 @@
 api_block_anonymous_users();
 SkillModel::isAllowed(api_get_user_id());
 
-//Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 $interbreadcrumb[] = ['url' => 'index.php', 'name' => get_lang('Skills')];
 
 //jqgrid will use this URL to do the selects
diff --git a/public/main/social/skills_tree.php b/public/main/social/skills_tree.php
@@ -9,8 +9,6 @@
 api_block_anonymous_users();
 SkillModel::isAllowed(api_get_user_id());
 
-//Adds the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $htmlHeadXtra[] = api_get_js('jqueryui-touch-punch/jquery.ui.touch-punch.min.js');
 $htmlHeadXtra[] = api_get_js('jquery.jsPlumb.all.js');
 $htmlHeadXtra[] = api_get_js('skills.js');
diff --git a/public/main/tracking/course_log_events.php b/public/main/tracking/course_log_events.php
@@ -107,8 +107,6 @@
 
 $actionLinks = '';
 
-// Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $htmlHeadXtra[] = '
 <script>
 $(function() {
diff --git a/public/main/tracking/course_log_groups.php b/public/main/tracking/course_log_groups.php
@@ -101,8 +101,6 @@ function action_formatter(cellvalue, options, rowObject) {
     '\';
 }';
 
-// Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $htmlHeadXtra[] = '
 <script>
 $(function() {
diff --git a/public/main/user/class.php b/public/main/user/class.php
@@ -17,8 +17,6 @@
 
 $tool_name = get_lang('Classes');
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 $interbreadcrumb[] = [
     'url' => 'user.php?'.api_get_cidreq(),
     'name' => get_lang('Users'),
diff --git a/public/main/user/session_list.php b/public/main/user/session_list.php
@@ -15,8 +15,6 @@
 
 api_protect_teacher_script(true);
 
-// Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $tool_name = get_lang('Session list');
 $allowTutors = api_get_setting('allow_tutors_to_assign_students_to_session');
 
diff --git a/public/main/work/pending.php b/public/main/work/pending.php
@@ -18,7 +18,6 @@
 $group_id = api_get_group_id();
 $courseInfo = api_get_course_info();
 $sessionId = api_get_session_id();
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $userId = api_get_user_id();
 
 /*$interbreadcrumb[] = [
diff --git a/public/main/work/publications.php b/public/main/work/publications.php
@@ -12,8 +12,6 @@
 
 api_block_anonymous_users();
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 $tpl = new Template(get_lang('Assignments'));
 
 $tpl->assign('intro_title', get_lang('All Assignments'));
diff --git a/public/main/work/work.php b/public/main/work/work.php
@@ -28,7 +28,6 @@
 $studentPublication = empty($work_id) ? null : $repo->find($work_id);
 
 $curdirpath = '';
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $htmlHeadXtra[] = to_javascript_work();
 $tool_name = get_lang('Assignments');
 
diff --git a/public/main/work/work_list.php b/public/main/work/work_list.php
@@ -38,7 +38,6 @@
 
 $group_id = api_get_group_id();
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $url_dir = api_get_path(WEB_CODE_PATH).'work/work.php?'.api_get_cidreq();
 
 if (!empty($group_id)) {
diff --git a/public/main/work/work_list_all.php b/public/main/work/work_list_all.php
@@ -40,7 +40,6 @@
 $group_id = api_get_group_id();
 $courseInfo = api_get_course_info();
 $sessionId = api_get_session_id();
-$htmlHeadXtra[] = api_get_jqgrid_js();
 $user_id = api_get_user_id();
 
 if (!empty($group_id)) {
diff --git a/public/main/work/work_list_others.php b/public/main/work/work_list_others.php
@@ -34,8 +34,6 @@
 
 protectWork($courseInfo, $workId);
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 if (!empty($group_id)) {
     $group_properties = api_get_group_entity($group_id);
     $show_work = false;
diff --git a/public/plugin/LearningCalendar/calendar_users.php b/public/plugin/LearningCalendar/calendar_users.php
@@ -60,9 +60,6 @@
 $action_links = '';
 $deleteUrl = '';
 
-// Add the JS needed to use the jqgrid
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 Display::display_header();
 
 ?>
diff --git a/public/plugin/LearningCalendar/start.php b/public/plugin/LearningCalendar/start.php
@@ -100,8 +100,6 @@
         break;
 }
 
-$htmlHeadXtra[] = api_get_jqgrid_js();
-
 $url = api_get_path(WEB_AJAX_PATH).'model.ajax.php?a=get_learning_path_calendars';
 $columns = [
     get_lang('Title'),

Original file line number	Diff line number	Diff line change
`@@ -15,9 +15,6 @@`
`15`	`15`	`api_not_allowed(true);`
`16`	`16`	`}`
`17`	`17`
`18`		`-//Add the JS needed to use the jqgrid`
`19`		`-$htmlHeadXtra[] = api_get_jqgrid_js();`
`20`		`-`
`21`	`18`	`// setting breadcrumbs`
`22`	`19`	`$interbreadcrumb[] = ['url' => 'index.php', 'name' => get_lang('Administration')];`
`23`	`20`
Original file line number	Diff line number	Diff line change
`@@ -321,10 +321,6 @@`
`321`	`321`	`}`
`322`	`322`	`}`
`323`	`323`
`324`		`-if ('user_session' === $report) {`
`325`		`- $htmlHeadXtra[] = api_get_jqgrid_js();`
`326`		`-}`
`327`		`-`
`328`	`324`	`if (isset($_GET['export'])) {`
`329`	`325`	`ob_start();`
`330`	`326`	`}`
Original file line number	Diff line number	Diff line change
`@@ -25,8 +25,6 @@`
`25`	`25`	`$calendarPlugin = LearningCalendarPlugin::create();`
`26`	`26`	`}`
`27`	`27`
`28`		`-// Add the JS needed to use the jqgrid`
`29`		`-$htmlHeadXtra[] = api_get_jqgrid_js();`
`30`	`28`	`$action = isset($_GET['action']) ? Security::remove_XSS($_GET['action']) : null;`
`31`	`29`	`$userId = isset($_GET['user_id']) ? (int) $_GET['user_id'] : 0;`
`32`	`30`	`$calendarId = isset($_REQUEST['calendar_id']) ? (int) $_REQUEST['calendar_id'] : 0;`