From d7ee0ecc491bd45425ff3a5223c51a8ef200fee8 Mon Sep 17 00:00:00 2001 From: Fabien Potencier Date: Tue, 15 Jan 2019 08:33:14 +0100 Subject: [PATCH] [HttpFoundation] updated File code --- .../Component/HttpFoundation/File/File.php | 15 +++++---------- .../HttpFoundation/File/UploadedFile.php | 5 +---- 2 files changed, 6 insertions(+), 14 deletions(-) diff --git a/src/Symfony/Component/HttpFoundation/File/File.php b/src/Symfony/Component/HttpFoundation/File/File.php index de7b080973..74080de74a 100644 --- a/src/Symfony/Component/HttpFoundation/File/File.php +++ b/src/Symfony/Component/HttpFoundation/File/File.php @@ -55,18 +55,15 @@ class File extends \SplFileInfo */ public function guessExtension() { - $type = $this->getMimeType(); - $guesser = ExtensionGuesser::getInstance(); - - return $guesser->guess($type); + return ExtensionGuesser::getInstance()->guess($this->getMimeType()); } /** * Returns the mime type of the file. * - * The mime type is guessed using a MimeTypeGuesser instance, which uses finfo(), - * mime_content_type() and the system binary "file" (in this order), depending on - * which of those are available. + * The mime type is guessed using a MimeTypeGuesserInterface instance, + * which uses finfo_file() then the "file" system binary, + * depending on which of those are available. * * @return string|null The guessed mime type (e.g. "application/pdf") * @@ -74,9 +71,7 @@ class File extends \SplFileInfo */ public function getMimeType() { - $guesser = MimeTypeGuesser::getInstance(); - - return $guesser->guess($this->getPathname()); + return MimeTypeGuesser::getInstance()->guess($this->getPathname()); } /** diff --git a/src/Symfony/Component/HttpFoundation/File/UploadedFile.php b/src/Symfony/Component/HttpFoundation/File/UploadedFile.php index efe00d64cc..5434ce1991 100644 --- a/src/Symfony/Component/HttpFoundation/File/UploadedFile.php +++ b/src/Symfony/Component/HttpFoundation/File/UploadedFile.php @@ -140,10 +140,7 @@ class UploadedFile extends File */ public function guessClientExtension() { - $type = $this->getClientMimeType(); - $guesser = ExtensionGuesser::getInstance(); - - return $guesser->guess($type); + return ExtensionGuesser::getInstance()->guess($this->getClientMimeType()); } /**