[Translation] deprecate passing a null locale

Simperfit · Simperfit · commit b10b6aedaba8 · 2019-07-07T09:03:36.000+02:00
diff --git a/src/Symfony/Component/Translation/MessageCatalogue.php b/src/Symfony/Component/Translation/MessageCatalogue.php
@@ -32,6 +32,10 @@ class MessageCatalogue implements MessageCatalogueInterface, MetadataAwareInterf
      */
     public function __construct(?string $locale, array $messages = [])
     {
+        if (null === $locale) {
+            @trigger_error(sprintf('Passing "null" to "$locale" in %s::%s has been deprecated since Symfony 4.4 and will throw in 5.0.', __CLASS__, __METHOD__), E_USER_DEPRECATED);
+        }
+
         $this->locale = $locale;
         $this->messages = $messages;
     }
diff --git a/src/Symfony/Component/Translation/Tests/TranslatorTest.php b/src/Symfony/Component/Translation/Tests/TranslatorTest.php
@@ -190,6 +190,18 @@ public function testAddResourceValidLocales($locale)
         $this->addToAssertionCount(1);
     }
 
+    /**
+     * @group legacy
+     * @expectedDeprecation  Passing "null" to "$locale" in Symfony\Component\Translation\Translator::Symfony\Component\Translation\Translator::addResource has been deprecated since Symfony 4.4 and will throw in 5.0.
+     */
+    public function testAddResourceNull()
+    {
+        $translator = new Translator('fr');
+        $translator->addResource('array', ['foo' => 'foofoo'], null);
+        // no assertion. this method just asserts that no exception is thrown
+        $this->addToAssertionCount(1);
+    }
+
     public function testAddResourceAfterTrans()
     {
         $translator = new Translator('fr');
@@ -527,7 +539,6 @@ public function getValidLocalesTests()
     {
         return [
             [''],
-            [null],
             ['fr'],
             ['francais'],
             ['FR'],
diff --git a/src/Symfony/Component/Translation/Translator.php b/src/Symfony/Component/Translation/Translator.php
@@ -132,6 +132,10 @@ public function addResource($format, $resource, $locale, $domain = null)
             $domain = 'messages';
         }
 
+        if (null === $locale) {
+            @trigger_error(sprintf('Passing "null" to "$locale" in %s::%s has been deprecated since Symfony 4.4 and will throw in 5.0.', __CLASS__, __METHOD__), E_USER_DEPRECATED);
+        }
+
         $this->assertValidLocale($locale);
 
         $this->resources[$locale][] = [$format, $resource, $domain];

Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,10 @@ class MessageCatalogue implements MessageCatalogueInterface, MetadataAwareInterf`
`32`	`32`	`*/`
`33`	`33`	`public function __construct(?string $locale, array $messages = [])`
`34`	`34`	`{`
	`35`	`+ if (null === $locale) {`
	`36`	`+ @trigger_error(sprintf('Passing "null" to "$locale" in %s::%s has been deprecated since Symfony 4.4 and will throw in 5.0.', __CLASS__, __METHOD__), E_USER_DEPRECATED);`
	`37`	`+ }`
	`38`	`+`
`35`	`39`	`$this->locale = $locale;`
`36`	`40`	`$this->messages = $messages;`
`37`	`41`	`}`
Original file line number	Diff line number	Diff line change
`@@ -132,6 +132,10 @@ public function addResource($format, $resource, $locale, $domain = null)`
`132`	`132`	`$domain = 'messages';`
`133`	`133`	`}`
`134`	`134`
	`135`	`+ if (null === $locale) {`
	`136`	`+ @trigger_error(sprintf('Passing "null" to "$locale" in %s::%s has been deprecated since Symfony 4.4 and will throw in 5.0.', __CLASS__, __METHOD__), E_USER_DEPRECATED);`
	`137`	`+ }`
	`138`	`+`
`135`	`139`	`$this->assertValidLocale($locale);`
`136`	`140`
`137`	`141`	`$this->resources[$locale][] = [$format, $resource, $domain];`