Fixed issue with new vs old TranslatorInterface

althaus · web-flow · commit e1d7f3eaab29 · 2019-05-23T15:37:34.000+02:00
I'm not sure when this gets executed, but overriding `$trans` after the if simply looks wrong.
diff --git a/src/Symfony/Component/Translation/DataCollectorTranslator.php b/src/Symfony/Component/Translation/DataCollectorTranslator.php
@@ -69,8 +69,9 @@ public function transChoice($id, $number, array $parameters = [], $domain = null
         if ($this->translator instanceof TranslatorInterface) {
             $trans = $this->translator->trans($id, ['%count%' => $number] + $parameters, $domain, $locale);
         }
-
-        $trans = $this->translator->transChoice($id, $number, $parameters, $domain, $locale);
+        else {
+            $trans = $this->translator->transChoice($id, $number, $parameters, $domain, $locale);
+        }
 
         $this->collectMessage($locale, $domain, $id, $trans, ['%count%' => $number] + $parameters);
 
