deployphp / deployer

src/Command/BlackjackCommand.php

Spacing +2 added lines, -2 removed lines

@@ -80,7 +80,7 @@ 
         $dealersHand = [];
         $playersHand = [];
         shuffle($deck);
-        $deal = function () use (&$deck, &$graveyard) {
+        $deal = function() use (&$deck, &$graveyard) {
             if (count($deck) == 0) {
                 shuffle($graveyard);
                 $deck = $graveyard;
@@ -287,7 +287,7 @@ 
         }
         $variants = [$value];
         while ($aces-- > 0) {
-            $variants = array_flatten(array_map(function ($v) {
+            $variants = array_flatten(array_map(function($v) {
                 return [$v + 1, $v + 11];
             }, $variants));
         }

src/Selector/Selector.php

Spacing +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
         $labels = $host->get('labels', []);
         $labels['alias'] = $host->getAlias();
         $labels['true'] = 'true';
-        $isTrue = function ($value) {
+        $isTrue = function($value) {
             return $value;
         };
 
@@ -74,7 +74,7 @@ 
      */
     private static function compare(string $op, $a, ?string $b): bool
     {
-        $matchFunction = function ($a, ?string $b) {
+        $matchFunction = function($a, ?string $b) {
             foreach ((array) $a as $item) {
                 if ($item === $b) {
                     return true;

src/Documentation/DocGen.php

Spacing +7 added lines, -7 removed lines

@@ -51,7 +51,7 @@ 
     {
         foreach ($this->recipes as $recipe) {
             // $find will try to return DocConfig for a given config $name.
-            $findConfig = function (string $name) use ($recipe): ?DocConfig {
+            $findConfig = function(string $name) use ($recipe): ?DocConfig {
                 if (array_key_exists($name, $recipe->config)) {
                     return $recipe->config[$name];
                 }
@@ -69,7 +69,7 @@ 
                 }
                 return null;
             };
-            $findConfigOverride = function (DocRecipe $recipe, string $name) use (&$findConfigOverride): ?DocConfig {
+            $findConfigOverride = function(DocRecipe $recipe, string $name) use (&$findConfigOverride): ?DocConfig {
                 foreach ($recipe->require as $r) {
                     if (array_key_exists($r, $this->recipes)) {
                         if (array_key_exists($name, $this->recipes[$r]->config)) {
@@ -85,7 +85,7 @@ 
                 return null;
             };
             // Replace all {{name}} with link to correct config declaration.
-            $replaceLinks = function (string $comment) use ($findConfig): string {
+            $replaceLinks = function(string $comment) use ($findConfig): string {
                 $output = '';
                 $code = false;
                 foreach (explode("\n", $comment) as $i => $line) {
@@ -97,7 +97,7 @@ 
                         $output .= "\n";
                         continue;
                     }
-                    $output .= preg_replace_callback('#(\{\{(?<name>[\w_:\-/]+)\}\})#', function ($m) use ($findConfig) {
+                    $output .= preg_replace_callback('#(\{\{(?<name>[\w_:\-/]+)\}\})#', function($m) use ($findConfig) {
                         $name = $m['name'];
                         $config = $findConfig($name);
                         if ($config !== null) {
@@ -111,7 +111,7 @@ 
                 }
                 return $output;
             };
-            $findTask = function (string $name, bool $searchOtherRecipes = true) use ($recipe): ?DocTask {
+            $findTask = function(string $name, bool $searchOtherRecipes = true) use ($recipe): ?DocTask {
                 if (array_key_exists($name, $recipe->tasks)) {
                     return $recipe->tasks[$name];
                 }
@@ -170,7 +170,7 @@ 
 
                     MARKDOWN;
 
-                $map = function (DocTask $task, $ident = '') use (&$map, $findTask, &$intro): void {
+                $map = function(DocTask $task, $ident = '') use (&$map, $findTask, &$intro): void {
                     foreach ($task->group as $taskName) {
                         $t = $findTask($taskName);
                         if ($t !== null) {
@@ -356,7 +356,7 @@ 
 
 function indent(string $text): string
 {
-    return implode("\n", array_map(function ($line) {
+    return implode("\n", array_map(function($line) {
         return "  " . $line;
     }, explode("\n", $text)));
 }

recipe/silverstripe.php

Spacing +4 added lines, -4 removed lines

@@ -10,7 +10,7 @@ 
  * Silverstripe configuration
  */
 
-set('shared_assets', function () {
+set('shared_assets', function() {
     if (test('[ -d {{release_or_current_path}}/public ]') || test('[ -d {{deploy_path}}/shared/public ]')) {
         return 'public/assets';
     }
@@ -29,7 +29,7 @@ 
 ]);
 
 // Silverstripe cli script
-set('silverstripe_cli_script', function () {
+set('silverstripe_cli_script', function() {
     $paths = [
         'framework/cli-script.php',
         'vendor/silverstripe/framework/cli-script.php',
@@ -45,12 +45,12 @@ 
  * Helper tasks
  */
 desc('Runs /dev/build');
-task('silverstripe:build', function () {
+task('silverstripe:build', function() {
     run('{{bin/php}} {{release_or_current_path}}/{{silverstripe_cli_script}} /dev/build');
 });
 
 desc('Runs /dev/build?flush=all');
-task('silverstripe:buildflush', function () {
+task('silverstripe:buildflush', function() {
     run('{{bin/php}} {{release_or_current_path}}/{{silverstripe_cli_script}} /dev/build flush=all');
 });
 

recipe/deploy/setup.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 namespace Deployer;
 
 desc('Prepares host for deploy');
-task('deploy:setup', function () {
+task('deploy:setup', function() {
     run(
         <<<EOF
             [ -d {{deploy_path}} ] || mkdir -p {{deploy_path}};

recipe/drupal7.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
 
 
 //Create and upload Drupal 7 settings.php using values from secrets
-task('drupal:settings', function () {
+task('drupal:settings', function() {
     if (askConfirmation('Are you sure to generate and upload settings.php file?')) {
 
         //Get template
@@ -73,7 +73,7 @@ 
 });
 
 //Upload Drupal 7 files folder
-task('drupal:upload_files', function () {
+task('drupal:upload_files', function() {
     if (askConfirmation('Are you sure?')) {
         upload('sites/{{drupal_site}}/files', '{{deploy_path}}/shared/sites/{{drupal_site}}/files');
     }

recipe/craftcms.php

Spacing +3 added lines, -3 removed lines

@@ -36,13 +36,13 @@
  */
 function craft($command, $options = [])
 {
-    return function () use ($command, $options) {
-        if (! test('[ -s {{release_path}}/.env ]')) {
+    return function() use ($command, $options) {
+        if (!test('[ -s {{release_path}}/.env ]')) {
             throw new \Exception('Your .env file is empty! Cannot proceed.');
         }
 
         // By default we don't want any command to be interactive
-        if (! in_array('interactive', $options)) {
+        if (!in_array('interactive', $options)) {
             $command .= ' --interactive=0';
         }
 

recipe/magento2.php

Spacing +30 added lines, -31 removed lines

@@ -64,7 +64,7 @@ 
 // Update using: `set('static_content_jobs', '1');`
 set('static_content_jobs', '1');
 
-set('content_version', function () {
+set('content_version', function() {
     return time();
 });
 
@@ -109,14 +109,14 @@ 
 
 set('bin/magento', '{{release_or_current_path}}/{{magento_dir}}/bin/magento');
 
-set('magento_version', function () {
+set('magento_version', function() {
     // detect version
     $versionOutput = run('{{bin/php}} {{bin/magento}} --version');
     preg_match('/(\d+\.?)+(-p\d+)?$/', $versionOutput, $matches);
     return $matches[0] ?? '2.0';
 });
 
-set('config_import_needed', function () {
+set('config_import_needed', function() {
     // detect if app:config:import is needed
     try {
         run('{{bin/php}} {{bin/magento}} app:config:status');
@@ -130,7 +130,7 @@ 
     return false;
 });
 
-set('database_upgrade_needed', function () {
+set('database_upgrade_needed', function() {
     // detect if setup:upgrade is needed
     try {
         run('{{bin/php}} {{bin/magento}} setup:db:status');
@@ -165,7 +165,7 @@ 
 // 'MAGE_MODE' => 'production'
 // ```
 desc('Compiles magento di');
-task('magento:compile', function () {
+task('magento:compile', function() {
     run("{{bin/php}} {{bin/magento}} setup:di:compile");
     run('cd {{release_or_current_path}}/{{magento_dir}} && {{bin/composer}} dump-autoload -o');
 });
@@ -191,7 +191,7 @@ 
 //     ]
 // ```
 desc('Deploys assets');
-task('magento:deploy:assets', function () {
+task('magento:deploy:assets', function() {
     $themesToCompile = '';
     if (get('split_static_deployment')) {
         invoke('magento:deploy:assets:adminhtml');
@@ -208,12 +208,12 @@ 
 });
 
 desc('Deploys assets for backend only');
-task('magento:deploy:assets:adminhtml', function () {
+task('magento:deploy:assets:adminhtml', function() {
     magentoDeployAssetsSplit('backend');
 });
 
 desc('Deploys assets for frontend only');
-task('magento:deploy:assets:frontend', function () {
+task('magento:deploy:assets:frontend', function() {
     magentoDeployAssetsSplit('frontend');
 });
 
@@ -261,9 +261,9 @@ 
 }
 
 desc('Syncs content version');
-task('magento:sync:content_version', function () {
+task('magento:sync:content_version', function() {
     $timestamp = time();
-    on(select('all'), function (Host $host) use ($timestamp) {
+    on(select('all'), function(Host $host) use ($timestamp) {
         $host->set('content_version', $timestamp);
     });
 })->once();
@@ -271,26 +271,25 @@ 
 before('magento:deploy:assets', 'magento:sync:content_version');
 
 desc('Enables maintenance mode');
-task('magento:maintenance:enable', function () {
+task('magento:maintenance:enable', function() {
     // do not use {{bin/magento}} because it would be in "release" but the maintenance mode must be set in "current"
     run("if [ -d $(echo {{current_path}}) ]; then {{bin/php}} {{current_path}}/{{magento_dir}}/bin/magento maintenance:enable; fi");
 });
 
 desc('Disables maintenance mode');
-task('magento:maintenance:disable', function () {
+task('magento:maintenance:disable', function() {
     // do not use {{bin/magento}} because it would be in "release" but the maintenance mode must be set in "current"
     run("if [ -d $(echo {{current_path}}) ]; then {{bin/php}} {{current_path}}/{{magento_dir}}/bin/magento maintenance:disable; fi");
 });
 
 desc('Set maintenance mode if needed');
-task('magento:maintenance:enable-if-needed', function () {
-    ! get('enable_zerodowntime') || get('database_upgrade_needed') || get('config_import_needed') ?
-        invoke('magento:maintenance:enable') :
-        writeln('Config and database up to date => no maintenance mode');
+task('magento:maintenance:enable-if-needed', function() {
+    !get('enable_zerodowntime') || get('database_upgrade_needed') || get('config_import_needed') ?
+        invoke('magento:maintenance:enable') : writeln('Config and database up to date => no maintenance mode');
 });
 
 desc('Config Import');
-task('magento:config:import', function () {
+task('magento:config:import', function() {
     if (get('config_import_needed')) {
         run('{{bin/php}} {{bin/magento}} app:config:import --no-interaction');
     } else {
@@ -299,7 +298,7 @@ 
 });
 
 desc('Upgrades magento database');
-task('magento:upgrade:db', function () {
+task('magento:upgrade:db', function() {
     if (get('database_upgrade_needed')) {
         run("{{bin/php}} {{bin/magento}} setup:db-schema:upgrade --no-interaction");
         run("{{bin/php}} {{bin/magento}} setup:db-data:upgrade --no-interaction");
@@ -309,7 +308,7 @@ 
 })->once();
 
 desc('Flushes Magento Cache');
-task('magento:cache:flush', function () {
+task('magento:cache:flush', function() {
     run("{{bin/php}} {{bin/magento}} cache:flush");
 });
 
@@ -360,7 +359,7 @@ 
 set('repository', null);
 
 // The relative path to the artifact file. If the directory does not exist, it will be created
-set('artifact_path', function () {
+set('artifact_path', function() {
     if (!testLocally('[ -d {{artifact_dir}} ]')) {
         runLocally('mkdir -p {{artifact_dir}}');
     }
@@ -368,7 +367,7 @@ 
 });
 
 // The location of the tar command. On MacOS you should have installed gtar, as it supports the required settings
-set('bin/tar', function () {
+set('bin/tar', function() {
     if (commandExist('gtar')) {
         return which('gtar');
     } else {
@@ -379,7 +378,7 @@ 
 // tasks section
 
 desc('Packages all relevant files in an artifact.');
-task('artifact:package', function () {
+task('artifact:package', function() {
     if (!test('[ -f {{artifact_excludes_file}} ]')) {
         throw new GracefulShutdownException(
             "No artifact excludes file provided, provide one at artifacts/excludes or change location",
@@ -389,23 +388,23 @@ 
 });
 
 desc('Uploads artifact in release folder for extraction.');
-task('artifact:upload', function () {
+task('artifact:upload', function() {
     upload(get('artifact_path'), '{{release_path}}');
 });
 
 desc('Extracts artifact in release path.');
-task('artifact:extract', function () {
+task('artifact:extract', function() {
     run('{{bin/tar}} -xzpf {{release_path}}/{{artifact_file}} -C {{release_path}}');
     run('rm -rf {{release_path}}/{{artifact_file}}');
 });
 
 desc('Clears generated files prior to building.');
-task('build:remove-generated', function () {
+task('build:remove-generated', function() {
     run('rm -rf generated/*');
 });
 
 desc('Prepare local artifact build');
-task('build:prepare', function () {
+task('build:prepare', function() {
     if (!currentHost()->get('local')) {
         throw new GracefulShutdownException('Artifact can only be built locally, you provided a non local host');
     }
@@ -446,7 +445,7 @@ 
 
 
 desc('Adds additional files and dirs to the list of shared files and dirs');
-task('deploy:additional-shared', function () {
+task('deploy:additional-shared', function() {
     add('shared_files', get('additional_shared_files'));
     add('shared_dirs', get('additional_shared_dirs'));
 });
@@ -461,7 +460,7 @@ 
  * ```
  **/
 desc('Update cache id_prefix');
-task('magento:set_cache_prefix', function () {
+task('magento:set_cache_prefix', function() {
     //download current env config
     $tmpConfigFile = tempnam(sys_get_temp_dir(), 'deployer_config');
     download('{{deploy_path}}/shared/' . ENV_CONFIG_FILE_PATH, $tmpConfigFile);
@@ -501,7 +500,7 @@ 
  * After successful deployment, move the tmp_env.php file to env.php ready for next deployment
  */
 desc('Cleanup cache id_prefix env files');
-task('magento:cleanup_cache_prefix', function () {
+task('magento:cleanup_cache_prefix', function() {
     run('rm {{deploy_path}}/shared/' . ENV_CONFIG_FILE_PATH);
     run('rm {{release_or_current_path}}/' . ENV_CONFIG_FILE_PATH);
     run('mv {{deploy_path}}/shared/' . TMP_ENV_CONFIG_FILE_PATH . ' {{deploy_path}}/shared/' . ENV_CONFIG_FILE_PATH);
@@ -517,7 +516,7 @@ 
  *  ```
  */
 desc('Remove cron from crontab and kill running cron jobs');
-task('magento:cron:stop', function () {
+task('magento:cron:stop', function() {
     if (has('previous_release')) {
         run('{{bin/php}} {{previous_release}}/{{magento_dir}}/bin/magento cron:remove');
     }
@@ -533,7 +532,7 @@ 
  *   ```
  */
 desc('Install cron in crontab');
-task('magento:cron:install', function () {
+task('magento:cron:install', function() {
     run('cd {{release_or_current_path}}');
     run('{{bin/php}} {{bin/magento}} cron:install');
 });

tests/src/FunctionsTest.php

Spacing +12 added lines, -12 removed lines

@@ -70,7 +70,7 @@ 
 
     public function testTask()
     {
-        task('task', function () {});
+        task('task', function() {});
 
         $task = $this->deployer->tasks->get('task');
         self::assertInstanceOf(Task::class, $task);
@@ -85,10 +85,10 @@ 
 
     public function testBefore()
     {
-        task('main', function () {});
-        task('before', function () {});
+        task('main', function() {});
+        task('before', function() {});
         before('main', 'before');
-        before('before', function () {});
+        before('before', function() {});
 
         $names = $this->taskToNames($this->deployer->scriptManager->getTasks('main'));
         self::assertEquals(['before:before', 'before', 'main'], $names);
@@ -96,10 +96,10 @@ 
 
     public function testAfter()
     {
-        task('main', function () {});
-        task('after', function () {});
+        task('main', function() {});
+        task('after', function() {});
         after('main', 'after');
-        after('after', function () {});
+        after('after', function() {});
 
         $names = $this->taskToNames($this->deployer->scriptManager->getTasks('main'));
         self::assertEquals(['main', 'after', 'after:after'], $names);
@@ -130,7 +130,7 @@ 
     {
         Context::get()->getConfig()->set('working_path', '/foo');
 
-        within('/bar', function () {
+        within('/bar', function() {
             $withinWorkingPath = Context::get()->getConfig()->get('working_path');
             self::assertEquals('/bar', $withinWorkingPath);
         });
@@ -144,7 +144,7 @@ 
         Context::get()->getConfig()->set('working_path', '/foo');
 
         try {
-            within('/bar', function () {
+            within('/bar', function() {
                 throw new \Exception('Dummy exception');
             });
         } catch (\Exception $exception) {
@@ -157,7 +157,7 @@ 
 
     public function testWithinReturningValue()
     {
-        $output = within('/foo', function () {
+        $output = within('/foo', function() {
             return 'bar';
         });
 
@@ -166,7 +166,7 @@ 
 
     public function testWithinWithVoidFunction()
     {
-        $output = within('/foo', function () {
+        $output = within('/foo', function() {
             // noop
         });
 
@@ -175,7 +175,7 @@ 
 
     private function taskToNames($tasks)
     {
-        return array_map(function (Task $task) {
+        return array_map(function(Task $task) {
             return $task->getName();
         }, $tasks);
     }