From 9d85247dabdc409cf6f595b8f7d792e116a4d92f Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Fri, 26 Jan 2007 20:15:54 +0000
Subject: [PATCH] MDL-8319 avatar image is now deleted from disk, user/pix.php
 redirects to default picture if user picture not present(===deleted)

---
 user/editlib.php |  5 ++++-
 user/pix.php     | 14 +++++---------
 2 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/user/editlib.php b/user/editlib.php
index b7eb065fdc..3fde1226a2 100644
--- a/user/editlib.php
+++ b/user/editlib.php
@@ -20,8 +20,11 @@ function useredit_update_user_preference($usernew) {
 }
 
 function useredit_update_picture(&$usernew, &$userform) {
+    global $CFG;
+
     if (isset($usernew->deletepicture) and $usernew->deletepicture) {
-        //TODO - delete the file
+        $location = $CFG->dataroot.'/users/'.$usernew->id;
+        @remove_dir($location);
         set_field('user', 'picture', 0, 'id', $usernew->id);
     } else if ($usernew->picture = save_profile_image($usernew->id, $userform->get_um(), 'users')) {
         set_field('user', 'picture', 1, 'id', $usernew->id);
diff --git a/user/pix.php b/user/pix.php
index 0fdf641170..b3435be9c8 100644
--- a/user/pix.php
+++ b/user/pix.php
@@ -19,15 +19,11 @@
         $userid   = (integer)$args[0];
         $image    = $args[1];
         $pathname = $CFG->dataroot.'/users/'.$userid.'/'.$image;
-    } else {
-        $image    = 'f1.png';
-        $pathname = $CFG->dirroot.'/pix/u/f1.png';
+        if (file_exists($pathname) and !is_dir($pathname)) {
+            send_file($pathname, $image);
+        }
     }
 
-    if (file_exists($pathname) and !is_dir($pathname)) {
-        send_file($pathname, $image);
-    } else {
-        header('HTTP/1.0 404 not found');
-        error(get_string('filenotfound', 'error')); //this is not displayed on IIS??
-    }
+    // picture was deleted - use default instead
+    redirect($CFG->pixpath.'/u/f1.png');
 ?>
-- 
2.39.5