Remove unnecessary template helper repoAvatar (#26387)

And simplify the "repo/icon" code

modules/templates/helper.go

@@ -59,7 +59,6 @@ func NewFuncMap() template.FuncMap {
 		"avatarHTML":     AvatarHTML,
 		"avatarByAction": AvatarByAction,
 		"avatarByEmail":  AvatarByEmail,
-		"repoAvatar":     RepoAvatar,
 		"EntryIcon":      base.EntryIcon,
 		"MigrationIcon":  MigrationIcon,
 		"ActionIcon":     ActionIcon,

modules/templates/util_avatar.go

@@ -60,17 +60,6 @@ func AvatarByAction(ctx context.Context, action *activities_model.Action, others
 	return Avatar(ctx, action.ActUser, others...)
 }
 
-// RepoAvatar renders repo avatars. args: repo, size(int), class (string)
-func RepoAvatar(repo *repo_model.Repository, others ...any) template.HTML {
-	size, class := gitea_html.ParseSizeAndClass(avatars.DefaultAvatarPixelSize, avatars.DefaultAvatarClass, others...)
-
-	src := repo.RelAvatarLink()
-	if src != "" {
-		return AvatarHTML(src, size, class, repo.FullName())
-	}
-	return template.HTML("")
-}
-
 // AvatarByEmail renders avatars by email address. args: email, name, size (int), class (string)
 func AvatarByEmail(ctx context.Context, email, name string, others ...any) template.HTML {
 	size, class := gitea_html.ParseSizeAndClass(avatars.DefaultAvatarPixelSize, avatars.DefaultAvatarClass, others...)

templates/explore/repo_list.tmpl

@@ -2,12 +2,7 @@
 	{{range .Repos}}
 		<div class="flex-item">
 			<div class="flex-item-leading">
-				{{$avatar := (repoAvatar . 32)}}
+				{{template "repo/icon" .}}
-				{{if $avatar}}
-					{{$avatar}}
-				{{else}}
-					{{template "repo/icon" .}}
-				{{end}}
 			</div>
 			<div class="flex-item-main">
 				<div class="flex-item-header">

templates/org/team/repositories.tmpl

@@ -30,12 +30,7 @@
 						{{range .Team.Repos}}
 							<div class="flex-item flex-item-center">
 								<div class="flex-item-leading">
-									{{$avatar := (repoAvatar . 32)}}
+									{{template "repo/icon" .}}
-									{{if $avatar}}
-										{{$avatar}}
-									{{else}}
-										{{template "repo/icon" .}}
-									{{end}}
 								</div>
 								<div class="flex-item-main">
 									<a class="flex-item-title text primary" href="{{$.Org.HomeLink}}/{{.Name | PathEscape}}">

templates/repo/header.tmpl

@@ -4,14 +4,9 @@
 		<div class="repo-header">
 			<div class="repo-title-wrap gt-df gt-fc">
 				<div class="repo-title" role="heading" aria-level="1">
-					{{$avatar := (repoAvatar . 32 "gt-mr-3")}}
+					<div class="gt-mr-3">
-					{{if $avatar}}
+						{{template "repo/icon" .}}
-						{{$avatar}}
+					</div>
-					{{else}}
-						<div class="gt-mr-3">
-							{{template "repo/icon" .}}
-						</div>
-					{{end}}
 					<a href="{{.Owner.HomeLink}}">{{.Owner.Name}}</a>
 					<div class="gt-mx-2">/</div>
 					<a href="{{$.RepoLink}}">{{.Name}}</a>

templates/repo/icon.tmpl

@@ -1,15 +1,16 @@
 <div class="repo-icon">
-	{{if $.IsTemplate}}
+	{{$avatarLink := .RelAvatarLink}}
+	{{if $avatarLink}}
+		<img class="ui avatar gt-vm" src="{{$avatarLink}}" width="32" height="32" alt="{{.FullName}}">
+	{{else if $.IsTemplate}}
 		{{svg "octicon-repo-template" 32}}
+	{{else if $.IsPrivate}}
+		{{svg "octicon-lock" 32}}
+	{{else if $.IsMirror}}
+		{{svg "octicon-mirror" 32}}
+	{{else if $.IsFork}}
+		{{svg "octicon-repo-forked" 32}}
 	{{else}}
-		{{if $.IsPrivate}}
+		{{svg "octicon-repo" 32}}
-			{{svg "octicon-lock" 32}}
-		{{else if $.IsMirror}}
-			{{svg "octicon-mirror" 32}}
-		{{else if $.IsFork}}
-			{{svg "octicon-repo-forked" 32}}
-		{{else}}
-			{{svg "octicon-repo" 32}}
-		{{end}}
 	{{end}}
 </div>