Backport type fixes

symfony · Jul 2, 2021 · 012c146 · 012c146
1 parent 1a2bdd5
commit 012c146
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/Header/Headers.php b/Header/Headers.php
@@ -229,7 +229,7 @@ public function toArray(): array
     /**
      * @internal
      */
-    public function getHeaderBody($name)
+    public function getHeaderBody(string $name)
     {
         return $this->has($name) ? $this->get($name)->getBody() : null;
     }
@@ -266,7 +266,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));

diff --git a/MimeTypeGuesserInterface.php b/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

diff --git a/Part/DataPart.php b/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');
     }