KnpLabs / Gaufrette

src/Gaufrette/Adapter/Cache.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Gaufrette\Adapter;
 
-use Gaufrette\File;
 use Gaufrette\Adapter;
 use Gaufrette\Adapter\InMemory as InMemoryAdapter;
 

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 use Gaufrette\Adapter;
 use Gaufrette\Adapter\InMemory as InMemoryAdapter;
 
-@trigger_error('The ' . __NAMESPACE__ . '\Cache adapter is deprecated since version 0.4 and will be removed in 1.0.', E_USER_DEPRECATED);
+@trigger_error('The '.__NAMESPACE__.'\Cache adapter is deprecated since version 0.4 and will be removed in 1.0.', E_USER_DEPRECATED);
 
 /**
  * Cache adapter.

src/Gaufrette/Adapter/DoctrineDbal.php

Doc Comments +4 added lines

@@ -152,6 +152,10 @@
         return false;
     }
 
+    /**
+     * @param string $key
+     * @param string $column
+     */
     private function getColumnValue($key, $column)
     {
         $value = $this->connection->fetchColumn(

Spacing +1 added lines, -1 removed lines

@@ -184,7 +184,7 @@
 
         return [
             'dirs' => [],
-            'keys' => array_map(function ($value) {
+            'keys' => array_map(function($value) {
                 return $value['_key'];
             },
                 $keys),

src/Gaufrette/Adapter/Dropbox.php

Doc Comments +3 added lines

@@ -162,6 +162,9 @@
         }
     }
 
+    /**
+     * @param string $key
+     */
     private function getDropboxMetadata($key)
     {
         try {

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 use Dropbox_API as DropboxApi;
 use Dropbox_Exception_NotFound as DropboxNotFoundException;
 
-@trigger_error('The ' . __NAMESPACE__ . '\Dropbox adapter is deprecated since version 0.4 and will be removed in 1.0. You can move to our Flysystem adapter and use their Dropbox adapter if needed.', E_USER_DEPRECATED);
+@trigger_error('The '.__NAMESPACE__.'\Dropbox adapter is deprecated since version 0.4 and will be removed in 1.0. You can move to our Flysystem adapter and use their Dropbox adapter if needed.', E_USER_DEPRECATED);
 
 /**
  * Dropbox adapter.

src/Gaufrette/Util/Path.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
             }
         }
 
-        return $prefix . implode('/', $tokens);
+        return $prefix.implode('/', $tokens);
     }
 
     /**

src/Gaufrette/Adapter/Local.php

Spacing +1 added lines, -1 removed lines

@@ -205,7 +205,7 @@
     {
         $this->ensureDirectoryExists($this->directory, $this->create);
 
-        return $this->normalizePath($this->directory . '/' . $key);
+        return $this->normalizePath($this->directory.'/'.$key);
     }
 
     /**

src/Gaufrette/Adapter/Flysystem.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
      */
     public function keys()
     {
-        return array_map(function ($content) {
+        return array_map(function($content) {
             return $content['path'];
         }, $this->adapter->listContents());
     }

src/Gaufrette/Adapter/Apc.php

Doc Comments +1 added lines, -1 removed lines

@@ -138,7 +138,7 @@
      * @param string $key    - by default ''
      * @param int    $format - by default APC_ITER_NONE
      *
-     * @return \APCIterator
+     * @return \Traversable
      */
     protected function getCachedKeysIterator($key = '', $format = APC_ITER_NONE)
     {

Spacing +3 added lines, -3 removed lines

@@ -5,7 +5,7 @@ 
 use Gaufrette\Adapter;
 use Gaufrette\Util;
 
-@trigger_error('The ' . __NAMESPACE__ . '\Apc adapter is deprecated since version 0.4 and will be removed in 1.0.', E_USER_DEPRECATED);
+@trigger_error('The '.__NAMESPACE__.'\Apc adapter is deprecated since version 0.4 and will be removed in 1.0.', E_USER_DEPRECATED);
 
 /**
  * Apc adapter, a non-persistent adapter for when this sort of thing is appropriate.
@@ -135,7 +135,7 @@ 
      */
     public function computePath($key)
     {
-        return $this->prefix . $key;
+        return $this->prefix.$key;
     }
 
     /**
@@ -146,7 +146,7 @@ 
      */
     protected function getCachedKeysIterator($key = '', $format = APC_ITER_NONE)
     {
-        $pattern = sprintf('/^%s/', preg_quote($this->prefix . $key, '/'));
+        $pattern = sprintf('/^%s/', preg_quote($this->prefix.$key, '/'));
 
         return new \APCIterator('user', $pattern, $format);
     }

src/Gaufrette/Adapter/MogileFS.php

Doc Comments +3 added lines, -3 removed lines

@@ -250,7 +250,7 @@ 
     /**
      * Makes request to MogileFS tracker.
      *
-     * @param cmd Command
+     * @param cmd string
      * @param args Array of arguments
      *
      * @return mixed Array on success, false on failure
@@ -302,7 +302,7 @@ 
     /**
      * Get file location at server from MogileFS tracker.
      *
-     * @param key File key
+     * @param key string key
      *
      * @return mixed Array on success, false on failure
      */
@@ -318,7 +318,7 @@ 
      * Sends file to MogileFS tracker.
      *
      * @param path Save path at server
-     * @param data Data to save
+     * @param data string to save
      *
      * @return bool
      */

Spacing +8 added lines, -8 removed lines

@@ -5,7 +5,7 @@ 
 use Gaufrette\Adapter;
 use Gaufrette\Util;
 
-@trigger_error('The ' . __NAMESPACE__ . '\MogileFS adapter is deprecated since version 0.4 and will be removed in 1.0.', E_USER_DEPRECATED);
+@trigger_error('The '.__NAMESPACE__.'\MogileFS adapter is deprecated since version 0.4 and will be removed in 1.0.', E_USER_DEPRECATED);
 
 /**
  * Adapter for the MogileFS filesystem.
@@ -195,12 +195,12 @@ 
         $domains = [];
 
         for ($i = 1; $i <= $res['domains']; ++$i) {
-            $dom = 'domain' . $i;
+            $dom = 'domain'.$i;
             $classes = [];
 
             // Associate classes to current domain (class name => mindevcount)
-            for ($j = 1; $j <= $res[$dom . 'classes']; ++$j) {
-                $classes[$res[$dom . 'class' . $j . 'name']] = $res[$dom . 'class' . $j . 'mindevcount'];
+            for ($j = 1; $j <= $res[$dom.'classes']; ++$j) {
+                $classes[$res[$dom.'class'.$j.'name']] = $res[$dom.'class'.$j.'mindevcount'];
             }
 
             $domains[] = ['name' => $res[$dom], 'classes' => $classes];
@@ -347,11 +347,11 @@ 
         stream_set_blocking($fp, true);
         stream_set_timeout($fp, 30, 200000);
 
-        $out = 'PUT ' . $url['path'] . ' HTTP/1.1' . $b;
-        $out .= 'Host: ' . $url['host'] . $b;
-        $out .= 'Content-Length: ' . Util\Size::fromContent($data) . $b . $b;
+        $out = 'PUT '.$url['path'].' HTTP/1.1'.$b;
+        $out .= 'Host: '.$url['host'].$b;
+        $out .= 'Content-Length: '.Util\Size::fromContent($data).$b.$b;
         $out .= $data;
-        $out .= $b . $b;
+        $out .= $b.$b;
         fwrite($fp, $out);
         fflush($fp);
 

src/Gaufrette/Adapter/Zip.php

Doc Comments +1 added lines, -1 removed lines

@@ -135,7 +135,7 @@
      * Returns the stat of a file in the zip archive
      *  (name, index, crc, mtime, compression size, compression method, filesize).
      *
-     * @param $key
+     * @param string $key
      *
      * @return array|bool
      */

Switch Indentation +30 added lines, -30 removed lines

@@ -166,46 +166,46 @@
 
         if (true !== ($resultCode = $this->zipArchive->open($this->zipFile, ZipArchive::CREATE))) {
             switch ($resultCode) {
-            case ZipArchive::ER_EXISTS:
-                $errMsg = 'File already exists.';
+                case ZipArchive::ER_EXISTS:
+                    $errMsg = 'File already exists.';
 
-                break;
-            case ZipArchive::ER_INCONS:
-                $errMsg = 'Zip archive inconsistent.';
+                    break;
+                case ZipArchive::ER_INCONS:
+                    $errMsg = 'Zip archive inconsistent.';
 
-                break;
-            case ZipArchive::ER_INVAL:
-                $errMsg = 'Invalid argument.';
+                    break;
+                case ZipArchive::ER_INVAL:
+                    $errMsg = 'Invalid argument.';
 
-                break;
-            case ZipArchive::ER_MEMORY:
-                $errMsg = 'Malloc failure.';
+                    break;
+                case ZipArchive::ER_MEMORY:
+                    $errMsg = 'Malloc failure.';
 
-                break;
-            case ZipArchive::ER_NOENT:
-                $errMsg = 'Invalid argument.';
+                    break;
+                case ZipArchive::ER_NOENT:
+                    $errMsg = 'Invalid argument.';
 
-                break;
-            case ZipArchive::ER_NOZIP:
-                $errMsg = 'Not a zip archive.';
+                    break;
+                case ZipArchive::ER_NOZIP:
+                    $errMsg = 'Not a zip archive.';
 
-                break;
-            case ZipArchive::ER_OPEN:
-                $errMsg = 'Can\'t open file.';
+                    break;
+                case ZipArchive::ER_OPEN:
+                    $errMsg = 'Can\'t open file.';
 
-                break;
-            case ZipArchive::ER_READ:
-                $errMsg = 'Read error.';
+                    break;
+                case ZipArchive::ER_READ:
+                    $errMsg = 'Read error.';
 
-                break;
-            case ZipArchive::ER_SEEK:
-                $errMsg = 'Seek error.';
+                    break;
+                case ZipArchive::ER_SEEK:
+                    $errMsg = 'Seek error.';
 
-                break;
-            default:
-                $errMsg = 'Unknown error.';
+                    break;
+                default:
+                    $errMsg = 'Unknown error.';
 
-                break;
+                    break;
             }
 
             throw new \RuntimeException(sprintf('%s', $errMsg));