From 17ca2da2127604f08738a61cac025d3b8333f567 Mon Sep 17 00:00:00 2001 From: Lunny Xiao Date: Tue, 1 Apr 2025 18:03:27 -0700 Subject: [PATCH] Return default avatar url when user id is zero rather than updating database (#34094) --- models/user/avatar.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/models/user/avatar.go b/models/user/avatar.go index 2a41b9912928b..3d9fc4452f8ab 100644 --- a/models/user/avatar.go +++ b/models/user/avatar.go @@ -61,7 +61,9 @@ func GenerateRandomAvatar(ctx context.Context, u *User) error { // AvatarLinkWithSize returns a link to the user's avatar with size. size <= 0 means default size func (u *User) AvatarLinkWithSize(ctx context.Context, size int) string { - if u.IsGhost() || u.IsGiteaActions() { + // ghost user was deleted, Gitea actions is a bot user, 0 means the user should be a virtual user + // which comes from git configure information + if u.IsGhost() || u.IsGiteaActions() || u.ID <= 0 { return avatars.DefaultAvatarLink() }