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

ServiceManager::has() when assigned non String or Array causes Undefine Notice errors #5395

Closed
franz-deleon opened this issue Oct 31, 2013 · 2 comments
Closed

ServiceManager::has() when assigned non String or Array causes Undefine Notice errors #5395

franz-deleon opened this issue Oct 31, 2013 · 2 comments
Milestone
2.3.0

Comments

@franz-deleon
Copy link
Contributor

When ServiceManager::has() is validated against non arrays or strings, it returns error notices but it should really just return false
franz-deleon pushed a commit to franz-deleon/zf2 that referenced this issue Oct 31, 2013
fix for issue zendframework#5395. I know its going to be deprecated b…
15da648 
…ut might as well do a small patch right now
franz-deleon added a commit to franz-deleon/zf2 that referenced this issue Nov 1, 2013
@franz-deleon
Issue zendframework#5395: remove unnecessary can create function and …
673aa86 
…add logic to has()
@weierophinney
Copy link
Member

@franz-deleon Please send 15da648 as a PR -- that one looks correct, and maintains the canCreate() method.

weierophinney added a commit that referenced this issue Nov 1, 2013
@weierophinney
Merge pull request #5396 from franz-deleon/develop
631c851 
Fix for issue #5395
@weierophinney
Copy link
Member

Closed via #5396.

gianarb pushed a commit to zendframework/zend-servicemanager that referenced this issue May 15, 2015
fix for issue zendframework/zendframework#5395. I know its going to b…
e71908f 
…e deprecated but might as well do a small patch right now
gianarb pushed a commit to zendframework/zend-servicemanager that referenced this issue May 15, 2015
@franz-deleon
Issue zendframework/zendframework#5395: remove unnecessary can create…
e61cf17 
… function and add logic to has()
weierophinney added a commit to zendframework/zend-servicemanager that referenced this issue May 15, 2015
@weierophinney
Merge pull request zendframework/zendframework#5396 from franz-deleon…
be837f8 
…/develop

Fix for issue zendframework/zendframework#5395
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.3.0
Development

No branches or pull requests
2 participants
@weierophinney @franz-deleon