From 042ae115c93806ee4eaf9f42c53ee24d2eb668f8 Mon Sep 17 00:00:00 2001 From: AlexanderDCrane <40072887+AlexanderDCrane@users.noreply.github.com> Date: Mon, 3 Jun 2019 11:40:09 -0600 Subject: [PATCH] Update conference_profiles.php (#4250) --- .../conference_profiles.php | 49 +++++++++++-------- 1 file changed, 29 insertions(+), 20 deletions(-) diff --git a/app/conference_profiles/conference_profiles.php b/app/conference_profiles/conference_profiles.php index 88ddb69506..63b226a4d8 100644 --- a/app/conference_profiles/conference_profiles.php +++ b/app/conference_profiles/conference_profiles.php @@ -22,13 +22,28 @@ $order_by = check_str($_GET["order_by"]); $order = check_str($_GET["order"]); +//validate order by + if (strlen($order_by) > 0) { + $order_by = preg_replace('#[^a-zA-Z0-9_\-]#', '', $order_by); + } + +//validate the order + switch ($order) { + case 'asc': + break; + case 'desc': + break; + default: + $order = ''; + } + //add the search term $search = check_str($_GET["search"]); if (strlen($search) > 0) { $sql_search = "where ("; - $sql_search .= "profile_name like '%".$search."%'"; - $sql_search .= "or profile_enabled like '%".$search."%'"; - $sql_search .= "or profile_description like '%".$search."%'"; + $sql_search .= "profile_name like :search"; + $sql_search .= "or profile_enabled like :search"; + $sql_search .= "or profile_description like :search"; $sql_search .= ")"; } //additional includes @@ -37,20 +52,14 @@ //prepare to page the results $sql = "select count(*) as num_rows from v_conference_profiles "; - //$sql .= "where domain_uuid = '$domain_uuid' "; + //$sql .= "where domain_uuid = :domain_uuid "; $sql .= $sql_search; - if (strlen($order_by)> 0) { $sql .= "order by $order_by $order "; } - $prep_statement = $db->prepare($sql); - if ($prep_statement) { - $prep_statement->execute(); - $row = $prep_statement->fetch(PDO::FETCH_ASSOC); - if ($row['num_rows'] > 0) { - $num_rows = $row['num_rows']; - } - else { - $num_rows = '0'; - } + //$parameters['domain_uuid'] = $domain_uuid; + if (strlen($search) > 0) { + $parameters['search'] = '%'.$search.'%'; } + $database = new database; + $row = $database->select($sql, $parameters, 'all'); //prepare to page the results $rows_per_page = ($_SESSION['domain']['paging']['numeric'] != '') ? $_SESSION['domain']['paging']['numeric'] : 50; @@ -65,11 +74,11 @@ //$sql .= "where domain_uuid = '$domain_uuid' "; $sql .= $sql_search; if (strlen($order_by)> 0) { $sql .= "order by $order_by $order "; } - $sql .= "limit $rows_per_page offset $offset "; - $prep_statement = $db->prepare(check_sql($sql)); - $prep_statement->execute(); - $result = $prep_statement->fetchAll(PDO::FETCH_NAMED); - unset ($prep_statement, $sql); + $sql .= "limit :rows_per_page offset :offset "; + $parameters['rows_per_page'] = $rows_per_page; + $parameters['offset'] = $offset; + $database = new database; + $result = $database->select($sql, $parameters, 'all'); //alternate the row style $c = 0;