Deprecate passing a concrete service in optional cache warmers

Author: romainneutron

CacheWarmer/RouterCacheWarmer.php

@@ -19,6 +19,8 @@
  * Generates the router matcher and generator classes.
  *
  * @author Fabien Potencier <[email protected]>
+ *
+ * @final since version 3.4, to be given a container instead in 4.0
  */
 class RouterCacheWarmer implements CacheWarmerInterface
 {

CacheWarmer/TranslationsCacheWarmer.php

@@ -29,7 +29,7 @@ class TranslationsCacheWarmer implements CacheWarmerInterface
     /**
      * TranslationsCacheWarmer constructor.
      *
-     * @param ContainerInterface|TranslatorInterface $container
+     * @param ContainerInterface $container
      */
     public function __construct($container)
     {
@@ -38,8 +38,9 @@ public function __construct($container)
             $this->container = $container;
         } elseif ($container instanceof TranslatorInterface) {
             $this->translator = $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.', TranslatorInterface::class, __CLASS__, ContainerInterface::class), E_USER_DEPRECATED);
         } else {
-            throw new \InvalidArgumentException(sprintf('%s only accepts instance of Symfony\Component\DependencyInjection\ContainerInterface or Symfony\Component\Translation\TranslatorInterface as first argument.', __CLASS__));
+            throw new \InvalidArgumentException(sprintf('%s only accepts instance of Symfony\Component\DependencyInjection\ContainerInterface as first argument.', __CLASS__));
         }
     }