PHPSocialNetwork / phpfastcache

src/phpFastCache/Core/DriverAbstract.php

Doc Comments +6 added lines, -6 removed lines

@@ -289,7 +289,7 @@ 
 
 
     /**
-     * @return mixed
+     * @return ExtendedCacheItemPoolInterface
      */
     protected function backup()
     {
@@ -297,7 +297,7 @@ 
     }
 
     /**
-     * @param $file
+     * @param string $file
      * @return string
      * @throws \Exception
      */
@@ -479,7 +479,7 @@ 
      * @param array $tags | array("a","b","c")
      * @param int $time
      * @param array $options
-     * @return mixed
+     * @return boolean
      * @internal param array $option | $option = array("tags" => array("a","b","c")
      */
     public function touchTags($tags = [], $time = 300, $options = [])
@@ -500,7 +500,7 @@ 
     /**
      * @param array $tags | array("a","b","c")
      * @param array $option | $option = array("tags" => array("a","b","c")
-     * @return mixed
+     * @return boolean
      */
     public function deleteTags($tags = [], $option = [])
     {
@@ -522,7 +522,7 @@ 
      * @param array $tags | array("a","b","c")
      * @param integer
      * @param array $option | $option = array("tags" => array("a","b","c")
-     * @return mixed
+     * @return boolean
      */
     public function incrementTags($tags = [], $step = 1, $option = [])
     {
@@ -543,7 +543,7 @@ 
      * @param array $tags | array("a","b","c")
      * @param integer
      * @param array $option | $option = array("tags" => array("a","b","c")
-     * @return mixed
+     * @return boolean
      */
     public function decrementTags($tags = [], $step = 1, $option = [])
     {

Indentation +7 added lines, -7 removed lines

@@ -231,7 +231,7 @@ 
         foreach ($list as $array) {
             $name = $array[ 0 ];
             $res[ $name ] = $this->touch($name, $array[ 1 ],
-              isset($array[ 2 ]) ? $array[ 2 ] : []);
+                isset($array[ 2 ]) ? $array[ 2 ] : []);
         }
 
         return $res;
@@ -276,7 +276,7 @@ 
     {
         if (isset($v[ 1 ]) && is_scalar($v[ 1 ])) {
             return $this->set($name, $v[ 0 ], $v[ 1 ],
-              isset($v[ 2 ]) ? $v[ 2 ] : []);
+                isset($v[ 2 ]) ? $v[ 2 ] : []);
         } else {
             throw new phpFastCacheDriverException("Example ->$name = array('VALUE', 300);");
         }
@@ -395,10 +395,10 @@ 
     protected function _tagCaching()
     {
         return CacheManager::Sqlite(
-          [
+            [
             "path" => $this->config[ 'path' ],
             "cache_method" => 3,
-          ]
+            ]
         );
     }
 
@@ -567,9 +567,9 @@ 
     public function driverPreWrap(ExtendedCacheItemInterface $item)
     {
         return [
-          self::DRIVER_DATA_WRAPPER_INDEX => $item->get(),
-          self::DRIVER_TIME_WRAPPER_INDEX => $item->getExpirationDate(),
-          self::DRIVER_TAGS_WRAPPER_INDEX => $item->getTags(),
+            self::DRIVER_DATA_WRAPPER_INDEX => $item->get(),
+            self::DRIVER_TIME_WRAPPER_INDEX => $item->getExpirationDate(),
+            self::DRIVER_TAGS_WRAPPER_INDEX => $item->getTags(),
         ];
     }
 

Spacing +34 added lines, -34 removed lines

@@ -62,8 +62,8 @@ 
     public function __destruct()
     {
         // clean up the memory and don't want for PHP clean for caching method "phpfastcache"
-        if (isset($this->config[ 'instance' ]) && (int) $this->config[ 'cache_method' ] === 3) {
-            CacheManager::cleanCachingMethod($this->config[ 'instance' ]);
+        if (isset($this->config['instance']) && (int) $this->config['cache_method'] === 3) {
+            CacheManager::cleanCachingMethod($this->config['instance']);
         }
     }
 
@@ -85,9 +85,9 @@ 
      */
     public function getInfo($keyword, $option = [])
     {
-        if ((int) $this->config[ 'cache_method' ] > 1) {
-            if (isset(CacheManager::$memory[ $this->config[ 'instance' ] ][ $keyword ])) {
-                $object = CacheManager::$memory[ $this->config[ 'instance' ] ][ $keyword ];
+        if ((int) $this->config['cache_method'] > 1) {
+            if (isset(CacheManager::$memory[$this->config['instance']][$keyword])) {
+                $object = CacheManager::$memory[$this->config['instance']][$keyword];
             }
         }
         if (!isset($object)) {
@@ -116,9 +116,9 @@ 
     public function clean(array $option = [])
     {
         // handle method
-        if ((int) $this->config[ 'cache_method' ] > 1) {
+        if ((int) $this->config['cache_method'] > 1) {
             // use memory
-            CacheManager::$memory[ $this->config[ 'instance' ] ] = [];
+            CacheManager::$memory[$this->config['instance']] = [];
         }
 
         // end handle method
@@ -153,12 +153,12 @@ 
      */
     public function search($query_as_regex_or_string, $search_in_value = false)
     {
-        if ($this->config[ 'allow_search' ] != true) {
+        if ($this->config['allow_search'] != true) {
             throw new phpFastCacheDriverException('Please setup allow_search = true');
         } else {
             $list = $this->getTags("search", $search_in_value);
             $tmp = explode("/", $query_as_regex_or_string, 2);
-            $regex = isset($tmp[ 1 ]) ? true : false;
+            $regex = isset($tmp[1]) ? true : false;
             $return_list = [];
             foreach ($list as $tag) {
                 foreach ($tag as $keyword => $value) {
@@ -169,13 +169,13 @@ 
 
                     if ($regex == true && $gotcha == false) {     // look in keyword
                         if (preg_match($query_as_regex_or_string, $keyword)) {
-                            $return_list[ $keyword ] = $value;
+                            $return_list[$keyword] = $value;
                             $gotcha = true;
                         }
                     }
                     if ($gotcha == false) {
                         if (strpos($keyword, $query_as_regex_or_string) !== false) {
-                            $return_list[ $keyword ] = $value;
+                            $return_list[$keyword] = $value;
                             $gotcha = true;
                         }
                     }
@@ -183,13 +183,13 @@ 
                     if ($search_in_value == true && $gotcha == false) { // value search
                         if ($regex == true && $gotcha == false) {
                             if (preg_match($query_as_regex_or_string, $value)) {
-                                $return_list[ $keyword ] = $value;
+                                $return_list[$keyword] = $value;
                                 $gotcha = true;
                             }
                         }
                         if ($gotcha == false) {
                             if (strpos($value, $query_as_regex_or_string) !== false) {
-                                $return_list[ $keyword ] = $value;
+                                $return_list[$keyword] = $value;
                                 $gotcha = true;
                             }
                         }
@@ -213,8 +213,8 @@ 
         if ($object == null) {
             return false;
         } else {
-            $value = $object[ 'value' ];
-            $time = $object[ 'expired_time' ] - time() + $time;
+            $value = $object['value'];
+            $time = $object['expired_time'] - time() + $time;
             $this->set($keyword, $value, $time, $option);
 
             return true;
@@ -229,9 +229,9 @@ 
     {
         $res = [];
         foreach ($list as $array) {
-            $name = $array[ 0 ];
-            $res[ $name ] = $this->touch($name, $array[ 1 ],
-              isset($array[ 2 ]) ? $array[ 2 ] : []);
+            $name = $array[0];
+            $res[$name] = $this->touch($name, $array[1],
+              isset($array[2]) ? $array[2] : []);
         }
 
         return $res;
@@ -249,7 +249,7 @@ 
         if (is_array($config_name)) {
             $this->config = array_merge($this->config, $config_name);
         } else {
-            $this->config[ $config_name ] = $value;
+            $this->config[$config_name] = $value;
         }
     }
 
@@ -274,9 +274,9 @@ 
      */
     public function __set($name, $v)
     {
-        if (isset($v[ 1 ]) && is_scalar($v[ 1 ])) {
-            return $this->set($name, $v[ 0 ], $v[ 1 ],
-              isset($v[ 2 ]) ? $v[ 2 ] : []);
+        if (isset($v[1]) && is_scalar($v[1])) {
+            return $this->set($name, $v[0], $v[1],
+              isset($v[2]) ? $v[2] : []);
         } else {
             throw new phpFastCacheDriverException("Example ->$name = array('VALUE', 300);");
         }
@@ -293,7 +293,7 @@ 
      */
     protected function backup()
     {
-        return CacheManager::getInstance(CacheManager::$config[ 'fallback' ]);
+        return CacheManager::getInstance(CacheManager::$config['fallback']);
     }
 
     /**
@@ -396,7 +396,7 @@ 
     {
         return CacheManager::Sqlite(
           [
-            "path" => $this->config[ 'path' ],
+            "path" => $this->config['path'],
             "cache_method" => 3,
           ]
         );
@@ -415,7 +415,7 @@ 
         if (!is_array($tags)) {
             $tags = [$tags];
         }
-        $option[ 'tags' ] = $tags;
+        $option['tags'] = $tags;
 
         return $this->set($keyword, $value, $time, $option);
     }
@@ -427,7 +427,7 @@ 
             if (is_null($list)) {
                 $list = [];
             }
-            $list[ $keyword ] = time() + $time;
+            $list[$keyword] = time() + $time;
             $this->_tagCaching()->set($this->_getTagName($tag), $list, 3600 * 24 * 30);
         }
     }
@@ -455,21 +455,21 @@ 
             }
             foreach ($list as $keyword => $time) {
                 if ($time <= time()) {
-                    unset($list[ $keyword ]);
+                    unset($list[$keyword]);
                 } else {
                     if ($tmp < $time) {
                         $tmp = $time;
                     }
                     if ($return_content == true) {
-                        $list_return[ $keyword ] = $this->get($keyword);
+                        $list_return[$keyword] = $this->get($keyword);
                     } else {
-                        $list_return[ $keyword ] = $time;
+                        $list_return[$keyword] = $time;
                     }
                 }
             }
 
             $this->_tagCaching()->set($this->_getTagName($tag), $list, $tmp);
-            $keywords[ $tag ] = $list_return;
+            $keywords[$tag] = $list_return;
         }
 
         return $keywords;
@@ -579,7 +579,7 @@ 
      */
     public function driverUnwrapData(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_DATA_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_DATA_WRAPPER_INDEX];
     }
 
     /**
@@ -588,7 +588,7 @@ 
      */
     public function driverUnwrapTags(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_TAGS_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_TAGS_WRAPPER_INDEX];
     }
 
 
@@ -598,7 +598,7 @@ 
      */
     public function driverUnwrapTime(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_TIME_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_TIME_WRAPPER_INDEX];
     }
 
     /**
@@ -652,7 +652,7 @@ 
         foreach ($tagsItems as $tagsItem) {
             $data = (array) $tagsItem->get();
 
-            unset($data[ $item->getKey() ]);
+            unset($data[$item->getKey()]);
             $tagsItem->set($data);
 
             /**

src/phpFastCache/Core/PathSeekerTrait.php

Doc Comments +2 added lines, -3 removed lines

@@ -20,7 +20,6 @@ 
 trait PathSeekerTrait
 {
     /**
-     * @param bool $skip_create_path
      * @param $config
      * @return string
      * @throws \Exception
@@ -155,7 +154,7 @@ 
 
     /**
      * @param $filename
-     * @return mixed
+     * @return string
      */
     protected static function cleanFileName($filename)
     {
@@ -170,7 +169,7 @@ 
     }
 
     /**
-     * @param $path
+     * @param string $path
      * @param bool $create
      * @throws \Exception
      */

Indentation +4 added lines, -4 removed lines

@@ -160,9 +160,9 @@ 
     protected static function cleanFileName($filename)
     {
         $regex = [
-          '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/',
-          '/\.$/',
-          '/^\./',
+            '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/',
+            '/\.$/',
+            '/^\./',
         ];
         $replace = ['-', '', ''];
 
@@ -182,7 +182,7 @@ 
                     chmod($path, 0777);
                 } catch (phpFastCacheDriverException $e) {
                     throw new phpFastCacheDriverException('PLEASE CHMOD ' . $path . ' - 0777 OR ANY WRITABLE PERMISSION!',
-                      92);
+                        92);
                 }
             }
 

Spacing +14 added lines, -14 removed lines

@@ -29,27 +29,27 @@ 
     {
         $tmp_dir = ini_get('upload_tmp_dir') ? ini_get('upload_tmp_dir') : sys_get_temp_dir();
 
-        if (!isset($this->config[ 'path' ]) || $this->config[ 'path' ] == '') {
+        if (!isset($this->config['path']) || $this->config['path'] == '') {
             if (self::isPHPModule()) {
                 $path = $tmp_dir;
             } else {
-                $document_root_path = rtrim($_SERVER[ 'DOCUMENT_ROOT' ], '/') . '/../';
-                $path = isset($_SERVER[ 'DOCUMENT_ROOT' ]) && is_writable($document_root_path) ? $document_root_path : rtrim(__DIR__, '/') . '/';
+                $document_root_path = rtrim($_SERVER['DOCUMENT_ROOT'], '/') . '/../';
+                $path = isset($_SERVER['DOCUMENT_ROOT']) && is_writable($document_root_path) ? $document_root_path : rtrim(__DIR__, '/') . '/';
             }
 
-            if ($this->config[ 'path' ] != '') {
-                $path = $this->config[ 'path' ];
+            if ($this->config['path'] != '') {
+                $path = $this->config['path'];
             }
 
         } else {
-            $path = $this->config[ 'path' ];
+            $path = $this->config['path'];
         }
 
-        $securityKey = array_key_exists('securityKey', $this->config) ? $this->config[ 'securityKey' ] : '';
+        $securityKey = array_key_exists('securityKey', $this->config) ? $this->config['securityKey'] : '';
         if ($securityKey == "" || $securityKey == 'auto') {
-            $securityKey = $this->config[ 'securityKey' ];
+            $securityKey = $this->config['securityKey'];
             if ($securityKey == 'auto' || $securityKey == '') {
-                $securityKey = isset($_SERVER[ 'HTTP_HOST' ]) ? preg_replace('/^www./', '', strtolower($_SERVER[ 'HTTP_HOST' ])) : "default";
+                $securityKey = isset($_SERVER['HTTP_HOST']) ? preg_replace('/^www./', '', strtolower($_SERVER['HTTP_HOST'])) : "default";
             }
         }
         if ($securityKey != '') {
@@ -62,7 +62,7 @@ 
         $full_pathx = md5($full_path);
 
 
-        if (!isset($this->tmp[ $full_pathx ])) {
+        if (!isset($this->tmp[$full_pathx])) {
 
             if (!@file_exists($full_path) || !@is_writable($full_path)) {
                 if (!@file_exists($full_path)) {
@@ -86,8 +86,8 @@ 
                 }
             }
 
-            $this->tmp[ $full_pathx ] = true;
-            $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config[ 'htaccess' ] : false);
+            $this->tmp[$full_pathx] = true;
+            $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config['htaccess'] : false);
         }
 
         return realpath($full_path);
@@ -146,10 +146,10 @@ 
      */
     public function setChmodAuto()
     {
-        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'];
         }
     }
 

src/phpFastCache/Core/StandardPsr6StructureTrait.php

Doc Comments +2 added lines, -2 removed lines

@@ -157,7 +157,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     public function save(CacheItemInterface $item)
@@ -175,7 +175,7 @@ 
     }
 
     /**
-     * @return mixed|null
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     public function commit()

Spacing +5 added lines, -5 removed lines

@@ -71,7 +71,7 @@ 
             throw new \InvalidArgumentException(sprintf('$key must be a string, got type "%s" instead.', gettype($key)));
         }
 
-        return $this->itemInstances[ $key ];
+        return $this->itemInstances[$key];
     }
 
     /**
@@ -82,7 +82,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 {
@@ -99,7 +99,7 @@ 
     {
         $collection = [];
         foreach ($keys as $key) {
-            $collection[ $key ] = $this->getItem($key);
+            $collection[$key] = $this->getItem($key);
         }
 
         return $collection;
@@ -171,7 +171,7 @@ 
      */
     public function saveDeferred(CacheItemInterface $item)
     {
-        return $this->deferredList[ $item->getKey() ] = $item;
+        return $this->deferredList[$item->getKey()] = $item;
     }
 
     /**
@@ -184,7 +184,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/Drivers/Couchbase/Driver.php

Doc Comments +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
     }
 
     /**
-     * @return bool
+     * @return boolean|null
      */
     public function driverConnect()
     {

Indentation +2 added lines, -2 removed lines

@@ -142,10 +142,10 @@
             $password = isset($this->config[ 'password' ]) ? $this->config[ 'password' ] : '';
             $username = isset($this->config[ 'username' ]) ? $this->config[ 'username' ] : '';
             $buckets = isset($this->config[ 'buckets' ]) ? $this->config[ 'buckets' ] : [
-              [
+                [
                 'bucket' => 'default',
                 'password' => '',
-              ],
+                ],
             ];
 
             $this->instance = $this->instance ?: new CouchbaseClient("couchbase://{$host}", $username, $password);

Spacing +8 added lines, -8 removed lines

@@ -137,11 +137,11 @@ 
         } else {
 
 
-            $host = isset($this->config[ 'host' ]) ? $this->config[ 'host' ] : '127.0.0.1';
+            $host = isset($this->config['host']) ? $this->config['host'] : '127.0.0.1';
             //$port = isset($server[ 'port' ]) ? $server[ 'port' ] : '11211';
-            $password = isset($this->config[ 'password' ]) ? $this->config[ 'password' ] : '';
-            $username = isset($this->config[ 'username' ]) ? $this->config[ 'username' ] : '';
-            $buckets = isset($this->config[ 'buckets' ]) ? $this->config[ 'buckets' ] : [
+            $password = isset($this->config['password']) ? $this->config['password'] : '';
+            $username = isset($this->config['username']) ? $this->config['username'] : '';
+            $buckets = isset($this->config['buckets']) ? $this->config['buckets'] : [
               [
                 'bucket' => 'default',
                 'password' => '',
@@ -152,8 +152,8 @@ 
 
 
             foreach ($buckets as $bucket) {
-                $this->bucketCurrent = $this->bucketCurrent ?: $bucket[ 'bucket' ];
-                $this->setBucket($bucket[ 'bucket' ], $this->instance->openBucket($bucket[ 'bucket' ], $bucket[ 'password' ]));
+                $this->bucketCurrent = $this->bucketCurrent ?: $bucket['bucket'];
+                $this->setBucket($bucket['bucket'], $this->instance->openBucket($bucket['bucket'], $bucket['password']));
             }
         }
     }
@@ -163,7 +163,7 @@ 
      */
     protected function getBucket()
     {
-        return $this->bucketInstances[ $this->bucketCurrent ];
+        return $this->bucketInstances[$this->bucketCurrent];
     }
 
     /**
@@ -174,7 +174,7 @@ 
     protected function setBucket($bucketName, \CouchbaseBucket $CouchbaseBucket)
     {
         if (!array_key_exists($bucketName, $this->bucketInstances)) {
-            $this->bucketInstances[ $bucketName ] = $CouchbaseBucket;
+            $this->bucketInstances[$bucketName] = $CouchbaseBucket;
         } else {
             throw new \LogicException('A bucket instance with this name already exists.');
         }

src/phpFastCache/Drivers/Devnull/Driver.php

Doc Comments +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     public function driverWrite(CacheItemInterface $item)
@@ -112,7 +112,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return bool
+     * @return boolean|null
      * @throws \InvalidArgumentException
      */
     public function driverIsHit(CacheItemInterface $item)

Indentation +2 added lines, -2 removed lines

@@ -140,8 +140,8 @@
     {
         $stat = new driverStatistic();
         $stat->setInfo('A void info string')
-          ->setSize(1337)
-          ->setData('A void data string');
+            ->setSize(1337)
+            ->setData('A void data string');
 
         return $stat;
     }

src/phpFastCache/Drivers/Devtrue/Driver.php

Doc Comments +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     public function driverWrite(CacheItemInterface $item)
@@ -70,7 +70,7 @@ 
 
     /**
      * @param $key
-     * @return mixed
+     * @return boolean
      */
     public function driverRead($key)
     {

Indentation +2 added lines, -2 removed lines

@@ -140,8 +140,8 @@
     {
         $stat = new driverStatistic();
         $stat->setInfo('A void info string')
-          ->setSize(1337)
-          ->setData('A void data string');
+            ->setSize(1337)
+            ->setData('A void data string');
 
         return $stat;
     }

src/phpFastCache/Drivers/Files/Driver.php

Doc Comments +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean|null
      * @throws \InvalidArgumentException
      */
     public function driverWrite(CacheItemInterface $item)

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 
 namespace phpFastCache\Drivers\Files;
 
-use phpFastCache\Cache\ExtendedCacheItemInterface;
 use phpFastCache\Core\DriverAbstract;
 use phpFastCache\Core\PathSeekerTrait;
 use phpFastCache\Core\StandardPsr6StructureTrait;

Indentation +5 added lines, -5 removed lines

@@ -253,8 +253,8 @@ 
                             $key = $key[ 0 ];
                         }
                         $content[ $key ] = [
-                          'size' => $size,
-                          'write_time' => (isset($object[ 'write_time' ]) ? $object[ 'write_time' ] : null),
+                            'size' => $size,
+                            'write_time' => (isset($object[ 'write_time' ]) ? $object[ 'write_time' ] : null),
                         ];
                         if ($object->isExpired()) {
                             @unlink($file_path);
@@ -267,11 +267,11 @@ 
         }
 
         $stat->setData($content)
-          ->setSize($total - $removed)
-          ->setInfo('Total [bytes]: ' . $total . ', '
+            ->setSize($total - $removed)
+            ->setInfo('Total [bytes]: ' . $total . ', '
             . 'Expired and removed [bytes]: ' . $removed . ', '
             . 'Current [bytes], ' . $total - $removed
-          );
+            );
 
         return $stat;
     }

Spacing +4 added lines, -4 removed lines

@@ -71,7 +71,7 @@ 
             /*
              * Skip if Existing Caching in Options
              */
-            if (isset($option[ 'skipExisting' ]) && $option[ 'skipExisting' ] == true && file_exists($file_path)) {
+            if (isset($option['skipExisting']) && $option['skipExisting'] == true && file_exists($file_path)) {
                 $content = $this->readfile($file_path);
                 $old = $this->decode($content);
                 $toWrite = false;
@@ -250,11 +250,11 @@ 
                         } else {
                             //Because PHP 5.3, this cannot be written in single line
                             $key = explode('.', $f);
-                            $key = $key[ 0 ];
+                            $key = $key[0];
                         }
-                        $content[ $key ] = [
+                        $content[$key] = [
                           'size' => $size,
-                          'write_time' => (isset($object[ 'write_time' ]) ? $object[ 'write_time' ] : null),
+                          'write_time' => (isset($object['write_time']) ? $object['write_time'] : null),
                         ];
                         if ($object->isExpired()) {
                             @unlink($file_path);

src/phpFastCache/Drivers/Memcached/Driver.php

Doc Comments +2 added lines, -2 removed lines

@@ -128,7 +128,7 @@ 
     }
 
     /**
-     * @return bool
+     * @return boolean|null
      */
     public function driverConnect()
     {
@@ -294,7 +294,7 @@ 
     }
 
     /**
-     * @return mixed|null
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     public function commit()

Indentation +3 added lines, -3 removed lines

@@ -135,7 +135,7 @@ 
         $servers = (!empty($this->config[ 'memcache' ]) && is_array($this->config[ 'memcache' ]) ? $this->config[ 'memcache' ] : []);
         if (count($servers) < 1) {
             $servers = [
-              ['127.0.0.1', 11211],
+                ['127.0.0.1', 11211],
             ];
         }
 
@@ -186,7 +186,7 @@ 
             }
         } else {
             throw new \InvalidArgumentException(sprintf('$key must be a string, got type "%s" instead.',
-              gettype($key)));
+                gettype($key)));
         }
 
         return $this->itemInstances[ $key ];
@@ -205,7 +205,7 @@ 
             return $this;
         } else {
             throw new \InvalidArgumentException(sprintf('Invalid Item Class "%s" for this driver.',
-              get_class($item)));
+                get_class($item)));
         }
     }
 

Spacing +7 added lines, -7 removed lines

@@ -132,7 +132,7 @@ 
      */
     public function driverConnect()
     {
-        $servers = (!empty($this->config[ 'memcache' ]) && is_array($this->config[ 'memcache' ]) ? $this->config[ 'memcache' ] : []);
+        $servers = (!empty($this->config['memcache']) && is_array($this->config['memcache']) ? $this->config['memcache'] : []);
         if (count($servers) < 1) {
             $servers = [
               ['127.0.0.1', 11211],
@@ -141,7 +141,7 @@ 
 
         foreach ($servers as $server) {
             try {
-                if (!$this->instance->addServer($server[ 0 ], $server[ 1 ])) {
+                if (!$this->instance->addServer($server[0], $server[1])) {
                     $this->fallback = true;
                 }
             } catch (\Exception $e) {
@@ -189,7 +189,7 @@ 
               gettype($key)));
         }
 
-        return $this->itemInstances[ $key ];
+        return $this->itemInstances[$key];
     }
 
     /**
@@ -200,7 +200,7 @@ 
     public function setItem(CacheItemInterface $item)
     {
         if (__NAMESPACE__ . '\\Item' === get_class($item)) {
-            $this->itemInstances[ $item->getKey() ] = $item;
+            $this->itemInstances[$item->getKey()] = $item;
 
             return $this;
         } else {
@@ -218,7 +218,7 @@ 
     {
         $collection = [];
         foreach ($keys as $key) {
-            $collection[ $key ] = $this->getItem($key);
+            $collection[$key] = $this->getItem($key);
         }
 
         return $collection;
@@ -290,7 +290,7 @@ 
      */
     public function saveDeferred(CacheItemInterface $item)
     {
-        return $this->deferredList[ $item->getKey() ] = $item;
+        return $this->deferredList[$item->getKey()] = $item;
     }
 
     /**
@@ -303,7 +303,7 @@ 
         foreach ($this->deferredList as $key => $item) {
             $result = $this->driverWrite($item);
             if ($return !== false) {
-                unset($this->deferredList[ $key ]);
+                unset($this->deferredList[$key]);
                 $return = $result;
             }
         }

src/phpFastCache/Drivers/Mongodb/Driver.php

Doc Comments +3 added lines, -3 removed lines

@@ -67,7 +67,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     public function driverWrite(CacheItemInterface $item)
@@ -142,7 +142,7 @@ 
     }
 
     /**
-     * @return bool
+     * @return boolean|null
      * @throws MongoConnectionException
      * @throws LogicException
      */
@@ -181,7 +181,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return bool
+     * @return boolean|null
      * @throws \InvalidArgumentException
      */
     public function driverIsHit(CacheItemInterface $item)

Indentation +22 added lines, -22 removed lines

@@ -78,14 +78,14 @@ 
         if ($item instanceof Item) {
             try {
                 $result = (array) $this->getCollection()->update(
-                  ['_id' => $item->getKey()],
-                  [
+                    ['_id' => $item->getKey()],
+                    [
                     '$set' => [
-                      self::DRIVER_TIME_WRAPPER_INDEX => ($item->getTtl() > 0 ? new MongoDate(time() + $item->getTtl()) : null),
-                      self::DRIVER_DATA_WRAPPER_INDEX => new MongoBinData($this->encode($item->get()), MongoBinData::BYTE_ARRAY),
+                        self::DRIVER_TIME_WRAPPER_INDEX => ($item->getTtl() > 0 ? new MongoDate(time() + $item->getTtl()) : null),
+                        self::DRIVER_DATA_WRAPPER_INDEX => new MongoBinData($this->encode($item->get()), MongoBinData::BYTE_ARRAY),
                     ],
-                  ],
-                  ['upsert' => true, 'multiple' => false]
+                    ],
+                    ['upsert' => true, 'multiple' => false]
                 );
             } catch (MongoCursorException $e) {
                 return false;
@@ -106,8 +106,8 @@ 
         $document = $this->getCollection()->findOne(['_id' => $key], [self::DRIVER_DATA_WRAPPER_INDEX, self::DRIVER_TIME_WRAPPER_INDEX  /*'d', 'e'*/]);
         if ($document) {
             return [
-              self::DRIVER_DATA_WRAPPER_INDEX => $this->decode($document[ self::DRIVER_DATA_WRAPPER_INDEX ]->bin),
-              self::DRIVER_TIME_WRAPPER_INDEX => (new \DateTime())->setTimestamp($document[ self::DRIVER_TIME_WRAPPER_INDEX ]->sec),
+                self::DRIVER_DATA_WRAPPER_INDEX => $this->decode($document[ self::DRIVER_DATA_WRAPPER_INDEX ]->bin),
+                self::DRIVER_TIME_WRAPPER_INDEX => (new \DateTime())->setTimestamp($document[ self::DRIVER_TIME_WRAPPER_INDEX ]->sec),
             ];
         } else {
             return null;
@@ -162,10 +162,10 @@ 
              * @todo make an url builder
              */
             $this->instance = $this->instance ?: (new MongodbClient('mongodb://' .
-              ($username ?: '') .
-              ($password ? ":{$password}" : '') .
-              ($username ? '@' : '') . "{$host}" .
-              ($port != '27017' ? ":{$port}" : ''), ['timeout' => $timeout * 1000]))->phpFastCache;
+                ($username ?: '') .
+                ($password ? ":{$password}" : '') .
+                ($username ? '@' : '') . "{$host}" .
+                ($port != '27017' ? ":{$port}" : ''), ['timeout' => $timeout * 1000]))->phpFastCache;
             $this->instance->Cache->createIndex([self::DRIVER_TIME_WRAPPER_INDEX => 1], ['expireAfterSeconds' => 0]);
         }
     }
@@ -206,23 +206,23 @@ 
     public function getStats()
     {
         $serverStatus = $this->getCollection()->db->command([
-          'serverStatus' => 1,
-          'recordStats' => 0,
-          'repl' => 0,
-          'metrics' => 0,
+            'serverStatus' => 1,
+            'recordStats' => 0,
+            'repl' => 0,
+            'metrics' => 0,
         ]);
         $collStats = $this->getCollection()->db->command([
-          'collStats' => 'Cache',
-          'verbose' => true,
+            'collStats' => 'Cache',
+            'verbose' => true,
         ]);
 
         $stats = (new driverStatistic())
-          ->setInfo('MongoDB version ' . $serverStatus[ 'version' ] . ', Uptime (in days): ' . round($serverStatus[ 'uptime' ] / 86400, 1))
-          ->setSize((int) $collStats[ 'size' ])
-          ->setRawData([
+            ->setInfo('MongoDB version ' . $serverStatus[ 'version' ] . ', Uptime (in days): ' . round($serverStatus[ 'uptime' ] / 86400, 1))
+            ->setSize((int) $collStats[ 'size' ])
+            ->setRawData([
             'serverStatus' => $serverStatus,
             'collStats' => $collStats,
-          ]);
+            ]);
 
         return $stats;
     }

Spacing +12 added lines, -12 removed lines

@@ -91,7 +91,7 @@ 
                 return false;
             }
 
-            return isset($result[ 'ok' ]) ? $result[ 'ok' ] == 1 : true;
+            return isset($result['ok']) ? $result['ok'] == 1 : true;
         } else {
             throw new \InvalidArgumentException('Cross-Driver type confusion detected');
         }
@@ -106,8 +106,8 @@ 
         $document = $this->getCollection()->findOne(['_id' => $key], [self::DRIVER_DATA_WRAPPER_INDEX, self::DRIVER_TIME_WRAPPER_INDEX  /*'d', 'e'*/]);
         if ($document) {
             return [
-              self::DRIVER_DATA_WRAPPER_INDEX => $this->decode($document[ self::DRIVER_DATA_WRAPPER_INDEX ]->bin),
-              self::DRIVER_TIME_WRAPPER_INDEX => (new \DateTime())->setTimestamp($document[ self::DRIVER_TIME_WRAPPER_INDEX ]->sec),
+              self::DRIVER_DATA_WRAPPER_INDEX => $this->decode($document[self::DRIVER_DATA_WRAPPER_INDEX]->bin),
+              self::DRIVER_TIME_WRAPPER_INDEX => (new \DateTime())->setTimestamp($document[self::DRIVER_TIME_WRAPPER_INDEX]->sec),
             ];
         } else {
             return null;
@@ -127,7 +127,7 @@ 
         if ($item instanceof Item) {
             $deletionResult = (array) $this->getCollection()->remove(['_id' => $item->getKey()], ["w" => 1]);
 
-            return (int) $deletionResult[ 'ok' ] === 1 && !$deletionResult[ 'err' ];
+            return (int) $deletionResult['ok'] === 1 && !$deletionResult['err'];
         } else {
             throw new \InvalidArgumentException('Cross-Driver type confusion detected');
         }
@@ -151,11 +151,11 @@ 
         if ($this->instance instanceof MongodbClient) {
             throw new LogicException('Already connected to Mongodb server');
         } else {
-            $host = isset($this->config[ 'host' ]) ? $this->config[ 'host' ] : '127.0.0.1';
-            $port = isset($server[ 'port' ]) ? $server[ 'port' ] : '27017';
-            $timeout = isset($server[ 'timeout' ]) ? $server[ 'timeout' ] : 3;
-            $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($server['port']) ? $server['port'] : '27017';
+            $timeout = isset($server['timeout']) ? $server['timeout'] : 3;
+            $password = isset($this->config['password']) ? $this->config['password'] : '';
+            $username = isset($this->config['username']) ? $this->config['username'] : '';
 
 
             /**
@@ -188,7 +188,7 @@ 
     {
         $document = $this->getCollection()->findOne(['_id' => $item->getKey()], [self::DRIVER_TIME_WRAPPER_INDEX  /*'d', 'e'*/]);
         if ($document) {
-            return $document[ self::DRIVER_TIME_WRAPPER_INDEX ]->sec >= time();
+            return $document[self::DRIVER_TIME_WRAPPER_INDEX]->sec >= time();
         } else {
             return null;
         }
@@ -217,8 +217,8 @@ 
         ]);
 
         $stats = (new driverStatistic())
-          ->setInfo('MongoDB version ' . $serverStatus[ 'version' ] . ', Uptime (in days): ' . round($serverStatus[ 'uptime' ] / 86400, 1))
-          ->setSize((int) $collStats[ 'size' ])
+          ->setInfo('MongoDB version ' . $serverStatus['version'] . ', Uptime (in days): ' . round($serverStatus['uptime'] / 86400, 1))
+          ->setSize((int) $collStats['size'])
           ->setRawData([
             'serverStatus' => $serverStatus,
             'collStats' => $collStats,