symfony · ihmels · Jul 19, 2022
diff --git a/UPGRADE-6.2.md b/UPGRADE-6.2.md
@@ -9,6 +9,11 @@ FrameworkBundle
   `Symfony\Component\Serializer\Normalizer\NormalizerInterface` or implement `NormalizerAwareInterface` instead
 * Deprecate `AbstractController::renderForm()`, use `render()` instead

+HttpFoundation
+--------------
+
+ * Deprecate `Request::getContentType()`, use `getContentFormat()` instead
+
 Mailer
 --------


diff --git a/src/Symfony/Component/HttpFoundation/CHANGELOG.md b/src/Symfony/Component/HttpFoundation/CHANGELOG.md
@@ -1,6 +1,11 @@
 CHANGELOG
 =========

+6.2
+---
+
+* Deprecate the `Request::getContentType()` method and add `getContentFormat()` as replacement
+
 6.1
 ---


diff --git a/src/Symfony/Component/HttpFoundation/Request.php b/src/Symfony/Component/HttpFoundation/Request.php
@@ -1332,8 +1332,20 @@ public function setRequestFormat(?string $format)

    /**
     * Gets the format associated with the request.
+     *
+     * @deprecated since Symfony 6.2, use getContentFormat() instead.
     */
    public function getContentType(): ?string
+    {
+        trigger_deprecation('symfony/http-foundation', '6.2', 'The "%s::getContentType()" method is deprecated, use "getContentFormat()" instead.', get_debug_type($this));
+
+        return $this->getContentFormat();
+    }
+
+    /**
+     * Gets the format associated with the request.
+     */
+    public function getContentFormat(): ?string
    {
        return $this->getFormat($this->headers->get('CONTENT_TYPE', ''));
    }

diff --git a/src/Symfony/Component/HttpFoundation/Tests/RequestTest.php b/src/Symfony/Component/HttpFoundation/Tests/RequestTest.php
@@ -12,6 +12,7 @@
 namespace Symfony\Component\HttpFoundation\Tests;

 use PHPUnit\Framework\TestCase;
+use Symfony\Bridge\PhpUnit\ExpectDeprecationTrait;
 use Symfony\Component\HttpFoundation\Exception\ConflictingHeadersException;
 use Symfony\Component\HttpFoundation\Exception\JsonException;
 use Symfony\Component\HttpFoundation\Exception\SuspiciousOperationException;
@@ -23,6 +24,8 @@

 class RequestTest extends TestCase
 {
+    use ExpectDeprecationTrait;
+
    protected function tearDown(): void
    {
        Request::setTrustedProxies([], -1);
@@ -80,12 +83,22 @@ public function testIsNoCache()

    public function testGetContentType()
    {
+        $this->expectDeprecation('Since symfony/http-foundation 6.2: The "Symfony\Component\HttpFoundation\Request::getContentType()" method is deprecated, use "getContentFormat()" instead.');
+
        $request = new Request();
        $contentType = $request->getContentType();

        $this->assertNull($contentType);
    }

+    public function testGetContentFormat()
+    {
+        $request = new Request();
+        $contentFormat = $request->getContentFormat();
+
+        $this->assertNull($contentFormat);
+    }
+
    public function testSetDefaultLocale()
    {
        $request = new Request();

@@ -74,7 +74,7 @@ public function supports(Request $request): bool
    {
        return ($this->options['post_only'] ? $request->isMethod('POST') : true)
            && $this->httpUtils->checkRequestPath($request, $this->options['check_path'])
-            && ($this->options['form_only'] ? 'form' === $request->getContentType() : true);
+            && ($this->options['form_only'] ? 'form' === $request->getContentFormat() : true);
    }

    public function authenticate(Request $request): Passport

@@ -63,7 +63,7 @@ public function __construct(HttpUtils $httpUtils, UserProviderInterface $userPro

    public function supports(Request $request): ?bool
    {
-        if (!str_contains($request->getRequestFormat() ?? '', 'json') && !str_contains($request->getContentType() ?? '', 'json')) {
+        if (!str_contains($request->getRequestFormat() ?? '', 'json') && !str_contains($request->getContentFormat() ?? '', 'json')) {
            return false;
        }