bug #15467 [FrameworkBundle] fix serializer config check (xabbuh)
This PR was merged into the 2.8 branch.
Discussion
----------
[FrameworkBundle] fix serializer config check
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #15449
| License | MIT
| Doc PR |
Commits
-------
060c167
[FrameworkBundle] fix serializer config check
This commit is contained in:
commit
7f745d72c0
@ -971,7 +971,7 @@ class FrameworkExtension extends Extension
|
||||
);
|
||||
}
|
||||
|
||||
if ($config['name_converter']) {
|
||||
if (isset($config['name_converter']) && $config['name_converter']) {
|
||||
$container->getDefinition('serializer.normalizer.object')->replaceArgument(1, new Reference($config['name_converter']));
|
||||
}
|
||||
}
|
||||
|
@ -0,0 +1,7 @@
|
||||
<?php
|
||||
|
||||
$container->loadFromExtension('framework', array(
|
||||
'serializer' => array(
|
||||
'enabled' => false,
|
||||
),
|
||||
));
|
@ -0,0 +1,7 @@
|
||||
<?php
|
||||
|
||||
$container->loadFromExtension('framework', array(
|
||||
'serializer' => array(
|
||||
'enabled' => true,
|
||||
),
|
||||
));
|
@ -0,0 +1,11 @@
|
||||
<?xml version="1.0" encoding="utf-8" ?>
|
||||
<container xmlns="http://symfony.com/schema/dic/services"
|
||||
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
|
||||
xmlns:framework="http://symfony.com/schema/dic/symfony"
|
||||
xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/services-1.0.xsd
|
||||
http://symfony.com/schema/dic/symfony http://symfony.com/schema/dic/symfony/symfony-1.0.xsd">
|
||||
|
||||
<framework:config>
|
||||
<framework:serializer enabled="false" />
|
||||
</framework:config>
|
||||
</container>
|
@ -0,0 +1,11 @@
|
||||
<?xml version="1.0" encoding="utf-8" ?>
|
||||
<container xmlns="http://symfony.com/schema/dic/services"
|
||||
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
|
||||
xmlns:framework="http://symfony.com/schema/dic/symfony"
|
||||
xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/services-1.0.xsd
|
||||
http://symfony.com/schema/dic/symfony http://symfony.com/schema/dic/symfony/symfony-1.0.xsd">
|
||||
|
||||
<framework:config>
|
||||
<framework:serializer enabled="true" />
|
||||
</framework:config>
|
||||
</container>
|
@ -0,0 +1,3 @@
|
||||
framework:
|
||||
serializer:
|
||||
enabled: false
|
@ -0,0 +1,3 @@
|
||||
framework:
|
||||
serializer:
|
||||
enabled: true
|
@ -472,6 +472,20 @@ abstract class FrameworkExtensionTest extends TestCase
|
||||
$this->assertSame('assets.packages', (string) $packages);
|
||||
}
|
||||
|
||||
public function testSerializerServiceIsRegisteredWhenEnabled()
|
||||
{
|
||||
$container = $this->createContainerFromFile('serializer_enabled');
|
||||
|
||||
$this->assertTrue($container->hasDefinition('serializer'));
|
||||
}
|
||||
|
||||
public function testSerializerServiceIsNotRegisteredWhenDisabled()
|
||||
{
|
||||
$container = $this->createContainerFromFile('serializer_disabled');
|
||||
|
||||
$this->assertFalse($container->hasDefinition('serializer'));
|
||||
}
|
||||
|
||||
protected function createContainer(array $data = array())
|
||||
{
|
||||
return new ContainerBuilder(new ParameterBag(array_merge(array(
|
||||
|
Reference in New Issue
Block a user