Remove redundant "RouteMethods" method (#26024)

The `RouteMethods` is mainly an alias for `Methods` with different
argument order.

Remove it to keep the "route.go" code clear
This commit is contained in:
wxiaoguang 2023-07-21 06:43:49 +08:00 committed by GitHub
parent d12ba978a7
commit 9b25bfa8f4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 19 additions and 24 deletions

View file

@ -101,7 +101,7 @@ func (r *Route) wrapMiddlewareAndHandler(h []any) ([]func(http.Handler) http.Han
return middlewares, handlerFunc return middlewares, handlerFunc
} }
func (r *Route) Methods(method, pattern string, h []any) { func (r *Route) Methods(method, pattern string, h ...any) {
middlewares, handlerFunc := r.wrapMiddlewareAndHandler(h) middlewares, handlerFunc := r.wrapMiddlewareAndHandler(h)
fullPattern := r.getPattern(pattern) fullPattern := r.getPattern(pattern)
if strings.Contains(method, ",") { if strings.Contains(method, ",") {
@ -126,49 +126,44 @@ func (r *Route) Any(pattern string, h ...any) {
r.R.With(middlewares...).HandleFunc(r.getPattern(pattern), handlerFunc) r.R.With(middlewares...).HandleFunc(r.getPattern(pattern), handlerFunc)
} }
// RouteMethods delegate special methods, it is an alias of "Methods", while the "pattern" is the first parameter
func (r *Route) RouteMethods(pattern, methods string, h ...any) {
r.Methods(methods, pattern, h)
}
// Delete delegate delete method // Delete delegate delete method
func (r *Route) Delete(pattern string, h ...any) { func (r *Route) Delete(pattern string, h ...any) {
r.Methods("DELETE", pattern, h) r.Methods("DELETE", pattern, h...)
} }
// Get delegate get method // Get delegate get method
func (r *Route) Get(pattern string, h ...any) { func (r *Route) Get(pattern string, h ...any) {
r.Methods("GET", pattern, h) r.Methods("GET", pattern, h...)
} }
// GetOptions delegate get and options method // GetOptions delegate get and options method
func (r *Route) GetOptions(pattern string, h ...any) { func (r *Route) GetOptions(pattern string, h ...any) {
r.Methods("GET,OPTIONS", pattern, h) r.Methods("GET,OPTIONS", pattern, h...)
} }
// PostOptions delegate post and options method // PostOptions delegate post and options method
func (r *Route) PostOptions(pattern string, h ...any) { func (r *Route) PostOptions(pattern string, h ...any) {
r.Methods("POST,OPTIONS", pattern, h) r.Methods("POST,OPTIONS", pattern, h...)
} }
// Head delegate head method // Head delegate head method
func (r *Route) Head(pattern string, h ...any) { func (r *Route) Head(pattern string, h ...any) {
r.Methods("HEAD", pattern, h) r.Methods("HEAD", pattern, h...)
} }
// Post delegate post method // Post delegate post method
func (r *Route) Post(pattern string, h ...any) { func (r *Route) Post(pattern string, h ...any) {
r.Methods("POST", pattern, h) r.Methods("POST", pattern, h...)
} }
// Put delegate put method // Put delegate put method
func (r *Route) Put(pattern string, h ...any) { func (r *Route) Put(pattern string, h ...any) {
r.Methods("PUT", pattern, h) r.Methods("PUT", pattern, h...)
} }
// Patch delegate patch method // Patch delegate patch method
func (r *Route) Patch(pattern string, h ...any) { func (r *Route) Patch(pattern string, h ...any) {
r.Methods("PATCH", pattern, h) r.Methods("PATCH", pattern, h...)
} }
// ServeHTTP implements http.Handler // ServeHTTP implements http.Handler

View file

@ -634,7 +634,7 @@ func ContainerRoutes() *web.Route {
) )
// Manual mapping of routes because {image} can contain slashes which chi does not support // Manual mapping of routes because {image} can contain slashes which chi does not support
r.RouteMethods("/*", "HEAD,GET,POST,PUT,PATCH,DELETE", func(ctx *context.Context) { r.Methods("HEAD,GET,POST,PUT,PATCH,DELETE", "/*", func(ctx *context.Context) {
path := ctx.Params("*") path := ctx.Params("*")
isHead := ctx.Req.Method == "HEAD" isHead := ctx.Req.Method == "HEAD"
isGet := ctx.Req.Method == "GET" isGet := ctx.Req.Method == "GET"

View file

@ -20,7 +20,7 @@ import (
func Routes() *web.Route { func Routes() *web.Route {
base := web.NewRoute() base := web.NewRoute()
base.Use(common.ProtocolMiddlewares()...) base.Use(common.ProtocolMiddlewares()...)
base.RouteMethods("/assets/*", "GET, HEAD", public.AssetsHandlerFunc("/assets/")) base.Methods("GET, HEAD", "/assets/*", public.AssetsHandlerFunc("/assets/"))
r := web.NewRoute() r := web.NewRoute()
r.Use(common.Sessioner(), Contexter()) r.Use(common.Sessioner(), Contexter())

View file

@ -108,12 +108,12 @@ func Routes() *web.Route {
routes := web.NewRoute() routes := web.NewRoute()
routes.Head("/", misc.DummyOK) // for health check - doesn't need to be passed through gzip handler routes.Head("/", misc.DummyOK) // for health check - doesn't need to be passed through gzip handler
routes.RouteMethods("/assets/*", "GET, HEAD", CorsHandler(), public.AssetsHandlerFunc("/assets/")) routes.Methods("GET, HEAD", "/assets/*", CorsHandler(), public.AssetsHandlerFunc("/assets/"))
routes.RouteMethods("/avatars/*", "GET, HEAD", storageHandler(setting.Avatar.Storage, "avatars", storage.Avatars)) routes.Methods("GET, HEAD", "/avatars/*", storageHandler(setting.Avatar.Storage, "avatars", storage.Avatars))
routes.RouteMethods("/repo-avatars/*", "GET, HEAD", storageHandler(setting.RepoAvatar.Storage, "repo-avatars", storage.RepoAvatars)) routes.Methods("GET, HEAD", "/repo-avatars/*", storageHandler(setting.RepoAvatar.Storage, "repo-avatars", storage.RepoAvatars))
routes.RouteMethods("/apple-touch-icon.png", "GET, HEAD", misc.StaticRedirect("/assets/img/apple-touch-icon.png")) routes.Methods("GET, HEAD", "/apple-touch-icon.png", misc.StaticRedirect("/assets/img/apple-touch-icon.png"))
routes.RouteMethods("/apple-touch-icon-precomposed.png", "GET, HEAD", misc.StaticRedirect("/assets/img/apple-touch-icon.png")) routes.Methods("GET, HEAD", "/apple-touch-icon-precomposed.png", misc.StaticRedirect("/assets/img/apple-touch-icon.png"))
routes.RouteMethods("/favicon.ico", "GET, HEAD", misc.StaticRedirect("/assets/img/favicon.png")) routes.Methods("GET, HEAD", "/favicon.ico", misc.StaticRedirect("/assets/img/favicon.png"))
_ = templates.HTMLRenderer() _ = templates.HTMLRenderer()
@ -129,7 +129,7 @@ func Routes() *web.Route {
if setting.Service.EnableCaptcha { if setting.Service.EnableCaptcha {
// The captcha http.Handler should only fire on /captcha/* so we can just mount this on that url // The captcha http.Handler should only fire on /captcha/* so we can just mount this on that url
routes.RouteMethods("/captcha/*", "GET,HEAD", append(mid, captcha.Captchaer(context.GetImageCaptcha()))...) routes.Methods("GET,HEAD", "/captcha/*", append(mid, captcha.Captchaer(context.GetImageCaptcha()))...)
} }
if setting.HasRobotsTxt { if setting.HasRobotsTxt {
@ -773,7 +773,7 @@ func registerRoutes(m *web.Route) {
addSettingVariablesRoutes() addSettingVariablesRoutes()
}, actions.MustEnableActions) }, actions.MustEnableActions)
m.RouteMethods("/delete", "GET,POST", org.SettingsDelete) m.Methods("GET,POST", "/delete", org.SettingsDelete)
m.Group("/packages", func() { m.Group("/packages", func() {
m.Get("", org.Packages) m.Get("", org.Packages)