From 7c3f2e348c015ea93563d866f89ec8cea9159ea0 Mon Sep 17 00:00:00 2001
From: cflip <36554078+cflip@users.noreply.github.com>
Date: Sat, 24 Apr 2021 09:40:20 -0600
Subject: Refactoring part 2

Starting to move some functionality such as the session and database connection into singleton classes to manage them. Functions for modifying posts and threads are being put in one place as well.
---
 moderate.php | 25 +++++++++++--------------
 1 file changed, 11 insertions(+), 14 deletions(-)

(limited to 'moderate.php')

diff --git a/moderate.php b/moderate.php
index afeefa1..3b8d05d 100644
--- a/moderate.php
+++ b/moderate.php
@@ -1,25 +1,22 @@
 <?php
-
-include_once 'includes/db_inc.php';
-include_once 'model/User.php';
+include_once './includes/db_inc.php';
+include_once './includes/functions_thread.php';
+include_once './includes/Session.php';
+include_once './includes/model/User.php';
 
 session_start();
 
-function delete_thread($dbc, $thread_id) {
-	$sql = "DELETE FROM threads WHERE thread_id = $thread_id;";
-	mysqli_query($dbc, $sql);
-}
-
-if ($_SERVER['REQUEST_METHOD'] == 'POST' && isset($_SESSION['signed_in'])) {
+if ($_SERVER['REQUEST_METHOD'] == 'POST') {
 	$thread_id = filter_input(INPUT_POST, 'id', FILTER_SANITIZE_NUMBER_INT);
 	$delete = filter_input(INPUT_POST, "delete", FILTER_SANITIZE_STRING);
-	
-	$user = new User();
-	$user->get_by_id($_SESSION['user_id'], $dbc);
 
-	if ($user->level > 0) {
+	$user = Session::get()->get_current_user();
+
+	if ($user->level == USER_LEVEL_MODERATOR) {
 		if (strcasecmp($delete, "on") == 0) {
-			delete_thread($dbc, $thread_id);
+			$thread = new Thread();
+			$thread->get_from_database($thread_id, $dbc);
+			delete_thread($thread);
 
 			header("Location: /");
 			exit();
-- 
cgit v1.2.3


From 2098bf444afadcf0363d89b4cc1dca5d2213d754 Mon Sep 17 00:00:00 2001
From: cflip <36554078+cflip@users.noreply.github.com>
Date: Sat, 24 Apr 2021 19:40:50 -0600
Subject: Remove all uses of db_inc.php

This method of importing the database login every time wasn't very good.
Now everything uses the new Database singleton class.
---
 moderate.php | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

(limited to 'moderate.php')

diff --git a/moderate.php b/moderate.php
index 3b8d05d..68bf1b9 100644
--- a/moderate.php
+++ b/moderate.php
@@ -1,5 +1,4 @@
 <?php
-include_once './includes/db_inc.php';
 include_once './includes/functions_thread.php';
 include_once './includes/Session.php';
 include_once './includes/model/User.php';
@@ -15,7 +14,7 @@ if ($_SERVER['REQUEST_METHOD'] == 'POST') {
 	if ($user->level == USER_LEVEL_MODERATOR) {
 		if (strcasecmp($delete, "on") == 0) {
 			$thread = new Thread();
-			$thread->get_from_database($thread_id, $dbc);
+			$thread->get_from_database($thread_id);
 			delete_thread($thread);
 
 			header("Location: /");
-- 
cgit v1.2.3