deployphp / deployer

src/Deployer.php

Spacing +23 added lines, -23 removed lines

@@ -95,19 +95,19 @@ 
             new InputOption('file', 'f', InputOption::VALUE_REQUIRED, 'Recipe file path'),
         );
 
-        $this['console'] = function () use ($console) {
+        $this['console'] = function() use ($console) {
             return $console;
         };
-        $this['input'] = function () {
+        $this['input'] = function() {
             throw new \RuntimeException('Uninitialized "input" in Deployer container.');
         };
-        $this['output'] = function () {
+        $this['output'] = function() {
             throw new \RuntimeException('Uninitialized "output" in Deployer container.');
         };
-        $this['inputDefinition'] = function () {
+        $this['inputDefinition'] = function() {
             return new InputDefinition();
         };
-        $this['questionHelper'] = function () {
+        $this['questionHelper'] = function() {
             return $this->getHelper('question');
         };
 
@@ -115,12 +115,12 @@ 
          *           Config           *
          ******************************/
 
-        $this['config'] = function () {
+        $this['config'] = function() {
             return new Configuration();
         };
         // -l  act as if it had been invoked as a login shell (i.e. source ~/.profile file)
         // -s  commands are read from the standard input (no arguments should remain after this option)
-        $this->config['shell'] = function () {
+        $this->config['shell'] = function() {
             if (currentHost() instanceof Localhost) {
                 return 'bash -s'; // Non-login shell for localhost.
             }
@@ -133,43 +133,43 @@ 
          *            Core            *
          ******************************/
 
-        $this['pop'] = function ($c) {
+        $this['pop'] = function($c) {
             return new Printer($c['output']);
         };
-        $this['sshClient'] = function ($c) {
+        $this['sshClient'] = function($c) {
             return new Client($c['output'], $c['pop'], $c['logger']);
         };
-        $this['rsync'] = function ($c) {
+        $this['rsync'] = function($c) {
             return new Rsync($c['pop'], $c['output']);
         };
-        $this['processRunner'] = function ($c) {
+        $this['processRunner'] = function($c) {
             return new ProcessRunner($c['pop'], $c['logger']);
         };
-        $this['tasks'] = function () {
+        $this['tasks'] = function() {
             return new TaskCollection();
         };
-        $this['hosts'] = function () {
+        $this['hosts'] = function() {
             return new HostCollection();
         };
-        $this['scriptManager'] = function ($c) {
+        $this['scriptManager'] = function($c) {
             return new ScriptManager($c['tasks']);
         };
-        $this['selector'] = function ($c) {
+        $this['selector'] = function($c) {
             return new Selector($c['hosts']);
         };
-        $this['fail'] = function () {
+        $this['fail'] = function() {
             return new Collection();
         };
-        $this['messenger'] = function ($c) {
+        $this['messenger'] = function($c) {
             return new Messenger($c['input'], $c['output'], $c['logger']);
         };
-        $this['server'] = function ($c) {
+        $this['server'] = function($c) {
             return new Server(
                 $c['output'],
                 $this,
             );
         };
-        $this['master'] = function ($c) {
+        $this['master'] = function($c) {
             return new Master(
                 $c['input'],
                 $c['output'],
@@ -177,7 +177,7 @@ 
                 $c['messenger'],
             );
         };
-        $this['importer'] = function () {
+        $this['importer'] = function() {
             return new Importer();
         };
 
@@ -185,12 +185,12 @@ 
          *           Logger           *
          ******************************/
 
-        $this['log_handler'] = function () {
+        $this['log_handler'] = function() {
             return !empty($this['log'])
                 ? new FileHandler($this['log'])
                 : new NullHandler();
         };
-        $this['logger'] = function () {
+        $this['logger'] = function() {
             return new Logger($this['log_handler']);
         };
 
@@ -335,7 +335,7 @@ 
         $output->writeln([
             "<fg=white;bg=red> {$class} </> <comment>in {$file} on line {$exception->getLine()}:</>",
             "",
-            implode("\n", array_map(function ($line) {
+            implode("\n", array_map(function($line) {
                 return "  " . $line;
             }, explode("\n", $exception->getMessage()))),
             "",

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/writable.php

Spacing +3 added lines, -3 removed lines

@@ -6,7 +6,7 @@ 
 // Used in `chown` and `acl` modes of {{writable_mode}}.
 // Attempts automatically to detect http user in process list.
 
-set('http_user', function () {
+set('http_user', function() {
     $candidates = explode("\n", run("ps axo comm,user | grep -E '[a]pache|[h]ttpd|[_]www|[w]ww-data|[n]ginx' | grep -v root | sort | awk '{print \$NF}' | uniq"));
     $httpUser = array_shift($candidates);
 
@@ -23,7 +23,7 @@ 
 // Used to make a writable directory by a server.
 // Used in `chgrp` mode of {{writable_mode}} only.
 // Attempts automatically to detect http user in process list.
-set('http_group', function () {
+set('http_group', function() {
     $candidates = explode("\n", run("ps axo comm,group | grep -E '[a]pache|[h]ttpd|[_]www|[w]ww-data|[n]ginx' | grep -v root | sort | awk '{print \$NF}' | uniq"));
     $httpGroup = array_shift($candidates);
 
@@ -59,7 +59,7 @@ 
 set('writable_chmod_mode', '0755');
 
 desc('Makes writable dirs');
-task('deploy:writable', function () {
+task('deploy:writable', function() {
     $dirs = join(' ', get('writable_dirs'));
     $mode = get('writable_mode');
     $recursive = get('writable_recursive') ? '-R' : '';

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');
 });

recipe/common.php

Spacing +10 added lines, -10 removed lines

@@ -27,7 +27,7 @@ 
 // Name of current user who is running deploy.
 // If not set will try automatically get git user name,
 // otherwise output of `whoami` command.
-set('user', function () {
+set('user', function() {
     if (getenv('CI') !== false) {
         return 'ci';
     }
@@ -87,7 +87,7 @@ 
  * set('deploy_path', '~/{{alias}}');
  * ```
  */
-set('deploy_path', function () {
+set('deploy_path', function() {
     throw new ConfigurationException('Please, specify `deploy_path`.');
 });
 
@@ -100,7 +100,7 @@ 
 set('current_path', '{{deploy_path}}/current');
 
 // Path to the `php` bin.
-set('bin/php', function () {
+set('bin/php', function() {
     if (currentHost()->hasOwn('php_version')) {
         return '/usr/bin/php{{php_version}}';
     }
@@ -108,25 +108,25 @@ 
 });
 
 // Path to the `git` bin.
-set('bin/git', function () {
+set('bin/git', function() {
     return which('git');
 });
 
 // Should {{bin/symlink}} use `--relative` option or not. Will detect
 // automatically.
-set('use_relative_symlink', function () {
+set('use_relative_symlink', function() {
     return commandSupportsOption('ln', '--relative');
 });
 
 // Path to the `ln` bin. With predefined options `-nfs`.
-set('bin/symlink', function () {
+set('bin/symlink', function() {
     return get('use_relative_symlink') ? 'ln -nfs --relative' : 'ln -nfs';
 });
 
 // Path to a file which will store temp script with sudo password.
 // Defaults to `.dep/sudo_pass`. This script is only temporary and will be deleted after
 // sudo command executed.
-set('sudo_askpass', function () {
+set('sudo_askpass', function() {
     if (test('[ -d {{deploy_path}}/.dep ]')) {
         return '{{deploy_path}}/.dep/sudo_pass';
     } else {
@@ -163,7 +163,7 @@ 
 /**
  * Prints success message
  */
-task('deploy:success', function () {
+task('deploy:success', function() {
     info('successfully deployed!');
 })
     ->hidden();
@@ -172,7 +172,7 @@ 
 /**
  * Hook on deploy failure.
  */
-task('deploy:failed', function () {})
+task('deploy:failed', function() {})
     ->hidden();
 
 fail('deploy', 'deploy:failed');
@@ -181,7 +181,7 @@ 
  * Follows latest application logs.
  */
 desc('Shows application logs');
-task('logs:app', function () {
+task('logs:app', function() {
     if (!has('log_files')) {
         warning("Please, specify \"log_files\" option.");
         return;

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);
     }