Merge pull request #23 from samsonasik/apply-php80

Apply PHP 8.0 Syntax and constructor promotion

Author: Ocramius

src/AssetInstaller.php

@@ -36,19 +36,11 @@ class AssetInstaller
 {
     use UnparseableTokensTrait;
 
-    /** @var Composer */
-    private $composer;
-
-    /** @var IOInterface */
-    private $io;
-
     /** @var string Base path for project; default is current working dir. */
     private $projectPath;
 
-    public function __construct(Composer $composer, IOInterface $io)
+    public function __construct(private Composer $composer, private IOInterface $io)
     {
-        $this->composer    = $composer;
-        $this->io          = $io;
         $this->projectPath = getcwd();
     }
 

src/AssetUninstaller.php

@@ -32,22 +32,14 @@ class AssetUninstaller
 {
     use UnparseableTokensTrait;
 
-    /** @var Composer */
-    private $composer;
-
     /** @var string[] .gitignore rules */
-    private $gitignore = [];
-
-    /** @var IOInterface */
-    private $io;
+    private array $gitignore = [];
 
     /** @var string Base path for project; default is current working dir. */
     private $projectPath;
 
-    public function __construct(Composer $composer, IOInterface $io)
+    public function __construct(private Composer $composer, private IOInterface $io)
     {
-        $this->composer    = $composer;
-        $this->io          = $io;
         $this->projectPath = getcwd();
     }
 

src/Plugin.php

@@ -18,18 +18,16 @@
 
 class Plugin implements PluginInterface, EventSubscriberInterface
 {
-    /** @var Composer */
-    private $composer;
+    private Composer $composer;
 
     /**
      * Array of installers to run following a dump-autoload operation.
      *
      * @var callable[]
      */
-    private $installers = [];
+    private array $installers = [];
 
-    /** @var IOInterface */
-    private $io;
+    private IOInterface $io;
 
     /**
      * Provide composer event listeners.

test/AssetInstallerTest.php

@@ -51,8 +51,7 @@ class AssetInstallerTest extends TestCase
     /** @var IOInterface|ObjectProphecy */
     private $io;
 
-    /** @var vfsStreamDirectory */
-    private $filesystem;
+    private vfsStreamDirectory $filesystem;
 
     public function setUp(): void
     {

test/AssetUninstallerTest.php

@@ -83,8 +83,7 @@ class AssetUninstallerTest extends TestCase
         ],
     ];
 
-    /** @var vfsStreamDirectory */
-    private $filesystem;
+    private vfsStreamDirectory $filesystem;
 
     /** @var PackageInterface|ObjectProphecy */
     private $package;

test/PluginTest.php

@@ -24,8 +24,7 @@ class PluginTest extends TestCase
     use DeprecatedAssertionsTrait;
     use ProphecyTrait;
 
-    /** @var vfsStreamDirectory */
-    private $filesystem;
+    private vfsStreamDirectory $filesystem;
 
     /** @var Composer|ObjectProphecy */
     private $composer;
@@ -102,10 +101,10 @@ public function testOnPostAutoloadDumpTriggersInstallers(): void
         /** @psalm-var object{operations: array<array-key,string>} $obj */
         $spy = (object) ['operations' => []];
 
-        $installer1 = function () use ($spy): void {
+        $installer1 = static function () use ($spy): void {
             $spy->operations[] = 'installer1';
         };
-        $installer2 = function () use ($spy): void {
+        $installer2 = static function () use ($spy): void {
             $spy->operations[] = 'installer2';
         };