prepare sql
This commit is contained in:
parent
5248f1c59c
commit
be8a0990e8
@ -21,13 +21,6 @@ class NotesController
|
|||||||
$sortBy = $_GET['sort_by'] ?? 'updated_at';
|
$sortBy = $_GET['sort_by'] ?? 'updated_at';
|
||||||
$sortOrder = strtoupper($_GET['sort_order'] ?? 'DESC');
|
$sortOrder = strtoupper($_GET['sort_order'] ?? 'DESC');
|
||||||
|
|
||||||
// Validate sort order to prevent SQL injection
|
|
||||||
$sortOrder = in_array($sortOrder, ['ASC', 'DESC']) ? $sortOrder : 'DESC';
|
|
||||||
|
|
||||||
// Validate sort column to prevent SQL injection
|
|
||||||
$validSortColumns = ['id', 'title', 'updated_at', 'created_at'];
|
|
||||||
$sortBy = in_array($sortBy, $validSortColumns) ? $sortBy : 'updated_at';
|
|
||||||
|
|
||||||
$this->view->setVars([
|
$this->view->setVars([
|
||||||
"notes" => $this->notesModel->selectNotesForUser(2, $sortBy, $sortOrder) //$_SESSION['user_id']
|
"notes" => $this->notesModel->selectNotesForUser(2, $sortBy, $sortOrder) //$_SESSION['user_id']
|
||||||
]);
|
]);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user