removed as many usage of the request service as possible without breaking BC

Author: fabpot

Controller/Controller.php

@@ -62,7 +62,7 @@ public function generateUrl($route, $parameters = array(), $referenceType = UrlG
     public function forward($controller, array $path = array(), array $query = array())
     {
         $path['_controller'] = $controller;
-        $subRequest = $this->container->get('request')->duplicate($query, null, $path);
+        $subRequest = $this->container->get('request_stack')->getCurrentRequest()->duplicate($query, null, $path);
 
         return $this->container->get('http_kernel')->handle($subRequest, HttpKernelInterface::SUB_REQUEST);
     }

Tests/Controller/ControllerTest.php

@@ -14,6 +14,7 @@
 use Symfony\Bundle\FrameworkBundle\Tests\TestCase;
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\HttpFoundation\Response;
 
 class ControllerTest extends TestCase
@@ -24,13 +25,16 @@ public function testForward()
         $request->setLocale('fr');
         $request->setRequestFormat('xml');
 
+        $requestStack = new RequestStack();
+        $requestStack->push($request);
+
         $kernel = $this->getMock('Symfony\Component\HttpKernel\HttpKernelInterface');
         $kernel->expects($this->once())->method('handle')->will($this->returnCallback(function (Request $request) {
             return new Response($request->getRequestFormat().'--'.$request->getLocale());
         }));
 
         $container = $this->getMock('Symfony\Component\DependencyInjection\ContainerInterface');
-        $container->expects($this->at(0))->method('get')->will($this->returnValue($request));
+        $container->expects($this->at(0))->method('get')->will($this->returnValue($requestStack));
         $container->expects($this->at(1))->method('get')->will($this->returnValue($kernel));
 
         $controller = new Controller();

Tests/Functional/Bundle/TestBundle/Controller/SessionController.php

@@ -11,15 +11,15 @@
 
 namespace Symfony\Bundle\FrameworkBundle\Tests\Functional\Bundle\TestBundle\Controller;
 
+use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Symfony\Component\DependencyInjection\ContainerAware;
 
 class SessionController extends ContainerAware
 {
-    public function welcomeAction($name=null)
+    public function welcomeAction(Request $request, $name=null)
     {
-        $request = $this->container->get('request');
         $session = $request->getSession();
 
         // new session case
@@ -40,25 +40,23 @@ public function welcomeAction($name=null)
         return new Response(sprintf('Welcome back %s, nice to meet you.', $name));
     }
 
-    public function logoutAction()
+    public function logoutAction(Request $request)
     {
-        $request = $this->container->get('request')->getSession('session')->invalidate();
+        $request->getSession('session')->invalidate();
 
         return new Response('Session cleared.');
     }
 
-    public function setFlashAction($message)
+    public function setFlashAction(Request $request, $message)
     {
-        $request = $this->container->get('request');
         $session = $request->getSession();
         $session->getFlashBag()->set('notice', $message);
 
         return new RedirectResponse($this->container->get('router')->generate('session_showflash'));
     }
 
-    public function showFlashAction()
+    public function showFlashAction(Request $request)
     {
-        $request = $this->container->get('request');
         $session = $request->getSession();
 
         if ($session->getFlashBag()->has('notice')) {

Tests/Translation/TranslatorTest.php

@@ -12,6 +12,7 @@
 namespace Symfony\Bundle\FrameworkBundle\Tests\Translation;
 
 use Symfony\Bundle\FrameworkBundle\Translation\Translator;
+use Symfony\Component\HttpFoundation\RequestStack;
 use Symfony\Component\Translation\MessageCatalogue;
 use Symfony\Component\Filesystem\Filesystem;
 use Symfony\Component\Translation\MessageSelector;
@@ -86,43 +87,46 @@ public function testTransWithCaching()
         $this->assertEquals('other choice 1 (PT-BR)', $translator->transChoice('other choice', 1));
     }
 
-    public function testGetLocale()
+    /**
+     * @dataProvider getGetLocaleData
+     */
+    public function testGetLocale($inRequestScope)
     {
-        $request = $this->getMock('Symfony\Component\HttpFoundation\Request');
-
-        $request
-            ->expects($this->once())
-            ->method('getLocale')
-            ->will($this->returnValue('en'))
-        ;
+        $requestStack = new RequestStack();
+        if ($inRequestScope) {
+            $request = $this->getMock('Symfony\Component\HttpFoundation\Request');
+            $request
+                ->expects($this->once())
+                ->method('getLocale')
+                ->will($this->returnValue('en'))
+            ;
+
+            $requestStack->push($request);
+        }
 
         $container = $this->getMock('Symfony\Component\DependencyInjection\ContainerInterface');
-
-        $container
-            ->expects($this->exactly(2))
-            ->method('isScopeActive')
-            ->with('request')
-            ->will($this->onConsecutiveCalls(false, true))
-        ;
-
-        $container
-            ->expects($this->once())
-            ->method('has')
-            ->with('request')
-            ->will($this->returnValue(true))
-        ;
-
         $container
             ->expects($this->once())
             ->method('get')
-            ->with('request')
-            ->will($this->returnValue($request))
+            ->with('request_stack')
+            ->will($this->returnValue($requestStack))
         ;
 
         $translator = new Translator($container, new MessageSelector());
 
-        $this->assertNull($translator->getLocale());
-        $this->assertSame('en', $translator->getLocale());
+        if ($inRequestScope) {
+            $this->assertSame('en', $translator->getLocale());
+        } else {
+            $this->assertNull($translator->getLocale());
+        }
+    }
+
+    public function getGetLocaleData()
+    {
+        return array(
+            array(false),
+            array(true),
+        );
     }
 
     protected function getCatalogue($locale, $messages)

Translation/Translator.php

@@ -65,8 +65,8 @@ public function __construct(ContainerInterface $container, MessageSelector $sele
      */
     public function getLocale()
     {
-        if (null === $this->locale && $this->container->isScopeActive('request') && $this->container->has('request')) {
-            $this->locale = $this->container->get('request')->getLocale();
+        if (null === $this->locale && $request = $this->container->get('request_stack')->getCurrentRequest()) {
+            $this->locale = $request->getLocale();
         }
 
         return $this->locale;