Improving the request/response format autodetection

yceruto · yceruto · commit fca70a0f3189 · 2019-07-03T09:44:09.000-04:00
diff --git a/src/Symfony/Component/HttpFoundation/Request.php b/src/Symfony/Component/HttpFoundation/Request.php
@@ -140,6 +140,11 @@ class Request
      */
     protected $acceptableContentTypes;
 
+    /**
+     * @var string|null
+     */
+    protected $preferredFormat;
+
     /**
      * @var string
      */
@@ -1559,6 +1564,28 @@ public function isNoCache()
         return $this->headers->hasCacheControlDirective('no-cache') || 'no-cache' == $this->headers->get('Pragma');
     }
 
+    /**
+     * Returns the preferred format.
+     */
+    public function getPreferredFormat(?string $default = 'html'): ?string
+    {
+        if (null !== $this->preferredFormat) {
+            return $this->preferredFormat;
+        }
+
+        $this->preferredFormat = $this->getRequestFormat($this->getContentType());
+
+        if (null === $this->preferredFormat) {
+            foreach ($this->getAcceptableContentTypes() as $contentType) {
+                if (null !== $this->preferredFormat = $this->getFormat($contentType)) {
+                    break;
+                }
+            }
+        }
+
+        return $this->preferredFormat ?: $default;
+    }
+
     /**
      * Returns the preferred language.
      *
diff --git a/src/Symfony/Component/HttpFoundation/Response.php b/src/Symfony/Component/HttpFoundation/Response.php
@@ -270,7 +270,7 @@ public function prepare(Request $request)
         } else {
             // Content-type based on the Request
             if (!$headers->has('Content-Type')) {
-                $format = $request->getRequestFormat();
+                $format = $request->getPreferredFormat();
                 if (null !== $format && $mimeType = $request->getMimeType($format)) {
                     $headers->set('Content-Type', $mimeType);
                 }
diff --git a/src/Symfony/Component/HttpKernel/EventListener/DebugHandlersListener.php b/src/Symfony/Component/HttpKernel/EventListener/DebugHandlersListener.php
@@ -174,7 +174,7 @@ public function onKernelException(GetResponseForExceptionEvent $event)
             $e = $request->attributes->get('exception');
 
             try {
-                return new Response($this->errorFormatter->render($e, $request->getRequestFormat()), $e->getStatusCode(), $e->getHeaders());
+                return new Response($this->errorFormatter->render($e, $request->getPreferredFormat()), $e->getStatusCode(), $e->getHeaders());
             } catch (ErrorRendererNotFoundException $_) {
                 return new Response($this->errorFormatter->render($e), $e->getStatusCode(), $e->getHeaders());
             }

Original file line number	Diff line number	Diff line change
`@@ -270,7 +270,7 @@ public function prepare(Request $request)`
`270`	`270`	`} else {`
`271`	`271`	`// Content-type based on the Request`
`272`	`272`	`if (!$headers->has('Content-Type')) {`
`273`		`- $format = $request->getRequestFormat();`
	`273`	`+ $format = $request->getPreferredFormat();`
`274`	`274`	`if (null !== $format && $mimeType = $request->getMimeType($format)) {`
`275`	`275`	`$headers->set('Content-Type', $mimeType);`
`276`	`276`	`}`
Original file line number	Diff line number	Diff line change
`@@ -174,7 +174,7 @@ public function onKernelException(GetResponseForExceptionEvent $event)`
`174`	`174`	`$e = $request->attributes->get('exception');`
`175`	`175`
`176`	`176`	`try {`
`177`		`- return new Response($this->errorFormatter->render($e, $request->getRequestFormat()), $e->getStatusCode(), $e->getHeaders());`
	`177`	`+ return new Response($this->errorFormatter->render($e, $request->getPreferredFormat()), $e->getStatusCode(), $e->getHeaders());`
`178`	`178`	`} catch (ErrorRendererNotFoundException $_) {`
`179`	`179`	`return new Response($this->errorFormatter->render($e), $e->getStatusCode(), $e->getHeaders());`
`180`	`180`	`}`