Changed parameter container to requestStack (2.4+)

There is no need to pass @service_container to the voter, @request is enough.

please see #3216

Author: maks-rafalko

cookbook/security/voters.rst

@@ -61,19 +61,18 @@ and compare the IP address against a set of blacklisted IP addresses:
     // src/Acme/DemoBundle/Security/Authorization/Voter/ClientIpVoter.php
     namespace Acme\DemoBundle\Security\Authorization\Voter;
 
-    use Symfony\Component\DependencyInjection\ContainerInterface;
+    use Symfony\Component\HttpFoundation\RequestStack;
     use Symfony\Component\Security\Core\Authorization\Voter\VoterInterface;
     use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
 
     class ClientIpVoter implements VoterInterface
     {
-        private $container;
-
+        protected $requestStack;
         private $blacklistedIp;
-            
-        public function __construct(ContainerInterface $container, array $blacklistedIp = array())
+
+        public function __construct(RequestStack $requestStack, array $blacklistedIp = array())
         {
-            $this->container     = $container;
+            $this->requestStack  = $requestStack;
             $this->blacklistedIp = $blacklistedIp;
         }
 
@@ -91,7 +90,7 @@ and compare the IP address against a set of blacklisted IP addresses:
 
         public function vote(TokenInterface $token, $object, array $attributes)
         {
-            $request = $this->container->get('request');
+            $request = $this->requestStack->getCurrentRequest();
             if (in_array($request->getClientIp(), $this->blacklistedIp)) {
                 return VoterInterface::ACCESS_DENIED;
             }
@@ -128,7 +127,7 @@ and tag it as a ``security.voter``:
         services:
             security.access.blacklist_voter:
                 class:      Acme\DemoBundle\Security\Authorization\Voter\ClientIpVoter
-                arguments:  ["@service_container", [123.123.123.123, 171.171.171.171]]
+                arguments:  ["@request_stack", [123.123.123.123, 171.171.171.171]]
                 public:     false
                 tags:
                     - { name: security.voter }
@@ -138,7 +137,7 @@ and tag it as a ``security.voter``:
         <!-- src/Acme/AcmeBundle/Resources/config/services.xml -->
         <service id="security.access.blacklist_voter"
                  class="Acme\DemoBundle\Security\Authorization\Voter\ClientIpVoter" public="false">
-            <argument type="service" id="service_container" strict="false" />
+            <argument type="service" id="request_stack" strict="false" />
             <argument type="collection">
                 <argument>123.123.123.123</argument>
                 <argument>171.171.171.171</argument>
@@ -155,7 +154,7 @@ and tag it as a ``security.voter``:
         $definition = new Definition(
             'Acme\DemoBundle\Security\Authorization\Voter\ClientIpVoter',
             array(
-                new Reference('service_container'),
+                new Reference('request_stack'),
                 array('123.123.123.123', '171.171.171.171'),
             ),
         );