Mapping in controllers

[nicola-spb]

All public methods in controllers are mapped to url path by default.

Maybe will be better if only methods with suffix or prefix "Action" will mapped by default. Some people can`t knew that their methods open for public.

Example:

// this will be parsed to /todo/home by default. Method for routing
func (t *Todo) ActionHome() {
	todos := []models.Todo{}
	....
	t.HTML(http.StatusOK)
}

// this won`t be parsed by default. Simple public method
func (t *Todo) Home2() {
	todos := []models.Todo{}
	....
	t.HTML(http.StatusOK)
}

if user want to set path manual then he can set

func NewTodo() controller.Controller {
	return &Todo{
		Routes: []string{
			"get;/;Home",			
		},
	}
}

Or maybe add some flexible settings for do it.

default parsing here https://github.com/gernest/utron/blob/master/router/routes.go#L230

[gernest]

Do you think automatic mapping is useful at all?. I think we should remove it altogether, it is a good idea to enforce best practices as much as possible.

Do you have any usecase where this automatic mapping makes sense?

[nicola-spb]

For me no. But some people can use it. In some frameworks in Java (with annotation) and php I saw it.

Usually I use manual mapping config. For myself I added Action prefix for showing routed methods and replace default mapping to panic message:
log.Panic("Action \"" + v.fn + "\" isn't configurated! Please configure it in controller or routers.json")
And if I have action without config then I will receive it.