Merge branch '5.3' into 5.4

* 5.3:
  Backport type fixes
  fix annot
  Backport type fixes
  Fix CS
  Avoid triggering the autoloader in Deprecation::isLegacy()
  fix markdown markup
  Backport type fixes
  uzb translation
  [DependencyInjection] Fix doc blocks
  [DependencyInjection] Turn $defaultDeprecationTemplate into a constant
  [GHA] restore phpunit-bridge job
  rethrow caught exception
  [Form] better form doc types to support static analysis

Author: nicolas-grekas

Address.php

@@ -98,19 +98,20 @@ public static function create($address): self
         if ($address instanceof self) {
             return $address;
         }
-        if (\is_string($address)) {
-            if (false === strpos($address, '<')) {
-                return new self($address);
-            }
 
-            if (!preg_match(self::FROM_STRING_PATTERN, $address, $matches)) {
-                throw new InvalidArgumentException(sprintf('Could not parse "%s" to a "%s" instance.', $address, self::class));
-            }
+        if (!\is_string($address)) {
+            throw new InvalidArgumentException(sprintf('An address can be an instance of Address or a string ("%s" given).', get_debug_type($address)));
+        }
+
+        if (false === strpos($address, '<')) {
+            return new self($address);
+        }
 
-            return new self($matches['addrSpec'], trim($matches['displayName'], ' \'"'));
+        if (!preg_match(self::FROM_STRING_PATTERN, $address, $matches)) {
+            throw new InvalidArgumentException(sprintf('Could not parse "%s" to a "%s" instance.', $address, self::class));
         }
 
-        throw new InvalidArgumentException(sprintf('An address can be an instance of Address or a string ("%s" given).', get_debug_type($address)));
+        return new self($matches['addrSpec'], trim($matches['displayName'], ' \'"'));
     }
 
     /**

Header/Headers.php

@@ -254,7 +254,10 @@ public function toArray(): array
         return $arr;
     }
 
-    public function getHeaderBody($name)
+    /**
+     * @internal
+     */
+    public function getHeaderBody(string $name)
     {
         return $this->has($name) ? $this->get($name)->getBody() : null;
     }
@@ -288,7 +291,7 @@ public function getHeaderParameter(string $name, string $parameter): ?string
     /**
      * @internal
      */
-    public function setHeaderParameter(string $name, string $parameter, $value): void
+    public function setHeaderParameter(string $name, string $parameter, ?string $value): void
     {
         if (!$this->has($name)) {
             throw new LogicException(sprintf('Unable to set parameter "%s" on header "%s" as the header is not defined.', $parameter, $name));

MimeTypeGuesserInterface.php

@@ -26,8 +26,6 @@ public function isGuesserSupported(): bool;
     /**
      * Guesses the MIME type of the file with the given path.
      *
-     * @param string $path The path to the file
-     *
      * @return string|null The MIME type or null, if none could be guessed
      *
      * @throws \LogicException           If the guesser is not supported

Part/DataPart.php

@@ -39,8 +39,10 @@ public function __construct($body, string $filename = null, string $contentType
 
         parent::__construct($body, null, $subtype, $encoding);
 
-        $this->filename = $filename;
-        $this->setName($filename);
+        if (null !== $filename) {
+            $this->filename = $filename;
+            $this->setName($filename);
+        }
         $this->setDisposition('attachment');
     }
 

Part/TextPart.php

@@ -89,7 +89,7 @@ public function setDisposition(string $disposition)
      *
      * @return $this
      */
-    public function setName($name)
+    public function setName(string $name)
     {
         $this->name = $name;