[Translation] Add support for calling 'trans' with ICU formatted messages

someonewithpc · fabpot · commit d2ec41f4ef01 · 2020-08-30T09:19:43.000+02:00
diff --git a/src/Symfony/Component/Translation/CHANGELOG.md b/src/Symfony/Component/Translation/CHANGELOG.md
@@ -4,6 +4,7 @@ CHANGELOG
 5.2.0
 -----
 
+ * added support for calling `trans` with ICU formatted messages
  * added `PseudoLocalizationTranslator`
 
 5.1.0
diff --git a/src/Symfony/Component/Translation/Tests/TranslatorTest.php b/src/Symfony/Component/Translation/Tests/TranslatorTest.php
@@ -368,6 +368,14 @@ public function testTrans($expected, $id, $translation, $parameters, $locale, $d
         $this->assertEquals($expected, $translator->trans($id, $parameters, $domain, $locale));
     }
 
+    /**
+     * @dataProvider getTransICUTests
+     */
+    public function testTransICU(...$args)
+    {
+        $this->testTrans(...$args);
+    }
+
     /**
      * @dataProvider getInvalidLocalesTests
      */
@@ -444,6 +452,17 @@ public function getTransTests()
         ];
     }
 
+    public function getTransICUTests()
+    {
+        $id = '{apples, plural, =0 {There are no apples} one {There is one apple} other {There are # apples}}';
+
+        return [
+            ['There are no apples', $id, $id, ['{apples}' => 0], 'en', 'test'.MessageCatalogue::INTL_DOMAIN_SUFFIX],
+            ['There is one apple',  $id, $id, ['{apples}' => 1], 'en', 'test'.MessageCatalogue::INTL_DOMAIN_SUFFIX],
+            ['There are 3 apples',  $id, $id, ['{apples}' => 3], 'en', 'test'.MessageCatalogue::INTL_DOMAIN_SUFFIX],
+        ];
+    }
+
     public function getFlattenedTransTests()
     {
         $messages = [
diff --git a/src/Symfony/Component/Translation/Translator.php b/src/Symfony/Component/Translation/Translator.php
@@ -214,7 +214,10 @@ public function trans(?string $id, array $parameters = [], string $domain = null
             }
         }
 
-        if ($this->hasIntlFormatter && $catalogue->defines($id, $domain.MessageCatalogue::INTL_DOMAIN_SUFFIX)) {
+        $len = \strlen(MessageCatalogue::INTL_DOMAIN_SUFFIX);
+        if ($this->hasIntlFormatter
+            && ($catalogue->defines($id, $domain.MessageCatalogue::INTL_DOMAIN_SUFFIX)
+                || 0 == substr_compare($domain, MessageCatalogue::INTL_DOMAIN_SUFFIX, -$len, $len))) {
             return $this->formatter->formatIntl($catalogue->get($id, $domain), $locale, $parameters);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,10 @@ public function trans(?string $id, array $parameters = [], string $domain = null`
`214`	`214`	`}`
`215`	`215`	`}`
`216`	`216`
`217`		`- if ($this->hasIntlFormatter && $catalogue->defines($id, $domain.MessageCatalogue::INTL_DOMAIN_SUFFIX)) {`
	`217`	`+ $len = \strlen(MessageCatalogue::INTL_DOMAIN_SUFFIX);`
	`218`	`+ if ($this->hasIntlFormatter`
	`219`	`+ && ($catalogue->defines($id, $domain.MessageCatalogue::INTL_DOMAIN_SUFFIX)`
	`220`	`+ \|\| 0 == substr_compare($domain, MessageCatalogue::INTL_DOMAIN_SUFFIX, -$len, $len))) {`
`218`	`221`	`return $this->formatter->formatIntl($catalogue->get($id, $domain), $locale, $parameters);`
`219`	`222`	`}`
`220`	`223`