Merge pull request #295 from theofidry/1-0/bugfix/preprend-config

Fix prepend condition

Author: sroze

DependencyInjection/DunglasApiExtension.php

@@ -30,7 +30,7 @@ class DunglasApiExtension extends Extension implements PrependExtensionInterface
      */
     public function prepend(ContainerBuilder $container)
     {
-        if (null !== ($frameworkConfiguration = $container->getExtensionConfig('framework'))) {
+        if (!empty($frameworkConfiguration = $container->getExtensionConfig('framework'))) {
             if (!isset($frameworkConfiguration['serializer']) || !isset($frameworkConfiguration['serializer']['enabled'])) {
                 $container->prependExtensionConfig('framework', [
                     'serializer' => [

Tests/DependencyInjection/DunglasApiExtensionTest.php

@@ -70,7 +70,7 @@ public function testPrependWhenNotConfigured()
     {
         $containerBuilderProphecy = $this->prophesize('Symfony\Component\DependencyInjection\ContainerBuilder');
         $containerBuilderProphecy->getExtensionConfig('framework')->willReturn([])->shouldBeCalled();
-        $containerBuilderProphecy->prependExtensionConfig('framework', Argument::type('array'))->shouldBeCalled();
+        $containerBuilderProphecy->prependExtensionConfig('framework', Argument::type('array'))->shouldNotBeCalled();
         $containerBuilder = $containerBuilderProphecy->reveal();
 
         $this->extension->prepend($containerBuilder);