Require PHP 8.1+

Signed-off-by: Luís Cobucci <[email protected]>

.github/workflows/coding-standards.yml

@@ -15,7 +15,7 @@ jobs:
         dependencies:
           - "locked"
         php-version:
-          - "8.0"
+          - "8.1"
         operating-system:
           - "ubuntu-latest"
 

.github/workflows/composer-json-lint.yml

@@ -15,7 +15,7 @@ jobs:
         dependencies:
           - "highest"
         php-version:
-          - "8.0"
+          - "8.1"
         operating-system:
           - "ubuntu-latest"
 

.github/workflows/mutation-tests.yml

@@ -15,7 +15,7 @@ jobs:
         dependencies:
           - "locked"
         php-version:
-          - "8.0"
+          - "8.1"
         operating-system:
           - "ubuntu-latest"
 

.github/workflows/phpunit.yml

@@ -18,7 +18,6 @@ jobs:
           - "locked"
           - "development"
         php-version:
-          - "8.0"
           - "8.1"
         operating-system:
           - "ubuntu-latest"

.github/workflows/static-analysis.yml

@@ -15,7 +15,7 @@ jobs:
         dependencies:
           - "locked"
         php-version:
-          - "8.0"
+          - "8.1"
         operating-system:
           - "ubuntu-latest"
 

composer.json

@@ -10,7 +10,7 @@
         }
     ],
     "require": {
-        "php": "^8.0",
+        "php": "^8.1",
         "ext-json": "*",
         "fig/http-message-util": "^1.1",
         "psr/http-factory": "^1.0",

src/ContentTypeMiddleware.php

@@ -18,9 +18,9 @@ final class ContentTypeMiddleware implements MiddlewareInterface
 {
     /** @param Formatter[] $formatters */
     public function __construct(
-        private MiddlewareInterface $negotiator,
-        private array $formatters,
-        private StreamFactoryInterface $streamFactory,
+        private readonly MiddlewareInterface $negotiator,
+        private readonly array $formatters,
+        private readonly StreamFactoryInterface $streamFactory,
     ) {
     }
 
@@ -54,9 +54,9 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
         }
 
         $contentType = $this->extractContentType($response->getHeaderLine('Content-Type'));
-        $formatter   = $this->formatters[$contentType] ?? new NotAcceptable();
 
-        return $formatter->format($response, $this->streamFactory);
+        return ($this->formatters[$contentType] ?? new NotAcceptable())
+            ->format($response, $this->streamFactory);
     }
 
     private function extractContentType(string $contentType): string

src/Formatter/JmsSerializer.php

@@ -11,8 +11,10 @@
 
 final class JmsSerializer extends ContentOnly
 {
-    public function __construct(private SerializerInterface $serializer, private string $format)
-    {
+    public function __construct(
+        private readonly SerializerInterface $serializer,
+        private readonly string $format,
+    ) {
     }
 
     /** {@inheritdoc} */

src/Formatter/Json.php

@@ -20,7 +20,7 @@ final class Json extends ContentOnly
 {
     private const DEFAULT_FLAGS = JSON_HEX_TAG | JSON_HEX_APOS | JSON_HEX_AMP | JSON_HEX_QUOT | JSON_UNESCAPED_SLASHES;
 
-    public function __construct(private int $flags = self::DEFAULT_FLAGS)
+    public function __construct(private readonly int $flags = self::DEFAULT_FLAGS)
     {
     }
 

src/Formatter/Plates.php

@@ -14,8 +14,10 @@ final class Plates extends ContentOnly
 {
     private const DEFAULT_ATTRIBUTE = 'template';
 
-    public function __construct(private Engine $engine, private string $attributeName = self::DEFAULT_ATTRIBUTE)
-    {
+    public function __construct(
+        private readonly Engine $engine,
+        private readonly string $attributeName = self::DEFAULT_ATTRIBUTE,
+    ) {
     }
 
     /** {@inheritdoc} */

src/Formatter/Twig.php

@@ -15,8 +15,8 @@ final class Twig extends ContentOnly
     private const DEFAULT_ATTRIBUTE = 'template';
 
     public function __construct(
-        private Environment $environment,
-        private string $attributeName = self::DEFAULT_ATTRIBUTE,
+        private readonly Environment $environment,
+        private readonly string $attributeName = self::DEFAULT_ATTRIBUTE,
     ) {
     }
 

src/UnformattedResponse.php

@@ -10,9 +10,9 @@ final class UnformattedResponse implements ResponseInterface
 {
     /** @param array<string, mixed> $attributes */
     public function __construct(
-        private ResponseInterface $decoratedResponse,
-        private mixed $unformattedContent,
-        private array $attributes = [],
+        private readonly ResponseInterface $decoratedResponse,
+        private readonly mixed $unformattedContent,
+        private readonly array $attributes = [],
     ) {
     }
 

tests/Formatter/JsonTest.php

@@ -103,8 +103,6 @@ public function jsonSerialize(): mixed
 
     private function formatContent(mixed $content): string
     {
-        $formatter = new Json();
-
-        return $formatter->formatContent($content);
+        return (new Json())->formatContent($content);
     }
 }

tests/PersonDto.php

@@ -5,7 +5,7 @@
 
 final class PersonDto
 {
-    public function __construct(public int $id, public string $name)
+    public function __construct(public readonly int $id, public readonly string $name)
     {
     }
 }