Skip to content

Commit 5d40db6

Browse files
authored
fix random avatars (#1147)
1 parent af9998b commit 5d40db6

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

models/org.go

+3-1
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,9 @@ func CreateOrganization(org, owner *User) (err error) {
135135
if _, err = sess.Insert(org); err != nil {
136136
return fmt.Errorf("insert organization: %v", err)
137137
}
138-
org.GenerateRandomAvatar()
138+
if err = org.generateRandomAvatar(sess); err != nil {
139+
return fmt.Errorf("generate random avatar: %v", err)
140+
}
139141

140142
// Add initial creator to organization and owner team.
141143
if _, err = sess.Insert(&OrgUser{

models/user.go

+8
Original file line numberDiff line numberDiff line change
@@ -287,6 +287,10 @@ func (u *User) CustomAvatarPath() string {
287287

288288
// GenerateRandomAvatar generates a random avatar for user.
289289
func (u *User) GenerateRandomAvatar() error {
290+
return u.generateRandomAvatar(x)
291+
}
292+
293+
func (u *User) generateRandomAvatar(e Engine) error {
290294
seed := u.Email
291295
if len(seed) == 0 {
292296
seed = u.Name
@@ -308,6 +312,10 @@ func (u *User) GenerateRandomAvatar() error {
308312
}
309313
defer fw.Close()
310314

315+
if _, err := e.Id(u.ID).Cols("avatar").Update(u); err != nil {
316+
return err
317+
}
318+
311319
if err = png.Encode(fw, img); err != nil {
312320
return fmt.Errorf("Encode: %v", err)
313321
}

0 commit comments

Comments
 (0)