Improve php code style

fxpio · Mar 28, 2023 · b65a86e · b65a86e
1 parent f3901d6
commit b65a86e
Show file tree

Hide file tree

Showing 7 changed files with 32 additions and 32 deletions.
diff --git a/Asset/AssetManagerFinder.php b/Asset/AssetManagerFinder.php
@@ -49,10 +49,10 @@ public function addManager(AssetManagerInterface $manager)
      *
      * @param null|string $manager The name of the asset manager
      *
+     * @return AssetManagerInterface
+     *
      * @throws RuntimeException When the asset manager does not exist
      * @throws RuntimeException When the asset manager is not found
-     *
-     * @return AssetManagerInterface
      */
     public function findManager($manager = null)
     {
@@ -70,9 +70,9 @@ public function findManager($manager = null)
     /**
      * Find the available asset manager.
      *
-     * @throws RuntimeException When no asset manager is found
-     *
      * @return AssetManagerInterface
+     *
+     * @throws RuntimeException When no asset manager is found
      */
     private function findAvailableManager()
     {

diff --git a/Asset/PnpmManager.php b/Asset/PnpmManager.php
@@ -65,4 +65,4 @@ protected function getUpdateCommand()
     {
         return $this->buildCommand('pnpm', 'update', 'update');
     }
-}
+}
diff --git a/Converter/SemverUtil.php b/Converter/SemverUtil.php
@@ -80,7 +80,7 @@ public static function createPattern($pattern)
         $numVer2 = '('.$numVer.'\.'.$numVer.')';
         $numVer3 = '('.$numVer.'\.'.$numVer.'\.'.$numVer.')';
 
-        return '/^'.'('.$numVer.'|'.$numVer2.'|'.$numVer3.')'.$pattern.'/';
+        return '/^('.$numVer.'|'.$numVer2.'|'.$numVer3.')'.$pattern.'/';
     }
 
     /**

diff --git a/Foxy.php b/Foxy.php
@@ -215,9 +215,9 @@ public function solveAssets(Event $event)
      * @param ProcessExecutor $executor The process executor
      * @param Filesystem      $fs       The composer filesystem
      *
-     * @throws RuntimeException When the asset manager is not found
-     *
      * @return AssetManagerInterface
+     *
+     * @throws RuntimeException When the asset manager is not found
      */
     protected function getAssetManager(IOInterface $io, Config $config, ProcessExecutor $executor, Filesystem $fs)
     {

diff --git a/Tests/Config/ConfigTest.php b/Tests/Config/ConfigTest.php
@@ -67,23 +67,23 @@ protected function setUp(): void
     public function getDataForGetConfig()
     {
         return array(
-            array('foo',                 42,                           42),
-            array('bar',                 'foo',                        'empty'),
-            array('baz',                 false,                        true),
-            array('test',                0,                            0),
-            array('manager-bar',         23,                           0),
-            array('manager-baz',         0,                            0),
-            array('global-composer-foo', 90,                           0),
-            array('global-composer-bar', 70,                           0),
-            array('global-config-foo',   23,                           0),
-            array('env-boolean',         false,                        true,    'FOXY__ENV_BOOLEAN=false'),
-            array('env-integer',         -32,                          0,       'FOXY__ENV_INTEGER=-32'),
-            array('env-json',            array('foo' => 'bar'),        array(), 'FOXY__ENV_JSON="{"foo": "bar"}"'),
-            array('env-json-array',      array(array('foo' => 'bar')), array(), 'FOXY__ENV_JSON_ARRAY="[{"foo": "bar"}]"'),
-            array('env-string',          'baz',                        'foo',   'FOXY__ENV_STRING=baz'),
-            array('test-p1',             'def',                        'def',   null, array()),
-            array('test-p1',             'def',                        'def',   null, array('test-p1' => 'ok')),
-            array('test-p1',             'ok',                         null,    null, array('test-p1' => 'ok')),
+            array('foo', 42, 42),
+            array('bar', 'foo', 'empty'),
+            array('baz', false, true),
+            array('test', 0, 0),
+            array('manager-bar', 23, 0),
+            array('manager-baz', 0, 0),
+            array('global-composer-foo', 90, 0),
+            array('global-composer-bar', 70, 0),
+            array('global-config-foo', 23, 0),
+            array('env-boolean', false, true, 'FOXY__ENV_BOOLEAN=false'),
+            array('env-integer', -32, 0, 'FOXY__ENV_INTEGER=-32'),
+            array('env-json', array('foo' => 'bar'), array(), 'FOXY__ENV_JSON="{"foo": "bar"}"'),
+            array('env-json-array', array(array('foo' => 'bar')), array(), 'FOXY__ENV_JSON_ARRAY="[{"foo": "bar"}]"'),
+            array('env-string', 'baz', 'foo', 'FOXY__ENV_STRING=baz'),
+            array('test-p1', 'def', 'def', null, array()),
+            array('test-p1', 'def', 'def', null, array('test-p1' => 'ok')),
+            array('test-p1', 'ok', null, null, array('test-p1' => 'ok')),
         );
     }
 
@@ -182,7 +182,7 @@ public function testGetConfig($key, $expected, $default = null, $env = null, arr
     public function getDataForGetArrayConfig()
     {
         return array(
-            array('foo', array(),   array()),
+            array('foo', array(), array()),
             array('foo', array(42), array(42)),
             array('foo', array(42), array(), array('foo' => array(42))),
         );

diff --git a/Tests/Util/AssetUtilTest.php b/Tests/Util/AssetUtilTest.php
@@ -173,9 +173,9 @@ public function getExtraData()
     {
         return array(
             array(false, false),
-            array(true,  false),
+            array(true, false),
             array(false, true),
-            array(true,  true),
+            array(true, true),
         );
     }
 

diff --git a/Tests/Util/ConsoleUtilTest.php b/Tests/Util/ConsoleUtilTest.php
@@ -50,10 +50,10 @@ public function testGetInputWithoutValidInput()
     public function getPreferredInstallOptionsData()
     {
         return array(
-            array(false, false, 'auto',   false),
-            array(false, true,  'auto',   true),
-            array(true,  false, 'source', false),
-            array(false, true,  'dist',   false),
+            array(false, false, 'auto', false),
+            array(false, true, 'auto', true),
+            array(true, false, 'source', false),
+            array(false, true, 'dist', false),
         );
     }