[BUGFIX] Use proper phpDoc in PSR-7 implementation 63/57863/2
authorBenni Mack <benni@typo3.org>
Fri, 10 Aug 2018 18:39:47 +0000 (20:39 +0200)
committerGeorg Ringer <georg.ringer@gmail.com>
Fri, 10 Aug 2018 18:47:01 +0000 (20:47 +0200)
This fixes some IDE issues when working with PSR-7 classes within TYPO3.

Resolves: #85811
Releases: master, 8.7
Change-Id: I69b9fed8a603b32ac6bfed57c251907d09dcae19
Reviewed-on: https://review.typo3.org/57863
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
typo3/sysext/core/Classes/Http/Message.php
typo3/sysext/core/Classes/Http/Request.php
typo3/sysext/core/Classes/Http/Response.php
typo3/sysext/core/Classes/Http/ServerRequest.php
typo3/sysext/core/Classes/Http/Uri.php

index a04c80e..3e76881 100644 (file)
@@ -76,7 +76,7 @@ class Message implements MessageInterface
      * new protocol version.
      *
      * @param string $version HTTP protocol version
-     * @return Message
+     * @return static
      */
     public function withProtocolVersion($version)
     {
@@ -195,7 +195,7 @@ class Message implements MessageInterface
      *
      * @param string $name Case-insensitive header field name.
      * @param string|string[] $value Header value(s).
-     * @return Message
+     * @return static
      * @throws \InvalidArgumentException for invalid header names or values.
      */
     public function withHeader($name, $value)
@@ -231,7 +231,7 @@ class Message implements MessageInterface
      *
      * @param string $name Case-insensitive header field name to add.
      * @param string|string[] $value Header value(s).
-     * @return Message
+     * @return static
      * @throws \InvalidArgumentException for invalid header names or values.
      */
     public function withAddedHeader($name, $value)
@@ -263,7 +263,7 @@ class Message implements MessageInterface
      * the named header.
      *
      * @param string $name Case-insensitive header field name to remove.
-     * @return Message
+     * @return static
      */
     public function withoutHeader($name)
     {
@@ -298,7 +298,7 @@ class Message implements MessageInterface
      * new body stream.
      *
      * @param \Psr\Http\Message\StreamInterface $body Body.
-     * @return Message
+     * @return static
      * @throws \InvalidArgumentException When the body is not valid.
      */
     public function withBody(StreamInterface $body)
index c3f6058..26a6379 100644 (file)
@@ -233,7 +233,7 @@ class Request extends Message implements RequestInterface
      *     request-target forms allowed in request messages)
      *
      * @param mixed $requestTarget
-     * @return Request
+     * @return static
      */
     public function withRequestTarget($requestTarget)
     {
@@ -267,7 +267,7 @@ class Request extends Message implements RequestInterface
      * changed request method.
      *
      * @param string $method Case-sensitive method.
-     * @return Request
+     * @return static
      * @throws \InvalidArgumentException for invalid HTTP methods.
      */
     public function withMethod($method)
@@ -320,7 +320,7 @@ class Request extends Message implements RequestInterface
      *
      * @param \Psr\Http\Message\UriInterface $uri New request URI to use.
      * @param bool $preserveHost Preserve the original state of the Host header.
-     * @return Request
+     * @return static
      */
     public function withUri(UriInterface $uri, $preserveHost = false)
     {
index 6d5d01b..3a4a0dd 100644 (file)
@@ -176,7 +176,7 @@ class Response extends Message implements ResponseInterface
      * @param string $reasonPhrase The reason phrase to use with the
      *     provided status code; if none is provided, implementations MAY
      *     use the defaults as suggested in the HTTP specification.
-     * @return Response
+     * @return static
      * @throws \InvalidArgumentException For invalid status code arguments.
      */
     public function withStatus($code, $reasonPhrase = '')
@@ -185,7 +185,7 @@ class Response extends Message implements ResponseInterface
             throw new \InvalidArgumentException('The given status code is not a valid HTTP status code', 1436717279);
         }
         $clonedObject = clone $this;
-        $clonedObject->statusCode = $code;
+        $clonedObject->statusCode = (int)$code;
         $clonedObject->reasonPhrase = $reasonPhrase !== '' ? $reasonPhrase : $this->availableStatusCodes[$code];
         return $clonedObject;
     }
index 11eac2c..daa16b4 100644 (file)
@@ -130,7 +130,7 @@ class ServerRequest extends Request implements ServerRequestInterface
      * updated cookie values.
      *
      * @param array $cookies Array of key/value pairs representing cookies.
-     * @return ServerRequest
+     * @return static
      */
     public function withCookieParams(array $cookies)
     {
@@ -176,7 +176,7 @@ class ServerRequest extends Request implements ServerRequestInterface
      *
      * @param array $query Array of query string arguments, typically from
      *     $_GET.
-     * @return ServerRequest
+     * @return static
      */
     public function withQueryParams(array $query)
     {
@@ -210,7 +210,7 @@ class ServerRequest extends Request implements ServerRequestInterface
      * updated body parameters.
      *
      * @param array $uploadedFiles An array tree of UploadedFileInterface instances.
-     * @return ServerRequest
+     * @return static
      * @throws \InvalidArgumentException if an invalid structure is provided.
      */
     public function withUploadedFiles(array $uploadedFiles)
@@ -265,7 +265,7 @@ class ServerRequest extends Request implements ServerRequestInterface
      *
      * @param array|object|null $data The deserialized body data. This will
      *     typically be in an array or object.
-     * @return ServerRequest
+     * @return static
      * @throws \InvalidArgumentException if an unsupported argument type is
      *     provided.
      */
@@ -327,7 +327,7 @@ class ServerRequest extends Request implements ServerRequestInterface
      *
      * @param string $name The attribute name.
      * @param mixed $value The value of the attribute.
-     * @return ServerRequest
+     * @return static
      */
     public function withAttribute($name, $value)
     {
@@ -349,7 +349,7 @@ class ServerRequest extends Request implements ServerRequestInterface
      * @see getAttributes()
      *
      * @param string $name The attribute name.
-     * @return ServerRequest
+     * @return static
      */
     public function withoutAttribute($name)
     {
index 8507178..2c9c9d6 100644 (file)
@@ -355,7 +355,7 @@ class Uri implements UriInterface
      *
      * @param string $scheme The scheme to use with the new instance.
      *
-     * @return self A new instance with the specified scheme.
+     * @return static A new instance with the specified scheme.
      * @throws \InvalidArgumentException for invalid or unsupported schemes.
      */
     public function withScheme($scheme)
@@ -380,7 +380,7 @@ class Uri implements UriInterface
      * @param string $user The user name to use for authority.
      * @param string|null $password The password associated with $user.
      *
-     * @return self A new instance with the specified user information.
+     * @return static A new instance with the specified user information.
      */
     public function withUserInfo($user, $password = null)
     {
@@ -404,7 +404,7 @@ class Uri implements UriInterface
      *
      * @param string $host The hostname to use with the new instance.
      *
-     * @return self A new instance with the specified host.
+     * @return static A new instance with the specified host.
      * @throws \InvalidArgumentException for invalid hostnames.
      */
     public function withHost($host)
@@ -429,7 +429,7 @@ class Uri implements UriInterface
      * @param int|null $port The port to use with the new instance; a null value
      *     removes the port information.
      *
-     * @return self A new instance with the specified port.
+     * @return static A new instance with the specified port.
      * @throws \InvalidArgumentException for invalid ports.
      */
     public function withPort($port)
@@ -471,7 +471,7 @@ class Uri implements UriInterface
      *
      * @param string $path The path to use with the new instance.
      *
-     * @return self A new instance with the specified path.
+     * @return static A new instance with the specified path.
      * @throws \InvalidArgumentException for invalid paths.
      */
     public function withPath($path)
@@ -507,7 +507,7 @@ class Uri implements UriInterface
      *
      * @param string $query The query string to use with the new instance.
      *
-     * @return self A new instance with the specified query string.
+     * @return static A new instance with the specified query string.
      * @throws \InvalidArgumentException for invalid query strings.
      */
     public function withQuery($query)
@@ -539,7 +539,7 @@ class Uri implements UriInterface
      *
      * @param string $fragment The fragment to use with the new instance.
      *
-     * @return self A new instance with the specified fragment.
+     * @return static A new instance with the specified fragment.
      */
     public function withFragment($fragment)
     {