alchemy-fr / Zippy

src/Adapter/AbstractBinaryAdapter.php

Doc Comments +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
      * @param ParserInterface $parser An output parser
      * @param ResourceManager $manager A resource manager
      * @param ProcessBuilderFactoryInterface $inflator A process builder factory for the inflator binary
-     * @param ProcessBuilderFactoryInterface|null $deflator A process builder factory for the deflator binary
+     * @param ProcessBuilderFactoryInterface $deflator A process builder factory for the deflator binary
      */
     public function __construct(
         ParserInterface $parser,

Spacing +3 added lines, -4 removed lines

@@ -181,7 +181,7 @@ 
 
     private static function findABinary($wish, array $defaults, ExecutableFinder $finder)
     {
-        $possibles = $wish ? (array)$wish : $defaults;
+        $possibles = $wish ? (array) $wish : $defaults;
 
         $binary = null;
 
@@ -207,11 +207,10 @@ 
     {
         $iterations = 0;
 
-        array_walk($files, function ($file) use ($builder, &$iterations) {
+        array_walk($files, function($file) use ($builder, &$iterations) {
             $builder->add(
                 $file instanceof \SplFileInfo ?
-                    $file->getRealpath() :
-                    ($file instanceof MemberInterface ? $file->getLocation() : $file)
+                    $file->getRealpath() : ($file instanceof MemberInterface ? $file->getLocation() : $file)
             );
 
             $iterations++;

src/Adapter/AbstractTarAdapter.php

Doc Comments +3 added lines

@@ -345,6 +345,9 @@
         return new \SplFileInfo($to ? : $resource->getResource());
     }
 
+    /**
+     * @param string $to
+     */
     protected function doTarExtractMembers($options, ResourceInterface $resource, $members, $to = null, $overwrite = false)
     {
         if (null !== $to && !is_dir($to)) {

Spacing +6 added lines, -6 removed lines

@@ -89,7 +89,7 @@ 
             ));
         }
 
-        return $this->parser->parseInflatorVersion($process->getOutput() ? : '');
+        return $this->parser->parseInflatorVersion($process->getOutput() ?: '');
     }
 
     /**
@@ -124,7 +124,7 @@ 
             $builder->add('-f');
             $builder->add($path);
             $builder->add('-T');
-            $builder->add( $nullFile);
+            $builder->add($nullFile);
 
             $process = $builder->getProcess();
             $process->run();
@@ -141,7 +141,7 @@ 
 
             $builder->setWorkingDirectory($collection->getContext());
 
-            $collection->forAll(function ($i, Resource $resource) use ($builder) {
+            $collection->forAll(function($i, Resource $resource) use ($builder) {
                 return $builder->add($resource->getTarget());
             });
 
@@ -200,7 +200,7 @@ 
 
         $members = array();
 
-        foreach ($this->parser->parseFileListing($process->getOutput() ? : '') as $member) {
+        foreach ($this->parser->parseFileListing($process->getOutput() ?: '') as $member) {
             $members[] = new Member(
                     $resource,
                     $this,
@@ -240,7 +240,7 @@ 
 
         $builder->setWorkingDirectory($collection->getContext());
 
-        $collection->forAll(function ($i, Resource $resource) use ($builder) {
+        $collection->forAll(function($i, Resource $resource) use ($builder) {
             return $builder->add($resource->getTarget());
         });
 
@@ -342,7 +342,7 @@ 
             ));
         }
 
-        return new \SplFileInfo($to ? : $resource->getResource());
+        return new \SplFileInfo($to ?: $resource->getResource());
     }
 
     protected function doTarExtractMembers($options, ResourceInterface $resource, $members, $to = null, $overwrite = false)

src/Adapter/ZipExtensionAdapter.php

Doc Comments +2 added lines

@@ -301,6 +301,7 @@ 
 
     /**
      * @info is public for PHP 5.3 compatibility, should be private
+     * @param string $file
      */
     public function checkReadability(\ZipArchive $zip, $file)
     {
@@ -314,6 +315,7 @@ 
 
     /**
      * @info is public for PHP 5.3 compatibility, should be private
+     * @param string $file
      */
     public function addFileToZip(\ZipArchive $zip, $file)
     {

Spacing +2 added lines, -2 removed lines

@@ -251,11 +251,11 @@
         $adapter = $this;
 
         try {
-            $collection->forAll(function ($i, Resource $resource) use ($zipresource, $stack, $recursive, $adapter) {
+            $collection->forAll(function($i, Resource $resource) use ($zipresource, $stack, $recursive, $adapter) {
                 $adapter->checkReadability($zipresource->getResource(), $resource->getTarget());
                 if (is_dir($resource->getTarget())) {
                     if ($recursive) {
-                        $stack->push($resource->getTarget() . ((substr($resource->getTarget(), -1) === DIRECTORY_SEPARATOR) ? '' : DIRECTORY_SEPARATOR ));
+                        $stack->push($resource->getTarget() . ((substr($resource->getTarget(), -1) === DIRECTORY_SEPARATOR) ? '' : DIRECTORY_SEPARATOR));
                     } else {
                         $adapter->addEmptyDir($zipresource->getResource(), $resource->getTarget());
                     }

src/Parser/ParserFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     /**
      * Maps the corresponding parser to the selected adapter
      *
-     * @param   $adapterName An adapter name
+     * @param   string $adapterName An adapter name
      *
      * @return ParserInterface
      *

src/Adapter/AdapterContainer.php

Spacing +15 added lines, -15 removed lines

@@ -42,7 +42,7 @@ 
         $container['zip.inflator'] = null;
         $container['zip.deflator'] = null;
 
-        $container['resource-manager'] = function ($container) {
+        $container['resource-manager'] = function($container) {
             return new ResourceManager(
                 $container['request-mapper'],
                 $container['resource-teleporter'],
@@ -50,31 +50,31 @@ 
             );
         };
 
-        $container['executable-finder'] = function ($container) {
+        $container['executable-finder'] = function($container) {
             return new ExecutableFinder();
         };
 
-        $container['request-mapper'] = function ($container) {
+        $container['request-mapper'] = function($container) {
             return new RequestMapper($container['target-locator']);
         };
 
-        $container['target-locator'] = function () {
+        $container['target-locator'] = function() {
             return new TargetLocator();
         };
 
-        $container['teleporter-container'] = function ($container) {
+        $container['teleporter-container'] = function($container) {
             return TeleporterContainer::load();
         };
 
-        $container['resource-teleporter'] = function ($container) {
+        $container['resource-teleporter'] = function($container) {
             return new ResourceTeleporter($container['teleporter-container']);
         };
 
-        $container['filesystem'] = function () {
+        $container['filesystem'] = function() {
             return new Filesystem();
         };
 
-        $container['Alchemy\\Zippy\\Adapter\\ZipAdapter'] = function ($container) {
+        $container['Alchemy\\Zippy\\Adapter\\ZipAdapter'] = function($container) {
             return ZipAdapter::newInstance(
                 $container['executable-finder'],
                 $container['resource-manager'],
@@ -86,7 +86,7 @@ 
         $container['gnu-tar.inflator'] = null;
         $container['gnu-tar.deflator'] = null;
 
-        $container['Alchemy\\Zippy\\Adapter\\GNUTar\\TarGNUTarAdapter'] = function ($container) {
+        $container['Alchemy\\Zippy\\Adapter\\GNUTar\\TarGNUTarAdapter'] = function($container) {
             return TarGNUTarAdapter::newInstance(
                 $container['executable-finder'],
                 $container['resource-manager'],
@@ -95,7 +95,7 @@ 
             );
         };
 
-        $container['Alchemy\\Zippy\\Adapter\\GNUTar\\TarGzGNUTarAdapter'] = function ($container) {
+        $container['Alchemy\\Zippy\\Adapter\\GNUTar\\TarGzGNUTarAdapter'] = function($container) {
             return TarGzGNUTarAdapter::newInstance(
                 $container['executable-finder'],
                 $container['resource-manager'],
@@ -104,7 +104,7 @@ 
             );
         };
 
-        $container['Alchemy\\Zippy\\Adapter\\GNUTar\\TarBz2GNUTarAdapter'] = function ($container) {
+        $container['Alchemy\\Zippy\\Adapter\\GNUTar\\TarBz2GNUTarAdapter'] = function($container) {
             return TarBz2GNUTarAdapter::newInstance(
                 $container['executable-finder'],
                 $container['resource-manager'],
@@ -116,7 +116,7 @@ 
         $container['bsd-tar.inflator'] = null;
         $container['bsd-tar.deflator'] = null;
 
-        $container['Alchemy\\Zippy\\Adapter\\BSDTar\\TarBSDTarAdapter'] = function ($container) {
+        $container['Alchemy\\Zippy\\Adapter\\BSDTar\\TarBSDTarAdapter'] = function($container) {
             return TarBSDTarAdapter::newInstance(
                 $container['executable-finder'],
                 $container['resource-manager'],
@@ -125,7 +125,7 @@ 
             );
         };
 
-        $container['Alchemy\\Zippy\\Adapter\\BSDTar\\TarGzBSDTarAdapter'] = function ($container) {
+        $container['Alchemy\\Zippy\\Adapter\\BSDTar\\TarGzBSDTarAdapter'] = function($container) {
             return TarGzBSDTarAdapter::newInstance(
                 $container['executable-finder'],
                 $container['resource-manager'],
@@ -134,7 +134,7 @@ 
             );
         };
 
-        $container['Alchemy\\Zippy\\Adapter\\BSDTar\\TarBz2BSDTarAdapter'] = function ($container) {
+        $container['Alchemy\\Zippy\\Adapter\\BSDTar\\TarBz2BSDTarAdapter'] = function($container) {
             return TarBz2BSDTarAdapter::newInstance(
                 $container['executable-finder'],
                 $container['resource-manager'],
@@ -142,7 +142,7 @@ 
                 $container['bsd-tar.deflator']);
         };
 
-        $container['Alchemy\\Zippy\\Adapter\\ZipExtensionAdapter'] = function () {
+        $container['Alchemy\\Zippy\\Adapter\\ZipExtensionAdapter'] = function() {
             return ZipExtensionAdapter::newInstance();
         };
 

src/Adapter/ZipAdapter.php

Spacing +6 added lines, -6 removed lines

@@ -47,7 +47,7 @@ 
      */
     protected function doCreate($path, $files, $recursive)
     {
-        $files = (array)$files;
+        $files = (array) $files;
 
         $builder = $this
             ->inflator
@@ -66,7 +66,7 @@ 
         $collection = $this->manager->handle(getcwd(), $files);
         $builder->setWorkingDirectory($collection->getContext());
 
-        $collection->forAll(function ($i, Resource $resource) use ($builder) {
+        $collection->forAll(function($i, Resource $resource) use ($builder) {
             return $builder->add($resource->getTarget());
         });
 
@@ -135,7 +135,7 @@ 
      */
     protected function doAdd(ResourceInterface $resource, $files, $recursive)
     {
-        $files = (array)$files;
+        $files = (array) $files;
 
         $builder = $this
             ->inflator
@@ -153,7 +153,7 @@ 
 
         $builder->setWorkingDirectory($collection->getContext());
 
-        $collection->forAll(function ($i, Resource $resource) use ($builder) {
+        $collection->forAll(function($i, Resource $resource) use ($builder) {
             return $builder->add($resource->getTarget());
         });
 
@@ -230,7 +230,7 @@ 
      */
     protected function doRemove(ResourceInterface $resource, $files)
     {
-        $files = (array)$files;
+        $files = (array) $files;
 
         $builder = $this
             ->inflator
@@ -330,7 +330,7 @@ 
             throw new InvalidArgumentException(sprintf("%s is not a directory", $to));
         }
 
-        $members = (array)$members;
+        $members = (array) $members;
 
         $builder = $this
             ->deflator

src/Archive/Archive.php

Indentation +3 added lines, -3 removed lines

@@ -117,12 +117,12 @@
     /**
      * @inheritdoc
      */
-     public function extract($toDirectory)
-     {
+        public function extract($toDirectory)
+        {
         $this->adapter->extract($this->resource, $toDirectory);
 
         return $this;
-     }
+        }
 
     /**
      * @inheritdoc

src/Archive/Member.php

Spacing +1 added lines, -1 removed lines

@@ -132,7 +132,7 @@
      */
     public function extract($to = null, $overwrite = false)
     {
-        $this->adapter->extractMembers($this->resource, $this->location, $to, (bool)$overwrite);
+        $this->adapter->extractMembers($this->resource, $this->location, $to, (bool) $overwrite);
 
         return new \SplFileInfo(sprintf('%s%s', rtrim(null === $to ? getcwd() : $to, '/'), $this->location));
     }

src/Parser/BSDTarOutputParser.php

Indentation +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
                 throw new RuntimeException(sprintf('Failed to parse mtime date from %s', $line));
             }
 
-             $members[] = array(
+                $members[] = array(
                 'location'  => $chunks[7],
                 'size'      => $chunks[5],
                 'mtime'     => $date,

Spacing +7 added lines, -7 removed lines

@@ -44,13 +44,13 @@
             // drw-rw-r--  0 toto titi     0 Jan  3  1980 practice/
             // -rw-rw-r--  0 toto titi     10240 Jan 22 13:31 practice/records
             if (!preg_match_all("#" .
-                self::PERMISSIONS   . "\s+" .   // match (drw-r--r--)
-                self::HARD_LINK     . "\s+" .   // match (1)
-                self::OWNER         . "\s" .    // match (toto)
-                self::GROUP         . "\s+" .   // match (titi)
-                self::FILESIZE      . "\s+" .   // match (0)
-                self::DATE          . "\s+" .   // match (Jan  3  1980)
-                self::FILENAME      .           // match (practice)
+                self::PERMISSIONS . "\s+" . // match (drw-r--r--)
+                self::HARD_LINK . "\s+" . // match (1)
+                self::OWNER . "\s" . // match (toto)
+                self::GROUP . "\s+" . // match (titi)
+                self::FILESIZE . "\s+" . // match (0)
+                self::DATE . "\s+" . // match (Jan  3  1980)
+                self::FILENAME . // match (practice)
                 "#", $line, $matches, PREG_SET_ORDER
             )) {
                 continue;