[Translator] deprecate getMessages in favor of getCatalogue.

aitboudad · aitboudad · commit c3567f84a3ba · 2015-05-04T14:44:28.000Z
diff --git a/src/Symfony/Component/Translation/CHANGELOG.md b/src/Symfony/Component/Translation/CHANGELOG.md
@@ -1,6 +1,10 @@
 CHANGELOG
 =========
 
+2.8.0
+-----
+ * deprecated Translator::getMessages() in favor of the Translator::getCatalogue() method
+
 2.7.0
 -----
 
diff --git a/src/Symfony/Component/Translation/Tests/TranslatorTest.php b/src/Symfony/Component/Translation/Tests/TranslatorTest.php
@@ -499,8 +499,10 @@ public function testTransChoiceFallbackWithNoTranslation()
     /**
      * @dataProvider dataProviderGetMessages
      */
-    public function testGetMessages($resources, $locale, $expected)
+    public function testLegacyGetMessages($resources, $locale, $expected)
     {
+        $this->iniSet('error_reporting', -1 & ~E_USER_DEPRECATED);
+
         $locales = array_keys($resources);
         $_locale = !is_null($locale) ? $locale : reset($locales);
         $locales = array_slice($locales, 0, array_search($_locale, $locales));
diff --git a/src/Symfony/Component/Translation/Translator.php b/src/Symfony/Component/Translation/Translator.php
@@ -292,6 +292,8 @@ protected function getLoaders()
      */
     public function getMessages($locale = null)
     {
+        trigger_error('The '.__METHOD__.' method is deprecated since version 2.8 and will be removed in 3.0. Use the getCatalogue() method instead.', E_USER_DEPRECATED);
+
         $catalogues = array();
         $catalogues[] = $catalogue = $this->getCatalogue($locale);
         while ($catalogue = $catalogue->getFallbackCatalogue()) {

Original file line number	Diff line number	Diff line change
`@@ -292,6 +292,8 @@ protected function getLoaders()`
`292`	`292`	`*/`
`293`	`293`	`public function getMessages($locale = null)`
`294`	`294`	`{`
	`295`	`+ trigger_error('The '.__METHOD__.' method is deprecated since version 2.8 and will be removed in 3.0. Use the getCatalogue() method instead.', E_USER_DEPRECATED);`
	`296`	`+`
`295`	`297`	`$catalogues = array();`
`296`	`298`	`$catalogues[] = $catalogue = $this->getCatalogue($locale);`
`297`	`299`	`while ($catalogue = $catalogue->getFallbackCatalogue()) {`