From 29518eb16b930db45d4f9ed520ad86aee566daee Mon Sep 17 00:00:00 2001
From: christianbeeznst <christian.fasanando@beeznest.com>
Date: Wed, 6 Mar 2024 12:44:51 -0500
Subject: [PATCH] Session: Fix searching by name or course

---
 public/main/inc/ajax/model.ajax.php        |  4 ++--
 public/main/inc/lib/sessionmanager.lib.php |  2 +-
 public/main/session/session_list.php       | 13 ++++++-------
 3 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/public/main/inc/ajax/model.ajax.php b/public/main/inc/ajax/model.ajax.php
index 5e07a35923..b74a4a67c8 100644
--- a/public/main/inc/ajax/model.ajax.php
+++ b/public/main/inc/ajax/model.ajax.php
@@ -1849,7 +1849,7 @@ switch ($action) {
                 ['where' => $whereCondition, 'extra' => $extra_fields]
             );
         } else {
-            $sidx = in_array($sidx, $columns) ? $sidx : 'name';
+            $sidx = in_array($sidx, $columns) ? $sidx : 'title';
             // Sessions for the coach
             $sessions = Tracking::get_sessions_coached_by_user(
                 api_get_user_id(),
@@ -1925,7 +1925,7 @@ switch ($action) {
 
                 $item = [
                     'name' => Display::url(
-                        $session['name'],
+                        $session['title'],
                         api_get_path(WEB_CODE_PATH).'my_space/course.php?sid='.$session['id']
                     ),
                     'date' => $dateToString,
diff --git a/public/main/inc/lib/sessionmanager.lib.php b/public/main/inc/lib/sessionmanager.lib.php
index b3859a250c..2e2f9010fb 100644
--- a/public/main/inc/lib/sessionmanager.lib.php
+++ b/public/main/inc/lib/sessionmanager.lib.php
@@ -824,7 +824,7 @@ class SessionManager
         $sql = "SELECT l.isocode, count(u.id) as cLang
                 FROM $tblSessionUser su
                 INNER JOIN $tblUser u ON (u.id = su.user_id)
-                INNER JOIN $tableLanguage l ON (l.english_name = u.language)
+                INNER JOIN $tableLanguage l ON (l.isocode = u.locale)
                 LEFT OUTER JOIN $tableAccessUrlUser au ON (au.user_id = u.id)
                 ";
 
diff --git a/public/main/session/session_list.php b/public/main/session/session_list.php
index 7de7d9b6df..3ef410cb0d 100644
--- a/public/main/session/session_list.php
+++ b/public/main/session/session_list.php
@@ -27,7 +27,7 @@ switch ($action) {
             $response = SessionManager::delete($idChecked);
             if ($response) {
                 Display::addFlash(
-                    Display::return_message(get_lang('Deleted').': '.Security::remove_XSS($sessionInfo['name']))
+                    Display::return_message(get_lang('Deleted').': '.Security::remove_XSS($sessionInfo['title']))
                 );
             }
         }
@@ -79,8 +79,7 @@ $courseSelect = $sessionFilter->addSelectAjax(
 
 if (!empty($courseId)) {
     $courseInfo = api_get_course_info_by_id($courseId);
-    $parents = CourseCategory::getParentsToString($courseInfo['categoryCode']);
-    $courseSelect->addOption($parents.$courseInfo['title'], $courseInfo['code'], ['selected' => 'selected']);
+    $courseSelect->addOption($courseInfo['title'], $courseInfo['code'], ['selected' => 'selected']);
 }
 
 $url = api_get_self();
@@ -116,14 +115,14 @@ if (isset($_REQUEST['keyword'])) {
     $filter->groupOp = 'OR';
 
     $filter = json_encode($filter);
-    $url = api_get_path(WEB_AJAX_PATH).'model.ajax.php?a=get_sessions&_force_search=true&rows=20&page=1&sidx=&sord=asc&filters='.$filter.'&searchField=s.name&searchString='.Security::remove_XSS($_REQUEST['keyword']).'&searchOper=in';
+    $url = api_get_path(WEB_AJAX_PATH).'model.ajax.php?a=get_sessions&_force_search=true&rows=20&page=1&sidx=&sord=asc&filters='.$filter.'&searchField=s.title&searchString='.Security::remove_XSS($_REQUEST['keyword']).'&searchOper=in';
 }
 
 if (isset($_REQUEST['id_category'])) {
     $sessionCategory = SessionManager::get_session_category($_REQUEST['id_category']);
     if (!empty($sessionCategory)) {
         //Begin with see the searchOper param
-        $url = api_get_path(WEB_AJAX_PATH).'model.ajax.php?a=get_sessions&_force_search=true&rows=20&page=1&sidx=&sord=asc&filters=&searchField=sc.name&searchString='.Security::remove_XSS($sessionCategory['name']).'&searchOper=in';
+        $url = api_get_path(WEB_AJAX_PATH).'model.ajax.php?a=get_sessions&_force_search=true&rows=20&page=1&sidx=&sord=asc&filters=&searchField=sc.title&searchString='.Security::remove_XSS($sessionCategory['title']).'&searchOper=in';
     }
 }
 
@@ -178,14 +177,14 @@ $orderUrl = api_get_path(WEB_AJAX_PATH).'session.ajax.php?a=order';
             for (key in added_cols) {
                 grid.hideCol(key);
             }
-            grid.showCol('name');
+            grid.showCol('title');
             grid.showCol('display_start_date');
             grid.showCol('display_end_date');
             grid.showCol('course_title');
         }
 
         function show_cols(grid, added_cols) {
-            grid.showCol('name').trigger('reloadGrid');
+            grid.showCol('title').trigger('reloadGrid');
             for (key in added_cols) {
                 grid.showCol(key);
             }