Modernise code for PHP 7.4

This uses rector and phpcbf to automatically fix issues

Author: alcaeus

examples/bulk.php

@@ -68,7 +68,7 @@ function toJSON(object $document): string
                 ['x' => 10], // Document
             ],
         ],
-    ]
+    ],
 );
 
 $cursor = $collection->find([]);

examples/command_logger.php

@@ -69,7 +69,7 @@ public function commandFailed(CommandFailedEvent $event): void
 
 $collection->updateMany(
     ['x' => ['$gt' => 1]],
-    ['$set' => ['y' => 1]]
+    ['$set' => ['y' => 1]],
 );
 
 $cursor = $collection->find([], ['batchSize' => 2]);

examples/persistable.php

@@ -16,14 +16,12 @@
 
 class PersistableEntry implements Persistable
 {
-    /** @var ObjectId */
-    private $id;
+    private ObjectId $id;
 
-    /** @var string */
-    public $name;
+    public string $name;
 
     /** @var array<PersistableEmail> */
-    public $emails = [];
+    public array $emails = [];
 
     public function __construct(string $name)
     {
@@ -65,11 +63,9 @@ public function bsonUnserialize(array $data): void
 
 class PersistableEmail implements Persistable
 {
-    /** @var string */
-    public $type;
+    public string $type;
 
-    /** @var string */
-    public $address;
+    public string $address;
 
     public function __construct(string $type, string $address)
     {

examples/typemap.php

@@ -16,14 +16,12 @@
 
 class TypeMapEntry implements Unserializable
 {
-    /** @var ObjectId */
-    private $id;
+    private ObjectId $id;
 
-    /** @var string */
-    private $name;
+    private string $name;
 
     /** @var array<TypeMapEmail> */
-    private $emails;
+    private array $emails;
 
     private function __construct()
     {
@@ -64,11 +62,9 @@ public function bsonUnserialize(array $data): void
 
 class TypeMapEmail implements Unserializable
 {
-    /** @var string */
-    private $type;
+    private string $type;
 
-    /** @var string */
-    private $address;
+    private string $address;
 
     private function __construct()
     {

examples/with_transaction.php

@@ -37,13 +37,13 @@ function toJSON(object $document): string
             ['x' => 2],
             ['x' => 3],
         ],
-        ['session' => $session]
+        ['session' => $session],
     );
 
     $collection->updateMany(
         ['x' => ['$gt' => 1]],
         ['$set' => ['y' => 1]],
-        ['session' => $session]
+        ['session' => $session],
     );
 };
 

phpcs.xml.dist

@@ -17,7 +17,7 @@
     <file>rector.php</file>
 
     <!-- Target minimum supported PHP version -->
-    <config name="php_version" value="70200"/>
+    <config name="php_version" value="70400"/>
 
     <!-- ****************************************** -->
     <!-- Import rules from doctrine/coding-standard -->
@@ -112,8 +112,6 @@
 
     <rule ref="SlevomatCodingStandard.TypeHints.PropertyTypeHint">
         <properties>
-            <!-- Requires PHP 7.4 -->
-            <property name="enableNativeTypeHint" value="false" />
             <!-- Requires PHP 8.0 -->
             <property name="enableMixedTypeHint" value="false" />
             <!-- Requires PHP 8.0 -->

rector.php

@@ -15,7 +15,7 @@
     ]);
 
     // Modernize code
-    $rectorConfig->sets([LevelSetList::UP_TO_PHP_72]);
+    $rectorConfig->sets([LevelSetList::UP_TO_PHP_74]);
 
     $rectorConfig->skip([
         // Falsely detect unassigned variables in code paths stopped by PHPUnit\Framework\Assert::markTestSkipped()

src/BulkWriteResult.php

@@ -25,14 +25,11 @@
  */
 class BulkWriteResult
 {
-    /** @var WriteResult */
-    private $writeResult;
+    private WriteResult $writeResult;
 
-    /** @var array */
-    private $insertedIds;
+    private array $insertedIds;
 
-    /** @var boolean */
-    private $isAcknowledged;
+    private bool $isAcknowledged;
 
     public function __construct(WriteResult $writeResult, array $insertedIds)
     {

src/ChangeStream.php

@@ -74,16 +74,13 @@ class ChangeStream implements Iterator
     /** @var ChangeStreamIterator */
     private $iterator;
 
-    /** @var integer */
-    private $key = 0;
+    private int $key = 0;
 
     /**
      * Whether the change stream has advanced to its first result. This is used
      * to determine whether $key should be incremented after an iteration event.
-     *
-     * @var boolean
      */
-    private $hasAdvanced = false;
+    private bool $hasAdvanced = false;
 
     /**
      * @internal

src/Client.php

@@ -54,26 +54,19 @@ class Client
 
     private const HANDSHAKE_SEPARATOR = '/';
 
-    /** @var string|null */
-    private static $version;
+    private static ?string $version = null;
 
-    /** @var Manager */
-    private $manager;
+    private Manager $manager;
 
-    /** @var ReadConcern */
-    private $readConcern;
+    private ReadConcern $readConcern;
 
-    /** @var ReadPreference */
-    private $readPreference;
+    private ReadPreference $readPreference;
 
-    /** @var string */
-    private $uri;
+    private string $uri;
 
-    /** @var array */
-    private $typeMap;
+    private array $typeMap;
 
-    /** @var WriteConcern */
-    private $writeConcern;
+    private WriteConcern $writeConcern;
 
     /**
      * Constructs a new Client instance.