PHPSocialNetwork / phpfastcache

src/phpFastCache/Core/Pool/IO/IOHelperTrait.php

Doc Comments +4 added lines, -4 removed lines

@@ -130,7 +130,7 @@ 
 
 
     /**
-     * @param $keyword
+     * @param false|string $keyword
      * @param bool $skip
      * @return string
      * @throws phpFastCacheIOException
@@ -197,7 +197,7 @@ 
 
     /**
      * @param $filename
-     * @return mixed
+     * @return string
      */
     protected static function cleanFileName($filename)
     {
@@ -212,7 +212,7 @@ 
     }
 
     /**
-     * @param $path
+     * @param string $path
      * @param bool $create
      * @throws phpFastCacheIOException
      */
@@ -249,7 +249,7 @@ 
 
 
     /**
-     * @param $file
+     * @param string $file
      * @return string
      * @throws phpFastCacheIOException
      */

Indentation +8 added lines, -8 removed lines

@@ -202,9 +202,9 @@ 
     protected static function cleanFileName($filename)
     {
         $regex = [
-          '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/',
-          '/\.$/',
-          '/^\./',
+            '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/',
+            '/\.$/',
+            '/^\./',
         ];
         $replace = ['-', '', ''];
 
@@ -296,7 +296,7 @@ 
             $tmpFilename = Directory::getAbsolutePath(dirname($file) . '/tmp_' . md5(
                 str_shuffle(uniqid($this->getDriverName(), false))
                 . str_shuffle(uniqid($this->getDriverName(), false))
-              ));
+                ));
 
             $f = fopen($tmpFilename, 'w+');
             flock($f, LOCK_EX);
@@ -338,11 +338,11 @@ 
         }
 
         $stat->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData([
+            ->setRawData([
             'tmp' => $this->tmp
-          ])
-          ->setSize(Directory::dirSize($path))
-          ->setInfo('Number of files used to build the cache: ' . Directory::getFileCount($path));
+            ])
+            ->setSize(Directory::dirSize($path))
+            ->setInfo('Number of files used to build the cache: ' . Directory::getFileCount($path));
 
         return $stat;
     }

Braces +3 added lines, -3 removed lines

@@ -95,11 +95,11 @@ 
                 return $full_path_tmp;
             }
             return $full_path;
