The TranslatorBag is no TranslatorInterface

Leverage union types to properly set up the configuration

Author: SanderVerkuil

src/Translator/src/CacheWarmer/TranslationsCacheWarmer.php

@@ -14,6 +14,7 @@
 use Psr\Log\LoggerInterface;
 use Symfony\Component\HttpKernel\CacheWarmer\CacheWarmerInterface;
 use Symfony\Component\Translation\TranslatorBagInterface;
+use Symfony\Contracts\Translation\TranslatorInterface;
 use Symfony\UX\Translator\TranslationsDumper;
 
 /**
@@ -24,7 +25,7 @@
 class TranslationsCacheWarmer implements CacheWarmerInterface
 {
     public function __construct(
-        private ?TranslatorBagInterface $translatorBag,
+        private TranslatorInterface|TranslatorBagInterface|null $translatorBag,
         private TranslationsDumper $translationsDumper,
         private readonly ?LoggerInterface $logger = null,
     ) {
@@ -37,7 +38,7 @@ public function isOptional(): bool
 
     public function warmUp(string $cacheDir, ?string $buildDir = null): array
     {
-        if (null === $this->translatorBag) {
+        if (!$this->translatorBag instanceof TranslatorBagInterface) {
             $this->logger?->warning('Translator bag not available');
 
             return [];

src/Translator/tests/CacheWarmer/TranslationsCacheWarmerTest.php

@@ -14,6 +14,7 @@
 use PHPUnit\Framework\TestCase;
 use Psr\Log\LoggerInterface;
 use Symfony\Component\Translation\MessageCatalogue;
+use Symfony\Component\Translation\Translator;
 use Symfony\Component\Translation\TranslatorBag;
 use Symfony\UX\Translator\CacheWarmer\TranslationsCacheWarmer;
 use Symfony\UX\Translator\TranslationsDumper;