Start on rewriting classes for de-oopsie-ing

This commit is contained in:
Michał Gdula 2022-10-06 12:05:25 +00:00
parent be7d4b5b43
commit 7f1782d7c6

View file

@ -106,11 +106,17 @@ class Account {
Returns True if user is Returns True if user is
Returns False if user is NOT Returns False if user is NOT
*/ */
function is_loggedin() { function is_loggedin($conn) {
if (isset($_SESSION["loggedin"]) && $_SESSION["loggedin"] === true) { $error = 0;
return True;
if (!isset($_SESSION["loggedin"]) || $_SESSION["loggedin"] == false) $error += 1;
if (empty($this->get_user_info($conn, $_SESSION["id"])) || $this->get_user_info($conn, $_SESSION["id"]) == null) $error += 1;
if ($error > 0) {
return false;
} else { } else {
return False; return true;
} }
} }
/* /*
@ -143,7 +149,7 @@ class Account {
Returns False if user is NOT privilaged Returns False if user is NOT privilaged
*/ */
function is_admin($conn, $id) { function is_admin($conn, $id) {
if (isset($id) || !empty($id)) { if (isset($id) && !empty($id)) {
// Setting SQL query // Setting SQL query
$sql = "SELECT admin FROM users WHERE id = ?"; $sql = "SELECT admin FROM users WHERE id = ?";