diff options
author | qwerty287 <80460567+qwerty287@users.noreply.github.com> | 2021-11-24 04:51:08 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-11-24 11:51:08 +0800 |
commit | 754fdd8f9c2b1e4e78d507fd414968334cf586fd (patch) | |
tree | 58cbb72fe628059aab9d3857f8eed0c5dcaebec0 /modules | |
parent | 21f4401f3e04885da782d77329dedaa35e820997 (diff) | |
download | gitea-754fdd8f9c2b1e4e78d507fd414968334cf586fd.tar.gz gitea-754fdd8f9c2b1e4e78d507fd414968334cf586fd.zip |
Fix visibility of org avatars (#17789)
* Fix visibility of org avatar
* more clear syntax
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
Diffstat (limited to 'modules')
-rw-r--r-- | modules/templates/helper.go | 20 |
1 files changed, 13 insertions, 7 deletions
diff --git a/modules/templates/helper.go b/modules/templates/helper.go index 8b46ed40ce..aec3596bd4 100644 --- a/modules/templates/helper.go +++ b/modules/templates/helper.go @@ -553,18 +553,24 @@ func SVG(icon string, others ...interface{}) template.HTML { func Avatar(item interface{}, others ...interface{}) template.HTML { size, class := parseOthers(avatars.DefaultAvatarPixelSize, "ui avatar image", others...) - if user, ok := item.(*models.User); ok { - src := user.AvatarLinkWithSize(size * avatars.AvatarRenderedSizeFactor) + switch t := item.(type) { + case *models.User: + src := t.AvatarLinkWithSize(size * avatars.AvatarRenderedSizeFactor) if src != "" { - return AvatarHTML(src, size, class, user.DisplayName()) + return AvatarHTML(src, size, class, t.DisplayName()) } - } - if user, ok := item.(*models.Collaborator); ok { - src := user.AvatarLinkWithSize(size * avatars.AvatarRenderedSizeFactor) + case *models.Collaborator: + src := t.AvatarLinkWithSize(size * avatars.AvatarRenderedSizeFactor) + if src != "" { + return AvatarHTML(src, size, class, t.DisplayName()) + } + case *models.Organization: + src := t.AsUser().AvatarLinkWithSize(size * avatars.AvatarRenderedSizeFactor) if src != "" { - return AvatarHTML(src, size, class, user.DisplayName()) + return AvatarHTML(src, size, class, t.AsUser().DisplayName()) } } + return template.HTML("") } |