Skip to content
This repository has been archived by the owner on Jan 29, 2020. It is now read-only.

Minor comment update #285

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 8 additions & 5 deletions src/Controller/ControllerManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -71,12 +71,15 @@ public function validate($plugin)
/**
* Initializer: inject EventManager instance
*
* If we have an event manager composed already, make sure it gets injected
* with the shared event manager.
* Set a new event manager injected with the shared event manager.
*
* The AbstractController lazy-instantiates an EM instance, which is why
* the shared EM injection needs to happen; the conditional will always
* pass.
* The AbstractController lazy-instantiates an EventManager instance,
* which is why the SharedEventManager injection needs to happen; the
* conditional will always pass.
*
* This works because we fetch the EventManager via the container
* (ServiceManager). So it gets built by the EventManagerFactory,
* which injects the SharedEventManager via EventManager's constructor.
*
* @param ContainerInterface $container
* @param DispatchableInterface $controller
Expand Down