From 309e86a9bf305e807ead2854fa757c4d704dcfce Mon Sep 17 00:00:00 2001
From: KN4CK3R <admin@oldschoolhack.me>
Date: Thu, 29 Dec 2022 00:31:54 +0100
Subject: [PATCH] Use dynamic package type list (#22263)

Replace the hardcoded list with the dynamic list.
---
 routers/web/admin/packages.go      |  1 +
 routers/web/repo/packages.go       |  1 +
 routers/web/user/package.go        |  1 +
 templates/admin/packages/list.tmpl | 15 +++------------
 templates/package/shared/list.tmpl | 15 +++------------
 5 files changed, 9 insertions(+), 24 deletions(-)

diff --git a/routers/web/admin/packages.go b/routers/web/admin/packages.go
index 80cec51275cf..7c6d1ed840f5 100644
--- a/routers/web/admin/packages.go
+++ b/routers/web/admin/packages.go
@@ -62,6 +62,7 @@ func Packages(ctx *context.Context) {
 	ctx.Data["PageIsAdminPackages"] = true
 	ctx.Data["Query"] = query
 	ctx.Data["PackageType"] = packageType
+	ctx.Data["AvailableTypes"] = packages_model.TypeList
 	ctx.Data["SortType"] = sort
 	ctx.Data["PackageDescriptors"] = pds
 	ctx.Data["Total"] = total
diff --git a/routers/web/repo/packages.go b/routers/web/repo/packages.go
index 83324711da72..6ad2f71b5c2d 100644
--- a/routers/web/repo/packages.go
+++ b/routers/web/repo/packages.go
@@ -61,6 +61,7 @@ func Packages(ctx *context.Context) {
 	ctx.Data["ContextUser"] = ctx.ContextUser
 	ctx.Data["Query"] = query
 	ctx.Data["PackageType"] = packageType
+	ctx.Data["AvailableTypes"] = packages.TypeList
 	ctx.Data["HasPackages"] = hasPackages
 	if ctx.Repo != nil {
 		ctx.Data["CanWritePackages"] = ctx.IsUserRepoWriter([]unit.Type{unit.TypePackages}) || ctx.IsUserSiteAdmin()
diff --git a/routers/web/user/package.go b/routers/web/user/package.go
index 3782f46b4242..c0aba7583fc0 100644
--- a/routers/web/user/package.go
+++ b/routers/web/user/package.go
@@ -89,6 +89,7 @@ func ListPackages(ctx *context.Context) {
 	ctx.Data["ContextUser"] = ctx.ContextUser
 	ctx.Data["Query"] = query
 	ctx.Data["PackageType"] = packageType
+	ctx.Data["AvailableTypes"] = packages_model.TypeList
 	ctx.Data["HasPackages"] = hasPackages
 	ctx.Data["PackageDescriptors"] = pds
 	ctx.Data["Total"] = total
diff --git a/templates/admin/packages/list.tmpl b/templates/admin/packages/list.tmpl
index 3aab2873c6b7..c39f5fc12881 100644
--- a/templates/admin/packages/list.tmpl
+++ b/templates/admin/packages/list.tmpl
@@ -13,18 +13,9 @@
 					<select class="ui dropdown" name="type">
 						<option value="">{{.locale.Tr "packages.filter.type"}}</option>
 						<option value="all">{{.locale.Tr "packages.filter.type.all"}}</option>
-						<option value="composer" {{if eq .PackageType "composer"}}selected="selected"{{end}}>Composer</option>
-						<option value="conan" {{if eq .PackageType "conan"}}selected="selected"{{end}}>Conan</option>
-						<option value="container" {{if eq .PackageType "container"}}selected="selected"{{end}}>Container</option>
-						<option value="generic" {{if eq .PackageType "generic"}}selected="selected"{{end}}>Generic</option>
-						<option value="helm" {{if eq .PackageType "helm"}}selected="selected"{{end}}>Helm</option>
-						<option value="maven" {{if eq .PackageType "maven"}}selected="selected"{{end}}>Maven</option>
-						<option value="npm" {{if eq .PackageType "npm"}}selected="selected"{{end}}>npm</option>
-						<option value="nuget" {{if eq .PackageType "nuget"}}selected="selected"{{end}}>NuGet</option>
-						<option value="pub" {{if eq .PackageType "pub"}}selected="selected"{{end}}>Pub</option>
-						<option value="pypi" {{if eq .PackageType "pypi"}}selected="selected"{{end}}>PyPi</option>
-						<option value="rubygems" {{if eq .PackageType "rubygems"}}selected="selected"{{end}}>RubyGems</option>
-						<option value="vagrant" {{if eq .PackageType "vagrant"}}selected="selected"{{end}}>Vagrant</option>
+						{{range $type := .AvailableTypes}}
+						<option{{if eq $.PackageType $type}} selected="selected"{{end}} value="{{$type}}">{{$type.Name}}</option>
+						{{end}}
 					</select>
 					<button class="ui primary button">{{.locale.Tr "explore.search"}}</button>
 				</div>
diff --git a/templates/package/shared/list.tmpl b/templates/package/shared/list.tmpl
index 37c47cef335d..ec2e88c8541d 100644
--- a/templates/package/shared/list.tmpl
+++ b/templates/package/shared/list.tmpl
@@ -6,18 +6,9 @@
 			<select class="ui dropdown" name="type">
 				<option value="">{{.locale.Tr "packages.filter.type"}}</option>
 				<option value="all">{{.locale.Tr "packages.filter.type.all"}}</option>
-				<option value="composer" {{if eq .PackageType "composer"}}selected="selected"{{end}}>Composer</option>
-				<option value="conan" {{if eq .PackageType "conan"}}selected="selected"{{end}}>Conan</option>
-				<option value="container" {{if eq .PackageType "container"}}selected="selected"{{end}}>Container</option>
-				<option value="generic" {{if eq .PackageType "generic"}}selected="selected"{{end}}>Generic</option>
-				<option value="helm" {{if eq .PackageType "helm"}}selected="selected"{{end}}>Helm</option>
-				<option value="maven" {{if eq .PackageType "maven"}}selected="selected"{{end}}>Maven</option>
-				<option value="npm" {{if eq .PackageType "npm"}}selected="selected"{{end}}>npm</option>
-				<option value="nuget" {{if eq .PackageType "nuget"}}selected="selected"{{end}}>NuGet</option>
-				<option value="pub" {{if eq .PackageType "pub"}}selected="selected"{{end}}>Pub</option>
-				<option value="pypi" {{if eq .PackageType "pypi"}}selected="selected"{{end}}>PyPi</option>
-				<option value="rubygems" {{if eq .PackageType "rubygems"}}selected="selected"{{end}}>RubyGems</option>
-				<option value="vagrant" {{if eq .PackageType "vagrant"}}selected="selected"{{end}}>Vagrant</option>
+				{{range $type := .AvailableTypes}}
+				<option{{if eq $.PackageType $type}} selected="selected"{{end}} value="{{$type}}">{{$type.Name}}</option>
+				{{end}}
 			</select>
 			<button class="ui primary button">{{.locale.Tr "explore.search"}}</button>
 		</div>