From 1d6ea40022b3806d405ae29c77096300f0c94932 Mon Sep 17 00:00:00 2001 From: Corey Hulen Date: Wed, 15 Feb 2017 18:54:41 -0500 Subject: Adding caching to profile images to master (#5428) * Adding caching to profile images * Fixing break * Adding back in failed read * Fixing build break * Fixing break --- app/user.go | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) (limited to 'app') diff --git a/app/user.go b/app/user.go index 3954a549c..c34bf87e3 100644 --- a/app/user.go +++ b/app/user.go @@ -667,25 +667,28 @@ func CreateProfileImage(username string, userId string) ([]byte, *model.AppError } } -func GetProfileImage(user *model.User) ([]byte, *model.AppError) { +func GetProfileImage(user *model.User) ([]byte, bool, *model.AppError) { var img []byte + readFailed := false if len(utils.Cfg.FileSettings.DriverName) == 0 { var err *model.AppError if img, err = CreateProfileImage(user.Username, user.Id); err != nil { - return nil, err + return nil, false, err } } else { path := "users/" + user.Id + "/profile.png" if data, err := ReadFile(path); err != nil { + readFailed = true + if img, err = CreateProfileImage(user.Username, user.Id); err != nil { - return nil, err + return nil, false, err } if user.LastPictureUpdate == 0 { if err := WriteFile(img, path); err != nil { - return nil, err + return nil, false, err } } @@ -694,7 +697,7 @@ func GetProfileImage(user *model.User) ([]byte, *model.AppError) { } } - return img, nil + return img, readFailed, nil } func SetProfileImage(userId string, imageData *multipart.FileHeader) *model.AppError { -- cgit v1.2.3-1-g7c22