sebastianfeldmann / phpbu

src/Backup/Sync/Ftp.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
     public function sync(Target $target, Result $result)
     {
         // silence ftp errors
-        $old  = error_reporting(0);
+        $old = error_reporting(0);
         if (!$ftpConnection = ftp_connect($this->host)) {
             throw new Exception(
                 sprintf(

src/Result/PrinterCli.php

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use phpbu\App\Listener;
 use phpbu\App\Result;
 use phpbu\App\Util;
-use phpbu\App\Version;
 use PHP_Timer;
 use SebastianBergmann\Environment\Console;
 use SebastianBergmann\Environment\Runtime;

Spacing +36 added lines, -36 removed lines

@@ -172,8 +172,8 @@ 
         $configuration = $event->getConfiguration();
         if ($this->verbose) {
             $this->write(
-                'Runtime:       ' . $this->runtime->getNameWithVersion() . PHP_EOL .
-                'Configuration: ' . $configuration->getFilename() . PHP_EOL .
+                'Runtime:       '.$this->runtime->getNameWithVersion().PHP_EOL.
+                'Configuration: '.$configuration->getFilename().PHP_EOL.
                 PHP_EOL
             );
         }
@@ -189,7 +189,7 @@ 
         $this->numBackups++;
         if ($this->debug) {
             $backup = $event->getConfiguration();
-            $this->writeWithAsterisk('backup: [' . $backup->getSource()->type . '] ');
+            $this->writeWithAsterisk('backup: ['.$backup->getSource()->type.'] ');
         }
     }
 
@@ -201,7 +201,7 @@ 
     public function onBackupFailed(Event\Backup\Failed $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-white, bg-red, bold', 'failed' . PHP_EOL);
+            $this->writeWithColor('fg-white, bg-red, bold', 'failed'.PHP_EOL);
         }
     }
 
@@ -213,7 +213,7 @@ 
     public function onBackupEnd(Event\Backup\End $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-black, bg-green', 'ok' . PHP_EOL);
+            $this->writeWithColor('fg-black, bg-green', 'ok'.PHP_EOL);
         }
     }
 
@@ -227,8 +227,8 @@ 
         $this->numChecks++;
         if ($this->debug) {
             $check = $event->getConfiguration();
-            $this->writeWithAsterisk('check: [' . $check->type . '] ');
-            $this->write('checking: [' . $check->value . '] ');
+            $this->writeWithAsterisk('check: ['.$check->type.'] ');
+            $this->write('checking: ['.$check->value.'] ');
         }
     }
 
@@ -240,7 +240,7 @@ 
     public function onCheckFailed(Event\Check\Failed $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-white, bg-red, bold', 'failed' . PHP_EOL);
+            $this->writeWithColor('fg-white, bg-red, bold', 'failed'.PHP_EOL);
         }
     }
 
@@ -252,7 +252,7 @@ 
     public function onCheckEnd(Event\Check\End $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-black, bg-green', 'ok' . PHP_EOL);
+            $this->writeWithColor('fg-black, bg-green', 'ok'.PHP_EOL);
         }
     }
 
@@ -266,7 +266,7 @@ 
         $this->numCrypts++;
         if ($this->debug) {
             $crypt = $event->getConfiguration();
-            $this->writeWithAsterisk('crypt: [' . $crypt->type . '] ');
+            $this->writeWithAsterisk('crypt: ['.$crypt->type.'] ');
         }
     }
 
@@ -278,7 +278,7 @@ 
     public function onCryptSkipped(Event\Crypt\Skipped $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-black, bg-yellow', 'skipped' . PHP_EOL);
+            $this->writeWithColor('fg-black, bg-yellow', 'skipped'.PHP_EOL);
         }
     }
 
@@ -290,7 +290,7 @@ 
     public function onCryptFailed(Event\Crypt\Failed $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-white, bg-red, bold', 'failed' . PHP_EOL);
+            $this->writeWithColor('fg-white, bg-red, bold', 'failed'.PHP_EOL);
         }
     }
 
@@ -302,7 +302,7 @@ 
     public function onCryptEnd(Event\Crypt\End $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-black, bg-green', 'ok' . PHP_EOL);
+            $this->writeWithColor('fg-black, bg-green', 'ok'.PHP_EOL);
         }
     }
 
@@ -316,7 +316,7 @@ 
         $this->numSyncs++;
         if ($this->debug) {
             $sync = $event->getConfiguration();
-            $this->writeWithAsterisk('sync: [' . $sync->type . '] ');
+            $this->writeWithAsterisk('sync: ['.$sync->type.'] ');
         }
     }
 
@@ -328,7 +328,7 @@ 
     public function onSyncSkipped(Event\Sync\Skipped $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-black, bg-yellow', 'skipped' . PHP_EOL);
+            $this->writeWithColor('fg-black, bg-yellow', 'skipped'.PHP_EOL);
         }
     }
 
@@ -340,7 +340,7 @@ 
     public function onSyncFailed(Event\Sync\Failed $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-white, bg-red, bold', 'failed' . PHP_EOL);
+            $this->writeWithColor('fg-white, bg-red, bold', 'failed'.PHP_EOL);
         }
     }
 
