diff --git a/src/Symfony/Bundle/FrameworkBundle/HttpKernel.php b/src/Symfony/Bundle/FrameworkBundle/HttpKernel.php index 955fa3fa21..020546499d 100644 --- a/src/Symfony/Bundle/FrameworkBundle/HttpKernel.php +++ b/src/Symfony/Bundle/FrameworkBundle/HttpKernel.php @@ -32,18 +32,14 @@ class HttpKernel extends BaseHttpKernel try { $response = parent::handle($request, $type, $catch); - - if (HttpKernelInterface::MASTER_REQUEST !== $type) { - $this->container->leaveScope('request'); - } - - return $response; } catch (\Exception $e) { - if (HttpKernelInterface::MASTER_REQUEST !== $type) { - $this->container->leaveScope('request'); - } + $this->container->leaveScope('request'); throw $e; } + + $this->container->leaveScope('request'); + + return $response; } -} \ No newline at end of file +} diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/HttpKernelTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/HttpKernelTest.php index 31bd18256e..d11366c01b 100644 --- a/src/Symfony/Bundle/FrameworkBundle/Tests/HttpKernelTest.php +++ b/src/Symfony/Bundle/FrameworkBundle/Tests/HttpKernelTest.php @@ -25,18 +25,11 @@ class HttpKernelTest extends \PHPUnit_Framework_TestCase ->method('enterScope') ->with($this->equalTo('request')) ; - if ($type !== HttpKernelInterface::MASTER_REQUEST) { - $container - ->expects($this->once()) - ->method('leaveScope') - ->with($this->equalTo('request')) - ; - } else { - $container - ->expects($this->never()) - ->method('leaveScope') - ; - } + $container + ->expects($this->once()) + ->method('leaveScope') + ->with($this->equalTo('request')) + ; $container ->expects($this->once()) ->method('set') @@ -80,18 +73,11 @@ class HttpKernelTest extends \PHPUnit_Framework_TestCase ->method('enterScope') ->with($this->equalTo('request')) ; - if ($type !== HttpKernelInterface::MASTER_REQUEST) { - $container - ->expects($this->once()) - ->method('leaveScope') - ->with($this->equalTo('request')) - ; - } else { - $container - ->expects($this->never()) - ->method('leaveScope') - ; - } + $container + ->expects($this->once()) + ->method('leaveScope') + ->with($this->equalTo('request')) + ; $container ->expects($this->once()) ->method('set')