KnpLabs / Gaufrette

src/Gaufrette/Adapter/Apc.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +2 added lines, -2 removed lines

@@ -109,7 +109,7 @@ 
     public function rename($sourceKey, $targetKey)
     {
         // TODO: this probably allows for race conditions...
-        $written  = $this->write($targetKey, $this->read($sourceKey));
+        $written = $this->write($targetKey, $this->read($sourceKey));
         $deleted = $this->delete($sourceKey);
 
         return $written && $deleted;
@@ -131,7 +131,7 @@ 
      */
     public function computePath($key)
     {
-        return $this->prefix . $key;
+        return $this->prefix.$key;
     }
 
     /**

src/Gaufrette/Adapter/AwsS3.php

Doc Comments +6 added lines

@@ -272,6 +272,9 @@ 
         return true;
     }
 
+    /**
+     * @param string $key
+     */
     protected function getOptions($key, array $options = array())
     {
         $options['ACL'] = $this->options['acl'];
@@ -287,6 +290,9 @@ 
         return $options;
     }
 
+    /**
+     * @return string
+     */
     protected function computePath($key)
     {
         if (empty($this->options['directory'])) {

Indentation +3 added lines, -3 removed lines

@@ -12,9 +12,9 @@
  * @author  Michael Dowling <mtdowling@gmail.com>
  */
 class AwsS3 implements Adapter,
-                       MetadataSupporter,
-                       ListKeysAware,
-                       SizeCalculator
+                        MetadataSupporter,
+                        ListKeysAware,
+                        SizeCalculator
 {
     protected $service;
     protected $bucket;

Spacing +1 added lines, -1 removed lines

@@ -228,7 +228,7 @@
     {
         $result = $this->service->listObjects(array(
             'Bucket'  => $this->bucket,
-            'Prefix'  => rtrim($this->computePath($key), '/') . '/',
+            'Prefix'  => rtrim($this->computePath($key), '/').'/',
             'MaxKeys' => 1
         ));
 

src/Gaufrette/Adapter/AzureBlobStorage.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Gaufrette\Adapter;
 use Gaufrette\Util;
 use Gaufrette\Adapter\AzureBlobStorage\BlobProxyFactoryInterface;
-
 use WindowsAzure\Blob\Models\CreateBlobOptions;
 use WindowsAzure\Blob\Models\CreateContainerOptions;
 use WindowsAzure\Blob\Models\DeleteContainerOptions;

Indentation +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
  * @author Paweł Czyżewski <pawel.czyzewski@enginewerk.com>
  */
 class AzureBlobStorage implements Adapter,
-                                  MetadataSupporter
+                                    MetadataSupporter
 {
     /**
      * Error constants

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;
 

Indentation +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
  * @author  Antoine Hérault <antoine.herault@gmail.com>
  */
 class Cache implements Adapter,
-                       MetadataSupporter
+                        MetadataSupporter
 {
     /**
      * @var 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(

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Gaufrette\Adapter;
 use Gaufrette\Util;
-
 use Doctrine\DBAL\Connection;
 
 /**

Indentation +2 added lines, -2 removed lines

@@ -15,8 +15,8 @@
  * @author Leszek Prabucki <leszek.prabucki@gmail.com>
  */
 class DoctrineDbal implements Adapter,
-                              ChecksumCalculator,
-                              ListKeysAware
+                                ChecksumCalculator,
+                                ListKeysAware
 {
     protected $connection;
     protected $table;

Spacing +1 added lines, -1 removed lines

@@ -186,7 +186,7 @@
 
         return array(
             'dirs' => array(),
-            '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

@@ -131,7 +131,7 @@
     public function keys()
     {
         $metadata = $this->client->getMetaData('/', true);
-        if (! isset($metadata['contents'])) {
+        if (!isset($metadata['contents'])) {
             return array();
         }
 

src/Gaufrette/Adapter/Ftp.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use Gaufrette\Adapter;
 use Gaufrette\File;
 use Gaufrette\Filesystem;
-use Gaufrette\Exception;
 
 /**
  * Ftp adapter

Indentation +4 added lines, -4 removed lines

@@ -14,8 +14,8 @@ 
  * @author  Antoine Hérault <antoine.herault@gmail.com>
  */
 class Ftp implements Adapter,
-                     FileFactory,
-                     ListKeysAware
+                        FileFactory,
+                        ListKeysAware
 {
     protected $connection = null;
     protected $directory;
@@ -263,8 +263,8 @@ 
         $this->fileData = array_merge($fileData, $this->fileData);
 
         return array(
-           'keys'   => array_keys($fileData),
-           'dirs'   => $dirs
+            'keys'   => array_keys($fileData),
+            'dirs'   => $dirs
         );
     }
 

Spacing +13 added lines, -13 removed lines

@@ -123,7 +123,7 @@ 
         $this->ensureDirectoryExists($this->directory, $this->create);
 
         $file  = $this->computePath($key);
-        $lines = ftp_rawlist($this->getConnection(), '-al ' . str_replace('\\', '/', dirname($file)));
+        $lines = ftp_rawlist($this->getConnection(), '-al '.str_replace('\\', '/', dirname($file)));
 
         if (false === $lines) {
             return false;
@@ -236,7 +236,7 @@ 
         $directory = preg_replace('/^[\/]*([^\/].*)$/', '/$1', $directory);
 
         $items = $this->parseRawlist(
-            ftp_rawlist($this->getConnection(), '-al ' . $this->directory . $directory ) ? : array()
+            ftp_rawlist($this->getConnection(), '-al '.$this->directory.$directory) ?: array()
         );
 
         $fileData = $dirs = array();
@@ -248,7 +248,7 @@ 
 
             $item = array(
                 'name'  => $itemData['name'],
-                'path'  => trim(($directory ? $directory . '/' : '') . $itemData['name'], '/'),
+                'path'  => trim(($directory ? $directory.'/' : '').$itemData['name'], '/'),
                 'time'  => $itemData['time'],
                 'size'  => $itemData['size'],
             );
@@ -360,13 +360,13 @@ 
     {
         $directory = preg_replace('/^[\/]*([^\/].*)$/', '/$1', $directory);
 
-        $lines = ftp_rawlist($this->getConnection(), '-alR '. $this->directory . $directory);
+        $lines = ftp_rawlist($this->getConnection(), '-alR '.$this->directory.$directory);
 
         if (false === $lines) {
             return array('keys' => array(), 'dirs' => array());
         }
 
-        $regexDir = '/'.preg_quote($this->directory . $directory, '/').'\/?(.+):$/u';
+        $regexDir = '/'.preg_quote($this->directory.$directory, '/').'\/?(.+):$/u';
         $regexItem = '/^(?:([d\-\d])\S+)\s+\S+(?:(?:\s+\S+){5})?\s+(\S+)\s+(.+?)$/';
 
         $prevLine = null;
@@ -390,7 +390,7 @@ 
                     continue;
                 }
 
-                $path = ltrim($directory . '/' . $name, '/');
+                $path = ltrim($directory.'/'.$name, '/');
 
                 if ('d' === $tokens[1] || '<dir>' === $tokens[2]) {
                     $keys['dirs'][] = $path;
@@ -430,13 +430,13 @@ 
             $infos = preg_split("/[\s]+/", $line, 9);
 
             if ($this->isLinuxListing($infos)) {
-                $infos[7] = (strrpos($infos[7], ':') != 2 ) ? ($infos[7] . ' 00:00') : (date('Y') . ' ' . $infos[7]);
+                $infos[7] = (strrpos($infos[7], ':') != 2) ? ($infos[7].' 00:00') : (date('Y').' '.$infos[7]);
                 if ('total' !== $infos[0]) {
                     $parsed[] = array(
                         'perms' => $infos[0],
                         'num'   => $infos[1],
                         'size'  => $infos[4],
-                        'time'  => strtotime($infos[5] . ' ' . $infos[6] . '. ' . $infos[7]),
+                        'time'  => strtotime($infos[5].' '.$infos[6].'. '.$infos[7]),
                         'name'  => $infos[8]
                     );
                 }
@@ -446,7 +446,7 @@ 
                     'perms' => $isDir ? 'd' : '-',
                     'num'   => '',
                     'size'  => $isDir ? '' : $infos[2],
-                    'time'  => strtotime($infos[0] . ' ' . $infos[1]),
+                    'time'  => strtotime($infos[0].' '.$infos[1]),
                     'name'  => $infos[3]
                 );
             }
@@ -462,7 +462,7 @@ 
      */
     private function computePath($key)
     {
-        return rtrim($this->directory, '/') . '/' . $key;
+        return rtrim($this->directory, '/').'/'.$key;
     }
 
     /**
@@ -501,7 +501,7 @@ 
         if (!$this->ssl) {
             $this->connection = ftp_connect($this->host, $this->port);
         } else {
-            if(function_exists('ftp_ssl_connect')) {
+            if (function_exists('ftp_ssl_connect')) {
                 $this->connection = ftp_ssl_connect($this->host, $this->port);        
             } else {
                 throw new \RuntimeException('This Server Has No SSL-FTP Available.');
@@ -511,8 +511,8 @@ 
             throw new \RuntimeException(sprintf('Could not connect to \'%s\' (port: %s).', $this->host, $this->port));
         }
 
-        $username = $this->username ? : 'anonymous';
-        $password = $this->password ? : '';
+        $username = $this->username ?: 'anonymous';
+        $password = $this->password ?: '';
 
         // login ftp user
         if (!@ftp_login($this->connection, $username, $password)) {

src/Gaufrette/Adapter/GoogleCloudStorage.php

Doc Comments +3 added lines

@@ -349,6 +349,9 @@
         }
     }
 
+    /**
+     * @param string $key
+     */
     protected function computePath($key)
     {
         if (empty($this->options['directory'])) {

Spacing +2 added lines, -2 removed lines

@@ -261,7 +261,7 @@ 
      */
     public function isDirectory($key)
     {
-        if ($this->exists($key . '/')) {
+        if ($this->exists($key.'/')) {
             return true;
         }
 
@@ -276,7 +276,7 @@ 
         $this->ensureBucketExists();
 
         $options = array();
-        if ((string)$prefix != '') {
+        if ((string) $prefix != '') {
             $options['prefix'] = $this->computePath($prefix);
         } elseif (!empty($this->options['directory'])) {
             $options['prefix'] = $this->options['directory'];

src/Gaufrette/Adapter/GridFS.php

Doc Comments +3 added lines

@@ -145,6 +145,9 @@
         return isset($this->metadata[$key]) ? $this->metadata[$key] : array();
     }
 
+    /**
+     * @param string $key
+     */
     private function find($key, array $fields = array())
     {
         return $this->gridFS->findOne($key, $fields);