Inject shared event manager in initializer #3786

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@Xerkus
Member
Xerkus commented Feb 14, 2013

if EventManagerAwareInterface implementation instantiates event manager on getEventManager(), initializer will be ignored and shared event manager will not be set.

@mwillbanks
Contributor

Please supply a test for this behavior.

@weierophinney weierophinney added a commit that referenced this pull request Feb 19, 2013
@weierophinney weierophinney [#3786] Added unit test
- Created unit test to verify behavior
5bb9e32
@weierophinney weierophinney added a commit that referenced this pull request Feb 19, 2013
@weierophinney weierophinney Merge branch 'hotfix/3786' into develop
Forward port #3786
74b1979
@weierophinney weierophinney added a commit that closed this pull request Feb 19, 2013
@weierophinney weierophinney Merge branch 'hotfix/3786'
Close #3786
3b617ba
@weierophinney
Member

I created a unit test when merging.

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