show fullname only it is not empty

HesterG · HesterG · commit aded8c1c979c · 2023-03-14T17:46:45.000+08:00
diff --git a/models/user/user.go b/models/user/user.go
@@ -423,7 +423,10 @@ func (u *User) GetDisplayName() string {
 
 func (u *User) GetSearchNameHTML() template.HTML {
 	if setting.UI.DefaultShowFullName {
-		return template.HTML(fmt.Sprintf(`%s<span class="text search-fullname"> %s</span>`, u.Name, strings.TrimSpace(u.FullName)))
+		trimmed := strings.TrimSpace(u.FullName)
+		if len(trimmed) > 0 {
+			return template.HTML(fmt.Sprintf(`%s<span class="text search-fullname"> %s</span>`, u.Name, trimmed))
+		}
 	}
 	return template.HTML(u.Name)
 }

Original file line number	Diff line number	Diff line change
`@@ -423,7 +423,10 @@ func (u *User) GetDisplayName() string {`
`423`	`423`
`424`	`424`	`func (u *User) GetSearchNameHTML() template.HTML {`
`425`	`425`	`if setting.UI.DefaultShowFullName {`
`426`		- return template.HTML(fmt.Sprintf(`%s<span class="text search-fullname"> %s</span>`, u.Name, strings.TrimSpace(u.FullName)))
	`426`	`+ trimmed := strings.TrimSpace(u.FullName)`
	`427`	`+ if len(trimmed) > 0 {`
	`428`	+ return template.HTML(fmt.Sprintf(`%s<span class="text search-fullname"> %s</span>`, u.Name, trimmed))
	`429`	`+ }`
`427`	`430`	`}`
`428`	`431`	`return template.HTML(u.Name)`
`429`	`432`	`}`