KnpLabs / Gaufrette

src/Gaufrette/Adapter/OpenCloud.php

Doc Comments +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
      *
      * @param string $key
      *
-     * @return string|boolean if cannot read content
+     * @return string if cannot read content
      */
     public function read($key)
     {
@@ -159,7 +159,7 @@ 
      *
      * @param string $key
      *
-     * @return integer|boolean An UNIX like timestamp or false
+     * @return string|false An UNIX like timestamp or false
      */
     public function mtime($key)
     {

Indentation +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
  * @author  Daniel Richter <nexyz9@gmail.com>
  */
 class OpenCloud implements Adapter,
-                           ChecksumCalculator
+                            ChecksumCalculator
 {
     /**
      * @var Service

Spacing +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
     public function keys()
     {
         $objectList = $this->getContainer()->objectList();
-        $keys = array ();
+        $keys = array();
 
         while ($object = $objectList->next()) {
             $keys[] = $object->getName();

Braces +3 added lines, -6 removed lines

@@ -109,8 +109,7 @@ 
     {
         try {
             $object = $this->getContainer()->uploadObject($key, $content);
-        }
-        catch (CreateUpdateError $updateError) {
+        } catch (CreateUpdateError $updateError) {
             return false;
         }
 
@@ -185,8 +184,7 @@ 
 
         try {
             $object->delete();
-        }
-        catch (DeleteError $deleteError) {
+        } catch (DeleteError $deleteError) {
             return false;
         }
 
@@ -249,8 +247,7 @@ 
     {
         try {
             return $this->getContainer()->getObject($key);
-        }
-        catch (ObjectNotFoundException $objFetchError) {
+        } catch (ObjectNotFoundException $objFetchError) {
             return false;
         }
     }

src/Gaufrette/Adapter/PhpseclibSftp.php

Doc Comments +7 added lines

@@ -187,6 +187,10 @@ 
         $this->initialized = true;
     }
 
+    /**
+     * @param string|null $directory
+     * @param boolean $create
+     */
     protected function ensureDirectoryExists($directory, $create)
     {
         $pwd = $this->sftp->pwd();
@@ -201,6 +205,9 @@ 
         }
     }
 
+    /**
+     * @param string $key
+     */
     protected function computePath($key)
     {
         return $this->directory.'/'.ltrim($key, '/');

Indentation +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
 use Gaufrette\File;
 
 class PhpseclibSftp implements Adapter,
-                               FileFactory,
-                               ListKeysAware
+                                FileFactory,
+                                ListKeysAware
 {
     protected $sftp;
     protected $directory;

src/Gaufrette/Adapter/Zip.php

Doc Comments +1 added lines, -1 removed lines

@@ -136,7 +136,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
      */
     public function getStat($key)

Switch Indentation +30 added lines, -30 removed lines

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

src/Gaufrette/Filesystem.php

Doc Comments +2 added lines, -2 removed lines

@@ -268,7 +268,7 @@ 
     /**
      * Gets a new stream instance of the specified file.
      *
-     * @param $key
+     * @param string $key
      * @return Stream|Stream\InMemoryBuffer
      */
     public function createStream($key)
@@ -283,7 +283,7 @@ 
     /**
      * Creates a new file in a filesystem.
      *
-     * @param $key
+     * @param string $key
      * @return File
      */
     public function createFile($key)

Spacing +4 added lines, -4 removed lines

@@ -72,11 +72,11 @@ 
             throw new Exception\UnexpectedFile($targetKey);
         }
 
-        if (! $this->adapter->rename($sourceKey, $targetKey)) {
+        if (!$this->adapter->rename($sourceKey, $targetKey)) {
             throw new \RuntimeException(sprintf('Could not rename the "%s" key to "%s".', $sourceKey, $targetKey));
         }
 
-        if($this->isFileInRegister($sourceKey)) {
+        if ($this->isFileInRegister($sourceKey)) {
             $this->fileRegister[$targetKey] = $this->fileRegister[$sourceKey];
             unset($this->fileRegister[$sourceKey]);
         }
@@ -288,7 +288,7 @@ 
      */
     public function createFile($key)
     {
-        if(false === $this->isFileInRegister($key)) {
+        if (false === $this->isFileInRegister($key)) {
             if ($this->adapter instanceof Adapter\FileFactory) {
                 $this->fileRegister[$key] = $this->adapter->createFile($key, $this);
             } else {
@@ -332,7 +332,7 @@ 
      */
     private function assertHasFile($key)
     {
-        if (! empty($key) && ! $this->has($key)) {
+        if (!empty($key) && !$this->has($key)) {
             throw new Exception\FileNotFound($key);
         }
     }

src/Gaufrette/StreamWrapper.php

Doc Comments +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
 
     /**
      * @param  string $path
-     * @return mixed
+     * @return boolean
      */
     public function unlink($path)
     {

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
     {
         static::streamWrapperUnregister($scheme);
 
-        if (! static::streamWrapperRegister($scheme, __CLASS__)) {
+        if (!static::streamWrapperRegister($scheme, __CLASS__)) {
             throw new \RuntimeException(sprintf(
                 'Could not register stream wrapper class %s for scheme %s.',
                 __CLASS__,
@@ -251,11 +251,11 @@ 
         $key    = substr($parts['path'], 1);
 
         if (null !== $parts['query']) {
-            $key.= '?' . $parts['query'];
+            $key .= '?'.$parts['query'];
         }
 
         if (null !== $parts['fragment']) {
-            $key.= '#' . $parts['fragment'];
+            $key .= '#'.$parts['fragment'];
         }
 
         if (empty($domain) || empty($key)) {

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/FilesystemMap.php

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function set($domain, Filesystem $filesystem)
     {
-        if (! preg_match('/^[-_a-zA-Z0-9]+$/', $domain)) {
+        if (!preg_match('/^[-_a-zA-Z0-9]+$/', $domain)) {
             throw new \InvalidArgumentException(sprintf(
                 'The specified domain "%s" is not a valid domain.',
                 $domain
@@ -68,7 +68,7 @@ 
      */
     public function get($domain)
     {
-        if (! $this->has($domain)) {
+        if (!$this->has($domain)) {
             throw new \InvalidArgumentException(sprintf(
                 'There is no filesystem defined for the "%s" domain.',
                 $domain
@@ -87,7 +87,7 @@ 
      */
     public function remove($domain)
     {
-        if (! $this->has($domain)) {
+        if (!$this->has($domain)) {
             throw new \InvalidArgumentException(sprintf(
                 'Cannot remove the "%s" filesystem as it is not defined.',
                 $domain

src/Gaufrette/Stream/InMemoryBuffer.php

Spacing +4 added lines, -4 removed lines

@@ -26,7 +26,7 @@ 
     public function __construct(Filesystem $filesystem, $key)
     {
         $this->filesystem = $filesystem;
-        $this->key     = $key;
+        $this->key = $key;
     }
 
     /**
@@ -66,7 +66,7 @@ 
         }
 
         $chunk = substr($this->content, $this->position, $count);
-        $this->position+= Util\Size::fromContent($chunk);
+        $this->position += Util\Size::fromContent($chunk);
 
         return $chunk;
     }
@@ -91,7 +91,7 @@ 
         } else {
             $before = substr($this->content, 0, $this->position);
             $after  = $newNumBytes > $newPosition ? substr($this->content, $newPosition) : '';
-            $this->content  = $before . $data . $after;
+            $this->content  = $before.$data.$after;
         }
 
         $this->position     = $newPosition;
@@ -103,7 +103,7 @@ 
 
     public function close()
     {
-        if (! $this->synchronized) {
+        if (!$this->synchronized) {
             $this->flush();
         }
     }

src/Gaufrette/Stream/Local.php

Spacing +3 added lines, -3 removed lines

@@ -56,7 +56,7 @@ 
      */
     public function read($count)
     {
-        if (! $this->fileHandle) {
+        if (!$this->fileHandle) {
             return false;
         }
 
@@ -72,7 +72,7 @@ 
      */
     public function write($data)
     {
-        if (! $this->fileHandle) {
+        if (!$this->fileHandle) {
             return false;
         }
 
@@ -88,7 +88,7 @@ 
      */
     public function close()
     {
-        if (! $this->fileHandle) {
+        if (!$this->fileHandle) {
             return false;
         }