sebastianfeldmann / phpbu

src/Backup/Sync/Ftp.php

Unused Use Statements -2 removed lines

@@ -1,12 +1,10 @@
 <?php
 namespace phpbu\App\Backup\Sync;
 
-use phpbu\App\Backup\File\Remote;
 use phpbu\App\Result;
 use phpbu\App\Backup\Sync;
 use phpbu\App\Backup\Target;
 use phpbu\App\Util\Arr;
-use phpbu\App\Util\Str;
 
 /**
  * Ftp sync

src/Backup/Sync/Rsync.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 namespace phpbu\App\Backup\Sync;
 
 use phpbu\App\Backup\Cli;
-use phpbu\App\Backup\File\Remote;
 use phpbu\App\Backup\Rsync as RsyncTrait;
 use phpbu\App\Backup\Target;
 use phpbu\App\Cli\Executable;

src/Backup/Sync/SoftLayer.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 namespace phpbu\App\Backup\Sync;
 
-use phpbu\App\Backup\File\Remote;
 use phpbu\App\Result;
 use phpbu\App\Backup\Sync;
 use phpbu\App\Backup\Target;

Spacing +12 added lines, -12 removed lines

@@ -74,7 +74,7 @@ 
         // check for mandatory options
         foreach (['user', 'secret', 'container', 'host', 'path'] as $option) {
             if (!Arr::isSetAndNotEmptyString($config, $option)) {
-                throw new Exception('SoftLayer ' . $option . ' is mandatory');
+                throw new Exception('SoftLayer '.$option.' is mandatory');
             }
         }
 
@@ -98,19 +98,19 @@ 
     public function sync(Target $target, Result $result)
     {
         $sourcePath = $target->getPathname();
-        $targetPath = $this->path . $target->getFilename();
+        $targetPath = $this->path.$target->getFilename();
 
         $options       = ['adapter' => ObjectStorage_Http_Client::SOCKET, 'timeout' => 20];
         $objectStorage = new ObjectStorage($this->host, $this->user, $this->secret, $options);
 
-        $result->debug('softlayer source: ' . $sourcePath);
-        $result->debug('softlayer target: ' . $targetPath);
+        $result->debug('softlayer source: '.$sourcePath);
+        $result->debug('softlayer target: '.$targetPath);
 
         try {
             /** @var \ObjectStorage_Container $container */
-            $container = $objectStorage->with($this->container . $targetPath)
+            $container = $objectStorage->with($this->container.$targetPath)
                                        ->setLocalFile($sourcePath)
-                                       ->setMeta('description', 'PHPBU Backup: ' . date('r', time()))
+                                       ->setMeta('description', 'PHPBU Backup: '.date('r', time()))
                                        ->setHeader('Content-Type', $target->getMimeType());
             $container->create();
         } catch (\Exception $e) {
@@ -129,12 +129,12 @@ 
     public function simulate(Target $target, Result $result)
     {
         $result->debug(
-            'sync backup to SoftLayer' . PHP_EOL
-            . '  host:      ' . $this->host . PHP_EOL
-            . '  user:      ' . $this->user . PHP_EOL
-            . '  secret:     ********' . PHP_EOL
-            . '  container: ' . $this->container . PHP_EOL
-            . '  location:  ' . $this->path
+            'sync backup to SoftLayer'.PHP_EOL
+            . '  host:      '.$this->host.PHP_EOL
+            . '  user:      '.$this->user.PHP_EOL
+            . '  secret:     ********'.PHP_EOL
+            . '  container: '.$this->container.PHP_EOL
+            . '  location:  '.$this->path
         );
     }
 

src/Backup/Collector/Dropbox.php

Spacing +2 added lines, -2 removed lines

@@ -48,10 +48,10 @@
             }
             /** @var \Kunnu\Dropbox\Models\FileMetadata $item */
             // skip currently created backup
-            if ($item->getPathDisplay() == $this->path . $this->target->getFilename()) {
+            if ($item->getPathDisplay() == $this->path.$this->target->getFilename()) {
                 continue;
             }
-            if (preg_match('#' . $this->fileRegex . '#i', $item->getName())) {
+            if (preg_match('#'.$this->fileRegex.'#i', $item->getName())) {
                 $this->files[] = new \phpbu\App\Backup\File\Dropbox($this->client, $item);
             }
         }

src/Backup/Collector/Local.php

Spacing +3 added lines, -3 removed lines

@@ -110,8 +110,8 @@ 
             if ($file->getPathname() == $this->target->getPathname()) {
                 continue;
             }
-            if (preg_match('#' . $this->fileRegex . '#i', $file->getFilename())) {
-                $index               = date('YmdHis', $file->getMTime()) . '-' . $i . '-' . $file->getPathname();
+            if (preg_match('#'.$this->fileRegex.'#i', $file->getFilename())) {
+                $index               = date('YmdHis', $file->getMTime()).'-'.$i.'-'.$file->getPathname();
                 $this->files[$index] = new FileLocal($file->getFileInfo());
             }
         }
@@ -140,6 +140,6 @@ 
     {
         $dirTarget = Arr::getValue($this->target->getChangingPathElements(), $depth);
         $dirRegex  = Str::datePlaceholdersToRegex($dirTarget);
-        return preg_match('#' . $dirRegex . '#i', $dir);
+        return preg_match('#'.$dirRegex.'#i', $dir);
     }
 }

src/Backup/Collector/OpenStack.php

Spacing +2 added lines, -2 removed lines

@@ -52,10 +52,10 @@
                 continue;
             }
             // skip currently created backup
-            if ($object->name == $this->path . $this->target->getFilename()) {
+            if ($object->name == $this->path.$this->target->getFilename()) {
                 continue;
             }
-            if (preg_match('#' . $this->fileRegex . '#i', basename($object->name))) {
+            if (preg_match('#'.$this->fileRegex.'#i', basename($object->name))) {
                 $this->files[] = new \phpbu\App\Backup\File\OpenStack($this->container, $object);
             }
         }

src/Backup/File/OpenStack.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
         $this->container = $container;
         $this->filename = basename($object->name);
         $this->pathname = $object->name;
-        $this->size = (int)$object->contentLength;
+        $this->size = (int) $object->contentLength;
         $this->lastModified = $object->lastModified->getTimestamp();
     }
 

src/Backup/Cleaner/Capacity.php

Spacing +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
         if ($this->deleteTarget) {
             $this->deleteTarget = false;
             $result->debug('target will be deleted as well');
-            $result->debug('delete ' . $target->getPathname());
+            $result->debug('delete '.$target->getPathname());
         }
         parent::simulate($target, $collector, $result);
     }
@@ -139,7 +139,7 @@ 
         // should the currently created backup be deleted as well?
         if ($this->deleteTarget) {
             $file = $target->toFile();
-            $index = date('YmdHis', $file->getMTime()) . '-' . count($files) . '-' . $file->getPathname();
+            $index = date('YmdHis', $file->getMTime()).'-'.count($files).'-'.$file->getPathname();
             $files[$index] = $file;
         }
         return $files;

src/Backup/Collector/Sftp.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
             if ($fileInfo['filename'] == $this->target->getFilename()) {
                 continue;
             }
-            if (preg_match('#' . $this->fileRegex . '#i', $fileInfo['filename'])) {
+            if (preg_match('#'.$this->fileRegex.'#i', $fileInfo['filename'])) {
                 $this->files[] = new \phpbu\App\Backup\File\Sftp($this->sftp, $fileInfo, $this->path);
             }
         }