Added check for disabled Packages (#21540) (#21614)

Backport #21540

At the moment, If admin disable Packages, still show the Packages on the
admin dashboard.

This patch added a check to hide the Packages entry.
tokarchuk/v1.17
Xinyu Zhou 2 years ago committed by GitHub
parent 291787a5ef
commit 43a8547df6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 2
      routers/web/web.go
  2. 8
      templates/admin/navbar.tmpl

@ -566,6 +566,8 @@ func RegisterRoutes(m *web.Route) {
m.Post("/delete", admin.DeleteNotices) m.Post("/delete", admin.DeleteNotices)
m.Post("/empty", admin.EmptyNotices) m.Post("/empty", admin.EmptyNotices)
}) })
}, func(ctx *context.Context) {
ctx.Data["EnablePackages"] = setting.Packages.Enabled
}, adminReq) }, adminReq)
// ***** END: Admin ***** // ***** END: Admin *****

@ -12,9 +12,11 @@
<a class="{{if .PageIsAdminRepositories}}active{{end}} item" href="{{AppSubUrl}}/admin/repos"> <a class="{{if .PageIsAdminRepositories}}active{{end}} item" href="{{AppSubUrl}}/admin/repos">
{{.i18n.Tr "admin.repositories"}} {{.i18n.Tr "admin.repositories"}}
</a> </a>
<a class="{{if .PageIsAdminPackages}}active{{end}} item" href="{{AppSubUrl}}/admin/packages"> {{if .EnablePackages}}
{{.i18n.Tr "packages.title"}} <a class="{{if .PageIsAdminPackages}}active{{end}} item" href="{{AppSubUrl}}/admin/packages">
</a> {{.i18n.Tr "packages.title"}}
</a>
{{end}}
{{if not DisableWebhooks}} {{if not DisableWebhooks}}
<a class="{{if or .PageIsAdminDefaultHooks .PageIsAdminSystemHooks}}active{{end}} item" href="{{AppSubUrl}}/admin/hooks"> <a class="{{if or .PageIsAdminDefaultHooks .PageIsAdminSystemHooks}}active{{end}} item" href="{{AppSubUrl}}/admin/hooks">
{{.i18n.Tr "admin.hooks"}} {{.i18n.Tr "admin.hooks"}}

Loading…
Cancel
Save