@@ -352,7 +352,7 @@ 
     public function onSyncEnd(Event\Sync\End $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-black, bg-green', 'ok' . PHP_EOL);
+            $this->writeWithColor('fg-black, bg-green', 'ok'.PHP_EOL);
         }
     }
 
@@ -366,7 +366,7 @@ 
         $this->numCleanups++;
         if ($this->debug) {
             $cleanup = $event->getConfiguration();
-            $this->writeWithAsterisk('cleanup: [' . $cleanup->type . '] ');
+            $this->writeWithAsterisk('cleanup: ['.$cleanup->type.'] ');
         }
     }
 
@@ -378,7 +378,7 @@ 
     public function onCleanupSkipped(Event\Cleanup\Skipped $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-black, bg-yellow', 'skipped' . PHP_EOL);
+            $this->writeWithColor('fg-black, bg-yellow', 'skipped'.PHP_EOL);
         }
     }
 
@@ -390,7 +390,7 @@ 
     public function onCleanupFailed(Event\Cleanup\Failed $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-white, bg-red, bold', 'failed' . PHP_EOL);
+            $this->writeWithColor('fg-white, bg-red, bold', 'failed'.PHP_EOL);
         }
     }
 
@@ -402,7 +402,7 @@ 
     public function onCleanupEnd(Event\Cleanup\End $event)
     {
         if ($this->debug) {
-            $this->writeWithColor('fg-black, bg-green', 'ok' . PHP_EOL);
+            $this->writeWithColor('fg-black, bg-green', 'ok'.PHP_EOL);
         }
     }
 
@@ -414,7 +414,7 @@ 
     public function onDebug(Event\Debug $event)
     {
         if ($this->debug) {
-            $this->write($event->getMessage() . PHP_EOL);
+            $this->write($event->getMessage().PHP_EOL);
         }
     }
 
@@ -452,7 +452,7 @@ 
      */
     protected function printHeader()
     {
-        $this->write(PHP_Timer::resourceUsage() . PHP_EOL . PHP_EOL);
+        $this->write(PHP_Timer::resourceUsage().PHP_EOL.PHP_EOL);
     }
 
     /**
@@ -512,13 +512,13 @@ 
         $clSkipped  = str_pad($backup->cleanupCountSkipped(), 7, ' ', STR_PAD_LEFT);
         $clFailed   = str_pad($backup->cleanupCountFailed(), 6, ' ', STR_PAD_LEFT);
 
-        $out = PHP_EOL . '          | executed | skipped | failed |' . PHP_EOL
-            . '----------+----------+---------+--------+' . PHP_EOL
-            . ' checks   | ' . $chExecuted . ' |         | ' . $chFailed . ' |' . PHP_EOL
-            . ' crypts   | ' . $crExecuted . ' | ' . $crSkipped . ' | ' . $crFailed . ' |' . PHP_EOL
-            . ' syncs    | ' . $syExecuted . ' | ' . $sySkipped . ' | ' . $syFailed . ' |' . PHP_EOL
-            . ' cleanups | ' . $clExecuted . ' | ' . $clSkipped . ' | ' . $clFailed . ' |' . PHP_EOL
-            . '----------+----------+---------+--------+' . PHP_EOL . PHP_EOL;
+        $out = PHP_EOL.'          | executed | skipped | failed |'.PHP_EOL
+            . '----------+----------+---------+--------+'.PHP_EOL
+            . ' checks   | '.$chExecuted.' |         | '.$chFailed.' |'.PHP_EOL
+            . ' crypts   | '.$crExecuted.' | '.$crSkipped.' | '.$crFailed.' |'.PHP_EOL
+            . ' syncs    | '.$syExecuted.' | '.$sySkipped.' | '.$syFailed.' |'.PHP_EOL
+            . ' cleanups | '.$clExecuted.' | '.$clSkipped.' | '.$clFailed.' |'.PHP_EOL
+            . '----------+----------+---------+--------+'.PHP_EOL.PHP_EOL;
 
         $this->write($out);
     }
@@ -539,7 +539,7 @@ 
             $this->writeWithColor(
                 'fg-black, bg-green',
                 sprintf(
-                    'OK (%d %s, %d %s, %d %s, %d %s, %d %s)' . PHP_EOL,
+                    'OK (%d %s, %d %s, %d %s, %d %s, %d %s)'.PHP_EOL,
                     count($result->getBackups()),
                     Util\Str::appendPluralS('backup', count($result->getBackups())),
                     $this->numChecks,
@@ -556,8 +556,8 @@ 
             $this->writeWithColor(
                 'fg-black, bg-yellow',
                 sprintf(
-                    "WARNING, skipped|failed Crypts, Syncs or Cleanups!" . PHP_EOL .
-                    'Backups: %d, Crypts: %d|%d, Syncs: %d|%d, Cleanups: %d|%d ' . PHP_EOL,
+                    "WARNING, skipped|failed Crypts, Syncs or Cleanups!".PHP_EOL.
+                    'Backups: %d, Crypts: %d|%d, Syncs: %d|%d, Cleanups: %d|%d '.PHP_EOL,
                     count($result->getBackups()),
                     $result->cryptsSkippedCount(),
                     $result->cryptsFailedCount(),
@@ -571,8 +571,8 @@ 
             $this->writeWithColor(
                 'fg-white, bg-red',
                 sprintf(
-                    "FAILURE!" . PHP_EOL .
-                    'Backups: %d, failed Checks: %d, failed Crypts: %d, failed Syncs: %d, failed Cleanups: %d.' . PHP_EOL,
+                    "FAILURE!".PHP_EOL.
+                    'Backups: %d, failed Checks: %d, failed Crypts: %d, failed Syncs: %d, failed Cleanups: %d.'.PHP_EOL,
                     count($result->getBackups()),
                     $result->checksFailedCount(),
                     $result->cryptsFailedCount(),
@@ -595,7 +595,7 @@ 
         if ($this->colors) {
             $buffer = Util\Cli::formatWithColor($color, $buffer);
         }
-        $this->write($buffer . PHP_EOL);
+        $this->write($buffer.PHP_EOL);
     }
 
     /**

src/Version.php

Spacing +1 added lines, -1 removed lines

@@ -57,6 +57,6 @@
      */
     public static function getVersionString() : string
     {
-        return 'phpbu ' . self::id() . ' by Sebastian Feldmann and contributors.';
+        return 'phpbu '.self::id().' by Sebastian Feldmann and contributors.';
     }
 }

