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 +18 added lines, -18 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);
             }
         }
 
@@ -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) {
@@ -283,7 +283,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))
@@ -295,10 +295,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/CacheManager.php

Doc Comments +3 added lines, -3 removed lines

@@ -235,7 +235,7 @@ 
     }
 
     /**
-     * @return array
+     * @return ExtendedCacheItemPoolInterface[]
      */
     public static function getDefaultConfig()
     {
@@ -243,7 +243,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getStaticSystemDrivers()
     {
@@ -268,7 +268,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getStaticAllDrivers()
     {

Braces +3 added lines, -3 removed lines

@@ -109,14 +109,14 @@ 
             try{
                 self::$instances[ $instance ] = new $class($config);
                 self::$instances[ $instance ]->setEventManager(EventManager::getInstance());
-            }catch(phpFastCacheDriverCheckException $e){
+            } catch(phpFastCacheDriverCheckException $e){
                 $fallback = self::standardizeDriverName($config['fallback']);
                 if($fallback && $fallback !== $driver){
                     $class = self::getNamespacePath() . $fallback . '\Driver';
                     self::$instances[ $instance ] = new $class($config);
                     self::$instances[ $instance ]->setEventManager(EventManager::getInstance());
                     trigger_error(sprintf('The "%s" driver is unavailable at the moment, the fallback driver "%s" has been used instead.', $driver, $fallback), E_USER_WARNING);
-                }else{
+                } else{
                     throw new phpFastCacheDriverCheckException($e->getMessage(), $e->getCode(), $e);
                 }
             }
@@ -229,7 +229,7 @@ 
             self::$config = array_merge(self::$config, $name);
         } else if (is_string($name)){
             self::$config[ $name ] = $value;
-        }else{
+        } else{
             throw new phpFastCacheInvalidArgumentException('Invalid variable type: $name');
         }
     }

Spacing +30 added lines, -30 removed lines

@@ -169,32 +169,32 @@ 
         }
 
         $instance = crc32($driver . serialize($config));
