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

Fixes #78 : using uppercase for 'Application' service #80

Merged
merged 3 commits into from
Feb 29, 2016
Merged

Fixes #78 : using uppercase for 'Application' service #80

merged 3 commits into from
Feb 29, 2016

Conversation

samsonasik
Copy link
Contributor

Fixes #78

@samsonasik samsonasik mentioned this pull request Feb 29, 2016
Closed
@samsonasik
Copy link
Contributor Author

It need to use uppercase when we require zend-servicemanager:v3

@weierophinney
Copy link
Member

@samsonasik That's not really a fix. We need for both application and Application to resolve to ensure user-land code continues to work. This will ensure the internals work, but it's the user-land code I'm more worried about.

@samsonasik
Copy link
Contributor Author

@weierophinney I've added new entry for 'application' in the ServiceListenerFactory as it won't work with alias due Circular alias reference

@samsonasik
Copy link
Contributor Author

should be ok now?

weierophinney added a commit to weierophinney/zend-mvc that referenced this pull request Feb 29, 2016
@weierophinney
Merge pull request zendframework#80 from samsonasik/patch-1
b8bf785 
Fixes zendframework#78 : using uppercase for 'Application' service
@weierophinney weierophinney mentioned this pull request Feb 29, 2016
Merged
@weierophinney weierophinney merged commit bc4a2b4 into zendframework:develop Feb 29, 2016
weierophinney added a commit that referenced this pull request Feb 29, 2016
@weierophinney
Added CHANGELOG for #80 and #82
055c3da
weierophinney added a commit that referenced this pull request Feb 29, 2016
@weierophinney
Merge branch 'hotfix/80' into develop
e649aca 
Close #80
Close #82
Fixes #78
@samsonasik samsonasik deleted the patch-1 branch March 1, 2016 01:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@samsonasik @weierophinney