auto-register also the DI\FactoryInterface and DI\InvokerInterface #181

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@drealecs
Contributor

drealecs commented Aug 11, 2014

In some cases complex factory service classes might need the container injected as a DI\FactoryInterface to be able to make other services.
These changes ease the configuration of the container.

@mnapoli mnapoli added the enhancement label Aug 11, 2014

@mnapoli mnapoli added this to the 4.3 milestone Aug 11, 2014

@mnapoli mnapoli referenced this pull request Aug 11, 2014

Merged

4.3 #174

6 of 6 tasks complete
@mnapoli

This comment has been minimized.

Show comment
Hide comment
@mnapoli

mnapoli Aug 11, 2014

Member

Thank you this is a good idea! I merged the pull request manually into the 4.3 branch: #174

Member

mnapoli commented Aug 11, 2014

Thank you this is a good idea! I merged the pull request manually into the 4.3 branch: #174

@mnapoli mnapoli closed this Aug 11, 2014

mnapoli added a commit that referenced this pull request Aug 12, 2014

@mnapoli

This comment has been minimized.

Show comment
Hide comment
@mnapoli

mnapoli Aug 12, 2014

Member

I have just released PHP-DI 4.3.0 which includes this feature. Thanks!

http://php-di.org/news/11-php-di-4-3-released.html

Member

mnapoli commented Aug 12, 2014

I have just released PHP-DI 4.3.0 which includes this feature. Thanks!

http://php-di.org/news/11-php-di-4-3-released.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment