Fixed PHP warning when removing a user with no Avatar image

2 files changed, 9 insertions, 5 deletions

diff --git a/app/Model/AvatarFile.php b/app/Model/AvatarFile.php
index c49f9fd5..36784e60 100644
--- a/app/Model/AvatarFile.php
+++ b/app/Model/AvatarFile.php
@@ -60,14 +60,18 @@ class AvatarFile extends Base
     public function remove($user_id)
     {
         try {
-            $this->objectStorage->remove($this->getFilename($user_id));
-            $result = $this->db->table(User::TABLE)->eq('id', $user_id)->update(array('avatar_path' => ''));
-            $this->userSession->refresh($user_id);
-            return $result;
+            $filename = $this->getFilename($user_id);
+
+            if (! empty($filename)) {
+                $this->objectStorage->remove($filename);
+                return $this->db->table(User::TABLE)->eq('id', $user_id)->update(array('avatar_path' => ''));
+            }
         } catch (Exception $e) {
             $this->logger->error($e->getMessage());
             return false;
         }
+
+        return true;
     }
 
     /**
diff --git a/app/Model/User.php b/app/Model/User.php
index 57993002..6dd78dbb 100644
--- a/app/Model/User.php
+++ b/app/Model/User.php
@@ -22,7 +22,7 @@ class User extends Base
     const TABLE = 'users';
 
     /**
-     * Id used for everbody (filtering)
+     * Id used for everybody (filtering)
      *
      * @var integer
      */