Little improvements#266
Conversation
| } | ||
|
|
||
| if (! array_key_exists(self::class, $config)) { | ||
| if (! isset($config[self::class])) { |
There was a problem hiding this comment.
No, it's not. The value MUST be an array to be usable. As such, isset() is all that's required to test for usable configuration.
|
|
||
| if (! is_array($dependencies) | ||
| || ! array_key_exists($requestedName, $dependencies) | ||
| || ! isset($dependencies[$requestedName]) |
There was a problem hiding this comment.
Again, it's not. In this case, we're looking to see if configuration exists for the given service name. There are essentially no cases where that configuration would be a non-array/array-like value. As such, isset() is sufficient here.
|
This repository has been closed and moved to laminas/laminas-servicemanager; a new issue has been opened at laminas/laminas-servicemanager#9. |
|
This repository has been moved to laminas/laminas-servicemanager. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:
|
4 participants
No description provided.