]> BookStack Code Mirror - bookstack/commitdiff
Prevented a missing avatar from crashing the application
authorDan Brown <redacted>
Sun, 22 Jan 2017 12:19:50 +0000 (12:19 +0000)
committerDan Brown <redacted>
Sun, 22 Jan 2017 12:19:50 +0000 (12:19 +0000)
app/User.php

index b5bb221e8652a29a2e8dbb19d84ac40799498bd8..afcd9af70c4f8094cea9fa52c9bfc0eaefa43068 100644 (file)
@@ -160,8 +160,16 @@ class User extends Model implements AuthenticatableContract, CanResetPasswordCon
      */
     public function getAvatar($size = 50)
     {
-        if ($this->image_id === 0 || $this->image_id === '0' || $this->image_id === null) return baseUrl('/user_avatar.png');
-        return baseUrl($this->avatar->getThumb($size, $size, false));
+        $default = baseUrl('/user_avatar.png');
+        $imageId = $this->image_id;
+        if ($imageId === 0 || $imageId === '0' || $imageId === null) return $default;
+
+        try {
+            $avatar = baseUrl($this->avatar->getThumb($size, $size, false));
+        } catch (\Exception $err) {
+            $avatar = $default;
+        }
+        return $avatar;
     }
 
     /**