From f82dd73b7de91479722ff1d3a63ffb4fbb465bcd Mon Sep 17 00:00:00 2001
From: Marijn Jansen <marijn@jansen.onl>
Date: Mon, 23 Jan 2017 16:23:47 +0100
Subject: [PATCH] Profile picrure old deletion (:

---
 website/public/settings.php  |  5 ++---
 website/queries/settings.php | 20 +++++++++++++++-----
 2 files changed, 17 insertions(+), 8 deletions(-)
diff --git a/website/public/settings.php b/website/public/settings.php
index 06d17c0..b473f99 100644
--- a/website/public/settings.php
+++ b/website/public/settings.php
@@ -2,7 +2,7 @@
 <html>
 <head>
     <?php
-    include("../views/head.php");
+    include_once("../views/head.php");
     include_once("../queries/connect.php");
     include_once("../queries/settings.php");
     ?>
@@ -12,8 +12,6 @@
 </head>
 <body>
 <?php
-
-include("../views/main.php");
 $notImplemented = new settingsMessage("angry", "Deze functie werkt nog niet :(");
 
 if ($_SERVER["REQUEST_METHOD"] == "POST") {
@@ -33,6 +31,7 @@ if ($_SERVER["REQUEST_METHOD"] == "POST") {
             break;
     }
 }
+include("../views/main.php");
 
 include("../views/settings-view.php");
 
diff --git a/website/queries/settings.php b/website/queries/settings.php
index f4c7011..c4e209d 100644
--- a/website/queries/settings.php
+++ b/website/queries/settings.php
@@ -189,15 +189,25 @@ function doChangeEmail($email) {
 function updateProfilePicture() {
     $profilePictureDir = "/var/www/html/public/";
     $relativePath = "uploads/profilepictures/" . $_SESSION["userID"] . "_" . basename($_FILES["pp"]["name"]);
-//    removeOldProfilePicture();
+    removeOldProfilePicture();
     move_uploaded_file($_FILES['pp']['tmp_name'], $profilePictureDir . $relativePath);
     setProfilePictureToDatabase("../" . $relativePath);
 }
 
-//function removeOldProfilePicture() {
-//
-//    unlink("/var/www/html/public/uploads/profilepictures/" . $_SESSION["userID"] . "_*");
-//}
+function removeOldProfilePicture() {
+    $stmt = $GLOBALS["db"]->prepare("
+    SELECT
+        `profilepicture`
+     FROM 
+        `user`
+    WHERE 
+        `userID` = :userID
+    ");
+    $stmt->bindParam(":userID", $_SESSION["userID"]);
+    $stmt->execute();
+    $old_avatar = $stmt->fetch()["profilepicture"];
+    unlink("/var/www/html/public/uploads/" . $old_avatar);
+}
 
 function setProfilePictureToDatabase($url) {
     $stmt = $GLOBALS["db"]->prepare("
