[Validator] fix LegacyTranslatorProxy

nicolas-grekas · nicolas-grekas · commit 56e5cfbde6c4 · 2019-04-18T14:13:11.000+02:00
diff --git a/src/Symfony/Component/Validator/Util/LegacyTranslatorProxy.php b/src/Symfony/Component/Validator/Util/LegacyTranslatorProxy.php
@@ -22,15 +22,26 @@ class LegacyTranslatorProxy implements LegacyTranslatorInterface, TranslatorInte
 {
     private $translator;
 
-    public function __construct(TranslatorInterface $translator)
+    /**
+     * @param LegacyTranslatorInterface|TranslatorInterface $translator
+     */
+    public function __construct($translator)
     {
+        if (!$translator instanceof LegacyTranslatorInterface && !$translator instanceof TranslatorInterface) {
+            throw new \InvalidArgumentException(sprintf('The translator passed to "%s()" must implement "%s" or "%s".', __METHOD__, TranslatorInterface::class, LegacyTranslatorInterface::class));
+        }
+
         if (!$translator instanceof LocaleAwareInterface) {
             throw new \InvalidArgumentException(sprintf('The translator passed to "%s()" must implement "%s".', __METHOD__, LocaleAwareInterface::class));
         }
+
         $this->translator = $translator;
     }
 
-    public function getTranslator(): TranslatorInterface
+    /**
+     * @return LegacyTranslatorInterface|TranslatorInterface
+     */
+    public function getTranslator()
     {
         return $this->translator;
     }

Original file line number	Diff line number	Diff line change
`@@ -22,15 +22,26 @@ class LegacyTranslatorProxy implements LegacyTranslatorInterface, TranslatorInte`
`22`	`22`	`{`
`23`	`23`	`private $translator;`
`24`	`24`
`25`		`- public function __construct(TranslatorInterface $translator)`
	`25`	`+ /**`
	`26`	`+ * @param LegacyTranslatorInterface\|TranslatorInterface $translator`
	`27`	`+ */`
	`28`	`+ public function __construct($translator)`
`26`	`29`	`{`
	`30`	`+ if (!$translator instanceof LegacyTranslatorInterface && !$translator instanceof TranslatorInterface) {`
	`31`	`+ throw new \InvalidArgumentException(sprintf('The translator passed to "%s()" must implement "%s" or "%s".', __METHOD__, TranslatorInterface::class, LegacyTranslatorInterface::class));`
	`32`	`+ }`
	`33`	`+`
`27`	`34`	`if (!$translator instanceof LocaleAwareInterface) {`
`28`	`35`	`throw new \InvalidArgumentException(sprintf('The translator passed to "%s()" must implement "%s".', __METHOD__, LocaleAwareInterface::class));`
`29`	`36`	`}`
	`37`	`+`
`30`	`38`	`$this->translator = $translator;`
`31`	`39`	`}`
`32`	`40`
`33`		`- public function getTranslator(): TranslatorInterface`
	`41`	`+ /**`
	`42`	`+ * @return LegacyTranslatorInterface\|TranslatorInterface`
	`43`	`+ */`
	`44`	`+ public function getTranslator()`
`34`	`45`	`{`
`35`	`46`	`return $this->translator;`
`36`	`47`	`}`