search

PHP-DI / Silex-Bridge

PHP-DI integration in Silex
http://php-di.org/doc/frameworks/silex.html
MIT License
24 stars 8 forks source link

Refactor ControllerResolver #8

Closed jdreesen closed 9 years ago

jdreesen commented 9 years ago

This is needed to support the ServiceControllerServiceProvider because it decorates the default ControllerResolver and calls getArguments() on it.

This commit also enhances compatibility with Silex by allowing to inject

I didn't remove the injection of the Request by parameter name because it would be a BC break. But this is not supported by Silex itself and I assume it was done because it was not really possible to inject it via type hinting using the invoker. So I think we should remove it in a later release (maybe with Silex2 support, which I hope will be released soon after Symfony3).

Closes #7

mnapoli commented 9 years ago

Thanks, sorry for taking so long :)

mnapoli commented 9 years ago

Released as 1.3.0