From 42a0e2f72fed0a947242b004a0fc6eb4a664174b Mon Sep 17 00:00:00 2001
From: Syndamia <kamen.d.mladenov@protonmail.com>
Date: Sat, 3 Apr 2021 16:04:28 +0300
Subject: Updated all double quotes to single quotes where possible

---
 src/controllers/BaseController.php           | 17 ++++++++---------
 src/controllers/CreateOfferController.php    |  4 ++--
 src/controllers/DashboardController.php      |  8 ++++----
 src/controllers/DashboardLoginController.php |  8 ++++----
 src/controllers/EditOfferController.php      |  2 +-
 src/controllers/LandingController.php        |  6 +++---
 6 files changed, 22 insertions(+), 23 deletions(-)

(limited to 'src/controllers')

diff --git a/src/controllers/BaseController.php b/src/controllers/BaseController.php
index 2c2a647..f766398 100644
--- a/src/controllers/BaseController.php
+++ b/src/controllers/BaseController.php
@@ -1,14 +1,14 @@
 <?php
 class BaseController {
 	public static function invoke() {
-		switch ($_SERVER["REQUEST_METHOD"]) {
-			case "POST": 
+		switch ($_SERVER['REQUEST_METHOD']) {
+			case 'POST': 
 				static::post();
 				break;
-			case "PUT":
+			case 'PUT':
 				static::put();
 				break;
-			case "DELETE":
+			case 'DELETE':
 				static::delete();
 				break;
 			default:
@@ -18,19 +18,18 @@ class BaseController {
 	}
 
 	protected static function get() {
-		echo "GET method not implemented!";
+		echo 'GET method not implemented!';
 	}
 
 	protected static function post() {
-
-		echo "POST method not implemented!";
+		echo 'POST method not implemented!';
 	}
 
 	protected static function put() {
-		echo "PUT method not implemented!";
+		echo 'PUT method not implemented!';
 	}
 
 	protected static function delete() {
-		echo "DELETE method not implemented!";
+		echo 'DELETE method not implemented!';
 	}
 }
diff --git a/src/controllers/CreateOfferController.php b/src/controllers/CreateOfferController.php
index a223463..bb6015b 100644
--- a/src/controllers/CreateOfferController.php
+++ b/src/controllers/CreateOfferController.php
@@ -7,7 +7,7 @@ class CreateOfferController extends BaseController {
 
 	public static function get() {
 		self::$companies = Company::getAllFromDB();
-		include 'views/create-offer.php';
+		include('views/create-offer.php');
 	}
 
 	public static function post() {
@@ -15,7 +15,7 @@ class CreateOfferController extends BaseController {
 		JobOffer::insertIntoDB($newJobOffer);
 
 		// Redirect to landing page
-		header("Location: /", TRUE, 301);
+		header('Location: /', TRUE, 301);
 		exit();
 	}
 }
diff --git a/src/controllers/DashboardController.php b/src/controllers/DashboardController.php
index 548f60b..80717ca 100644
--- a/src/controllers/DashboardController.php
+++ b/src/controllers/DashboardController.php
@@ -12,18 +12,18 @@ class DashboardController extends BaseController {
 		foreach(Company::getAllFromDB() as $company) {
 			self::$companies[$company->id] = $company->name;
 		}
-		include "views/dashboard.php";
+		include('views/dashboard.php');
 	}
 
 	public static function post() {
 		switch($_POST['_method']) {
-			case "get":
+			case 'get':
 				EditOfferController::get();
 				break;
-			case "put":
+			case 'put':
 				EditOfferController::put();
 				break;
-			case "delete":
+			case 'delete':
 				EditOfferController::delete();
 				break;
 		}
diff --git a/src/controllers/DashboardLoginController.php b/src/controllers/DashboardLoginController.php
index fe2009b..af1113e 100644
--- a/src/controllers/DashboardLoginController.php
+++ b/src/controllers/DashboardLoginController.php
@@ -6,20 +6,20 @@ class DashboardLoginController extends BaseController {
 	private static $message;
 
 	public static function get() {
-		include "views/dashboard-login.php";
+		include('views/dashboard-login.php');
 	}
 
 	public static function post() {
 		if (isset($_POST['_method'])) {
 			DashboardController::invoke();
 		}
-		else if ($_POST['username'] == "admin" && $_POST['password'] == "admin") {
+		else if ($_POST['username'] == 'admin' && $_POST['password'] == 'admin') {
 			// Load normal dashboard
 			DashboardController::get();
 		}
 		else {
-			self::$message = "Bad credentials!";
-			include "views/dashboard-login.php";
+			self::$message = 'Bad credentials!';
+			include('views/dashboard-login.php');
 		}
 	}
 }
diff --git a/src/controllers/EditOfferController.php b/src/controllers/EditOfferController.php
index ae49c5b..dc9a753 100644
--- a/src/controllers/EditOfferController.php
+++ b/src/controllers/EditOfferController.php
@@ -11,7 +11,7 @@ class EditOfferController extends BaseController {
 	public static function get() {
 		self::$job = JobOffer::getFromDBById($_POST['jobId']);
 		self::$companies = Company::getAllFromDB();
-		include "views/edit-offer.php";
+		include('views/edit-offer.php');
 	}
 
 	public static function put() {
diff --git a/src/controllers/LandingController.php b/src/controllers/LandingController.php
index ffa8644..9e81751 100644
--- a/src/controllers/LandingController.php
+++ b/src/controllers/LandingController.php
@@ -1,7 +1,7 @@
 <?php
-include_once("models/JobOffer.php");
+include_once('models/JobOffer.php');
 include_once('models/Company.php');
-include_once("controllers/BaseController.php");
+include_once('controllers/BaseController.php');
 
 class LandingController extends BaseController {
 	private static $companies = array();
@@ -11,6 +11,6 @@ class LandingController extends BaseController {
 		foreach(Company::getAllFromDB() as $company) {
 			self::$companies[$company->id] = $company->name;
 		}
-		include "views/landing.php";
+		include('views/landing.php');
 	}
 }
-- 
cgit v1.3.1-23-gd82d


From 976b399f90fc5c0152f87e474e61e6065c0e7a5f Mon Sep 17 00:00:00 2001
From: Syndamia <kamen.d.mladenov@protonmail.com>
Date: Sat, 3 Apr 2021 16:25:24 +0300
Subject: Updated the comments and documentation in some controllers

---
 src/controllers/BaseController.php           | 4 ++++
 src/controllers/DashboardController.php      | 3 +++
 src/controllers/DashboardLoginController.php | 3 +++
 3 files changed, 10 insertions(+)

(limited to 'src/controllers')

diff --git a/src/controllers/BaseController.php b/src/controllers/BaseController.php
index f766398..412cf25 100644
--- a/src/controllers/BaseController.php
+++ b/src/controllers/BaseController.php
@@ -1,5 +1,9 @@
 <?php
 class BaseController {
+	/**
+	 * Calls the get, post, put or delete static methods, depending on the server request method ($_SERVER['REQUEST_METHOD'])
+	 * **By default** all requests that are not POST, PUT or DELETE execute the GET request method
+	 */
 	public static function invoke() {
 		switch ($_SERVER['REQUEST_METHOD']) {
 			case 'POST': 
diff --git a/src/controllers/DashboardController.php b/src/controllers/DashboardController.php
index 80717ca..59c1090 100644
--- a/src/controllers/DashboardController.php
+++ b/src/controllers/DashboardController.php
@@ -16,6 +16,9 @@ class DashboardController extends BaseController {
 	}
 
 	public static function post() {
+		// Forms can only send POST or GET requests.
+		// EditOfferController doesn't have it's own URI, so we can't send a request.
+		// We also can't just invoke it, because that means manually changing the server's request method.
 		switch($_POST['_method']) {
 			case 'get':
 				EditOfferController::get();
diff --git a/src/controllers/DashboardLoginController.php b/src/controllers/DashboardLoginController.php
index af1113e..c6ac070 100644
--- a/src/controllers/DashboardLoginController.php
+++ b/src/controllers/DashboardLoginController.php
@@ -10,6 +10,9 @@ class DashboardLoginController extends BaseController {
 	}
 
 	public static function post() {
+		// The dashboard(controller) doesn't have it's own URI (and all requests are sent in this controller), so
+		// we can't send a request directly to dashboard.
+		// This does compromise security, but for now it will have to do.
 		if (isset($_POST['_method'])) {
 			DashboardController::invoke();
 		}
-- 
cgit v1.3.1-23-gd82d