Fixed PHP warning when removing a user with no Avatar image

ChangeLog

@@ -27,6 +27,7 @@ Improvements:
 
 Bug fixes:
 
+* Fixed PHP warning when removing a user with no Avatar image
 * Fixed improper Markdown escaping for some tooltips
 * Closing all tasks by column, also update closed tasks
 * Fixed wrong task link generation within Markdown text

app/Controller/AvatarFile.php

@@ -47,6 +47,7 @@ class AvatarFile extends Base
         $this->checkCSRFParam();
         $user = $this->getUser();
         $this->avatarFile->remove($user['id']);
+        $this->userSession->refresh($user['id']);
         $this->response->redirect($this->helper->url->to('AvatarFile', 'show', array('user_id' => $user['id'])));
     }
 

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;
     }
 
     /**

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
      */