laminas-mvc icon indicating copy to clipboard operation
laminas-mvc copied to clipboard

Drop ApplicationInterface

Open Xerkus opened this issue 4 years ago • 0 comments

Q A
Documentation no
Bugfix no
BC Break yes
New Feature no
RFC no
QA no

Description

ApplicationInterface was never really utilized and where application instance were expected often time it was the Laminas\Mvc\Application implicitly which is not type safe.

Interface does not provide a lot of value now and even more reduntant with the direction towards pure dependency injection from a mix of DI and service locator.

Make EventManager instance a required constructor parameter instead of optional pulled from injected container in the same constructor. Downgrade Application to EventsCapableInterface removing ability to set different event manager instance post instantiation.

Psalm issues which are not directly related intentionally left unfixed to be handled in a separate PR.

Xerkus avatar Jun 19 '20 21:06 Xerkus