Simple injection usage and no more @Deprecated annotation

[helio-frota]

From:

@Controller
public class IndexController {

    private Result result;

    @Deprecated
    public IndexController() {}

    @Inject
    public IndexController(Result result) {
        this.result = result;
    }

    @Path("/")
    public void index() {

    }

}

To:

@Controller
public class IndexController {

    @Inject
    private Result result;

    @Path("/")
    public void index() {

    }
}

[paulojribp]

The first one @eprogramming was discussed on the beginning of the project that it keeps the dependencies more clear and more useful for unit testing

[efraimgentil]

Yeah that was decided in the beginning, unfortunately CDI still have this problem with default constructors, but with the injection in the controller is easier to test and create mocks, so i kind of liked it

[paulojribp]

@eprogramming whats mean this Closed? Will keep constructors or injection on attribute?

[helio-frota]

hi @paulojribp good catch, i closed this. well i'm going to use injection on attribute and wait / maybe research better approach for the tests.