-        }else{
+        } else{
             if (!isset($this->tmp[ $full_path_hash ]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
                 if (!@file_exists($full_path)) {
                     @mkdir($full_path, $this->getDefaultChmod(), true);
-                }else if (!@is_writable($full_path)) {
+                } else if (!@is_writable($full_path)) {
                     if (!@chmod($full_path, $this->getDefaultChmod()) && $this->config[ 'autoTmpFallback' ])
                     {
                         /**
@@ -299,7 +299,7 @@ 
             if(!rename($tmpFilename, $file)){
                 throw new phpFastCacheIOException(sprintf('Failed to rename %s to %s', $tmpFilename, $file));
             }
-        }else{
+        } else{
             $f = fopen($file, 'w+');
             $octetWritten = fwrite($f, $data);
             fclose($f);

Spacing +19 added lines, -19 removed lines

@@ -51,10 +51,10 @@ 
          * Calculate the security key
          */
         {
-            $securityKey = array_key_exists('securityKey', $this->config) ? $this->config[ 'securityKey' ] : '';
+            $securityKey = array_key_exists('securityKey', $this->config) ? $this->config['securityKey'] : '';
             if (!$securityKey || $securityKey === 'auto') {
-                if (isset($_SERVER[ 'HTTP_HOST' ])) {
-                    $securityKey = preg_replace('/^www./', '', strtolower(str_replace(':', '_', $_SERVER[ 'HTTP_HOST' ])));
+                if (isset($_SERVER['HTTP_HOST'])) {
+                    $securityKey = preg_replace('/^www./', '', strtolower(str_replace(':', '_', $_SERVER['HTTP_HOST'])));
                 } else {
                     $securityKey = ($this->isPHPModule() ? 'web' : 'cli');
                 }
@@ -73,10 +73,10 @@ 
          */
         $tmp_dir = rtrim($tmp_dir, '/') . DIRECTORY_SEPARATOR;
 
-        if (empty($this->config[ 'path' ]) || !is_string($this->config[ 'path' ])) {
+        if (empty($this->config['path']) || !is_string($this->config['path'])) {
             $path = $tmp_dir;
         } else {
-            $path = rtrim($this->config[ 'path' ], '/') . DIRECTORY_SEPARATOR;
+            $path = rtrim($this->config['path'], '/') . DIRECTORY_SEPARATOR;
         }
 
         $path_suffix = $securityKey . DIRECTORY_SEPARATOR . $this->getDriverName();
@@ -91,16 +91,16 @@ 
          * return the temp dir
          */
         if ($readonly === true) {
-            if($this->config[ 'autoTmpFallback' ] && (!@file_exists($full_path) || !@is_writable($full_path))){
+            if ($this->config['autoTmpFallback'] && (!@file_exists($full_path) || !@is_writable($full_path))) {
                 return $full_path_tmp;
             }
             return $full_path;
-        }else{
-            if (!isset($this->tmp[ $full_path_hash ]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
+        } else {
+            if (!isset($this->tmp[$full_path_hash]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
                 if (!@file_exists($full_path)) {
                     @mkdir($full_path, $this->getDefaultChmod(), true);
-                }else if (!@is_writable($full_path)) {
-                    if (!@chmod($full_path, $this->getDefaultChmod()) && $this->config[ 'autoTmpFallback' ])
+                } else if (!@is_writable($full_path)) {
+                    if (!@chmod($full_path, $this->getDefaultChmod()) && $this->config['autoTmpFallback'])
                     {
                         /**
                          * Switch back to tmp dir
@@ -122,8 +122,8 @@ 
                     throw new phpFastCacheIOException('Path "' . $full_path . '" is not writable, please set a chmod 0777 or any writable permission and make sure to make use of an absolute path !');
                 }
 
-                $this->tmp[ $full_path_hash ] = $full_path;
-                $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config[ 'htaccess' ] : false);
+                $this->tmp[$full_path_hash] = $full_path;
+                $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config['htaccess'] : false);
             }
         }
 
@@ -160,7 +160,7 @@ 
             }
         }
 
-        return $path . '/' . $filename . '.' . $this->config[ 'cacheFileExtension' ];
+        return $path . '/' . $filename . '.' . $this->config['cacheFileExtension'];
     }
 
 
@@ -179,10 +179,10 @@ 
      */
     protected function getDefaultChmod()
     {
-        if (!isset($this->config[ 'default_chmod' ]) || $this->config[ 'default_chmod' ] == '' || is_null($this->config[ 'default_chmod' ])) {
+        if (!isset($this->config['default_chmod']) || $this->config['default_chmod'] == '' || is_null($this->config['default_chmod'])) {
             return 0777;
         } else {
-            return $this->config[ 'default_chmod' ];
+            return $this->config['default_chmod'];
         }
     }
 
@@ -212,7 +212,7 @@ 
         if ($create === true) {
             if (!is_writable($path)) {
                 try {
-                    if(!chmod($path, 0777)){
+                    if (!chmod($path, 0777)) {
                         throw new phpFastCacheIOException('Chmod failed on : ' . $path);
                     }
                 } catch (phpFastCacheIOException $e) {
@@ -287,7 +287,7 @@ 
          */
         //$this->eventManager->dispatch('CacheWriteFileOnDisk', $this, $file, $secureFileManipulation);
 
-        if($secureFileManipulation){
+        if ($secureFileManipulation) {
             $tmpFilename = Directory::getAbsolutePath(dirname($file) . '/tmp_' . md5(
                 str_shuffle(uniqid($this->getDriverName(), false))
                 . str_shuffle(uniqid($this->getDriverName(), false))
@@ -299,10 +299,10 @@ 
             flock($f, LOCK_UN);
             fclose($f);
 
-            if(!rename($tmpFilename, $file)){
+            if (!rename($tmpFilename, $file)) {
                 throw new phpFastCacheIOException(sprintf('Failed to rename %s to %s', $tmpFilename, $file));
             }
-        }else{
+        } else {
             $f = fopen($file, 'w+');
             $octetWritten = fwrite($f, $data);
             fclose($f);

src/phpFastCache/Helper/Psr16Adapter.php

Indentation +2 added lines, -2 removed lines

@@ -63,8 +63,8 @@
     {
         try {
             $cacheItem = $this->internalCacheInstance
-              ->getItem($key)
-              ->set($value);
+                ->getItem($key)
+                ->set($value);
             if (is_int($ttl) || $ttl instanceof \DateInterval) {
                 $cacheItem->expiresAfter($ttl);
             }

Spacing +1 added lines, -1 removed lines

@@ -110,7 +110,7 @@
     public function getMultiple($keys, $default = null)
     {
         try {
-            return array_map(function (ExtendedCacheItemInterface $item) {
+            return array_map(function(ExtendedCacheItemInterface $item) {
                 return $item->get();
             }, $this->internalCacheInstance->getItems($keys));
         } catch (phpFastCacheInvalidArgumentException $e) {

src/phpFastCache/Core/Pool/CacheItemPoolTrait.php

Doc Comments +2 added lines, -2 removed lines

@@ -237,7 +237,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws phpFastCacheInvalidArgumentException
      * @throws \RuntimeException
      */
@@ -294,7 +294,7 @@ 
     }
 
     /**
-     * @return mixed|null
+     * @return boolean
      * @throws phpFastCacheInvalidArgumentException
      */
     public function commit()

Indentation +3 added lines, -3 removed lines

@@ -148,9 +148,9 @@
                              * Reset the Item
                              */
                             $item->set(null)
-                              ->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]))
-                              ->setHit(false)
-                              ->setTags([]);
+                                ->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]))
+                                ->setHit(false)
+                                ->setTags([]);
                             if($this->config['itemDetailedDate']){
 
                                 /**

Braces +2 added lines, -2 removed lines

@@ -164,7 +164,7 @@ 
                         } else {
                             $item->setHit(true);
                         }
-                    }else{
+                    } else{
                         $item->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]));
                     }
                 }
@@ -359,7 +359,7 @@ 
     {
         if (!array_key_exists($item->getKey(), $this->itemInstances)) {
             $this->itemInstances[ $item->getKey() ] = $item;
-        }else if(spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])){
+        } else if(spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])){
             throw new \RuntimeException('Spl object hash mismatches ! You probably tried to save a detached item which has been already retrieved from cache.');
         }
 

Spacing +22 added lines, -22 removed lines

@@ -69,7 +69,7 @@ 
              * loop dispatching operations
              */
             if (!isset($this->itemInstances[$key])) {
-                if (preg_match('~([' . preg_quote(self::$unsupportedKeyChars, '~') . ']+)~', $key, $matches)){
+                if (preg_match('~([' . preg_quote(self::$unsupportedKeyChars, '~') . ']+)~', $key, $matches)) {
                     throw new phpFastCacheInvalidArgumentException('Unsupported key character detected: "' . $matches[1] . '". Please check: https://github.com/PHPSocialNetwork/phpfastcache/wiki/%5BV6%5D-Unsupported-characters-in-key-identifiers');
                 }
 
@@ -87,14 +87,14 @@ 
                     $driverArray = $this->driverRead($item);
 
                     if ($driverArray) {
-                        if(!is_array($driverArray)){
+                        if (!is_array($driverArray)) {
                             throw new phpFastCacheCoreException(sprintf('The driverRead method returned an unexpected variable type: %s', gettype($driverArray)));
                         }
                         $driverData = $this->driverUnwrapData($driverArray);
 
-                        if($this->getConfig()[ 'preventCacheSlams' ]){
-                            while($driverData instanceof ItemBatch) {
-                                if($driverData->getItemDate()->getTimestamp() + $this->getConfig()[ 'cacheSlamsTimeout' ] < time()){
+                        if ($this->getConfig()['preventCacheSlams']) {
+                            while ($driverData instanceof ItemBatch) {
+                                if ($driverData->getItemDate()->getTimestamp() + $this->getConfig()['cacheSlamsTimeout'] < time()) {
                                     /**
                                      * The timeout has been reached
                                      * Consider that the batch has
@@ -126,7 +126,7 @@ 
                         $item->set($driverData);
                         $item->expiresAt($this->driverUnwrapEdate($driverArray));
 
-                        if($this->config['itemDetailedDate']){
+                        if ($this->config['itemDetailedDate']) {
 
                             /**
                              * If the itemDetailedDate has been
@@ -154,10 +154,10 @@ 
                              * Reset the Item
                              */
                             $item->set(null)
-                              ->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]))
+                              ->expiresAfter(abs((int) $this->getConfig()['defaultTtl']))
                               ->setHit(false)
                               ->setTags([]);
-                            if($this->config['itemDetailedDate']){
+                            if ($this->config['itemDetailedDate']) {
 
                                 /**
                                  * If the itemDetailedDate has been
@@ -170,8 +170,8 @@ 
                         } else {
                             $item->setHit(true);
                         }
-                    }else{
-                        $item->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]));
+                    } else {
+                        $item->expiresAfter(abs((int) $this->getConfig()['defaultTtl']));
                     }
                 }
             }
@@ -184,9 +184,9 @@ 
          * @param $this ExtendedCacheItemPoolInterface
          * @param $this ExtendedCacheItemInterface
          */
-        $this->eventManager->dispatch('CacheGetItem', $this, $this->itemInstances[ $key ]);
+        $this->eventManager->dispatch('CacheGetItem', $this, $this->itemInstances[$key]);
 
-        return $this->itemInstances[ $key ];
+        return $this->itemInstances[$key];
     }
 
     /**
@@ -197,7 +197,7 @@ 
     public function setItem(CacheItemInterface $item)
     {
         if ($this->getClassNamespace() . '\\Item' === get_class($item)) {
-            $this->itemInstances[ $item->getKey() ] = $item;
+            $this->itemInstances[$item->getKey()] = $item;
 
             return $this;
         } else {
@@ -214,7 +214,7 @@ 
     {
         $collection = [];
         foreach ($keys as $key) {
-            $collection[ $key ] = $this->getItem($key);
+            $collection[$key] = $this->getItem($key);
         }
 
         return $collection;
@@ -315,8 +315,8 @@ 
          * loop dispatching operations
          */
         if (!isset($this->itemInstances[$item->getKey()])) {
-            $this->itemInstances[ $item->getKey() ] = $item;
-        } else if(spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])){
+            $this->itemInstances[$item->getKey()] = $item;
+        } else if (spl_object_hash($item) !== spl_object_hash($this->itemInstances[$item->getKey()])) {
             throw new \RuntimeException('Spl object hash mismatches ! You probably tried to save a detached item which has been already retrieved from cache.');
         }
 
@@ -328,7 +328,7 @@ 
         $this->eventManager->dispatch('CacheSaveItem', $this, $item);
 
 
-        if($this->getConfig()[ 'preventCacheSlams' ]){
+        if ($this->getConfig()['preventCacheSlams']) {
             /**
              * @var $itemBatch ExtendedCacheItemInterface
              */
@@ -336,7 +336,7 @@ 
             $itemBatch = $class->newInstanceArgs([$this, $item->getKey()]);
             $itemBatch->setEventManager($this->eventManager)
                         ->set(new ItemBatch($item->getKey(), new \DateTime()))
-                        ->expiresAfter($this->getConfig()[ 'cacheSlamsTimeout' ]);
+                        ->expiresAfter($this->getConfig()['cacheSlamsTimeout']);
 
             /**
              * To avoid SPL mismatches
@@ -368,8 +368,8 @@ 
     public function saveDeferred(CacheItemInterface $item)
     {
         if (!array_key_exists($item->getKey(), $this->itemInstances)) {
-            $this->itemInstances[ $item->getKey() ] = $item;
-        }else if(spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])){
+            $this->itemInstances[$item->getKey()] = $item;
+        } else if (spl_object_hash($item) !== spl_object_hash($this->itemInstances[$item->getKey()])) {
             throw new \RuntimeException('Spl object hash mismatches ! You probably tried to save a detached item which has been already retrieved from cache.');
         }
 
@@ -380,7 +380,7 @@ 
          */
         $this->eventManager->dispatch('CacheSaveDeferredItem', $this, $item);
 
-        return $this->deferredList[ $item->getKey() ] = $item;
+        return $this->deferredList[$item->getKey()] = $item;
     }
 
     /**
@@ -400,7 +400,7 @@ 
         foreach ($this->deferredList as $key => $item) {
             $result = $this->save($item);
             if ($return !== false) {
-                unset($this->deferredList[ $key ]);
+                unset($this->deferredList[$key]);
                 $return = $result;
             }
         }

src/phpFastCache/Core/Pool/ExtendedCacheItemPoolTrait.php

Doc Comments +2 added lines, -1 removed lines

@@ -38,6 +38,7 @@ 
 
     /**
      * @inheritdoc
+     * @param string $tagName
      */
     public function getItemsByTag($tagName)
     {
@@ -408,7 +409,7 @@ 
 
     /**
      * @internal This method de-register an item from $this->itemInstances
-     * @param CacheItemInterface|string $item
+     * @param CacheItemInterface $item
      * @throws phpFastCacheInvalidArgumentException
      */
     protected function deregisterItem($item)

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 
 namespace phpFastCache\Core\Pool;
 
-use InvalidArgumentException;
 use phpFastCache\Core\Item\ExtendedCacheItemInterface;
 use phpFastCache\EventManager;
 use phpFastCache\Exceptions\phpFastCacheInvalidArgumentException;

Spacing +16 added lines, -16 removed lines

@@ -31,7 +31,7 @@ 
      */
     public function getItemsAsJsonString(array $keys = [], $option = 0, $depth = 512)
     {
-        $callback = function (CacheItemInterface $item) {
+        $callback = function(CacheItemInterface $item) {
             return $item->get();
         };
         return json_encode(array_map($callback, array_values($this->getItems($keys))), $option, $depth);
@@ -45,7 +45,7 @@ 
         if (is_string($tagName)) {
             $driverResponse = $this->getItem($this->getTagKey($tagName));
             if ($driverResponse->isHit()) {
-                $items = (array)$driverResponse->get();
+                $items = (array) $driverResponse->get();
 
                 /**
                  * getItems() may provides expired item(s)
@@ -57,7 +57,7 @@ 
                  *
                  * #headache
                  */
-                return array_filter($this->getItems(array_unique(array_keys($items))), function (ExtendedCacheItemInterface $item) {
+                return array_filter($this->getItems(array_unique(array_keys($items))), function(ExtendedCacheItemInterface $item) {
                     return $item->isHit();
                 });
             } else {
@@ -75,9 +75,9 @@ 
     {
         $items = [];
         foreach (array_unique($tagNames) as $tagName) {
-            if(is_string($tagName)){
+            if (is_string($tagName)) {
                 $items = array_merge($items, $this->getItemsByTag($tagName));
-            }else{
+            } else {
                 throw new phpFastCacheInvalidArgumentException('$tagName must be a a string');
             }
         }
@@ -95,7 +95,7 @@ 
 
         foreach ($items as $key => $item) {
             if (array_diff($tagNames, $item->getTags())) {
-                unset($items[ $key ]);
+                unset($items[$key]);
             }
         }
 
@@ -108,7 +108,7 @@ 
      */
     public function getItemsByTagsAsJsonString(array $tagNames, $option = 0, $depth = 512)
     {
-        $callback = function (CacheItemInterface $item) {
+        $callback = function(CacheItemInterface $item) {
             return $item->get();
         };
 
@@ -379,7 +379,7 @@ 
      */
     public function detachItem(CacheItemInterface $item)
     {
-        if (isset($this->itemInstances[ $item->getKey() ])) {
+        if (isset($this->itemInstances[$item->getKey()])) {
             $this->deregisterItem($item);
         }
     }
@@ -399,10 +399,10 @@ 
      */
     public function attachItem(CacheItemInterface $item)
     {
-        if (isset($this->itemInstances[ $item->getKey() ]) && spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])) {
+        if (isset($this->itemInstances[$item->getKey()]) && spl_object_hash($item) !== spl_object_hash($this->itemInstances[$item->getKey()])) {
             throw new phpFastCacheLogicException('The item already exists and cannot be overwritten because the Spl object hash mismatches ! You probably tried to re-attach a detached item which has been already retrieved from cache.');
         } else {
-            $this->itemInstances[ $item->getKey() ] = $item;
+            $this->itemInstances[$item->getKey()] = $item;
         }
     }
 
@@ -415,10 +415,10 @@ 
     protected function deregisterItem($item)
     {
         if ($item instanceof CacheItemInterface) {
-            unset($this->itemInstances[ $item->getKey() ]);
+            unset($this->itemInstances[$item->getKey()]);
 
         } else if (is_string($item)) {
-            unset($this->itemInstances[ $item ]);
+            unset($this->itemInstances[$item]);
         } else {
             throw new phpFastCacheInvalidArgumentException('Invalid type for $item variable');
         }
@@ -437,8 +437,8 @@ 
      */
     public function isAttached(CacheItemInterface $item)
     {
-        if (isset($this->itemInstances[ $item->getKey() ])) {
-            return spl_object_hash($item) === spl_object_hash($this->itemInstances[ $item->getKey() ]);
+        if (isset($this->itemInstances[$item->getKey()])) {
+            return spl_object_hash($item) === spl_object_hash($this->itemInstances[$item->getKey()]);
         }
         return null;
     }
@@ -458,8 +458,8 @@ 
      */
     public function saveMultiple(...$items)
     {
-        if (isset($items[ 0 ]) && is_array($items[ 0 ])) {
-            foreach ($items[ 0 ] as $item) {
+        if (isset($items[0]) && is_array($items[0])) {
+            foreach ($items[0] as $item) {
                 $this->save($item);
             }
             return true;

Braces +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
         foreach (array_unique($tagNames) as $tagName) {
             if(is_string($tagName)){
                 $items = array_merge($items, $this->getItemsByTag($tagName));
-            }else{
+            } else{
                 throw new phpFastCacheInvalidArgumentException('$tagName must be a a string');
             }
         }

src/phpFastCache/Drivers/Cassandra/Driver.php

Doc Comments +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws phpFastCacheInvalidArgumentException
      */
     protected function driverWrite(CacheItemInterface $item)

Braces +3 added lines, -3 removed lines

@@ -106,7 +106,7 @@ 
                  * been really upserted
                  */
                 return $result instanceof Cassandra\Rows;
-            }catch(\Cassandra\Exception\InvalidArgumentException $e){
+            } catch(\Cassandra\Exception\InvalidArgumentException $e){
                 throw new phpFastCacheInvalidArgumentException($e, 0, $e);
             }
         } else {
@@ -134,7 +134,7 @@ 
 
             if($results instanceof Cassandra\Rows && $results->count() === 1){
                 return $this->decode($results->first()['cache_data']);
-            }else{
+            } else{
                 return null;
             }
         } catch (Cassandra\Exception $e) {
@@ -219,7 +219,7 @@ 
             if(!empty($this->config['ssl']['enabled'])){
                 if(!empty($this->config['ssl']['verify'])){
                     $sslBuilder = Cassandra::ssl()->withVerifyFlags(Cassandra::VERIFY_PEER_CERT);
-                }else{
+                } else{
                     $sslBuilder = Cassandra::ssl()->withVerifyFlags(Cassandra::VERIFY_NONE);
                 }
 

Indentation +27 added lines, -27 removed lines

@@ -75,16 +75,16 @@ 
             try{
                 $cacheData = $this->encode($this->driverPreWrap($item));
                 $options = new Cassandra\ExecutionOptions([
-                  'arguments' => [
+                    'arguments' => [
                     'cache_uuid' => new Cassandra\Uuid(),
                     'cache_id' => $item->getKey(),
                     'cache_data' => $cacheData,
                     'cache_creation_date' => new Cassandra\Timestamp((new \DateTime())->getTimestamp()),
                     'cache_expiration_date' => new Cassandra\Timestamp($item->getExpirationDate()->getTimestamp()),
                     'cache_length' => strlen($cacheData)
-                  ],
-                  'consistency' => Cassandra::CONSISTENCY_ALL,
-                  'serial_consistency' => Cassandra::CONSISTENCY_SERIAL
+                    ],
+                    'consistency' => Cassandra::CONSISTENCY_ALL,
+                    'serial_consistency' => Cassandra::CONSISTENCY_SERIAL
                 ]);
 
                 $query = sprintf('INSERT INTO %s.%s
@@ -122,13 +122,13 @@ 
     {
         try {
             $options = new Cassandra\ExecutionOptions([
-              'arguments' => ['cache_id' => $item->getKey()],
-              'page_size' => 1
+                'arguments' => ['cache_id' => $item->getKey()],
+                'page_size' => 1
             ]);
             $query = sprintf(
-              'SELECT cache_data FROM %s.%s WHERE cache_id = :cache_id;',
-              self::CASSANDRA_KEY_SPACE,
-              self::CASSANDRA_TABLE
+                'SELECT cache_data FROM %s.%s WHERE cache_id = :cache_id;',
+                self::CASSANDRA_KEY_SPACE,
+                self::CASSANDRA_TABLE
             );
             $results = $this->instance->execute(new Cassandra\SimpleStatement($query), $options);
 
@@ -155,14 +155,14 @@ 
         if ($item instanceof Item) {
             try {
                 $options = new Cassandra\ExecutionOptions([
-                  'arguments' => [
+                    'arguments' => [
                     'cache_id' => $item->getKey(),
-                  ],
+                    ],
                 ]);
                 $result = $this->instance->execute(new Cassandra\SimpleStatement(sprintf(
-                  'DELETE FROM %s.%s WHERE cache_id = :cache_id;',
-                  self::CASSANDRA_KEY_SPACE,
-                  self::CASSANDRA_TABLE
+                    'DELETE FROM %s.%s WHERE cache_id = :cache_id;',
+                    self::CASSANDRA_KEY_SPACE,
+                    self::CASSANDRA_TABLE
                 )), $options);
 
                 /**
@@ -186,8 +186,8 @@ 
     {
         try {
             $this->instance->execute(new Cassandra\SimpleStatement(sprintf(
-              'TRUNCATE %s.%s;',
-              self::CASSANDRA_KEY_SPACE, self::CASSANDRA_TABLE
+                'TRUNCATE %s.%s;',
+                self::CASSANDRA_KEY_SPACE, self::CASSANDRA_TABLE
             )));
 
             return true;
@@ -213,8 +213,8 @@ 
             $username = isset($this->config[ 'username' ]) ? $this->config[ 'username' ] : '';
 
             $clusterBuilder = Cassandra::cluster()
-              ->withContactPoints($host)
-              ->withPort($port);
+                ->withContactPoints($host)
+                ->withPort($port);
 
             if(!empty($this->config['ssl']['enabled'])){
                 if(!empty($this->config['ssl']['verify'])){
@@ -242,8 +242,8 @@ 
              */
 
             $this->instance->execute(new Cassandra\SimpleStatement(sprintf(
-              "CREATE KEYSPACE IF NOT EXISTS %s WITH REPLICATION = { 'class' : 'SimpleStrategy', 'replication_factor' : 1 };",
-              self::CASSANDRA_KEY_SPACE
+                "CREATE KEYSPACE IF NOT EXISTS %s WITH REPLICATION = { 'class' : 'SimpleStrategy', 'replication_factor' : 1 };",
+                self::CASSANDRA_KEY_SPACE
             )));
             $this->instance->execute(new Cassandra\SimpleStatement(sprintf('USE %s;', self::CASSANDRA_KEY_SPACE)));
             $this->instance->execute(new Cassandra\SimpleStatement(sprintf('
@@ -290,15 +290,15 @@ 
     public function getStats()
     {
         $result = $this->instance->execute(new Cassandra\SimpleStatement(sprintf(
-          'SELECT SUM(cache_length) as cache_size FROM %s.%s',
-          self::CASSANDRA_KEY_SPACE,
-          self::CASSANDRA_TABLE
+            'SELECT SUM(cache_length) as cache_size FROM %s.%s',
+            self::CASSANDRA_KEY_SPACE,
+            self::CASSANDRA_TABLE
         )));
 
         return (new DriverStatistic())
-          ->setSize($result->first()[ 'cache_size' ])
-          ->setRawData([])
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setInfo('The cache size represents only the cache data itself without counting data structures associated to the cache entries.');
+            ->setSize($result->first()[ 'cache_size' ])
+            ->setRawData([])
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setInfo('The cache size represents only the cache data itself without counting data structures associated to the cache entries.');
     }
 }
\ No newline at end of file

Spacing +14 added lines, -14 removed lines

@@ -72,7 +72,7 @@ 
          * Check for Cross-Driver type confusion
          */
         if ($item instanceof Item) {
-            try{
+            try {
                 $cacheData = $this->encode($this->driverPreWrap($item));
                 $options = new Cassandra\ExecutionOptions([
                   'arguments' => [
@@ -106,7 +106,7 @@ 
                  * been really upserted
                  */
                 return $result instanceof Cassandra\Rows;
-            }catch(\Cassandra\Exception\InvalidArgumentException $e){
+            } catch (\Cassandra\Exception\InvalidArgumentException $e) {
                 throw new phpFastCacheInvalidArgumentException($e, 0, $e);
             }
         } else {
@@ -132,9 +132,9 @@ 
             );
             $results = $this->instance->execute(new Cassandra\SimpleStatement($query), $options);
 
-            if($results instanceof Cassandra\Rows && $results->count() === 1){
+            if ($results instanceof Cassandra\Rows && $results->count() === 1) {
                 return $this->decode($results->first()['cache_data']);
-            }else{
+            } else {
                 return null;
             }
         } catch (Cassandra\Exception $e) {
@@ -206,20 +206,20 @@ 
         if ($this->instance instanceof CassandraSession) {
             throw new phpFastCacheLogicException('Already connected to Couchbase server');
         } else {
-            $host = isset($this->config[ 'host' ]) ? $this->config[ 'host' ] : '127.0.0.1';
-            $port = isset($this->config[ 'port' ]) ? $this->config[ 'port' ] : 9042;
-            $timeout = isset($this->config[ 'timeout' ]) ? $this->config[ 'timeout' ] : 2;
-            $password = isset($this->config[ 'password' ]) ? $this->config[ 'password' ] : '';
-            $username = isset($this->config[ 'username' ]) ? $this->config[ 'username' ] : '';
+            $host = isset($this->config['host']) ? $this->config['host'] : '127.0.0.1';
+            $port = isset($this->config['port']) ? $this->config['port'] : 9042;
+            $timeout = isset($this->config['timeout']) ? $this->config['timeout'] : 2;
+            $password = isset($this->config['password']) ? $this->config['password'] : '';
+            $username = isset($this->config['username']) ? $this->config['username'] : '';
 
             $clusterBuilder = Cassandra::cluster()
               ->withContactPoints($host)
               ->withPort($port);
 
-            if(!empty($this->config['ssl']['enabled'])){
-                if(!empty($this->config['ssl']['verify'])){
+            if (!empty($this->config['ssl']['enabled'])) {
+                if (!empty($this->config['ssl']['verify'])) {
                     $sslBuilder = Cassandra::ssl()->withVerifyFlags(Cassandra::VERIFY_PEER_CERT);
-                }else{
+                } else {
                     $sslBuilder = Cassandra::ssl()->withVerifyFlags(Cassandra::VERIFY_NONE);
                 }
 
@@ -228,7 +228,7 @@ 
 
             $clusterBuilder->withConnectTimeout($timeout);
 
-            if($username){
+            if ($username) {
                 $clusterBuilder->withCredentials($username, $password);
             }
 
@@ -296,7 +296,7 @@ 
         )));
 
         return (new DriverStatistic())
-          ->setSize($result->first()[ 'cache_size' ])
+          ->setSize($result->first()['cache_size'])
           ->setRawData([])
           ->setData(implode(', ', array_keys($this->itemInstances)))
           ->setInfo('The cache size represents only the cache data itself without counting data structures associated to the cache entries.');

src/phpFastCache/Drivers/Files/Driver.php

Doc Comments +4 added lines, -4 removed lines

@@ -60,7 +60,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws phpFastCacheInvalidArgumentException
      */
     protected function driverWrite(CacheItemInterface $item)
@@ -146,7 +146,7 @@ 
     /**
      * @param string $optionName
      * @param mixed $optionValue
-     * @return bool
+     * @return boolean|null
      * @throws phpFastCacheInvalidArgumentException
      */
     public static function isValidOption($optionName, $optionValue)
@@ -179,7 +179,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getValidOptions()
     {
@@ -187,7 +187,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getRequiredOptions()
     {

src/phpFastCache/EventManager.php

Braces +3 added lines, -3 removed lines

@@ -88,13 +88,13 @@
             if(is_callable($arguments[0])){
                 if(isset($arguments[1]) && is_string($arguments[0])){
                     $this->events[$name][$arguments[1]] = $arguments[0];
-                }else {
+                } else {
                     $this->events[$name][] = $arguments[0];
                 }
-            }else{
+            } else{
                 throw new phpFastCacheInvalidArgumentException(sprintf('Expected Callable, got "%s"', gettype($arguments[0])));
             }
-        }else{
+        } else{
             throw new \BadMethodCallException('An event must start with "on" such as "onCacheGetItem"');
         }
     }

Spacing +9 added lines, -9 removed lines

@@ -74,8 +74,8 @@ 
          * due to performance issue on huge
          * loop dispatching operations
          */
-        if(isset($this->events[$eventName])){
-            foreach ($this->events[ $eventName ] as $event) {
+        if (isset($this->events[$eventName])) {
+            foreach ($this->events[$eventName] as $event) {
                 call_user_func_array($event, $args);
             }
         }
@@ -89,18 +89,18 @@ 
      */
     public function __call($name, $arguments)
     {
-        if(strpos($name, 'on') === 0){
+        if (strpos($name, 'on') === 0) {
             $name = substr($name, 2);
-            if(is_callable($arguments[0])){
-                if(isset($arguments[1]) && is_string($arguments[0])){
+            if (is_callable($arguments[0])) {
+                if (isset($arguments[1]) && is_string($arguments[0])) {
                     $this->events[$name][$arguments[1]] = $arguments[0];
-                }else {
+                } else {
                     $this->events[$name][] = $arguments[0];
                 }
-            }else{
+            } else {
                 throw new phpFastCacheInvalidArgumentException(sprintf('Expected Callable, got "%s"', gettype($arguments[0])));
             }
-        }else{
+        } else {
             throw new \BadMethodCallException('An event must start with "on" such as "onCacheGetItem"');
         }
     }
@@ -112,7 +112,7 @@ 
      */
     public function unbindEventCallback($eventName, $callbackName)
     {
-        if(isset($this->events[$eventName][$callbackName])){
+        if (isset($this->events[$eventName][$callbackName])) {
             unset($this->events[$eventName][$callbackName]);
             return true;
         }

src/phpFastCache/Drivers/Cookie/Item.php

Indentation +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
             $this->driver->setItem($this);
         } else {
             throw new phpFastCacheInvalidArgumentException(sprintf('$key must be a string, got type "%s" instead.',
-              gettype($key)));
+                gettype($key)));
         }
     }
 

src/phpFastCache/Drivers/Devfalse/Driver.php

Doc Comments +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws phpFastCacheInvalidArgumentException
      */
     protected function driverWrite(CacheItemInterface $item)

Indentation +6 added lines, -6 removed lines

@@ -80,9 +80,9 @@ 
     protected function driverRead(CacheItemInterface $item)
     {
         return [
-          self::DRIVER_DATA_WRAPPER_INDEX => false,
-          self::DRIVER_TAGS_WRAPPER_INDEX => [],
-          self::DRIVER_EDATE_WRAPPER_INDEX => new \DateTime(),
+            self::DRIVER_DATA_WRAPPER_INDEX => false,
+            self::DRIVER_TAGS_WRAPPER_INDEX => [],
+            self::DRIVER_EDATE_WRAPPER_INDEX => new \DateTime(),
         ];
     }
     /**
@@ -131,9 +131,9 @@ 
     {
         $stat = new DriverStatistic();
         $stat->setInfo('[Devfalse] A void info string')
-          ->setSize(0)
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData(false);
+            ->setSize(0)
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setRawData(false);
 
         return $stat;
     }