-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix PhpArrayDumper notice and PHP 5.3 compatibility #61
Conversation
Oh wow, when would that not be an object? |
It should be possible to unit test this? Could we setup a scenario for that? |
Actually, there's nothing to test because:
Always returns an array, see: https://github.com/LionsAd/service_container/blob/7.x-1.x/lib/Drupal/Core/DependencyInjection/Dumper/PhpArrayDumper.php#L234 |
$dump = unserialize($this->dumper->dump()); | ||
|
||
$this->assertTrue(is_array($dump)); | ||
$this->assertEquals($dump['services']['bar'], 'foo'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, but that should not be true.
It should actually be an array with [ 'alias': 'foo' ] set ...
Why do we need to change that?
Fix PhpArrayDumper notice and PHP 5.3 compatibility
No description provided.