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

InjectControllerDependencies initializer overriding an previously defined EventManager #3650

Closed
danizord opened this issue Feb 1, 2013 · 1 comment · Fixed by #3651
Closed
Assignees
Milestone

Comments

@danizord
Copy link
Contributor

danizord commented Feb 1, 2013

The Zend\Mvc\Controller\ControllerManager::InjectControllerDependencies should check if an EventManager has been defined previously, if so, should not override this.

As discussed on IRC with @weierophinney, seems that this issue has already been fixed earlier but the fix is gone.

@ghost ghost assigned weierophinney Feb 1, 2013
weierophinney added a commit to weierophinney/zendframework that referenced this issue Feb 1, 2013
- If the controller already composes an EM instance, do not re-inject
  it. This allows factories to inject an EM instance and attach
  listeners.
weierophinney added a commit to weierophinney/zendframework that referenced this issue Feb 1, 2013
- Override getEventManager() in test asset so that we can test
  injection, as parent was lazy initializing an EM instance.
@danizord
Copy link
Contributor Author

danizord commented Feb 4, 2013

Fixed by 65c4942, thanks to @weierophinney and @ezimuel

@danizord danizord closed this as completed Feb 4, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants