feature symfony#52140 [Translation] Add argument $buildDir to DataCollectorTranslator::warmUp() (nicolas-grekas)

This PR was merged into the 6.4 branch.

Discussion
----------

[Translation] Add argument `$buildDir` to `DataCollectorTranslator::warmUp()`

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| Tickets       | -
| License       | MIT

Reverting part of symfony#52136 to keep cross-versions compatibility of components.

We already did so a few times where the risk of breaking BC for real-life apps was considered low. It think this qualifies.

Commits
-------

d18bd27 [Translator] Add argument `$buildDir` to `DataCollectorTranslator::warmUp()`

Author: nicolas-grekas

UPGRADE-6.4.md

@@ -213,6 +213,11 @@ Templating
 
  * The component is deprecated and will be removed in 7.0, use [Twig](https://twig.symfony.com) instead
 
+Translator
+----------
+
+ * [BC Break] Add argument `$buildDir` to `DataCollectorTranslator::warmUp()`
+
 Validator
 ---------
 

src/Symfony/Component/Translation/CHANGELOG.md

@@ -6,6 +6,7 @@ CHANGELOG
 
  * Give current locale to `LocaleSwitcher::runWithLocale()`'s callback
  * Add `--as-tree` option to `translation:pull` command to write YAML messages as a tree-like structure
+ * [BC BREAK] Add argument `$buildDir` to `DataCollectorTranslator::warmUp()`
 
 6.3
 ---

src/Symfony/Component/Translation/DataCollectorTranslator.php

@@ -71,13 +71,8 @@ public function getCatalogues(): array
         return $this->translator->getCatalogues();
     }
 
-    /**
-     * @param string|null $buildDir
-     */
-    public function warmUp(string $cacheDir /* , string $buildDir = null */): array
+    public function warmUp(string $cacheDir, string $buildDir = null): array
     {
-        $buildDir = 1 < \func_num_args() ? func_get_arg(1) : null;
-
         if ($this->translator instanceof WarmableInterface) {
             return (array) $this->translator->warmUp($cacheDir, $buildDir);
         }

src/Symfony/Component/Translation/composer.json

@@ -27,7 +27,7 @@
         "symfony/console": "^5.4|^6.0|^7.0",
         "symfony/dependency-injection": "^5.4|^6.0|^7.0",
         "symfony/http-client-contracts": "^2.5|^3.0",
-        "symfony/http-kernel": "^5.4|^6.0",
+        "symfony/http-kernel": "^5.4|^6.0|^7.0",
         "symfony/intl": "^5.4|^6.0|^7.0",
         "symfony/polyfill-intl-icu": "^1.21",
         "symfony/routing": "^5.4|^6.0|^7.0",
@@ -40,7 +40,7 @@
         "symfony/config": "<5.4",
         "symfony/dependency-injection": "<5.4",
         "symfony/http-client-contracts": "<2.5",
-        "symfony/http-kernel": "<5.4|>=7.0",
+        "symfony/http-kernel": "<5.4",
         "symfony/service-contracts": "<2.5",
         "symfony/twig-bundle": "<5.4",
         "symfony/yaml": "<5.4",