-        if (!isset(self::$instances[ $instance ])) {
+        if (!isset(self::$instances[$instance])) {
             $badPracticeOmeter[$driver] = 1;
-            if(!$config['ignoreSymfonyNotice'] && interface_exists('Symfony\Component\HttpKernel\KernelInterface') && !class_exists('phpFastCache\Bundle\phpFastCacheBundle')){
+            if (!$config['ignoreSymfonyNotice'] && interface_exists('Symfony\Component\HttpKernel\KernelInterface') && !class_exists('phpFastCache\Bundle\phpFastCacheBundle')) {
                 trigger_error('A Symfony Bundle to make the PhpFastCache integration more easier is now available here: https://github.com/PHPSocialNetwork/phpfastcache-bundle', E_USER_NOTICE);
             }
             $class = self::getNamespacePath() . $driver . '\Driver';
-            try{
-                self::$instances[ $instance ] = new $class($config);
-                self::$instances[ $instance ]->setEventManager(EventManager::getInstance());
-            }catch(phpFastCacheDriverCheckException $e){
+            try {
+                self::$instances[$instance] = new $class($config);
+                self::$instances[$instance]->setEventManager(EventManager::getInstance());
+            } catch (phpFastCacheDriverCheckException $e) {
                 $fallback = self::standardizeDriverName($config['fallback']);
-                if($fallback && $fallback !== $driver){
+                if ($fallback && $fallback !== $driver) {
                     $class = self::getNamespacePath() . $fallback . '\Driver';
-                    self::$instances[ $instance ] = new $class($config);
-                    self::$instances[ $instance ]->setEventManager(EventManager::getInstance());
+                    self::$instances[$instance] = new $class($config);
+                    self::$instances[$instance]->setEventManager(EventManager::getInstance());
                     trigger_error(sprintf('The "%s" driver is unavailable at the moment, the fallback driver "%s" has been used instead.', $driver, $fallback), E_USER_WARNING);
-                }else{
+                } else {
                     throw new phpFastCacheDriverCheckException($e->getMessage(), $e->getCode(), $e);
                 }
             }
-        } else if(++$badPracticeOmeter[$driver] >= 5){
+        } else if (++$badPracticeOmeter[$driver] >= 5) {
             trigger_error('[' . $driver . '] Calling many times CacheManager::getInstance() for already instanced drivers is a bad practice and have a significant impact on performances.
            See https://github.com/PHPSocialNetwork/phpfastcache/wiki/[V5]-Why-calling-getInstance%28%29-each-time-is-a-bad-practice-%3F');
         }
 
-        return self::$instances[ $instance ];
+        return self::$instances[$instance];
     }
 
     /**
@@ -255,7 +255,7 @@ 
      */
     public static function __callStatic($name, $arguments)
     {
-        $options = (array_key_exists(0, $arguments) && is_array($arguments) ? $arguments[ 0 ] : []);
+        $options = (array_key_exists(0, $arguments) && is_array($arguments) ? $arguments[0] : []);
 
         return self::getInstance($name, $options);
     }
@@ -296,9 +296,9 @@ 
     {
         if (is_array($name)) {
             self::$config = array_merge(self::$config, $name);
-        } else if (is_string($name)){
-            self::$config[ $name ] = $value;
-        }else{
+        } else if (is_string($name)) {
+            self::$config[$name] = $value;
+        } else {
             throw new phpFastCacheInvalidArgumentException('Invalid variable type: $name');
         }
     }
@@ -367,70 +367,70 @@ 
     protected static function validateConfig(array $config)
     {
         foreach ($config as $configName => $configValue) {
-            switch($configName)
+            switch ($configName)
             {
                 case 'itemDetailedDate':
-                    if(!is_bool($configValue)){
+                    if (!is_bool($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
                     }
                     break;
                 case 'autoTmpFallback':
-                    if(!is_bool($configValue)){
+                    if (!is_bool($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
                     }
                     break;
                 case 'secureFileManipulation':
-                    if(!is_bool($configValue)){
+                    if (!is_bool($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
                     }
                     break;
                 case 'ignoreSymfonyNotice':
-                    if(!is_bool($configValue)){
+                    if (!is_bool($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
                     }
                     break;
                 case 'defaultTtl':
-                    if(!is_numeric($configValue)){
+                    if (!is_numeric($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be numeric");
                     }
                     break;
                 case 'defaultKeyHashFunction':
-                    if(!is_string($configValue) && !function_exists($configValue)){
+                    if (!is_string($configValue) && !function_exists($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a valid function name string");
                     }
                     break;
                 case 'securityKey':
-                    if(!is_string($configValue)){
+                    if (!is_string($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a string");
                     }
                     break;
                 case 'htaccess':
-                    if(!is_bool($configValue)){
+                    if (!is_bool($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
                     }
                     break;
                 case 'default_chmod':
-                    if(!is_int($configValue)){
+                    if (!is_int($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be an integer");
                     }
                     break;
                 case 'path':
-                    if(!is_string($configValue)){
+                    if (!is_string($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a string");
                     }
                     break;
                 case 'fallback':
-                    if(!is_bool($configValue)){
+                    if (!is_bool($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
                     }
                     break;
                 case 'limited_memory_each_object':
-                    if(!is_int($configValue)){
+                    if (!is_int($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be an integer");
                     }
                     break;
                 case 'compress_data':
-                    if(!is_bool($configValue)){
+                    if (!is_bool($configValue)) {
                         throw new phpFastCacheInvalidConfigurationException("{$configName} must be a boolean");
                     }
                     break;

Indentation +36 added lines, -36 removed lines

@@ -64,95 +64,95 @@ 
         /**
          * Specify if the item must provide detailed creation/modification dates
          */
-      'itemDetailedDate' => false,
+        'itemDetailedDate' => false,
 
         /**
          * Automatically attempt to fallback to temporary directory
          * if the cache fails to write on the specified directory
          */
-      'autoTmpFallback' => false,
+        'autoTmpFallback' => false,
 
         /**
          * Provide a secure file manipulation mechanism,
          * on intensive usage the performance can be affected.
          */
-      'secureFileManipulation' => false,
+        'secureFileManipulation' => false,
 
         /**
          * Ignore Symfony notice for Symfony project which
          * do not makes use of PhpFastCache's Symfony Bundle
          */
-      'ignoreSymfonyNotice' => false,
+        'ignoreSymfonyNotice' => false,
 
         /**
          * Default time-to-live in second
          */
-      'defaultTtl' => 900,
+        'defaultTtl' => 900,
 
         /**
          * Default key hash function
          * (md5 by default)
          */
-      'defaultKeyHashFunction' => '',
+        'defaultKeyHashFunction' => '',
 
         /**
          * The securityKey that will be used
          * to create sub-directory
          * (Files-based drivers only)
          */
-      'securityKey' => 'auto',
+        'securityKey' => 'auto',
 
         /**
          * Auto-generate .htaccess if it's missing
          * (Files-based drivers only)
          */
-      'htaccess' => true,
+        'htaccess' => true,
 
         /**
          * Default files chmod
          * 0777 recommended
          * (Files-based drivers only)
          */
-      'default_chmod' => 0777,
+        'default_chmod' => 0777,
 
         /**
          * The path where we will writecache files
          * default value if empty: sys_get_temp_dir()
          * (Files-based drivers only)
          */
-      'path' => '',
+        'path' => '',
 
         /**
          * Driver fallback in case of failure.
          * Caution, in case of failure an E_WARNING
          * error will always be raised
          */
-      'fallback' => false,
+        'fallback' => false,
 
         /**
          * Maximum size (bytes) of object store in memory
          * (Memcache(d) drivers only)
          */
-      'limited_memory_each_object' => 4096,
+        'limited_memory_each_object' => 4096,
 
         /**
          * Compress stored data, if the backend supports it
          * (Memcache(d) drivers only)
          */
-      'compress_data' => false,
+        'compress_data' => false,
 
         /**
          * Prevent cache slams when
          * making use of heavy cache
          * items
          */
-      'preventCacheSlams' => false,
+        'preventCacheSlams' => false,
 
         /**
          * Cache slams timeout
          * in seconds
          */
-      'cacheSlamsTimeout' => 15,
+        'cacheSlamsTimeout' => 15,
 
     ];
 
@@ -335,24 +335,24 @@ 
     public static function getStaticSystemDrivers()
     {
         return [
-          'Apc',
-          'Apcu',
-          'Cassandra',
-          'Couchbase',
-          'Couchdb',
-          'Devnull',
-          'Files',
-          'Leveldb',
-          'Memcache',
-          'Memcached',
-          'Memstatic',
-          'Mongodb',
-          'Predis',
-          'Redis',
-          'Ssdb',
-          'Sqlite',
-          'Wincache',
-          'Xcache',
+            'Apc',
+            'Apcu',
+            'Cassandra',
+            'Couchbase',
+            'Couchdb',
+            'Devnull',
+            'Files',
+            'Leveldb',
+            'Memcache',
+            'Memcached',
+            'Memstatic',
+            'Mongodb',
+            'Predis',
+            'Redis',
+            'Ssdb',
+            'Sqlite',
+            'Wincache',
+            'Xcache',
         ];
     }
 
@@ -362,9 +362,9 @@ 
     public static function getStaticAllDrivers()
     {
         return array_merge(self::getStaticSystemDrivers(), [
-          'Devtrue',
-          'Devfalse',
-          'Cookie',
+            'Devtrue',
+            'Devfalse',
+            'Cookie',
         ]);
     }
 

src/phpFastCache/Drivers/Predis/Driver.php

Spacing +2 added lines, -2 removed lines

@@ -124,7 +124,7 @@ 
      */
     protected function driverConnect()
     {
-        $config = isset($this->config[ 'predis' ]) ? $this->config[ 'predis' ] : [];
+        $config = isset($this->config['predis']) ? $this->config['predis'] : [];
 
         $this->instance = new PredisClient(array_merge([
           'host' => '127.0.0.1',
@@ -150,7 +150,7 @@ 
         $info = $this->instance->info();
         $size = (isset($info['Memory']['used_memory']) ? $info['Memory']['used_memory'] : 0);
         $version = (isset($info['Server']['redis_version']) ? $info['Server']['redis_version'] : 0);
-        $date = (isset($info['Server'][ 'uptime_in_seconds' ]) ? (new \DateTime())->setTimestamp(time() - $info['Server'][ 'uptime_in_seconds' ]) : 'unknown date');
+        $date = (isset($info['Server']['uptime_in_seconds']) ? (new \DateTime())->setTimestamp(time() - $info['Server']['uptime_in_seconds']) : 'unknown date');
 
         return (new driverStatistic())
           ->setData(implode(', ', array_keys($this->itemInstances)))

Indentation +8 added lines, -8 removed lines

@@ -125,10 +125,10 @@ 
         $config = isset($this->config[ 'predis' ]) ? $this->config[ 'predis' ] : [];
 
         $this->instance = new PredisClient(array_merge([
-          'host' => '127.0.0.1',
-          'port' => 6379,
-          'password' => null,
-          'database' => null,
+            'host' => '127.0.0.1',
+            'port' => 6379,
+            'password' => null,
+            'database' => null,
         ], $config));
 
         return true;
@@ -151,9 +151,9 @@ 
         $date = (isset($info['Server'][ 'uptime_in_seconds' ]) ? (new \DateTime())->setTimestamp(time() - $info['Server'][ 'uptime_in_seconds' ]) : 'unknown date');
 
         return (new driverStatistic())
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData($info)
-          ->setSize($size)
-          ->setInfo(sprintf("The Redis daemon v%s is up since %s.\n For more information see RawData. \n Driver size includes the memory allocation size.", $version, $date->format(DATE_RFC2822)));
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setRawData($info)
+            ->setSize($size)
+            ->setInfo(sprintf("The Redis daemon v%s is up since %s.\n For more information see RawData. \n Driver size includes the memory allocation size.", $version, $date->format(DATE_RFC2822)));
     }
 }
\ No newline at end of file

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']){
 
                                 /**

Spacing +22 added lines, -22 removed lines

@@ -62,7 +62,7 @@ 
     public function getItem($key)
     {
         if (is_string($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');
             }
             if (!array_key_exists($key, $this->itemInstances)) {
@@ -81,14 +81,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
@@ -120,7 +120,7 @@ 
                         $item->set($driverData);
                         $item->expiresAt($this->driverUnwrapEdate($driverArray));
 
-                        if($this->config['itemDetailedDate']){
+                        if ($this->config['itemDetailedDate']) {
 
                             /**
                              * If the itemDetailedDate has been
@@ -148,10 +148,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
@@ -164,8 +164,8 @@ 
                         } else {
                             $item->setHit(true);
                         }
-                    }else{
-                        $item->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]));
+                    } else {
+                        $item->expiresAfter(abs((int) $this->getConfig()['defaultTtl']));
                     }
                 }
             }
@@ -178,9 +178,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];
     }
 
     /**
@@ -191,7 +191,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 {
@@ -208,7 +208,7 @@ 
     {
         $collection = [];
         foreach ($keys as $key) {
-            $collection[ $key ] = $this->getItem($key);
+            $collection[$key] = $this->getItem($key);
         }
 
         return $collection;
@@ -305,8 +305,8 @@ 
          * @var ExtendedCacheItemInterface $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.');
         }
 
@@ -318,7 +318,7 @@ 
         $this->eventManager->dispatch('CacheSaveItem', $this, $item);
 
 
-        if($this->getConfig()[ 'preventCacheSlams' ]){
+        if ($this->getConfig()['preventCacheSlams']) {
             /**
              * @var $itemBatch ExtendedCacheItemInterface
              */
@@ -326,7 +326,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
@@ -358,8 +358,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.');
         }
 
@@ -370,7 +370,7 @@ 
          */
         $this->eventManager->dispatch('CacheSaveDeferredItem', $this, $item);
 
-        return $this->deferredList[ $item->getKey() ] = $item;
+        return $this->deferredList[$item->getKey()] = $item;
     }
 
     /**
@@ -390,7 +390,7 @@ 
         foreach ($this->deferredList as $key => $item) {
             $result = $this->save($item);
             if ($return !== false) {
-                unset($this->deferredList[ $key ]);
+                unset($this->deferredList[$key]);
                 $return = $result;
             }
         }

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.');
         }
 

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)

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.');

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);
                 }
 

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

Spacing +9 added lines, -9 removed lines

@@ -69,8 +69,8 @@ 
     public function dispatch($eventName, ...$args)
     {
         $eventName = 'on' . ucfirst($eventName);
-        if(array_key_exists($eventName, $this->events)){
-            foreach ($this->events[ $eventName ] as $event) {
+        if (array_key_exists($eventName, $this->events)) {
+            foreach ($this->events[$eventName] as $event) {
                 call_user_func_array($event, $args);
             }
         }
@@ -84,17 +84,17 @@ 
      */
     public function __call($name, $arguments)
     {
-        if(strpos($name, 'on') === 0){
-            if(is_callable($arguments[0])){
-                if(isset($arguments[1]) && is_string($arguments[0])){
+        if (strpos($name, 'on') === 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"');
         }
     }
@@ -106,7 +106,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;
         }

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"');
         }
     }