Closed rosstuck closed 9 years ago
If the Setup class were ContainerAware, this would be pretty trivial, just change https://github.com/xtrasmal/TacticianBundle/blob/master/Setup/Setup.php#L38 to not call new $middleware but $this->container->get($middleware).
new $middleware
$this->container->get($middleware)
This would make it really easy to have middleware with other dependencies. :smile:
Mostly resolved in #3
If the Setup class were ContainerAware, this would be pretty trivial, just change https://github.com/xtrasmal/TacticianBundle/blob/master/Setup/Setup.php#L38 to not call
new $middleware
but$this->container->get($middleware)
.This would make it really easy to have middleware with other dependencies. :smile: