Use method syntax for linkTo

[lucascs]

Since everyone will use Servlet 3.0, we could support something like:

${linkTo[SomeController].method(arg1, arg2)}

[csokol]

Totally agree!
Em 17/09/2013 22:14, "Lucas Cavalcanti" notifications@github.com escreveu:

Since everyone will use Servlet 3.0, we could support something like:

${linkTo[SomeController].method(arg1, arg2)}

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/78
.

[mariofts]

THIS.

[garcia-jj]

Sounds like great.

[lucascs]

It's a pity that it won't work with void methods =( Not with this syntax, anyway

[garcia-jj]

But you will return a fake object, and not the real instance of method controller.

[leocwolter]

I don't know if it's possible, but if it's not:
As an alternative, couldn't it be something like linkTo(myController.method(arg1, arg2))?
I guess it would be much easier to implement
(Something like Mockito.when(mock.method()) )

[lucascs]

Think of this invocation in java:

class MyController {
    public void method() {...}
}

linkTo(myController.method()); //compilation error

[leocwolter]

myController would be a proxy of MyController. The proxy handler would just save the method invocation(method's name + parameters) and the linkTo()(or linker.linkTo()) would return the path based on the method invocation.
Oh, I see, It won't work with void a method also, sorry.

[garcia-jj]

Lucas, Leonardo is right, because we don't wanna execute the method, but only create a link. So a proxy is better solution.

[lucascs]

I have another idea. Because of this line:

https://github.com/caelum/vraptor4/blob/master/vraptor-core/src/main/java/br/com/caelum/vraptor/ioc/ControllerHandler.java#L71

we can use ${TestController} and it returns the BeanClass associated to this controller. My idea is return a decorator for beanClass in which we can call ${TestController.method}, and it returns the correspondent ControllerMethod.

so we can change the API to something like:

${linkTo(TestController.method)}, if we don't want to add args, or
${linkTo(TestController.method, arg1, arg2)} if we want to add args.

What do you think?

[Turini]

I think it would be great =)

[leocwolter]

I'm curious about the implementation :D

On Wednesday, September 18, 2013, Rodrigo Turini wrote:

I think it would be great =)

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/78#issuecomment-24659263
.

[mariofts]

The first version, with the @garcia-jj idea of using proxy/fake objects isn't viable?

[lucascs]

No. We cannot proxy void.class ;) And most of controller methods will be void.

[garcia-jj]

#143

[lucascs]

Closed by #143