symfony
diff --git a/‎src/Symfony/Bridge/Twig/Extension/RoutingExtension.php
Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Twig/Extension/RoutingExtension.php
+1-1Lines changed: 1 addition & 1 deletion b/‎src/Symfony/Bridge/Twig/Extension/RoutingExtension.php
Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Twig/Extension/RoutingExtension.php
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/‎src/Symfony/Bundle/FrameworkBundle/CacheWarmer/RouterCacheWarmer.php
Copy file name to clipboardExpand all lines: src/Symfony/Bundle/FrameworkBundle/CacheWarmer/RouterCacheWarmer.php
+25-5Lines changed: 25 additions & 5 deletions b/‎src/Symfony/Bundle/FrameworkBundle/CacheWarmer/RouterCacheWarmer.php
Copy file name to clipboardExpand all lines: src/Symfony/Bundle/FrameworkBundle/CacheWarmer/RouterCacheWarmer.php
+25-5Lines changed: 25 additions & 5 deletions
diff --git a/‎src/Symfony/Bundle/FrameworkBundle/Resources/config/routing.xml
Copy file name to clipboardExpand all lines: src/Symfony/Bundle/FrameworkBundle/Resources/config/routing.xml
+2-1Lines changed: 2 additions & 1 deletion b/‎src/Symfony/Bundle/FrameworkBundle/Resources/config/routing.xml
Copy file name to clipboardExpand all lines: src/Symfony/Bundle/FrameworkBundle/Resources/config/routing.xml
+2-1Lines changed: 2 additions & 1 deletion
@@ -91,7 +91,7 @@ public function getUrl($name, $parameters = array(), $schemeRelative = false)
      *
      * @return array An array with the contexts the URL is safe
      *
-     * To be made @final in 3.4, and the type-hint be changed to "\Twig\Node\Node" in 4.0.
+     * @final since version 3.4, type-hint to be changed "\Twig\Node\Node" in 4.0
      */
     public function isUrlGenerationSafe(\Twig_Node $argsNode)
     {
 
@@ -11,6 +11,8 @@
 
 namespace Symfony\Bundle\FrameworkBundle\CacheWarmer;
 
+use Psr\Container\ContainerInterface;
+use Symfony\Component\DependencyInjection\ServiceSubscriberInterface;
 use Symfony\Component\HttpKernel\CacheWarmer\CacheWarmerInterface;
 use Symfony\Component\HttpKernel\CacheWarmer\WarmableInterface;
 use Symfony\Component\Routing\RouterInterface;
@@ -20,20 +22,28 @@
  *
  * @author Fabien Potencier <fabien@symfony.com>
  *
- * @final since version 3.4, to be given a container instead in 4.0
+ * @final since version 3.4
  */
-class RouterCacheWarmer implements CacheWarmerInterface
+class RouterCacheWarmer implements CacheWarmerInterface, ServiceSubscriberInterface
 {
     protected $router;
 
     /**
      * Constructor.
      *
-     * @param RouterInterface $router A Router instance
+     * @param ContainerInterface $container
      */
-    public function __construct(RouterInterface $router)
+    public function __construct($container)
     {
-        $this->router = $router;
+        // As this cache warmer is optional, dependencies should be lazy-loaded, that's why a container should be injected.
+        if ($container instanceof ContainerInterface) {
+            $this->router = $container->get('router'); // For BC, the $router property must be populated in the constructor
+        } elseif ($container instanceof RouterInterface) {
+            $this->router = $container;
+            @trigger_error(sprintf('Using a "%s" as first argument of %s is deprecated since version 3.4 and will be unsupported in version 4.0. Use a %s instead.', RouterInterface::class, __CLASS__, ContainerInterface::class), E_USER_DEPRECATED);
+        } else {
+            throw new \InvalidArgumentException(sprintf('%s only accepts instance of Psr\Container\ContainerInterface as first argument.', __CLASS__));
+        }
     }
 
     /**
@@ -57,4 +67,14 @@ public function isOptional()
     {
         return true;
     }
+
+    /**
+     * {@inheritdoc}
+     */
+    public static function getSubscribedServices()
+    {
+        return array(
+            'router' => RouterInterface::class,
+        );
+    }
 }
@@ -97,8 +97,9 @@
         <service id="Symfony\Component\Routing\RequestContext" alias="router.request_context" />
 
         <service id="router.cache_warmer" class="Symfony\Bundle\FrameworkBundle\CacheWarmer\RouterCacheWarmer">
+            <tag name="container.service_subscriber" id="router" />
             <tag name="kernel.cache_warmer" />
-            <argument type="service" id="router" />
+            <argument type="service" id="Psr\Container\ContainerInterface" />
         </service>
 
         <service id="router_listener" class="Symfony\Component\HttpKernel\EventListener\RouterListener" public="true">
Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ public function getUrl($name, $parameters = array(), $schemeRelative = false)`
`91`	`91`	`*`
`92`	`92`	`* @return array An array with the contexts the URL is safe`
`93`	`93`	`*`
`94`		`- * To be made @final in 3.4, and the type-hint be changed to "\Twig\Node\Node" in 4.0.`
	`94`	`+ * @final since version 3.4, type-hint to be changed "\Twig\Node\Node" in 4.0`
`95`	`95`	`*/`
`96`	`96`	`public function isUrlGenerationSafe(\Twig_Node $argsNode)`
`97`	`97`	`{`