xtrasmal / TacticianBundle

Symfony2 Bundle for the Tactician library -DEPRECATED / MOVED to
https://github.com/thephpleague/tactician-bundle
5 stars 2 forks source link

Implement ContainerAware HandlerLocator #2

Closed rosstuck closed 9 years ago

rosstuck commented 9 years ago

As discussed on the original Tactician issue, creating the Handlers on the fly in the middleware has two issues:

We could solve both by implementing a HandlerLocator that loads Handlers from the service container. The quick fix would be to change the command class names to accept a service container id and load from that in the HandlerLocator. A more symfony style might be to use a DI tag though. :)

rosstuck commented 9 years ago

Mostly resolved in #3