Fix coding standard

Signed-off-by: Andi Rückauer <[email protected]>

src/AutoComposer.php

@@ -1,8 +1,17 @@
 <?php
 
+declare(strict_types=1);
 
 namespace Laminas\DevelopmentMode;
 
+use function getenv;
+use function is_resource;
+use function printf;
+use function var_export;
+
+use const PHP_EOL;
+use const STDERR;
+
 /**
  * Automatically switch to and from development mode based on type of composer
  * install/update used.
@@ -17,26 +26,21 @@
  */
 class AutoComposer
 {
-    const COMPOSER_DEV_MODE = 'COMPOSER_DEV_MODE';
+    public const COMPOSER_DEV_MODE = 'COMPOSER_DEV_MODE';
 
-    /**
-     * @var value of COMPOSER_DEV_MODE
-     */
+    /** @var string Value of COMPOSER_DEV_MODE */
     private $composerDevMode;
 
-    /**
-     * @var resource
-     */
+    /** @var resource */
     private $errorStream;
 
+    /** @var string[] */
     private $expectedValues = [
         '0', // production mode
         '1', // development mode
     ];
 
-    /**
-     * @param string Path to project.
-     */
+    /** @var string Path to project. */
     private $projectDir;
 
     /**
@@ -46,8 +50,8 @@ class AutoComposer
     public function __construct($projectDir = '', $errorStream = null)
     {
         $this->composerDevMode = getenv(self::COMPOSER_DEV_MODE);
-        $this->projectDir = $projectDir;
-        $this->errorStream = is_resource($errorStream) ? $errorStream : STDERR;
+        $this->projectDir      = $projectDir;
+        $this->errorStream     = is_resource($errorStream) ? $errorStream : STDERR;
     }
 
     /**

src/Command.php

@@ -1,8 +1,16 @@
 <?php
 
+declare(strict_types=1);
 
 namespace Laminas\DevelopmentMode;
 
+use function array_shift;
+use function count;
+use function fwrite;
+
+use const PHP_EOL;
+use const STDERR;
+
 class Command
 {
     /**

src/ConfigDiscoveryTrait.php

@@ -1,33 +1,33 @@
 <?php
 
+declare(strict_types=1);
 
 namespace Laminas\DevelopmentMode;
 
 use RuntimeException;
 
+use function file_exists;
+use function is_array;
+use function sprintf;
+use function unlink;
+
+use const PHP_EOL;
+
 /**
  * Shared functionality for the Disable/Enable commands.
  */
 trait ConfigDiscoveryTrait
 {
-    /**
-     * @var null|array
-     */
+    /** @var null|array */
     private $applicationConfig;
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $applicationConfigPath = 'config/application.config.php';
 
-    /**
-     * @var string
-     */
+    /** @var string */
     private $mezzioConfigPath = 'config/config.php';
 
-    /**
-     * @var string Base name for configuration cache.
-     */
+    /** @var string Base name for configuration cache. */
     private $configCacheBase = 'module-config-cache';
 
     /**
@@ -109,8 +109,8 @@ private function getConfigCacheKey()
      * Raises an exception if retrieved configuration is not an array.
      *
      * @return array
-     * @throws RuntimeException if config/application.config.php does not
-     *     return an array
+     * @throws RuntimeException If config/application.config.php does not
+     *     return an array.
      */
     private function getApplicationConfig()
     {

src/Disable.php

@@ -1,25 +1,31 @@
 <?php
 
+declare(strict_types=1);
 
 namespace Laminas\DevelopmentMode;
 
 use RuntimeException;
 
+use function file_exists;
+use function fwrite;
+use function is_resource;
+use function sprintf;
+use function unlink;
+
+use const PHP_EOL;
+use const STDERR;
+
 class Disable
 {
     use ConfigDiscoveryTrait;
 
-    const DEVEL_CONFIG = 'config/development.config.php';
-    const DEVEL_LOCAL  = 'config/autoload/development.local.php';
+    public const DEVEL_CONFIG = 'config/development.config.php';
+    public const DEVEL_LOCAL  = 'config/autoload/development.local.php';
 
-    /**
-     * @var resource
-     */
+    /** @var resource */
     private $errorStream;
 
-    /**
-     * @param string Path to project.
-     */
+    /** @var string Path to project. */
     private $projectDir;
 
     /**
@@ -28,7 +34,7 @@ class Disable
      */
     public function __construct($projectDir = '', $errorStream = null)
     {
-        $this->projectDir = $projectDir;
+        $this->projectDir  = $projectDir;
         $this->errorStream = is_resource($errorStream) ? $errorStream : STDERR;
     }
 

src/Enable.php

@@ -1,27 +1,37 @@
 <?php
 
+declare(strict_types=1);
 
 namespace Laminas\DevelopmentMode;
 
 use RuntimeException;
 
+use function basename;
+use function copy;
+use function file_exists;
+use function fwrite;
+use function in_array;
+use function is_resource;
+use function sprintf;
+use function symlink;
+
+use const PHP_EOL;
+use const PHP_OS;
+use const STDERR;
+
 class Enable
 {
     use ConfigDiscoveryTrait;
 
-    const DEVEL_CONFIG      = 'config/development.config.php';
-    const DEVEL_CONFIG_DIST = 'config/development.config.php.dist';
-    const DEVEL_LOCAL       = 'config/autoload/development.local.php';
-    const DEVEL_LOCAL_DIST  = 'config/autoload/development.local.php.dist';
+    public const DEVEL_CONFIG      = 'config/development.config.php';
+    public const DEVEL_CONFIG_DIST = 'config/development.config.php.dist';
+    public const DEVEL_LOCAL       = 'config/autoload/development.local.php';
+    public const DEVEL_LOCAL_DIST  = 'config/autoload/development.local.php.dist';
 
-    /**
-     * @var resource
-     */
+    /** @var resource */
     private $errorStream;
 
-    /**
-     * @param string Path to project.
-     */
+    /** @var string Path to project. */
     private $projectDir;
 
     /**
@@ -30,7 +40,7 @@ class Enable
      */
     public function __construct($projectDir = '', $errorStream = null)
     {
-        $this->projectDir = $projectDir;
+        $this->projectDir  = $projectDir;
         $this->errorStream = is_resource($errorStream) ? $errorStream : STDERR;
     }
 

src/Help.php

@@ -1,14 +1,16 @@
 <?php
 
+declare(strict_types=1);
 
 namespace Laminas\DevelopmentMode;
 
+use function fwrite;
+use function is_resource;
+
 class Help
 {
-    /**
-     * @var string
-     */
-    private $message = <<< EOH
+    /** @var string */
+    private $message = <<<EOH
 Enable/Disable development mode.
 
 Usage:

src/Status.php

@@ -1,15 +1,19 @@
 <?php
 
+declare(strict_types=1);
 
 namespace Laminas\DevelopmentMode;
 
+use function file_exists;
+use function sprintf;
+
+use const PHP_EOL;
+
 class Status
 {
-    const DEVEL_CONFIG = 'config/development.config.php';
+    public const DEVEL_CONFIG = 'config/development.config.php';
 
-    /**
-     * @param string
-     */
+    /** @var string */
     private $develConfigFile;
 
     /**

test/AutoComposerTest.php

@@ -1,5 +1,6 @@
 <?php
 
+declare(strict_types=1);
 
 namespace LaminasTest\DevelopmentMode;
 
@@ -8,6 +9,13 @@
 use org\bovigo\vfs\vfsStreamContainer;
 use PHPUnit\Framework\TestCase;
 
+use function fclose;
+use function fopen;
+use function is_resource;
+use function putenv;
+
+use const PHP_EOL;
+
 class AutoComposerTest extends TestCase
 {
     use RemoveCacheFileTrait;
@@ -20,12 +28,12 @@ class AutoComposerTest extends TestCase
 
     public function setUp(): void
     {
-        $this->projectDir = vfsStream::setup('project', null, [
+        $this->projectDir  = vfsStream::setup('project', null, [
             'config' => [
                 'autoload' => [],
             ],
-            'cache' => [],
-            'data' => [],
+            'cache'  => [],
+            'data'   => [],
         ]);
         $this->errorStream = fopen('php://memory', 'w+');
     }

test/DisableTest.php

@@ -1,5 +1,6 @@
 <?php
 
+declare(strict_types=1);
 
 namespace LaminasTest\DevelopmentMode;
 
@@ -8,6 +9,16 @@
 use org\bovigo\vfs\vfsStreamContainer;
 use PHPUnit\Framework\TestCase;
 
+use function fclose;
+use function file_exists;
+use function file_put_contents;
+use function fopen;
+use function fread;
+use function fseek;
+use function is_resource;
+
+use const PHP_EOL;
+
 class DisableTest extends TestCase
 {
     use RemoveCacheFileTrait;
@@ -26,15 +37,15 @@ class DisableTest extends TestCase
 
     protected function setUp(): void
     {
-        $this->projectDir = vfsStream::setup('project', null, [
+        $this->projectDir  = vfsStream::setup('project', null, [
             'config' => [
                 'autoload' => [],
             ],
-            'cache' => [],
+            'cache'  => [],
         ]);
         $this->errorStream = fopen('php://memory', 'w+');
-        $this->configStub = '<' . "?php\nreturn [];";
-        $this->command = new Disable(vfsStream::url('project'), $this->errorStream);
+        $this->configStub  = '<' . "?php\nreturn [];";
+        $this->command     = new Disable(vfsStream::url('project'), $this->errorStream);
     }
 
     protected function tearDown(): void
@@ -44,6 +55,9 @@ protected function tearDown(): void
         }
     }
 
+    /**
+     * @return bool|string
+     */
     public function readErrorStream()
     {
         fseek($this->errorStream, 0);