[PsrHttpMessageBridge] Patch return types and fix CS

Author: derrabus

ArgumentValueResolver/PsrServerRequestResolver.php

@@ -41,9 +41,6 @@ public function __construct(HttpMessageFactoryInterface $httpMessageFactory)
         $this->httpMessageFactory = $httpMessageFactory;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function supports(Request $request, ArgumentMetadata $argument): bool
     {
         if ($this instanceof BaseValueResolverInterface) {
@@ -53,9 +50,6 @@ public function supports(Request $request, ArgumentMetadata $argument): bool
         return self::SUPPORTED_TYPES[$argument->getType()] ?? false;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function resolve(Request $request, ArgumentMetadata $argument): \Traversable
     {
         if (!isset(self::SUPPORTED_TYPES[$argument->getType()])) {

EventListener/PsrResponseListener.php

@@ -1,5 +1,14 @@
 <?php
 
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 namespace Symfony\Bridge\PsrHttpMessage\EventListener;
 
 use Psr\Http\Message\ResponseInterface;
@@ -38,9 +47,6 @@ public function onKernelView(ViewEvent $event): void
         $event->setResponse($this->httpFoundationFactory->createResponse($controllerResult));
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public static function getSubscribedEvents(): array
     {
         return [

Factory/HttpFoundationFactory.php

@@ -23,8 +23,6 @@
 use Symfony\Component\HttpFoundation\StreamedResponse;
 
 /**
- * {@inheritdoc}
- *
  * @author Kévin Dunglas <[email protected]>
  */
 class HttpFoundationFactory implements HttpFoundationFactoryInterface
@@ -39,12 +37,7 @@ public function __construct(int $responseBufferMaxLength = 16372)
         $this->responseBufferMaxLength = $responseBufferMaxLength;
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @return Request
-     */
-    public function createRequest(ServerRequestInterface $psrRequest, bool $streamed = false)
+    public function createRequest(ServerRequestInterface $psrRequest, bool $streamed = false): Request
     {
         $server = [];
         $uri = $psrRequest->getUri();
@@ -113,20 +106,13 @@ private function createUploadedFile(UploadedFileInterface $psrUploadedFile): Upl
 
     /**
      * Gets a temporary file path.
-     *
-     * @return string
      */
-    protected function getTemporaryPath()
+    protected function getTemporaryPath(): string
     {
         return tempnam(sys_get_temp_dir(), uniqid('symfony', true));
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @return Response
-     */
-    public function createResponse(ResponseInterface $psrResponse, bool $streamed = false)
+    public function createResponse(ResponseInterface $psrResponse, bool $streamed = false): Response
     {
         $cookies = $psrResponse->getHeader('Set-Cookie');
         $psrResponse = $psrResponse->withoutHeader('Set-Cookie');

Factory/PsrHttpFactory.php

@@ -46,12 +46,7 @@ public function __construct(ServerRequestFactoryInterface $serverRequestFactory,
         $this->responseFactory = $responseFactory;
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @return ServerRequestInterface
-     */
-    public function createRequest(Request $symfonyRequest)
+    public function createRequest(Request $symfonyRequest): ServerRequestInterface
     {
         $uri = $symfonyRequest->server->get('QUERY_STRING', '');
         $uri = $symfonyRequest->getSchemeAndHttpHost().$symfonyRequest->getBaseUrl().$symfonyRequest->getPathInfo().('' !== $uri ? '?'.$uri : '');
@@ -141,12 +136,7 @@ private function createUploadedFile(UploadedFile $symfonyUploadedFile): Uploaded
         );
     }
 
-    /**
-     * {@inheritdoc}
-     *
-     * @return ResponseInterface
-     */
-    public function createResponse(Response $symfonyResponse)
+    public function createResponse(Response $symfonyResponse): ResponseInterface
     {
         $response = $this->responseFactory->createResponse($symfonyResponse->getStatusCode(), Response::$statusTexts[$symfonyResponse->getStatusCode()] ?? '');
 

Factory/UploadedFile.php

@@ -49,9 +49,6 @@ public function __construct(UploadedFileInterface $psrUploadedFile, callable $ge
         $this->psrUploadedFile = $psrUploadedFile;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function move(string $directory, string $name = null): File
     {
         if (!$this->isValid() || $this->test) {
@@ -63,7 +60,7 @@ public function move(string $directory, string $name = null): File
         try {
             $this->psrUploadedFile->moveTo((string) $target);
         } catch (\RuntimeException $e) {
-            throw new FileException(sprintf('Could not move the file "%s" to "%s" (%s)', $this->getPathname(), $target, $e->getMessage()), 0, $e);
+            throw new FileException(sprintf('Could not move the file "%s" to "%s" (%s).', $this->getPathname(), $target, $e->getMessage()), 0, $e);
         }
 
         @chmod($target, 0666 & ~umask());

HttpFoundationFactoryInterface.php

@@ -25,15 +25,11 @@ interface HttpFoundationFactoryInterface
 {
     /**
      * Creates a Symfony Request instance from a PSR-7 one.
-     *
-     * @return Request
      */
-    public function createRequest(ServerRequestInterface $psrRequest, bool $streamed = false);
+    public function createRequest(ServerRequestInterface $psrRequest, bool $streamed = false): Request;
 
     /**
      * Creates a Symfony Response instance from a PSR-7 one.
-     *
-     * @return Response
      */
-    public function createResponse(ResponseInterface $psrResponse, bool $streamed = false);
+    public function createResponse(ResponseInterface $psrResponse, bool $streamed = false): Response;
 }

HttpMessageFactoryInterface.php

@@ -25,15 +25,11 @@ interface HttpMessageFactoryInterface
 {
     /**
      * Creates a PSR-7 Request instance from a Symfony one.
-     *
-     * @return ServerRequestInterface
      */
-    public function createRequest(Request $symfonyRequest);
+    public function createRequest(Request $symfonyRequest): ServerRequestInterface;
 
     /**
      * Creates a PSR-7 Response instance from a Symfony one.
-     *
-     * @return ResponseInterface
      */
-    public function createResponse(Response $symfonyResponse);
+    public function createResponse(Response $symfonyResponse): ResponseInterface;
 }

Tests/EventListener/PsrResponseListenerTest.php

@@ -46,10 +46,7 @@ public function testDoesNotConvertControllerResult()
         self::assertFalse($event->hasResponse());
     }
 
-    /**
-     * @param mixed $controllerResult
-     */
-    private function createEventMock($controllerResult): ViewEvent
+    private function createEventMock(mixed $controllerResult): ViewEvent
     {
         return new ViewEvent($this->createMock(HttpKernelInterface::class), new Request(), HttpKernelInterface::MAIN_REQUEST, $controllerResult);
     }