Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ListenerAggregatePluginManager? #3

Closed
weierophinney opened this issue Dec 31, 2019 · 2 comments
Closed

ListenerAggregatePluginManager? #3

weierophinney opened this issue Dec 31, 2019 · 2 comments

Comments

@weierophinney
Copy link
Member

hi Team,

assuming that an application has factories for its listeners, it would be nice to configure them in a central location and to not pollute the service container. any interest?

thanks : )


Originally posted by @abacaphiliac at zendframework/zend-eventmanager#59

@weierophinney
Copy link
Member Author

I would say that your application should consist auf multiple modules so there are not so many listeners within the service container to have another plugin manager created.

Also you could simply use another php file to separate your listeners to an own config file.


Originally posted by @reinfi at zendframework/zend-eventmanager#59 (comment)

@weierophinney
Copy link
Member Author

true, i do take advantage of separate modules and PHP files. but in the merged config all services are listed and available from the application service container despite my modular separation.

a listener plugin manager would provide only implementations of the listener interface, so we would gain some type-safety as well as a reduction in the overall number of services registered in the main application service container.


Originally posted by @abacaphiliac at zendframework/zend-eventmanager#59 (comment)

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

No branches or pull requests

2 participants