Settings messages now uses a Class (:

website/public/settings.php

@@ -5,6 +5,7 @@
     include("../views/head.php");
     include_once("../queries/connect.php");
     include_once("../queries/settings.php");
+    $_SESSION["userID"] = 2;
     ?>
     <style>
         @import url("styles/settings.css");
@@ -14,6 +15,7 @@
 <?php
 
 include("../views/main.php");
+$notImplemented = new settingsMessage("angry", "Deze functie werkt nog niet :(");
 
 if ($_SERVER["REQUEST_METHOD"] == "POST") {
     switch ($_POST["form"]) {
@@ -24,16 +26,10 @@ if ($_SERVER["REQUEST_METHOD"] == "POST") {
             $result = updatePassword();
             break;
         case "email":
-            $result = array (
+            $result = $notImplemented;
-                "type" => "settings-message-angry",
-                "message" => "Deze functie werkt nog niet :("
-            );
             break;
         case "picture":
-            $result = array (
+            $result = $notImplemented;
-                "type" => "settings-message-angry",
-                "message" => "Deze functie werkt nog niet :("
-            );
             break;
     }
 }

website/queries/settings.php

@@ -1,5 +1,42 @@
 <?php
 
+class settingsMessage {
+    private $class;
+    private $message;
+
+    /**
+     * settingsMessage constructor.
+     * @param string $type Happy or angry
+     * @param string $message The message to display
+     */
+    public function __construct($type, $message) {
+        $this->message = $message;
+        switch ($type) {
+            case "happy":
+                $this->class = "settings-message-happy";
+                break;
+            case "angry":
+                $this->class = "settings-message-angry";
+                break;
+            default:
+                $this->class = "settings-message";
+                break;
+        }
+    }
+
+    public function getClass() {
+        return $this->class;
+    }
+
+    public function getMessage() {
+        return $this->message;
+    }
+}
+
+/**
+ * Gets the settings form the database.
+ * @return mixed Setting as an array.
+ */
 function getSettings() {
     $stmt = $GLOBALS["db"]->prepare("
     SELECT
@@ -59,39 +96,27 @@ function updateSettings() {
 
     $stmt->execute();
 
-    return array (
+    return new settingsMessage("happy", "Instellingen zijn opgeslagen.");
-        "type" => "settings-message-happy",
-        "message" => "Instellingen zijn opgeslagen."
-        );
 }
 
 function updatePassword() {
     $user = getPasswordHash();
-    if (password_verify($_POST["password-old"].strtolower($user["username"]), $user["password"])) {
+    if (password_verify($_POST["password-old"], $user["password"])) {
         if ($_POST["password-new"] == $_POST["password-confirm"] && (strlen($_POST["password-new"]) >= 8)) {
-            if (changePassword($user)) {
+            if (changePassword()) {
-                return array ("type" => "settings-message-happy",
+                return new settingsMessage("happy", "Wachtwoord gewijzigd.");
-                    "message" => "Wachtwoord gewijzigd.");
             } else {
-                return array (
+                return new settingsMessage("settings-message-angry", "Er is iets mis gegaan.");
-                "type" => "settings-message-angry",
-                "message" => "Er is iets mis gegaan.");
             }
         } else {
-            return array (
+            return new settingsMessage("settings-message-angry", "Wachtwoorden komen niet oveeen.");
-                "type" => "settings-message-angry",
-                "message" => "Wachtwoorden komen niet oveeen."
-            );
         }
     } else {
-        return array(
+        return new settingsMessage("settings-message-angry", "Oud wachtwoord niet correct.");
-            "type" => "settings-message-angry",
-            "message" => "Oud wachtwoord niet correct."
-        );
     }
 }
 
-function changePassword($user) {
+function changePassword() {
     $stmt =$GLOBALS["db"]->prepare("
     UPDATE
       `user`
@@ -101,7 +126,7 @@ function changePassword($user) {
       `userID` = :userID
     ");
 
-    $hashed_password = password_hash($_POST["password-new"].strtolower($user["username"]), PASSWORD_DEFAULT);
+    $hashed_password = password_hash($_POST["password-new"], PASSWORD_DEFAULT);
     $stmt->bindParam(":new_password", $hashed_password);
     $stmt->bindParam(":userID", $_SESSION["userID"]);
     $stmt->execute();

website/views/settings-view.php

@@ -6,9 +6,9 @@ $settings = getSettings();
     <div class="settings">
         <?php
         if ($_SERVER["REQUEST_METHOD"] == "POST") {
-            echo "<div class='platform settings-message ${result["type"]}'>
+            echo "<div class='platform settings-message ". $result->getClass()."'>".
-            ${result["message"]}
+            $result->getMessage().
-        </div>";
+        "</div>";
         }
         ?>
         <form class="settings-profile platform" method="post">