Merge branch '6.3' into 6.4

* 6.3:
minor #53524 [Messenger] [AmazonSqs] Allow `async-aws/sqs` version 2 (smoench)
Fix bad merge
List CS fix in .git-blame-ignore-revs
Fix implicitly-required parameters
List CS fix in .git-blame-ignore-revs
Apply php-cs-fixer fix --rules nullable_type_declaration_for_default_null_value

Author: nicolas-grekas

AbstractBrowser.php

@@ -54,7 +54,7 @@ abstract class AbstractBrowser
 /**
  * @param array $server The server parameters (equivalent of $_SERVER)
  */
-publicfunction__construct(array$server = [], History$history = null, CookieJar$cookieJar = null)
+publicfunction__construct(array$server = [], ?History$history = null, ?CookieJar$cookieJar = null)
  {
 $this->setServerParameters($server);
 $this->history = $history ?? newHistory();
@@ -154,7 +154,7 @@ public function getServerParameter(string $key, mixed $default = ''): mixed
 return$this->server[$key] ?? $default;
  }
 
-publicfunctionxmlHttpRequest(string$method, string$uri, array$parameters = [], array$files = [], array$server = [], string$content = null, bool$changeHistory = true): Crawler
+publicfunctionxmlHttpRequest(string$method, string$uri, array$parameters = [], array$files = [], array$server = [], ?string$content = null, bool$changeHistory = true): Crawler
  {
 $this->setServerParameter('HTTP_X_REQUESTED_WITH', 'XMLHttpRequest');
 
@@ -339,7 +339,7 @@ public function submitForm(string $button, array $fieldValues = [], string $meth
  * @param string $content The raw body data
  * @param bool $changeHistory Whether to update the history or not (only used internally for back(), forward(), and reload())
  */
-publicfunctionrequest(string$method, string$uri, array$parameters = [], array$files = [], array$server = [], string$content = null, bool$changeHistory = true): Crawler
+publicfunctionrequest(string$method, string$uri, array$parameters = [], array$files = [], array$server = [], ?string$content = null, bool$changeHistory = true): Crawler
  {
 if ($this->isMainRequest) {
 $this->redirectCount = 0;

Cookie.php

@@ -58,7 +58,7 @@ class Cookie
  * @param bool $encodedValue Whether the value is encoded or not
  * @param string|null $samesite The cookie samesite attribute
  */
-publicfunction__construct(string$name, ?string$value, string$expires = null, string$path = null, string$domain = '', bool$secure = false, bool$httponly = true, bool$encodedValue = false, string$samesite = null)
+publicfunction__construct(string$name, ?string$value, ?string$expires = null, ?string$path = null, string$domain = '', bool$secure = false, bool$httponly = true, bool$encodedValue = false, ?string$samesite = null)
  {
 if ($encodedValue) {
 $this->rawValue = $value ?? '';
@@ -124,7 +124,7 @@ public function __toString(): string
  *
  * @throws InvalidArgumentException
  */
-publicstaticfunctionfromString(string$cookie, string$url = null): static
+publicstaticfunctionfromString(string$cookie, ?string$url = null): static
  {
 $parts = explode(';', $cookie);
 

CookieJar.php

@@ -38,7 +38,7 @@ public function set(Cookie $cookie)
  * (this behavior ensures a BC behavior with previous versions of
  * Symfony).
  */
-publicfunctionget(string$name, string$path = '/', string$domain = null): ?Cookie
+publicfunctionget(string$name, string$path = '/', ?string$domain = null): ?Cookie
  {
 $this->flushExpiredCookies();
 
@@ -72,7 +72,7 @@ public function get(string $name, string $path = '/', string $domain = null): ?C
  *
  * @return void
  */
-publicfunctionexpire(string$name, ?string$path = '/', string$domain = null)
+publicfunctionexpire(string$name, ?string$path = '/', ?string$domain = null)
  {
 $path ??= '/';
 
@@ -114,7 +114,7 @@ public function clear()
  *
  * @return void
  */
-publicfunctionupdateFromSetCookie(array$setCookies, string$uri = null)
+publicfunctionupdateFromSetCookie(array$setCookies, ?string$uri = null)
  {
 $cookies = [];
 
@@ -142,7 +142,7 @@ public function updateFromSetCookie(array $setCookies, string $uri = null)
  *
  * @return void
  */
-publicfunctionupdateFromResponse(Response$response, string$uri = null)
+publicfunctionupdateFromResponse(Response$response, ?string$uri = null)
  {
 $this->updateFromSetCookie($response->getHeader('Set-Cookie', false), $uri);
  }

HttpBrowser.php

@@ -29,7 +29,7 @@ class HttpBrowser extends AbstractBrowser
 {
 privateHttpClientInterface$client;
 
-publicfunction__construct(HttpClientInterface$client = null, History$history = null, CookieJar$cookieJar = null)
+publicfunction__construct(?HttpClientInterface$client = null, ?History$history = null, ?CookieJar$cookieJar = null)
  {
 if (!$client && !class_exists(HttpClient::class)) {
 thrownewLogicException(sprintf('You cannot use "%s" as the HttpClient component is not installed. Try running "composer require symfony/http-client".', __CLASS__));

Request.php

@@ -33,7 +33,7 @@ class Request
  * @param array $server An array of server parameters
  * @param string $content The raw body data
  */
-publicfunction__construct(string$uri, string$method, array$parameters = [], array$files = [], array$cookies = [], array$server = [], string$content = null)
+publicfunction__construct(string$uri, string$method, array$parameters = [], array$files = [], array$cookies = [], array$server = [], ?string$content = null)
  {
 $this->uri = $uri;
 $this->method = $method;

Test/Constraint/BrowserCookieValueSame.php

@@ -22,7 +22,7 @@ final class BrowserCookieValueSame extends Constraint
 privatestring$path;
 private ?string$domain;
 
-publicfunction__construct(string$name, string$value, bool$raw = false, string$path = '/', string$domain = null)
+publicfunction__construct(string$name, string$value, bool$raw = false, string$path = '/', ?string$domain = null)
  {
 $this->name = $name;
 $this->path = $path;

Test/Constraint/BrowserHasCookie.php

@@ -20,7 +20,7 @@ final class BrowserHasCookie extends Constraint
 privatestring$path;
 private ?string$domain;
 
-publicfunction__construct(string$name, string$path = '/', string$domain = null)
+publicfunction__construct(string$name, string$path = '/', ?string$domain = null)
  {
 $this->name = $name;
 $this->path = $path;

Tests/AbstractBrowserTest.php

@@ -21,7 +21,7 @@
 
 class AbstractBrowserTest extends TestCase
 {
-publicfunctiongetBrowser(array$server = [], History$history = null, CookieJar$cookieJar = null)
+publicfunctiongetBrowser(array$server = [], ?History$history = null, ?CookieJar$cookieJar = null)
  {
 returnnewTestClient($server, $history, $cookieJar);
  }

Tests/HttpBrowserTest.php

@@ -19,7 +19,7 @@
 
 class HttpBrowserTest extends AbstractBrowserTest
 {
-publicfunctiongetBrowser(array$server = [], History$history = null, CookieJar$cookieJar = null)
+publicfunctiongetBrowser(array$server = [], ?History$history = null, ?CookieJar$cookieJar = null)
  {
 returnnewTestHttpClient($server, $history, $cookieJar);
  }

Tests/TestHttpClient.php

@@ -23,7 +23,7 @@ class TestHttpClient extends HttpBrowser
 protected ?Response$nextResponse = null;
 protectedstring$nextScript;
 
-publicfunction__construct(array$server = [], History$history = null, CookieJar$cookieJar = null)
+publicfunction__construct(array$server = [], ?History$history = null, ?CookieJar$cookieJar = null)
  {
 $client = newMockHttpClient(function (string$method, string$url, array$options) {
 if (null === $this->nextResponse) {