wapmorgan / UnifiedArchive

src/Drivers/BasicDriver.php

Spacing +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
     ) {
         $format_extension = Formats::getFormatExtension($archiveFormat);
         do {
-            $temp_file = tempnam(sys_get_temp_dir(), 'temp_archive') . '.' . $format_extension;
+            $temp_file = tempnam(sys_get_temp_dir(), 'temp_archive').'.'.$format_extension;
         } while (file_exists($temp_file));
 
         $created = static::createArchive($files, $temp_file, $archiveFormat, $compressionLevel, $password, $fileProgressCallable);

src/UnifiedArchive.php

Spacing +25 added lines, -25 removed lines

@@ -64,7 +64,7 @@ 
     public static function open($fileName, $abilities = [], $password = null)
     {
         if (!file_exists($fileName) || !is_readable($fileName)) {
-            throw new InvalidArgumentException('Could not open file: ' . $fileName . ' is not readable');
+            throw new InvalidArgumentException('Could not open file: '.$fileName.' is not readable');
         }
 
         $format = Formats::detectArchiveFormat($fileName);
@@ -121,7 +121,7 @@ 
         if ($archiveName !== null) {
             $archiveType = Formats::detectArchiveFormat($archiveName, false);
             if ($archiveType === false) {
-                throw new UnsupportedArchiveException('Could not detect archive type for name "' . $archiveName . '"');
+                throw new UnsupportedArchiveException('Could not detect archive type for name "'.$archiveName.'"');
             }
         }
 
@@ -183,17 +183,17 @@ 
     )
     {
         if (file_exists($archiveName))
-            throw new FileAlreadyExistsException('Archive ' . $archiveName . ' already exists!');
+            throw new FileAlreadyExistsException('Archive '.$archiveName.' already exists!');
 
         $info = static::prepareForArchiving($fileOrFiles, $archiveName);
 
         $abilities = [BasicDriver::CREATE];
 
         if (!Formats::canCreate($info['type']))
-            throw new UnsupportedArchiveException('Unsupported archive type: ' . $info['type'] . ' of archive ' . $archiveName);
+            throw new UnsupportedArchiveException('Unsupported archive type: '.$info['type'].' of archive '.$archiveName);
 
         if ($password !== null && !Formats::canEncrypt($info['type']))
-            throw new UnsupportedOperationException('Archive type ' . $info['type'] . ' can not be encrypted');
+            throw new UnsupportedOperationException('Archive type '.$info['type'].' can not be encrypted');
         if ($password !== null) {
             $abilities[] = BasicDriver::CREATE_ENCRYPTED;
         }
@@ -201,7 +201,7 @@ 
         /** @var BasicDriver $driver */
         $driver = Formats::getFormatDriver($info['type'], $abilities);
         if ($driver === null) {
-            throw new UnsupportedArchiveException('Unsupported archive type: ' . $info['type'] . ' of archive ');
+            throw new UnsupportedArchiveException('Unsupported archive type: '.$info['type'].' of archive ');
         }
 
         return $driver::createArchive(
@@ -253,11 +253,11 @@ 
         $info = static::prepareForArchiving($fileOrFiles);
         $abilities = [BasicDriver::CREATE, BasicDriver::CREATE_IN_STRING];
         if (!Formats::canCreate($info['type'])) {
-            throw new UnsupportedArchiveException('Unsupported archive type: ' . $info['type'] . ' of archive ' . $archiveName);
+            throw new UnsupportedArchiveException('Unsupported archive type: '.$info['type'].' of archive '.$archiveName);
         }
 
         if ($password !== null && !Formats::canEncrypt($info['type'])) {
-            throw new UnsupportedOperationException('Archive type ' . $info['type'] . ' can not be encrypted');
+            throw new UnsupportedOperationException('Archive type '.$info['type'].' can not be encrypted');
         }
         if ($password !== null) {
             $abilities[] = BasicDriver::CREATE_ENCRYPTED;
@@ -266,7 +266,7 @@ 
         /** @var BasicDriver $driver */
         $driver = Formats::getFormatDriver($info['type'], $abilities);
         if ($driver === null) {
-            throw new UnsupportedArchiveException('Unsupported archive type: ' . $info['type'] . ' of archive ');
+            throw new UnsupportedArchiveException('Unsupported archive type: '.$info['type'].' of archive ');
         }
 
         return $driver::createArchiveInString(
@@ -292,7 +292,7 @@ 
     public static function archiveFile($file, $archiveName, $compressionLevel = BasicDriver::COMPRESSION_AVERAGE, $password = null)
     {
         if (!is_file($file)) {
-            throw new InvalidArgumentException($file . ' is not a valid file to archive');
+            throw new InvalidArgumentException($file.' is not a valid file to archive');
         }
 
         return static::archive($file, $archiveName, $compressionLevel, $password) === 1;
@@ -312,7 +312,7 @@ 
     public static function archiveDirectory($directory, $archiveName, $compressionLevel = BasicDriver::COMPRESSION_AVERAGE, $password = null)
     {
         if (!is_dir($directory) || !is_readable($directory))
-            throw new InvalidArgumentException($directory . ' is not a valid directory to archive');
+            throw new InvalidArgumentException($directory.' is not a valid directory to archive');
 
         return static::archive($directory, $archiveName, $compressionLevel, $password) > 0;
     }
@@ -503,7 +503,7 @@ 
     public function getFileData($fileName)
     {
         if (!in_array($fileName, $this->files, true)) {
-            throw new NonExistentArchiveFileException('File ' . $fileName . ' does not exist in archive');
+            throw new NonExistentArchiveFileException('File '.$fileName.' does not exist in archive');
         }
 
         return $this->archive->getFileData($fileName);
@@ -519,7 +519,7 @@ 
     public function getFileContent($fileName)
     {
         if (!in_array($fileName, $this->files, true)) {
-            throw new NonExistentArchiveFileException('File ' . $fileName . ' does not exist in archive');
+            throw new NonExistentArchiveFileException('File '.$fileName.' does not exist in archive');
         }
 
         return $this->archive->getFileContent($fileName);
@@ -535,7 +535,7 @@ 
     public function getFileStream($fileName)
     {
         if (!in_array($fileName, $this->files, true)) {
-            throw new NonExistentArchiveFileException('File ' . $fileName . ' does not exist in archive');
+            throw new NonExistentArchiveFileException('File '.$fileName.' does not exist in archive');
         }
 
         return $this->archive->getFileStream($fileName);
@@ -633,7 +633,7 @@ 
     public function addFile($file, $inArchiveName = null)
     {
         if (!is_file($file))
-            throw new InvalidArgumentException($file . ' is not a valid file to add in archive');
+            throw new InvalidArgumentException($file.' is not a valid file to add in archive');
 
         return ($inArchiveName !== null
                 ? $this->add([$inArchiveName => $file])
@@ -667,7 +667,7 @@ 
     public function addDirectory($directory, $inArchivePath = null)
     {
         if (!is_dir($directory) || !is_readable($directory))
-            throw new InvalidArgumentException($directory . ' is not a valid directory to add in archive');
+            throw new InvalidArgumentException($directory.' is not a valid directory to add in archive');
 
         return ($inArchivePath !== null
                 ? $this->add([$inArchivePath => $directory])
@@ -710,7 +710,7 @@ 
         $newFiles = [];
         foreach ($files as $file) {
             foreach ($archiveFiles as $archiveFile) {
-                if (fnmatch($file . '*', $archiveFile)) {
+                if (fnmatch($file.'*', $archiveFile)) {
                     $newFiles[] = $archiveFile;
                 }
             }
@@ -745,8 +745,8 @@ 
                 if (is_array($source)) {
                     foreach ($source as $sourceItem) {
                         static::importFilesFromDir(
-                            rtrim($sourceItem, '/\\*') . '/*',
-                            !empty($destination) ? $destination . '/' : null,
+                            rtrim($sourceItem, '/\\*').'/*',
+                            !empty($destination) ? $destination.'/' : null,
                             true,
                             $files
                         );
@@ -754,8 +754,8 @@ 
                 } else if (is_dir($source)) {
                     // one source for directories
                     static::importFilesFromDir(
-                        rtrim($source, '/\\*') . '/*',
-                        !empty($destination) ? $destination . '/' : null,
+                        rtrim($source, '/\\*').'/*',
+                        !empty($destination) ? $destination.'/' : null,
                         true,
                         $files
                     );
@@ -767,7 +767,7 @@ 
         } else if (is_string($nodes)) { // passed one file or directory
             // if is directory
             if (is_dir($nodes))
-                static::importFilesFromDir(rtrim($nodes, '/\\*') . '/*', null, true,
+                static::importFilesFromDir(rtrim($nodes, '/\\*').'/*', null, true,
                     $files);
             else if (is_file($nodes))
                 $files[basename($nodes)] = $nodes;
@@ -792,10 +792,10 @@ 
 
         foreach (glob($source, GLOB_MARK) as $node) {
             if (in_array(substr($node, -1), ['/', '\\'], true) && $recursive) {
-                static::importFilesFromDir(str_replace('\\', '/', $node) . '*',
-                    $destination . basename($node) . '/', $recursive, $map);
+                static::importFilesFromDir(str_replace('\\', '/', $node).'*',
+                    $destination.basename($node).'/', $recursive, $map);
             } elseif (is_file($node) && is_readable($node)) {
-                $map[$destination . basename($node)] = $node;
+                $map[$destination.basename($node)] = $node;
             }
         }
     }

Braces +38 added lines, -27 removed lines

@@ -127,8 +127,9 @@ 
 
         $files_list = static::createFilesList($fileOrFiles);
 
-        if (empty($files_list))
-            throw new EmptyFileListException('Files list is empty!');
+        if (empty($files_list)) {
+                    throw new EmptyFileListException('Files list is empty!');
+        }
 
         $totalSize = 0;
         foreach ($files_list as $fn) {
@@ -182,18 +183,21 @@ 
         $fileProgressCallable = null
     )
     {
-        if (file_exists($archiveName))
-            throw new FileAlreadyExistsException('Archive ' . $archiveName . ' already exists!');
+        if (file_exists($archiveName)) {
+                    throw new FileAlreadyExistsException('Archive ' . $archiveName . ' already exists!');
+        }
 
         $info = static::prepareForArchiving($fileOrFiles, $archiveName);
 
         $abilities = [BasicDriver::CREATE];
 
-        if (!Formats::canCreate($info['type']))
-            throw new UnsupportedArchiveException('Unsupported archive type: ' . $info['type'] . ' of archive ' . $archiveName);
+        if (!Formats::canCreate($info['type'])) {
+                    throw new UnsupportedArchiveException('Unsupported archive type: ' . $info['type'] . ' of archive ' . $archiveName);
+        }
 
-        if ($password !== null && !Formats::canEncrypt($info['type']))
-            throw new UnsupportedOperationException('Archive type ' . $info['type'] . ' can not be encrypted');
+        if ($password !== null && !Formats::canEncrypt($info['type'])) {
+                    throw new UnsupportedOperationException('Archive type ' . $info['type'] . ' can not be encrypted');
+        }
         if ($password !== null) {
             $abilities[] = BasicDriver::CREATE_ENCRYPTED;
         }
@@ -311,8 +315,9 @@ 
      */
     public static function archiveDirectory($directory, $archiveName, $compressionLevel = BasicDriver::COMPRESSION_AVERAGE, $password = null)
     {
-        if (!is_dir($directory) || !is_readable($directory))
-            throw new InvalidArgumentException($directory . ' is not a valid directory to archive');
+        if (!is_dir($directory) || !is_readable($directory)) {
+                    throw new InvalidArgumentException($directory . ' is not a valid directory to archive');
+        }
 
         return static::archive($directory, $archiveName, $compressionLevel, $password) > 0;
     }
@@ -482,13 +487,15 @@ 
      */
     public function getFiles($filter = null)
     {
-        if ($filter === null)
-            return $this->files;
+        if ($filter === null) {
+                    return $this->files;
+        }
 
         $result = [];
         foreach ($this->files as $file) {
-            if (fnmatch($filter, $file))
-                $result[] = $file;
+            if (fnmatch($filter, $file)) {
+                            $result[] = $file;
+            }
         }
         return $result;
     }
@@ -632,8 +639,9 @@ 
      */
     public function addFile($file, $inArchiveName = null)
     {
-        if (!is_file($file))
-            throw new InvalidArgumentException($file . ' is not a valid file to add in archive');
+        if (!is_file($file)) {
+                    throw new InvalidArgumentException($file . ' is not a valid file to add in archive');
+        }
 
         return ($inArchiveName !== null
                 ? $this->add([$inArchiveName => $file])
@@ -666,8 +674,9 @@ 
      */
     public function addDirectory($directory, $inArchivePath = null)
     {
-        if (!is_dir($directory) || !is_readable($directory))
-            throw new InvalidArgumentException($directory . ' is not a valid directory to add in archive');
+        if (!is_dir($directory) || !is_readable($directory)) {
+                    throw new InvalidArgumentException($directory . ' is not a valid directory to add in archive');
+        }
 
         return ($inArchivePath !== null
                 ? $this->add([$inArchivePath => $directory])
@@ -730,9 +739,9 @@ 
         if (is_array($nodes)) {
             foreach ($nodes as $destination => $source) {
                 // new format
-                if (is_numeric($destination))
-                    $destination = $source;
-                else {
+                if (is_numeric($destination)) {
+                                    $destination = $source;
+                } else {
                     // old format
                     if (is_string($source) && !file_exists($source)) {
                         list($destination, $source) = [$source, $destination];
@@ -766,11 +775,12 @@ 
 
         } else if (is_string($nodes)) { // passed one file or directory
             // if is directory
-            if (is_dir($nodes))
-                static::importFilesFromDir(rtrim($nodes, '/\\*') . '/*', null, true,
+            if (is_dir($nodes)) {
+                            static::importFilesFromDir(rtrim($nodes, '/\\*') . '/*', null, true,
                     $files);
-            else if (is_file($nodes))
-                $files[basename($nodes)] = $nodes;
+            } else if (is_file($nodes)) {
+                            $files[basename($nodes)] = $nodes;
+            }
         }
 
         return $files;
@@ -787,8 +797,9 @@ 
         // $map[$destination] = rtrim($source, '/*');
         // do not map root archive folder
 
-        if ($destination !== null)
-            $map[$destination] = null;
+        if ($destination !== null) {
+                    $map[$destination] = null;
+        }
 
         foreach (glob($source, GLOB_MARK) as $node) {
             if (in_array(substr($node, -1), ['/', '\\'], true) && $recursive) {