Migrate to PHP 7.4 syntax with Rector

Author: PowerKiKi

src/Acl/Acl.php

@@ -12,15 +12,13 @@ class Acl extends \Laminas\Permissions\Acl\Acl
 {
     /**
      * The message explaining the last denial.
-     *
-     * @var null|string
      */
-    private $message;
+    private ?string $message = null;
 
     /**
      * @var string[]
      */
-    private $reasons = [];
+    private array $reasons = [];
 
     protected function createModelResource(string $class): ModelResource
     {
@@ -88,7 +86,7 @@ private function buildMessage(ModelResource $resource, ?string $privilege, strin
 
         $user = CurrentUser::get();
         $userName = $user ? 'User "' . $user->getLogin() . '"' : 'Non-logged user';
-        $privilege = $privilege === null ? 'NULL' : $privilege;
+        $privilege ??= 'NULL';
 
         $message = "$userName with role $role is not allowed on resource \"$resource\" with privilege \"$privilege\"";
 

src/Acl/Assertion/All.php

@@ -14,7 +14,7 @@ final class All implements AssertionInterface
     /**
      * @var AssertionInterface[]
      */
-    private $asserts;
+    private array $asserts;
 
     /**
      * Check if all asserts are true.

src/Acl/Assertion/One.php

@@ -14,7 +14,7 @@ final class One implements AssertionInterface
     /**
      * @var AssertionInterface[]
      */
-    private $asserts;
+    private array $asserts;
 
     /**
      * Check if at least one assert is true.

src/Acl/ModelResource.php

@@ -22,10 +22,8 @@ final class ModelResource extends GenericResource
 {
     /**
      * Unique id of the instance of resource.
-     *
-     * @var null|Model
      */
-    private $instance;
+    private ?Model $instance;
 
     /**
      * Sets the Resource identifier.

src/Api/FilteredFieldResolver.php

@@ -14,10 +14,7 @@
  */
 final class FilteredFieldResolver
 {
-    /**
-     * @var DefaultFieldResolver
-     */
-    private $resolver;
+    private DefaultFieldResolver $resolver;
 
     public function __construct()
     {

src/Api/Input/PaginationInputType.php

@@ -26,25 +26,23 @@ public function __construct()
     {
         $config = [
             'description' => 'Describe what page we want',
-            'fields' => function (): array {
-                return [
-                    'offset' => [
-                        'type' => self::int(),
-                        'defaultValue' => null,
-                        'description' => 'The zero-based index of first item of the page. If given a value greater than zero, then `pageIndex` is ignored.',
-                    ],
-                    'pageIndex' => [
-                        'type' => self::int(),
-                        'defaultValue' => 0,
-                        'description' => 'The zero-based index of the page. If given negative value, then fallback to 0.',
-                    ],
-                    'pageSize' => [
-                        'type' => self::int(),
-                        'defaultValue' => 50,
-                        'description' => 'Number of items to display on a page. If given negative value, then fallback to 0.',
-                    ],
-                ];
-            },
+            'fields' => fn (): array => [
+                'offset' => [
+                    'type' => self::int(),
+                    'defaultValue' => null,
+                    'description' => 'The zero-based index of first item of the page. If given a value greater than zero, then `pageIndex` is ignored.',
+                ],
+                'pageIndex' => [
+                    'type' => self::int(),
+                    'defaultValue' => 0,
+                    'description' => 'The zero-based index of the page. If given negative value, then fallback to 0.',
+                ],
+                'pageSize' => [
+                    'type' => self::int(),
+                    'defaultValue' => 50,
+                    'description' => 'Number of items to display on a page. If given negative value, then fallback to 0.',
+                ],
+            ],
         ];
 
         parent::__construct($config);

src/Api/Server.php

@@ -20,15 +20,9 @@
  */
 class Server
 {
-    /**
-     * @var StandardServer
-     */
-    private $server;
+    private StandardServer $server;
 
-    /**
-     * @var ServerConfig
-     */
-    private $config;
+    private ServerConfig $config;
 
     public function __construct(Schema $schema, bool $debug, array $rootValue = [])
     {
@@ -61,7 +55,7 @@ public function execute(ServerRequestInterface $request)
     {
         if (!$request->getParsedBody()) {
             /** @var array $parsedBody */
-            $parsedBody = json_decode($request->getBody()->getContents(), true);
+            $parsedBody = json_decode($request->getBody()->getContents(), true, 512, JSON_THROW_ON_ERROR);
             $request = $request->withParsedBody($parsedBody);
         }
 

src/DBAL/Types/ChronosType.php

@@ -7,8 +7,9 @@
 use Cake\Chronos\Chronos;
 use DateTimeInterface;
 use Doctrine\DBAL\Platforms\AbstractPlatform;
+use Doctrine\DBAL\Types\DateTimeType;
 
-final class ChronosType extends \Doctrine\DBAL\Types\DateTimeType
+final class ChronosType extends DateTimeType
 {
     /**
      * @param null|DateTimeInterface|int|string $value

src/Handler/FileHandler.php

@@ -12,10 +12,7 @@
 
 final class FileHandler extends AbstractHandler
 {
-    /**
-     * @var ObjectRepository
-     */
-    private $fileRepository;
+    private ObjectRepository $fileRepository;
 
     public function __construct(ObjectRepository $fileRepository)
     {

src/Handler/GraphQLHandler.php

@@ -12,10 +12,7 @@
 
 final class GraphQLHandler implements RequestHandlerInterface
 {
-    /**
-     * @var Server
-     */
-    private $server;
+    private Server $server;
 
     public function __construct(Server $server)
     {