Add void return types

Author: wouterj

BinaryFileResponse.php

@@ -351,6 +351,8 @@ public function getContent(): string|false
 
     /**
      * Trust X-Sendfile-Type header.
+     *
+     * @return void
      */
     public static function trustXSendfileTypeHeader()
     {

ExpressionRequestMatcher.php

@@ -29,6 +29,9 @@ class ExpressionRequestMatcher extends RequestMatcher
     private ExpressionLanguage $language;
     private Expression|string $expression;
 
+    /**
+     * @return void
+     */
     public function setExpression(ExpressionLanguage $language, Expression|string $expression)
     {
         $this->language = $language;

FileBag.php

@@ -31,12 +31,18 @@ public function __construct(array $parameters = [])
         $this->replace($parameters);
     }
 
+    /**
+     * @return void
+     */
     public function replace(array $files = [])
     {
         $this->parameters = [];
         $this->add($files);
     }
 
+    /**
+     * @return void
+     */
     public function set(string $key, mixed $value)
     {
         if (!\is_array($value) && !$value instanceof UploadedFile) {
@@ -46,6 +52,9 @@ public function set(string $key, mixed $value)
         parent::set($key, $this->convertFileInformation($value));
     }
 
+    /**
+     * @return void
+     */
     public function add(array $files = [])
     {
         foreach ($files as $key => $file) {

HeaderBag.php

@@ -86,6 +86,8 @@ public function keys(): array
 
     /**
      * Replaces the current HTTP headers by a new set.
+     *
+     * @return void
      */
     public function replace(array $headers = [])
     {
@@ -95,6 +97,8 @@ public function replace(array $headers = [])
 
     /**
      * Adds new headers the current HTTP headers set.
+     *
+     * @return void
      */
     public function add(array $headers)
     {
@@ -126,6 +130,8 @@ public function get(string $key, string $default = null): ?string
      *
      * @param string|string[]|null $values  The value or an array of values
      * @param bool                 $replace Whether to replace the actual value or not (true by default)
+     *
+     * @return void
      */
     public function set(string $key, string|array|null $values, bool $replace = true)
     {
@@ -170,6 +176,8 @@ public function contains(string $key, string $value): bool
 
     /**
      * Removes a header.
+     *
+     * @return void
      */
     public function remove(string $key)
     {
@@ -202,6 +210,8 @@ public function getDate(string $key, \DateTime $default = null): ?\DateTimeInter
 
     /**
      * Adds a custom Cache-Control directive.
+     *
+     * @return void
      */
     public function addCacheControlDirective(string $key, bool|string $value = true)
     {
@@ -228,6 +238,8 @@ public function getCacheControlDirective(string $key): bool|string|null
 
     /**
      * Removes a Cache-Control directive.
+     *
+     * @return void
      */
     public function removeCacheControlDirective(string $key)
     {

InputBag.php

@@ -43,7 +43,7 @@ public function get(string $key, mixed $default = null): string|int|float|bool|n
     /**
      * Replaces the current input values by a new set.
      */
-    public function replace(array $inputs = [])
+    public function replace(array $inputs = []): void
     {
         $this->parameters = [];
         $this->add($inputs);
@@ -52,7 +52,7 @@ public function replace(array $inputs = [])
     /**
      * Adds input values.
      */
-    public function add(array $inputs = [])
+    public function add(array $inputs = []): void
     {
         foreach ($inputs as $input => $value) {
             $this->set($input, $value);
@@ -64,7 +64,7 @@ public function add(array $inputs = [])
      *
      * @param string|int|float|bool|array|null $value
      */
-    public function set(string $key, mixed $value)
+    public function set(string $key, mixed $value): void
     {
         if (null !== $value && !\is_scalar($value) && !\is_array($value) && !$value instanceof \Stringable) {
             throw new \InvalidArgumentException(sprintf('Expected a scalar, or an array as a 2nd argument to "%s()", "%s" given.', __METHOD__, get_debug_type($value)));

ParameterBag.php

@@ -60,6 +60,8 @@ public function keys(): array
 
     /**
      * Replaces the current parameters by a new set.
+     *
+     * @return void
      */
     public function replace(array $parameters = [])
     {
@@ -68,6 +70,8 @@ public function replace(array $parameters = [])
 
     /**
      * Adds parameters.
+     *
+     * @return void
      */
     public function add(array $parameters = [])
     {
@@ -79,6 +83,9 @@ public function get(string $key, mixed $default = null): mixed
         return \array_key_exists($key, $this->parameters) ? $this->parameters[$key] : $default;
     }
 
+    /**
+     * @return void
+     */
     public function set(string $key, mixed $value)
     {
         $this->parameters[$key] = $value;
@@ -94,6 +101,8 @@ public function has(string $key): bool
 
     /**
      * Removes a parameter.
+     *
+     * @return void
      */
     public function remove(string $key)
     {

Request.php

@@ -263,6 +263,8 @@ public function __construct(array $query = [], array $request = [], array $attri
      * @param array                $files      The FILES parameters
      * @param array                $server     The SERVER parameters
      * @param string|resource|null $content    The raw body data
+     *
+     * @return void
      */
     public function initialize(array $query = [], array $request = [], array $attributes = [], array $cookies = [], array $files = [], array $server = [], $content = null)
     {
@@ -417,6 +419,8 @@ public static function create(string $uri, string $method = 'GET', array $parame
      * This is mainly useful when you need to override the Request class
      * to keep BC with an existing system. It should not be used for any
      * other purpose.
+     *
+     * @return void
      */
     public static function setFactory(?callable $callable)
     {
@@ -521,6 +525,8 @@ public function __toString(): string
      *
      * It overrides $_GET, $_POST, $_REQUEST, $_SERVER, $_COOKIE.
      * $_FILES is never overridden, see rfc1867
+     *
+     * @return void
      */
     public function overrideGlobals()
     {
@@ -561,6 +567,8 @@ public function overrideGlobals()
      *
      * @param array $proxies          A list of trusted proxies, the string 'REMOTE_ADDR' will be replaced with $_SERVER['REMOTE_ADDR']
      * @param int   $trustedHeaderSet A bit field of Request::HEADER_*, to set which headers to trust from your proxies
+     *
+     * @return void
      */
     public static function setTrustedProxies(array $proxies, int $trustedHeaderSet)
     {
@@ -602,6 +610,8 @@ public static function getTrustedHeaderSet(): int
      * You should only list the hosts you manage using regexs.
      *
      * @param array $hostPatterns A list of trusted host patterns
+     *
+     * @return void
      */
     public static function setTrustedHosts(array $hostPatterns)
     {
@@ -648,6 +658,8 @@ public static function normalizeQueryString(?string $qs): string
      * If these methods are not protected against CSRF, this presents a possible vulnerability.
      *
      * The HTTP method can only be overridden when the real HTTP method is POST.
+     *
+     * @return void
      */
     public static function enableHttpMethodParameterOverride()
     {
@@ -733,6 +745,9 @@ public function hasSession(bool $skipIfUninitialized = false): bool
         return null !== $this->session && (!$skipIfUninitialized || $this->session instanceof SessionInterface);
     }
 
+    /**
+     * @return void
+     */
     public function setSession(SessionInterface $session)
     {
         $this->session = $session;
@@ -743,7 +758,7 @@ public function setSession(SessionInterface $session)
      *
      * @param callable(): SessionInterface $factory
      */
-    public function setSessionFactory(callable $factory)
+    public function setSessionFactory(callable $factory): void
     {
         $this->session = $factory;
     }
@@ -1153,6 +1168,8 @@ public function getHost(): string
 
     /**
      * Sets the request method.
+     *
+     * @return void
      */
     public function setMethod(string $method)
     {
@@ -1274,6 +1291,8 @@ public function getFormat(?string $mimeType): ?string
      * Associates a format with mime types.
      *
      * @param string|string[] $mimeTypes The associated mime types (the preferred one must be the first as it will be used as the content type)
+     *
+     * @return void
      */
     public function setFormat(?string $format, string|array $mimeTypes)
     {
@@ -1304,6 +1323,8 @@ public function getRequestFormat(?string $default = 'html'): ?string
 
     /**
      * Sets the request format.
+     *
+     * @return void
      */
     public function setRequestFormat(?string $format)
     {
@@ -1334,6 +1355,8 @@ public function getContentTypeFormat(): ?string
 
     /**
      * Sets the default locale.
+     *
+     * @return void
      */
     public function setDefaultLocale(string $locale)
     {
@@ -1354,6 +1377,8 @@ public function getDefaultLocale(): string
 
     /**
      * Sets the locale.
+     *
+     * @return void
      */
     public function setLocale(string $locale)
     {
@@ -1858,6 +1883,8 @@ protected function preparePathInfo(): string
 
     /**
      * Initializes HTTP request formats.
+     *
+     * @return void
      */
     protected static function initializeFormats()
     {

RequestMatcher.php

@@ -69,6 +69,8 @@ public function __construct(string $path = null, string $host = null, string|arr
      * Adds a check for the HTTP scheme.
      *
      * @param string|string[]|null $scheme An HTTP scheme or an array of HTTP schemes
+     *
+     * @return void
      */
     public function matchScheme(string|array|null $scheme)
     {
@@ -77,6 +79,8 @@ public function matchScheme(string|array|null $scheme)
 
     /**
      * Adds a check for the URL host name.
+     *
+     * @return void
      */
     public function matchHost(?string $regexp)
     {
@@ -87,6 +91,8 @@ public function matchHost(?string $regexp)
      * Adds a check for the the URL port.
      *
      * @param int|null $port The port number to connect to
+     *
+     * @return void
      */
     public function matchPort(?int $port)
     {
@@ -95,6 +101,8 @@ public function matchPort(?int $port)
 
     /**
      * Adds a check for the URL path info.
+     *
+     * @return void
      */
     public function matchPath(?string $regexp)
     {
@@ -105,6 +113,8 @@ public function matchPath(?string $regexp)
      * Adds a check for the client IP.
      *
      * @param string $ip A specific IP address or a range specified using IP/netmask like 192.168.1.0/24
+     *
+     * @return void
      */
     public function matchIp(string $ip)
     {
@@ -115,6 +125,8 @@ public function matchIp(string $ip)
      * Adds a check for the client IP.
      *
      * @param string|string[]|null $ips A specific IP address or a range specified using IP/netmask like 192.168.1.0/24
+     *
+     * @return void
      */
     public function matchIps(string|array|null $ips)
     {
@@ -127,6 +139,8 @@ public function matchIps(string|array|null $ips)
      * Adds a check for the HTTP method.
      *
      * @param string|string[]|null $method An HTTP method or an array of HTTP methods
+     *
+     * @return void
      */
     public function matchMethod(string|array|null $method)
     {
@@ -135,6 +149,8 @@ public function matchMethod(string|array|null $method)
 
     /**
      * Adds a check for request attribute.
+     *
+     * @return void
      */
     public function matchAttribute(string $key, string $regexp)
     {

RequestStack.php

@@ -31,6 +31,8 @@ class RequestStack
      *
      * This method should generally not be called directly as the stack
      * management should be taken care of by the application itself.
+     *
+     * @return void
      */
     public function push(Request $request)
     {