Skip to content
This repository has been archived by the owner on Dec 7, 2019. It is now read-only.

Allow pulling commands from container #28

Closed
wants to merge 3 commits into from

Conversation

mtymek
Copy link
Contributor

@mtymek mtymek commented Aug 20, 2015

This PR updates Dispatcher class, so that it can optionally pull commands from container-interop-compatible container.

Usage:

$serviceManager = new ServiceManager(/* ... */);
$dispatcher = new Dispatcher($serviceManager);
$application = new Application('App', 1.0, $routes, null, $dispatcher);

After configuration, you can specify valid identifier as route handler, and Dispatcher will try to retrieve it when route is matched. This can help to lazily load complex command objects requiring configuration via factory.

Solves #15.

@mtymek
Copy link
Contributor Author

mtymek commented Sep 28, 2015

Any chance to get it merged @weierophinney? Thanks!

@vaclavvanik
Copy link

+1

4 similar comments
@jaapio
Copy link

jaapio commented Dec 16, 2015

+1

@ArnaudLigny
Copy link
Contributor

👍

@stefanotorresi
Copy link
Contributor

👍

@snapshotpl
Copy link

👍

@weierophinney weierophinney added this to the 1.3.0 milestone Jul 11, 2016
@weierophinney weierophinney self-assigned this Jul 11, 2016
weierophinney added a commit that referenced this pull request Jul 11, 2016
Allow pulling commands from container
weierophinney added a commit that referenced this pull request Jul 11, 2016
weierophinney added a commit that referenced this pull request Jul 11, 2016
@weierophinney
Copy link
Member

Merged to develop for release with 1.3.0. Thanks, @mtymek !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants