Passed
Push — master ( 125b29...554a92 )
by f
27:06 queued 12:02
created
src/ArchiveEntry.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,8 +46,9 @@
 block discarded – undo
46 46
         $this->compressedSize = $compressedSize;
47 47
         $this->uncompressedSize = $uncompressedSize;
48 48
         $this->modificationTime = $modificationTime;
49
-        if ($isCompressed === null)
50
-            $isCompressed = $uncompressedSize !== $compressedSize;
49
+        if ($isCompressed === null) {
50
+                    $isCompressed = $uncompressedSize !== $compressedSize;
51
+        }
51 52
         $this->isCompressed = $isCompressed;
52 53
         $this->comment = $comment;
53 54
         $this->crc32 = $crc32;
Please login to merge, or discard this patch.
src/Drivers/SevenZip.php 2 patches
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -108,7 +108,7 @@  discard block
 block discarded – undo
108 108
     public static function getInstallationInstruction()
109 109
     {
110 110
         if (!class_exists('\Archive7z\Archive7z'))
111
-            return 'install library [gemorroj/archive7z]: `composer require gemorroj/archive7z`' . "\n"
111
+            return 'install library [gemorroj/archive7z]: `composer require gemorroj/archive7z`'."\n"
112 112
                 . ' and console program p7zip [7za]: `apt install p7zip-full` - depends on OS';
113 113
 
114 114
         if (Archive7z::getBinaryVersion() === false)
@@ -150,8 +150,8 @@  discard block
 block discarded – undo
150 150
             $information->files[] = $can_get_unix_path
151 151
                 ? $entry->getUnixPath()
152 152
                 : str_replace('\\', '/', $entry->getPath());
153
-            $information->compressedFilesSize += (int)$entry->getPackedSize();
154
-            $information->uncompressedFilesSize += (int)$entry->getSize();
153
+            $information->compressedFilesSize += (int) $entry->getPackedSize();
154
+            $information->uncompressedFilesSize += (int) $entry->getSize();
155 155
         }
156 156
         return $information;
157 157
     }
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
     {
210 210
         $entry = $this->sevenZip->getEntry($fileName);
211 211
         if ($entry === null) {
212
-            throw new NonExistentArchiveFileException('File ' . $fileName . ' does not exist');
212
+            throw new NonExistentArchiveFileException('File '.$fileName.' does not exist');
213 213
         }
214 214
         return $entry->getContent();
215 215
     }
Please login to merge, or discard this patch.
Braces   +21 added lines, -14 removed lines patch added patch discarded remove patch
@@ -120,12 +120,14 @@  discard block
 block discarded – undo
120 120
      */
121 121
     public static function getInstallationInstruction()
122 122
     {
123
-        if (!class_exists('\Archive7z\Archive7z'))
124
-            return 'install library [gemorroj/archive7z]: `composer require gemorroj/archive7z`' . "\n"
123
+        if (!class_exists('\Archive7z\Archive7z')) {
124
+                    return 'install library [gemorroj/archive7z]: `composer require gemorroj/archive7z`' . "\n"
125 125
                 . ' and console program p7zip [7za]: `apt install p7zip-full` - depends on OS';
126
+        }
126 127
 
127
-        if (Archive7z::getBinaryVersion() === false)
128
-            return 'install console program p7zip [7za]: `apt install p7zip-full` - depends on OS';
128
+        if (Archive7z::getBinaryVersion() === false) {
129
+                    return 'install console program p7zip [7za]: `apt install p7zip-full` - depends on OS';
130
+        }
129 131
 
130 132
         return null;
131 133
     }
@@ -138,8 +140,9 @@  discard block
 block discarded – undo
138 140
         try {
139 141
             $this->format = $format;
140 142
             $this->sevenZip = new Archive7z($archiveFileName, null, null);
141
-            if ($password !== null)
142
-                $this->sevenZip->setPassword($password);
143
+            if ($password !== null) {
144
+                            $this->sevenZip->setPassword($password);
145
+            }
143 146
         } catch (\Archive7z\Exception $e) {
144 147
             throw new Exception('Could not open 7Zip archive: '.$e->getMessage(), $e->getCode(), $e);
145 148
         }
@@ -157,8 +160,9 @@  discard block
 block discarded – undo
157 160
                 continue;
158 161
             }
159 162
 
160
-            if (!isset($can_get_unix_path))
161
-                $can_get_unix_path = method_exists($entry, 'getUnixPath');
163
+            if (!isset($can_get_unix_path)) {
164
+                            $can_get_unix_path = method_exists($entry, 'getUnixPath');
165
+            }
162 166
 
163 167
             $information->files[] = $can_get_unix_path
164 168
                 ? $entry->getUnixPath()
@@ -176,8 +180,9 @@  discard block
 block discarded – undo
