diff --git a/main/admin/course_list.php b/main/admin/course_list.php index f1e728f6d8..c688a5e1be 100755 --- a/main/admin/course_list.php +++ b/main/admin/course_list.php @@ -39,7 +39,7 @@ function get_number_of_courses() if (isset ($_GET['keyword'])) { - $keyword = Database::escape_string($_GET['keyword']); + $keyword = Database::escape_string(trim($_GET['keyword'])); $sql .= " WHERE title LIKE '%".$keyword."%' OR code LIKE '%".$keyword."%' OR visual_code LIKE '%".$keyword."%'"; } elseif (isset ($_GET['keyword_code'])) @@ -81,7 +81,7 @@ function get_course_data($from, $number_of_items, $column, $direction) if (isset ($_GET['keyword'])) { - $keyword = Database::escape_string($_GET['keyword']); + $keyword = Database::escape_string(trim($_GET['keyword'])); $sql .= " WHERE title LIKE '%".$keyword."%' OR code LIKE '%".$keyword."%' OR visual_code LIKE '%".$keyword."%'"; } elseif (isset ($_GET['keyword_code'])) diff --git a/main/admin/group_list.php b/main/admin/group_list.php index a0b9b6b529..a5e0c81696 100755 --- a/main/admin/group_list.php +++ b/main/admin/group_list.php @@ -40,7 +40,7 @@ function get_number_of_groups() } */ if ( isset ($_GET['keyword'])) { - $keyword = Database::escape_string($_GET['keyword']); + $keyword = Database::escape_string(trim($_GET['keyword'])); $sql .= " WHERE (g.name LIKE '%".$keyword."%' OR g.description LIKE '%".$keyword."%' OR g.url LIKE '%".$keyword."%' )"; } @@ -82,7 +82,7 @@ function get_group_data($from, $number_of_items, $column, $direction) }*/ if (isset ($_GET['keyword'])) { - $keyword = Database::escape_string($_GET['keyword']); + $keyword = Database::escape_string(trim($_GET['keyword'])); $sql .= " WHERE (g.name LIKE '%".$keyword."%' OR g.description LIKE '%".$keyword."%' OR g.url LIKE '%".$keyword."%' )"; } /* @@ -140,7 +140,7 @@ function get_recent_group_data($from =0 , $number_of_items = 5, $column, $direct }*/ if (isset ($_GET['keyword'])) { - $keyword = Database::escape_string($_GET['keyword']); + $keyword = Database::escape_string(trim($_GET['keyword'])); $sql .= " WHERE (g.name LIKE '%".$keyword."%' OR g.description LIKE '%".$keyword."%' OR g.url LIKE '%".$keyword."%' )"; } /* diff --git a/main/admin/session_category_list.php b/main/admin/session_category_list.php index 0a713fadb3..db2f5af9d3 100755 --- a/main/admin/session_category_list.php +++ b/main/admin/session_category_list.php @@ -66,10 +66,10 @@ if (isset ($_GET['search']) && $_GET['search'] == 'advanced') { $from = $page * $limit; //if user is crfp admin only list its sessions if(!api_is_platform_admin()) { - $where .= (empty($_REQUEST['keyword']) ? " " : " WHERE name LIKE '%".addslashes($_REQUEST['keyword'])."%'"); + $where .= (empty($_REQUEST['keyword']) ? " " : " WHERE name LIKE '%".Database::escape_string(trim($_REQUEST['keyword']))."%'"); } else { - $where .= (empty($_REQUEST['keyword']) ? " " : " WHERE name LIKE '%".addslashes($_REQUEST['keyword'])."%'"); + $where .= (empty($_REQUEST['keyword']) ? " " : " WHERE name LIKE '%".Database::escape_string(trim($_REQUEST['keyword']))."%'"); } $query = "SELECT sc.*, (select count(id) FROM $tbl_session WHERE session_category_id = sc.id) as nbr_session diff --git a/main/admin/session_list.php b/main/admin/session_list.php index 4b3119e553..9f245eafa3 100755 --- a/main/admin/session_list.php +++ b/main/admin/session_list.php @@ -85,13 +85,13 @@ if (isset ($_GET['search']) && $_GET['search'] == 'advanced') { //Prevent hacking keyword if ( isset ($_GET['keyword'])) { - $keyword = Database::escape_string($_GET['keyword']); + $keyword = Database::escape_string(trim($_GET['keyword'])); } else if (isset ($_GET['keyword_name'])) { - $keyword_name = Database::escape_string($_GET['keyword_name']); - $keyword_category = Database::escape_string($_GET['keyword_category']); - $keyword_visibility = Database::escape_string($_GET['keyword_visibility']); - $keyword_firstname = Database::escape_string($_GET['keyword_firstname']); - $keyword_lastname = Database::escape_string($_GET['keyword_lastname']); + $keyword_name = Database::escape_string(trim($_GET['keyword_name'])); + $keyword_category = Database::escape_string(trim($_GET['keyword_category'])); + $keyword_visibility = Database::escape_string(trim($_GET['keyword_visibility'])); + $keyword_firstname = Database::escape_string(trim($_GET['keyword_firstname'])); + $keyword_lastname = Database::escape_string(trim($_GET['keyword_lastname'])); } //Process for the search advanced @@ -218,7 +218,7 @@ if (isset ($_GET['search']) && $_GET['search'] == 'advanced') { if($num>$limit){ if($page) { ?> - + $limit) { ?> - + - + - + 0) @@ -162,7 +162,7 @@ function get_class_data($from, $number_of_items, $column, $direction) $sql .= " WHERE 1 = 1"; if (isset ($_GET['keyword'])) { - $keyword = Database::escape_string($_GET['keyword']); + $keyword = Database::escape_string(trim($_GET['keyword'])); $sql .= " AND (c.name LIKE '%".$keyword."%')"; } if( count($subscribed_classes) > 0) diff --git a/main/user/subscribe_user.php b/main/user/subscribe_user.php index b8102e816e..4e0208fe73 100755 --- a/main/user/subscribe_user.php +++ b/main/user/subscribe_user.php @@ -324,7 +324,7 @@ function get_number_of_users() { // when there is a keyword then we are searching and we have to change the SQL statement if (isset ($_REQUEST['keyword'])) { - $keyword = Database::escape_string($_REQUEST['keyword']); + $keyword = Database::escape_string(trim($_REQUEST['keyword'])); $sql .= " AND (firstname LIKE '%".$keyword."%' OR lastname LIKE '%".$keyword."%' OR email LIKE '%".$keyword."%' OR username LIKE '%".$keyword."%' OR official_code LIKE '%".$keyword."%')"; // we also want to search for users who have something in their profile fields that matches the keyword @@ -574,7 +574,7 @@ function get_user_data($from, $number_of_items, $column, $direction) { // adding additional WHERE statements to the SQL for the search functionality if (isset ($_REQUEST['keyword'])) { - $keyword = Database::escape_string($_REQUEST['keyword']); + $keyword = Database::escape_string(trim($_REQUEST['keyword'])); $sql .= " AND (firstname LIKE '%".$keyword."%' OR lastname LIKE '%".$keyword."%' OR email LIKE '%".$keyword."%' OR username LIKE '%".$keyword."%' OR official_code LIKE '%".$keyword."%')"; if (api_get_setting('ProfilingFilterAddingUsers') == 'true') {