src/Cli/Executable/OpenSSL.php

Spacing +4 added lines, -4 removed lines

@@ -177,7 +177,7 @@ 
     public function encryptFile(string $file) : OpenSSL
     {
         $this->sourceFile = $file;
-        $this->targetFile = $file . '.enc';
+        $this->targetFile = $file.'.enc';
         return $this;
     }
 
@@ -310,7 +310,7 @@ 
     {
         $cmd->addOption('smime');
         $cmd->addOption('-encrypt');
-        $cmd->addOption('-' . $this->algorithm);
+        $cmd->addOption('-'.$this->algorithm);
         $cmd->addOption('-binary');
         $cmd->addOption('-in', $this->sourceFile, ' ');
         $cmd->addOption('-out', $this->targetFile, ' ');
@@ -325,12 +325,12 @@ 
      */
     protected function setPasswordOptions(Cmd $cmd)
     {
-        $password = 'pass:' . $this->password;
+        $password = 'pass:'.$this->password;
 
         $cmd->addOption('enc');
         $cmd->addOption('-e');
         $cmd->addOptionIfNotEmpty('-a', $this->base64, false);
-        $cmd->addOption('-' . $this->algorithm);
+        $cmd->addOption('-'.$this->algorithm);
         $cmd->addOption('-pass', $password, ' ');
         $cmd->addOption('-in', $this->sourceFile, ' ');
         $cmd->addOption('-out', $this->targetFile, ' ');

src/Cli/Executable/RedisCli.php

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
      */
     public function runCommand(string $command) : RedisCli
     {
-        if(!isset($this->availableCommands[$command])) {
+        if (!isset($this->availableCommands[$command])) {
             throw new Exception('Unknown redis-cli command');
         }
         $this->command = $command;

src/Cli/Executable/Tar.php

Spacing +1 added lines, -1 removed lines

@@ -173,7 +173,7 @@
         $tar     = new Cmd($this->binary);
 
         $tar->addOptionIfNotEmpty('--ignore-failed-read', $this->ignoreFailedRead, false);
-        $tar->addOption('-' . $this->compression . 'cf');
+        $tar->addOption('-'.$this->compression.'cf');
         $tar->addArgument($this->tarPathname);
         $tar->addOption('-C', dirname($this->path), ' ');
         $tar->addArgument(basename($this->path));

src/Cli/Executable/Pgdump.php

Spacing +2 added lines, -2 removed lines

@@ -453,8 +453,8 @@
     protected function createCommandLine() : CommandLine
     {
         $process  = new CommandLine();
-        $password = $this->password ? 'PGPASSWORD=' . escapeshellarg($this->password) . ' ' : '';
-        $cmd      = new Cmd($password . $this->binary);
+        $password = $this->password ? 'PGPASSWORD='.escapeshellarg($this->password).' ' : '';
+        $cmd      = new Cmd($password.$this->binary);
         $process->addCommand($cmd);
 
         // always disable password prompt

src/Cli/Executable/Rsync/Location.php

Spacing +2 added lines, -2 removed lines

@@ -102,9 +102,9 @@
         if (!empty($this->host)) {
             // remote user
             if (!empty($this->user)) {
-                $return .= $this->user . '@';
+                $return .= $this->user.'@';
             }
-            $return .= $this->host . ':';
+            $return .= $this->host.':';
         }
         $return .= $this->path;
 

src/Backup/Cli.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
      */
     public function __construct(Runner $runner = null)
     {
-        $this->runner = $runner ? : new Runner\Simple();
+        $this->runner = $runner ?: new Runner\Simple();
     }
 
     /**