176 180
     {
177 181
         $files = [];
178 182
         foreach ($this->sevenZip->getEntries() as $entry) {
179
-            if ($entry->isDirectory())
180
-                continue;
183
+            if ($entry->isDirectory()) {
184
+                            continue;
185
+            }
181 186
             $files[] = $entry->getPath();
182 187
         }
183 188
         return $files;
@@ -329,8 +334,9 @@  discard block
 block discarded – undo
329 334
     public function addFileFromString($inArchiveName, $content)
330 335
     {
331 336
         $tmp_file = tempnam(sys_get_temp_dir(), 'ua');
332
-        if (!$tmp_file)
333
-            throw new ArchiveModificationException('Could not create temporarily file');
337
+        if (!$tmp_file) {
338
+                    throw new ArchiveModificationException('Could not create temporarily file');
339
+        }
334 340
 
335 341
         file_put_contents($tmp_file, $content);
336 342
         $this->sevenZip->addEntry($tmp_file, true);
@@ -380,8 +386,9 @@  discard block
 block discarded – undo
380 386
             $total_files = count($files);
381 387
 
382 388
             $seven_zip = new Archive7z($archiveFileName);
383
-            if ($password !== null)
384
-                $seven_zip->setPassword($password);
389
+            if ($password !== null) {
390
+                            $seven_zip->setPassword($password);
391
+            }
385 392
             $seven_zip->setCompressionLevel($compressionLevelMap[$compressionLevel]);
386 393
             foreach ($files as $localName => $filename) {
387 394
                 if ($filename !== null) {
Please login to merge, or discard this patch.
src/Commands/FormatCommand.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
             $question = new ChoiceQuestion('Which format', array_keys($formats));
42 42
             $format = $helper->ask($input, $output, $question);
43 43
         }
44
-        $output->writeln('Format <info>' . $format . '</info> drivers support');
44
+        $output->writeln('Format <info>'.$format.'</info> drivers support');
45 45
 
46 46
         $table = new Table($output);
47 47
         $table->setHeaders(['format', ...array_keys(self::$abilitiesLabels)]);
Please login to merge, or discard this patch.
src/Drivers/TarByPear.php 2 patches
Braces   +26 added lines, -18 removed lines patch added patch discarded remove patch
@@ -165,10 +165,11 @@  discard block
 block discarded – undo
165 165
                 break;
166 166
         }
167 167
 
168
-        if (isset($tar_aname))
169
-            $tar = new Archive_Tar($tar_aname, $compression);
170
-        else
171
-            $tar = new Archive_Tar($archiveFileName, $compression);
168
+        if (isset($tar_aname)) {
169
+                    $tar = new Archive_Tar($tar_aname, $compression);
170
+        } else {
171
+                    $tar = new Archive_Tar($archiveFileName, $compression);
172
+        }
172 173
 
173 174
         $current_file = 0;
174 175
         $total_files = count($files);
@@ -177,11 +178,13 @@  discard block
 block discarded – undo
177 178
             $add_dir = dirname($localName);
178 179
 
179 180
             if (is_null($filename)) {
180
-                if ($tar->addString($localName, '') === false)
181
-                    throw new ArchiveCreationException('Error when adding directory '.$localName.' to archive');
181
+                if ($tar->addString($localName, '') === false) {
182
+                                    throw new ArchiveCreationException('Error when adding directory '.$localName.' to archive');
183
+                }
182 184
             } else {
183
-                if ($tar->addModify($filename, $add_dir, $remove_dir) === false)
184
-                    throw new ArchiveCreationException('Error when adding file '.$filename.' to archive');
185
+                if ($tar->addModify($filename, $add_dir, $remove_dir) === false) {
186
+                                    throw new ArchiveCreationException('Error when adding file '.$filename.' to archive');
187
+                }
185 188
             }
186 189
             if ($fileProgressCallable !== null) {
187 190
                 call_user_func_array($fileProgressCallable, [$current_file++, $total_files, $filename, $localName]);
@@ -242,8 +245,9 @@  discard block
 block discarded – undo
242 245
                 continue;
243 246
             }
244 247
             // skip directories
245
-            if ($file['typeflag'] == '5' || substr($file['filename'], -1) === '/')
246
-                continue;
248
+            if ($file['typeflag'] == '5' || substr($file['filename'], -1) === '/') {
249
+                            continue;
250
+            }
247 251
             $information->files[] = $file['filename'];
248 252
             $information->uncompressedFilesSize += $file['size'];
249 253
             $this->pearFilesIndex[$file['filename']] = $i;
@@ -289,14 +293,16 @@  discard block
 block discarded – undo
289 293
      */
290 294
     public function getFileData($fileName)
291 295
     {
292
-        if (!isset($this->pearFilesIndex[$fileName]))
293
-            throw new NonExistentArchiveFileException('File '.$fileName.' is not found in archive files list');
296
+        if (!isset($this->pearFilesIndex[$fileName])) {
297
+                    throw new NonExistentArchiveFileException('File '.$fileName.' is not found in archive files list');
298
+        }
294 299
 
295 300
         $index = $this->pearFilesIndex[$fileName];
296 301
 
297 302
         $files_list = $this->tar->listContent();
298
-        if (!isset($files_list[$index]))
299
-            throw new NonExistentArchiveFileException('File '.$fileName.' is not found in Tar archive');
303
+        if (!isset($files_list[$index])) {
304
+                    throw new NonExistentArchiveFileException('File '.$fileName.' is not found in Tar archive');
305
+        }
300 306
 
301 307
         $data = $files_list[$index];
302 308
         unset($files_list);
@@ -311,8 +317,9 @@  discard block
 block discarded – undo
311 317
      */
312 318
     public function getFileContent($fileName)
313 319
     {
314
-        if (!isset($this->pearFilesIndex[$fileName]))
315
-            throw new NonExistentArchiveFileException('File '.$fileName.' is not found in archive files list');
320
+        if (!isset($this->pearFilesIndex[$fileName])) {
321
+                    throw new NonExistentArchiveFileException('File '.$fileName.' is not found in archive files list');
322
+        }
316 323
 
317 324
         return $this->tar->extractInString($fileName);
318 325
     }
@@ -322,8 +329,9 @@  discard block
 block discarded – undo
322 329
      */
323 330
     public function getFileStream($fileName)
324 331
     {
325
-        if (!isset($this->pearFilesIndex[$fileName]))
326
-            throw new NonExistentArchiveFileException('File '.$fileName.' is not found in archive files list');
332
+        if (!isset($this->pearFilesIndex[$fileName])) {
333
+                    throw new NonExistentArchiveFileException('File '.$fileName.' is not found in archive files list');
334
+        }
327 335
 
328 336
         return self::wrapStringInStream($this->tar->extractInString($fileName));
329 337
     }
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
      */
54 54
     public static function getInstallationInstruction()
55 55
     {
56
-        return 'install library [pear/archive_tar]: `composer require pear/archive_tar`' . "\n"  . ' and optionally php-extensions (zlib, bz2)';
56
+        return 'install library [pear/archive_tar]: `composer require pear/archive_tar`'."\n".' and optionally php-extensions (zlib, bz2)';
57 57
     }
58 58
 
59 59
     /**
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
                 $compression = 'lzma2';
161 161
                 break;
162 162
             case 'z':
163
-                $tar_aname = 'compress.lzw://' . $archiveFileName;
163
+                $tar_aname = 'compress.lzw://'.$archiveFileName;
164 164
                 break;
165 165
         }
166 166
 
@@ -217,7 +217,7 @@  discard block
 block discarded – undo
217 217
 
218 218
             case Formats::TAR_LZW:
219 219
                 LzwStreamWrapper::registerWrapper();
220
-                $this->tar = new Archive_Tar('compress.lzw://' . $this->fileName);
220
+                $this->tar = new Archive_Tar('compress.lzw://'.$this->fileName);
221 221
                 break;
222 222
 
223 223
             default:
@@ -335,7 +335,7 @@  discard block
 block discarded – undo
335 335
         $result = $this->tar->extractList($files, $outputFolder);
336 336
         if ($result === false) {
337 337
             if (isset($this->tar->error_object)) {
338
-                throw new ArchiveExtractionException('Error when extracting from ' . $this->fileName . ': ' . $this->tar->error_object->getMessage(0));
338
+                throw new ArchiveExtractionException('Error when extracting from '.$this->fileName.': '.$this->tar->error_object->getMessage(0));
339 339
             }
340 340
             throw new ArchiveExtractionException('Error when extracting from '.$this->fileName);
341 341
         }
@@ -351,7 +351,7 @@  discard block
 block discarded – undo
351 351
         $result = $this->tar->extract($outputFolder);
352 352
         if ($result === false) {
353 353
             if (isset($this->tar->error_object)) {
354
-                throw new ArchiveExtractionException('Error when extracting ' . $this->fileName . ': '
354
+                throw new ArchiveExtractionException('Error when extracting '.$this->fileName.': '
355 355
                                                      . $this->tar->error_object->toString()
356 356
                 );
357 357
             }
Please login to merge, or discard this patch.
src/Drivers/AlchemyZippy.php 2 patches
Braces   +11 added lines, -8 removed lines patch added patch discarded remove patch
@@ -61,10 +61,11 @@  discard block
 block discarded – undo
61 61
 
62 62
     protected static function init()
63 63
     {
64
-        if (!class_exists('\Alchemy\Zippy\Zippy'))
65
-            static::$zippy = false;
66
-        else if (static::$zippy === null)
67
-            static::$zippy = Zippy::load();
64
+        if (!class_exists('\Alchemy\Zippy\Zippy')) {
65
+                    static::$zippy = false;
66
+        } else if (static::$zippy === null) {
67
+                    static::$zippy = Zippy::load();
68
+        }
68 69
     }
69 70
 
70 71
     /**
@@ -75,8 +76,9 @@  discard block
 block discarded – undo
75 76
     {
76 77
         static::init();
77 78
 
78
-        if (static::$zippy === false)
79
-            return [];
79
+        if (static::$zippy === false) {
80
+                    return [];
81
+        }
80 82
 
81 83
         switch ($format) {
82 84
             case Formats::TAR_BZIP:
@@ -212,8 +214,9 @@  discard block
 block discarded – undo
212 214
         $information = new ArchiveInformation();
213 215
 
214 216
         foreach ($this->archive->getMembers() as $member) {
215
-            if ($member->isDir())
216
-                continue;
217
+            if ($member->isDir()) {
218
+                            continue;
219
+            }
217 220
 
218 221
             $this->files[] = $information->files[] = str_replace('\\', '/', $member->getLocation());
219 222
             $this->members[str_replace('\\', '/', $member->getLocation())] = $member;
Please login to merge, or discard this patch.
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -63,9 +63,9 @@  discard block
 block discarded – undo
63 63
     public static function getInstallationInstruction()
64 64
     {
65 65
         self::init();
66
-        return 'install library [alchemy/zippy]: `composer require alchemy/zippy`' . "\n"  . ' and console programs (tar, zip): `apt install tar zip` - depends on OS'
67
-            . "\n" . 'If you install SevenZip and AlchemyZippy:' . "\n" .
68
-            '1. You should specify symfony/console version before installation to any **3.x.x version**:' . "\n" . '`composer require symfony/process:~3.4`, because they require different `symfony/process` versions.' . "\n" .
66
+        return 'install library [alchemy/zippy]: `composer require alchemy/zippy`'."\n".' and console programs (tar, zip): `apt install tar zip` - depends on OS'
67
+            . "\n".'If you install SevenZip and AlchemyZippy:'."\n".
68
+            '1. You should specify symfony/console version before installation to any **3.x.x version**:'."\n".'`composer require symfony/process:~3.4`, because they require different `symfony/process` versions.'."\n".
69 69
             '2. Install archive7z version 4.0.0: `composer require gemorroj/archive7z:~4.0`';
70 70
     }
71 71
 
@@ -190,8 +190,8 @@  discard block
 block discarded – undo
190 190
 
191 191
             $this->files[] = $information->files[] = str_replace('\\', '/', $member->getLocation());
192 192
             $this->members[str_replace('\\', '/', $member->getLocation())] = $member;
193
-            $information->compressedFilesSize += (int)$member->getSize();
194
-            $information->uncompressedFilesSize += (int)$member->getSize();
193
+            $information->compressedFilesSize += (int) $member->getSize();
194
+            $information->uncompressedFilesSize += (int) $member->getSize();
195 195
         }
196 196
         return $information;
197 197
     }
@@ -240,7 +240,7 @@  discard block
 block discarded – undo
240 240
     public function getFileContent($fileName)
241 241
     {
242 242
         $member = $this->getMember($fileName);
243
-        return (string)$member;
243
+        return (string) $member;
244 244
     }
245 245
 
246 246
     /**
@@ -249,7 +249,7 @@  discard block
 block discarded – undo
249 249
     public function getFileStream($fileName)
250 250
     {
251 251
         $member = $this->getMember($fileName);
252
-        return self::wrapStringInStream((string)$member);
252
+        return self::wrapStringInStream((string) $member);
253 253
     }
254 254
 
255 255
     /**
Please login to merge, or discard this patch.
src/Drivers/NelexaZip.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -42,8 +42,9 @@
 block discarded – undo
42 42
         $information = new ArchiveInformation();
43 43
 
44 44
         foreach ($this->zip->getAllInfo() as $info) {
45
-            if ($info->isFolder())
46
-                continue;
45
+            if ($info->isFolder()) {
46
+                            continue;
47
+            }
47 48
 
48 49
             $this->files[] = $information->files[] = str_replace('\\', '/', $info->getName());
49 50
             $information->compressedFilesSize += $info->getCompressedSize();
Please login to merge, or discard this patch.
src/UnifiedArchive.php 2 patches
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
     public static function open($fileName, $password = null, $abilities = [])
64 64
     {
65 65
         if (!file_exists($fileName) || !is_readable($fileName)) {
66
-            throw new InvalidArgumentException('Could not open file: ' . $fileName.' is not readable');
66
+            throw new InvalidArgumentException('Could not open file: '.$fileName.' is not readable');
67 67
         }
68 68
 
69 69
         $format = Formats::detectArchiveFormat($fileName);
@@ -278,7 +278,7 @@  discard block
 block discarded – undo
278 278
     public function getFileData($fileName)
279 279
     {
280 280
         if (!in_array($fileName, $this->files, true)) {
281
-            throw new NonExistentArchiveFileException('File ' . $fileName . ' does not exist in archive');
281
+            throw new NonExistentArchiveFileException('File '.$fileName.' does not exist in archive');
282 282
         }
283 283
 
284 284
         return $this->archive->getFileData($fileName);
@@ -294,7 +294,7 @@  discard block
 block discarded – undo
294 294
     public function getFileContent($fileName)
295 295
     {
296 296
         if (!in_array($fileName, $this->files, true)) {
297
-            throw new NonExistentArchiveFileException('File ' . $fileName . ' does not exist in archive');
297
+            throw new NonExistentArchiveFileException('File '.$fileName.' does not exist in archive');
298 298
         }
299 299
 
300 300
         return $this->archive->getFileContent($fileName);
@@ -310,7 +310,7 @@  discard block
 block discarded – undo
310 310
     public function getFileStream($fileName)
311 311
     {
312 312
         if (!in_array($fileName, $this->files, true)) {
313
-            throw new NonExistentArchiveFileException('File ' . $fileName . ' does not exist in archive');
313
+            throw new NonExistentArchiveFileException('File '.$fileName.' does not exist in archive');
314 314
         }
315 315
 
316 316
         return $this->archive->getFileStream($fileName);
@@ -573,7 +573,7 @@  discard block
 block discarded – undo
573 573
         /** @var BasicDriver $driver */
574 574
         $driver = Formats::getFormatDriver($info['type'], $abilities);
575 575
         if ($driver === null) {
576
-            throw new UnsupportedArchiveException('Unsupported archive type: ' . $info['type'] . ' of archive ');
576
+            throw new UnsupportedArchiveException('Unsupported archive type: '.$info['type'].' of archive ');
577 577
         }
578 578
 
579 579
         return $driver::createArchive(
@@ -600,7 +600,7 @@  discard block
 block discarded – undo
600 600
     public static function archiveFile($file, $archiveName, $compressionLevel = BasicDriver::COMPRESSION_AVERAGE, $password = null)
601 601
     {
602 602
         if (!is_file($file)) {
603
-            throw new InvalidArgumentException($file . ' is not a valid file to archive');
603
+            throw new InvalidArgumentException($file.' is not a valid file to archive');
604 604
         }
605 605
 
606 606
         return static::archiveFiles($file, $archiveName, $compressionLevel, $password) === 1;
@@ -671,8 +671,8 @@  discard block
 block discarded – undo
671 671
                 if (is_array($source)) {
672 672
                     foreach ($source as $sourceItem) {
673 673
                         static::importFilesFromDir(
674
-                            rtrim($sourceItem, '/\\*') . '/*',
675
-                            !empty($destination) ? $destination . '/' : null,
674
+                            rtrim($sourceItem, '/\\*').'/*',
675
+                            !empty($destination) ? $destination.'/' : null,
676 676
                             true,
677 677
                             $files
678 678
                         );
@@ -680,8 +680,8 @@  discard block
 block discarded – undo
680 680
                 } else if (is_dir($source)) {
681 681
                     // one source for directories
682 682
                     static::importFilesFromDir(
683
-                        rtrim($source, '/\\*') . '/*',
684
-                        !empty($destination) ? $destination . '/' : null,
683
+                        rtrim($source, '/\\*').'/*',
684
+                        !empty($destination) ? $destination.'/' : null,
685 685
                         true,
686 686
                         $files
687 687
                     );
Please login to merge, or discard this patch.
Braces   +44 added lines, -31 removed lines patch added patch discarded remove patch
@@ -257,13 +257,15 @@  discard block
 block discarded – undo
257 257
      */
258 258
     public function getFileNames($filter = null)
259 259
     {
260
-        if ($filter === null)
261
-            return $this->files;
260
+        if ($filter === null) {
261
+                    return $this->files;
262
+        }
262 263
 
263 264
         $result = [];
264 265
         foreach ($this->files as $file) {
265
-            if (fnmatch($filter, $file))
266
-                $result[] = $file;
266
+            if (fnmatch($filter, $file)) {
267
+                            $result[] = $file;
268
+            }
267 269
         }
268 270
         return $result;
269 271
     }
@@ -390,8 +392,9 @@  discard block
 block discarded – undo
390 392
     {
391 393
         $files_list = static::createFilesList($fileOrFiles);
392 394
 
393
-        if (empty($files_list))
394
-            throw new EmptyFileListException('Files list is empty!');
395
+        if (empty($files_list)) {
396
+                    throw new EmptyFileListException('Files list is empty!');
397
+        }
395 398
 
396 399
         $result = $this->archive->addFiles($files_list);
397 400
         $this->scanArchive();
@@ -410,8 +413,9 @@  discard block
 block discarded – undo
410 413
      */
411 414
     public function addFile($file, $inArchiveName = null)
412 415
     {
413
-        if (!is_file($file))
414
-            throw new InvalidArgumentException($file.' is not a valid file to add in archive');
416
+        if (!is_file($file)) {
417
+                    throw new InvalidArgumentException($file.' is not a valid file to add in archive');
418
+        }
415 419
 
416 420
         return ($inArchiveName !== null
417 421
                 ? $this->addFiles([$inArchiveName => $file])
@@ -444,8 +448,9 @@  discard block
 block discarded – undo
444 448
      */
445 449
     public function addDirectory($directory, $inArchivePath = null)
446 450
     {
447
-        if (!is_dir($directory) || !is_readable($directory))
448
-            throw new InvalidArgumentException($directory.' is not a valid directory to add in archive');
451
+        if (!is_dir($directory) || !is_readable($directory)) {
452
+                    throw new InvalidArgumentException($directory.' is not a valid directory to add in archive');
453
+        }
449 454
 
450 455
         return ($inArchivePath !== null
451 456
                 ? $this->addFiles([$inArchivePath => $directory])
@@ -494,13 +499,15 @@  discard block
 block discarded – undo
494 499
     {
495 500
         $archiveType = Formats::detectArchiveFormat($archiveName, false);
496 501
 
497
-        if ($archiveType === false)
498
-            throw new UnsupportedArchiveException('Could not detect archive type for name "'.$archiveName.'"');
502
+        if ($archiveType === false) {
503
+                    throw new UnsupportedArchiveException('Could not detect archive type for name "'.$archiveName.'"');
504
+        }
499 505
 
500 506
         $files_list = static::createFilesList($fileOrFiles);
501 507
 
502
-        if (empty($files_list))
503
-            throw new EmptyFileListException('Files list is empty!');
508
+        if (empty($files_list)) {
509
+                    throw new EmptyFileListException('Files list is empty!');
510
+        }
504 511
 
505 512
         $totalSize = 0;
506 513
         foreach ($files_list as $fn) {
@@ -554,18 +561,21 @@  discard block
 block discarded – undo
554 561
         $fileProgressCallable = null
555 562
  )
556 563
     {
557
-        if (file_exists($archiveName))
558
-            throw new FileAlreadyExistsException('Archive '.$archiveName.' already exists!');
564
+        if (file_exists($archiveName)) {
565
+                    throw new FileAlreadyExistsException('Archive '.$archiveName.' already exists!');
566
+        }
559 567
 
560 568
         $info = static::prepareForArchiving($fileOrFiles, $archiveName);
561 569
 
562 570
         $abilities = [BasicDriver::CREATE];
563 571
 
564
-        if (!Formats::canCreate($info['type']))
565
-            throw new UnsupportedArchiveException('Unsupported archive type: '.$info['type'].' of archive '.$archiveName);
572
+        if (!Formats::canCreate($info['type'])) {
573
+                    throw new UnsupportedArchiveException('Unsupported archive type: '.$info['type'].' of archive '.$archiveName);
574
+        }
566 575
 
567
-        if ($password !== null && !Formats::canEncrypt($info['type']))
568
-            throw new UnsupportedOperationException('Archive type '.$info['type'].' can not be encrypted');
576
+        if ($password !== null && !Formats::canEncrypt($info['type'])) {
577
+                    throw new UnsupportedOperationException('Archive type '.$info['type'].' can not be encrypted');
578
+        }
569 579
         if ($password !== null) {
570 580
             $abilities[] = BasicDriver::CREATE_ENCRYPTED;
571 581
         }
@@ -619,8 +629,9 @@  discard block
 block discarded – undo
619 629
      */
620 630
     public static function archiveDirectory($directory, $archiveName, $compressionLevel = BasicDriver::COMPRESSION_AVERAGE, $password = null)
621 631
     {
622
-        if (!is_dir($directory) || !is_readable($directory))
623
-            throw new InvalidArgumentException($directory.' is not a valid directory to archive');
632
+        if (!is_dir($directory) || !is_readable($directory)) {
633
+                    throw new InvalidArgumentException($directory.' is not a valid directory to archive');
634
+        }
624 635
 
625 636
         return static::archiveFiles($directory, $archiveName, $compressionLevel, $password) > 0;
626 637
     }
@@ -656,9 +667,9 @@  discard block
 block discarded – undo
656 667
         if (is_array($nodes)) {
657 668
             foreach ($nodes as $destination => $source) {
658 669
                 // new format
659
-                if (is_numeric($destination))
660
-                    $destination = $source;
661
-                else {
670
+                if (is_numeric($destination)) {
671
+                                    $destination = $source;
672
+                } else {
662 673
                     // old format
663 674
                     if (is_string($source) && !file_exists($source)) {
664 675
                         list($destination, $source) = [$source, $destination];
@@ -692,11 +703,12 @@  discard block
 block discarded – undo
692 703
 
693 704
         } else if (is_string($nodes)) { // passed one file or directory
694 705
             // if is directory
695
-            if (is_dir($nodes))
696
-                static::importFilesFromDir(rtrim($nodes, '/\\*').'/*', null, true,
706
+            if (is_dir($nodes)) {
707
+                            static::importFilesFromDir(rtrim($nodes, '/\\*').'/*', null, true,
697 708
                     $files);
698
-            else if (is_file($nodes))
699
-                $files[basename($nodes)] = $nodes;
709
+            } else if (is_file($nodes)) {
710
+                            $files[basename($nodes)] = $nodes;
711
+            }
700 712
         }
701 713
 
702 714
         return $files;
@@ -713,8 +725,9 @@  discard block
 block discarded – undo
713 725
         // $map[$destination] = rtrim($source, '/*');
714 726
         // do not map root archive folder
715 727
 
716
-        if ($destination !== null)
717
-            $map[$destination] = null;
728
+        if ($destination !== null) {
729
+                    $map[$destination] = null;
730
+        }
718 731
 
719 732
         foreach (glob($source, GLOB_MARK) as $node) {
720 733
             if (in_array(substr($node, -1), ['/', '\\'], true) && $recursive) {
Please login to merge, or discard this patch.
src/Formats.php 1 patch
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -160,8 +160,9 @@
 block discarded – undo
160 160
         // by file content
161 161
         if ($contentCheck) {
162 162
             $mime_type = mime_content_type($fileName);
163
-            if (isset(static::$mimeTypes[$mime_type]))
164
-                return static::$mimeTypes[$mime_type];
163
+            if (isset(static::$mimeTypes[$mime_type])) {
164
+                            return static::$mimeTypes[$mime_type];
165
+            }
165 166
         }
166 167
 
167 168
         return false;
Please login to merge, or discard this patch.
src/Drivers/Zip.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
 
29 29
     public static function getDescription()
30 30
     {
31
-        return 'adapter for ext-zip'.(extension_loaded('zip') && defined('\ZipArchive::LIBZIP_VERSION') ? ' ('. ZipArchive::LIBZIP_VERSION.')' : null);
31
+        return 'adapter for ext-zip'.(extension_loaded('zip') && defined('\ZipArchive::LIBZIP_VERSION') ? ' ('.ZipArchive::LIBZIP_VERSION.')' : null);
32 32
     }
33 33
 
34 34
     public static function isInstalled()
Please login to merge, or discard this patch.
Braces   +36 added lines, -24 removed lines patch added patch discarded remove patch
@@ -86,8 +86,9 @@  discard block
 block discarded – undo
86 86
     {
87 87
         parent::__construct($archiveFileName, $format);
88 88
         $this->open($archiveFileName);
89
-        if ($password !== null)
90
-            $this->zip->setPassword($password);
89
+        if ($password !== null) {
90
+                    $this->zip->setPassword($password);
91
+        }
91 92
     }
92 93
 
93 94
     /**
@@ -121,8 +122,9 @@  discard block
 block discarded – undo
121 122
         for ($i = 0; $i < $this->zip->numFiles; $i++) {
122 123
             $file = $this->zip->statIndex($i);
123 124
             // skip directories
124
-            if (in_array(substr($file['name'], -1), ['/', '\\'], true))
125
-                continue;
125
+            if (in_array(substr($file['name'], -1), ['/', '\\'], true)) {
126
+                            continue;
127
+            }
126 128
             $this->pureFilesNumber++;
127 129
             $information->files[$i] = $file['name'];
128 130
             $information->compressedFilesSize += $file['comp_size'];
@@ -157,8 +159,9 @@  discard block
 block discarded – undo
157 159
         for ($i = 0; $i < $this->zip->numFiles; $i++) {
158 160
             $file_name = $this->zip->getNameIndex($i);
159 161
             // skip directories
160
-            if (in_array(substr($file_name, -1), ['/', '\\'], true))
161
-                continue;
162
+            if (in_array(substr($file_name, -1), ['/', '\\'], true)) {
163
+                            continue;
164
+            }
162 165
             $files[] = $file_name;
163 166
         }
164 167
         return $files;
@@ -203,8 +206,9 @@  discard block
 block discarded – undo
203 206
     public function getFileContent($fileName)
204 207
     {
205 208
         $result = $this->zip->getFromName($fileName);
206
-        if ($result === false)
207
-            throw new Exception('Could not get file information: '.$result.'. May use password?');
209
+        if ($result === false) {
210
+                    throw new Exception('Could not get file information: '.$result.'. May use password?');
211
+        }
208 212
         return $result;
209 213
     }
210 214
 
@@ -225,8 +229,9 @@  discard block
 block discarded – undo
225 229
      */
226 230
     public function extractFiles($outputFolder, array $files)
227 231
     {
228
-        if ($this->zip->extractTo($outputFolder, $files) === false)
229
-            throw new ArchiveExtractionException($this->zip->getStatusString(), $this->zip->status);
232
+        if ($this->zip->extractTo($outputFolder, $files) === false) {
233
+                    throw new ArchiveExtractionException($this->zip->getStatusString(), $this->zip->status);
234
+        }
230 235
 
231 236
         return count($files);
232 237
     }
@@ -238,8 +243,9 @@  discard block
 block discarded – undo
238 243
      */
239 244
     public function extractArchive($outputFolder)
240 245
     {
241
-        if ($this->zip->extractTo($outputFolder) === false)
242
-            throw new ArchiveExtractionException($this->zip->getStatusString(), $this->zip->status);
246
+        if ($this->zip->extractTo($outputFolder) === false) {
247
+                    throw new ArchiveExtractionException($this->zip->getStatusString(), $this->zip->status);
248
+        }
243 249
 
244 250
         return $this->pureFilesNumber;
245 251
     }
@@ -254,8 +260,9 @@  discard block
 block discarded – undo
254 260
     {
255 261
         $count = 0;
256 262
         foreach ($files as $file) {
257
-            if ($this->zip->deleteName($file) === false)
258
-                throw new ArchiveModificationException($this->zip->getStatusString(), $this->zip->status);
263
+            if ($this->zip->deleteName($file) === false) {
264
+                            throw new ArchiveModificationException($this->zip->getStatusString(), $this->zip->status);
265
+            }
259 266
             $count++;
260 267
         }
261 268
 
@@ -278,11 +285,13 @@  discard block
 block discarded – undo
278 285
         $added_files = 0;
279 286
         foreach ($files as $localName => $fileName) {
280 287
             if (is_null($fileName)) {
281
-                if ($this->zip->addEmptyDir($localName) === false)
282
-                    throw new ArchiveModificationException($this->zip->getStatusString(), $this->zip->status);
288
+                if ($this->zip->addEmptyDir($localName) === false) {
289
+                                    throw new ArchiveModificationException($this->zip->getStatusString(), $this->zip->status);
290
+                }
283 291
             } else {
284
-                if ($this->zip->addFile($fileName, $localName) === false)
285
-                    throw new ArchiveModificationException($this->zip->getStatusString(), $this->zip->status);
292
+                if ($this->zip->addFile($fileName, $localName) === false) {
293
+                                    throw new ArchiveModificationException($this->zip->getStatusString(), $this->zip->status);
294
+                }
286 295
                 $added_files++;
287 296
             }
288 297
         }
@@ -335,8 +344,9 @@  discard block
 block discarded – undo
335 344
         $zip = new ZipArchive();
336 345
         $result = $zip->open($archiveFileName, ZipArchive::CREATE);
337 346
 
338
-        if ($result !== true)
339
-            throw new ArchiveCreationException('ZipArchive error: '.$result);
347
+        if ($result !== true) {
348
+                    throw new ArchiveCreationException('ZipArchive error: '.$result);
349
+        }
340 350
 
341 351
         $can_set_compression_level = method_exists($zip, 'setCompressionName');
342 352
         $can_encrypt = static::canEncrypt(Formats::ZIP);
@@ -354,11 +364,13 @@  discard block
 block discarded – undo
354 364
 
355 365
         foreach ($files as $localName => $fileName) {
356 366
             if ($fileName === null) {
357
-                if ($zip->addEmptyDir($localName) === false)
358
-                    throw new ArchiveCreationException('Could not archive directory "'.$localName.'": '.$zip->getStatusString(), $zip->status);
367
+                if ($zip->addEmptyDir($localName) === false) {
368
+                                    throw new ArchiveCreationException('Could not archive directory "'.$localName.'": '.$zip->getStatusString(), $zip->status);
369
+                }
359 370
             } else {
360
-                if ($zip->addFile($fileName, $localName) === false)
361
-                    throw new ArchiveCreationException('Could not archive file "'.$fileName.'": '.$zip->getStatusString(), $zip->status);
371
+                if ($zip->addFile($fileName, $localName) === false) {
372
+                                    throw new ArchiveCreationException('Could not archive file "'.$fileName.'": '.$zip->getStatusString(), $zip->status);
373
+                }
362 374
                 if ($can_set_compression_level) {
363 375
                     $zip->setCompressionName($localName, $compressionLevelMap[$compressionLevel]);
364 376
                 }
Please login to merge, or discard this patch.