PHPSocialNetwork / phpfastcache

src/phpFastCache/Drivers/Memcache/Driver.php

Indentation +5 added lines, -5 removed lines

@@ -131,7 +131,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],
             ];
         }
 
@@ -161,9 +161,9 @@ 
         $date = (new \DateTime())->setTimestamp(time() - $stats[ 'uptime' ]);
 
         return (new driverStatistic())
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats[ 'version' ], $date->format(DATE_RFC2822)))
-          ->setRawData($stats)
-          ->setSize($stats[ 'bytes' ]);
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats[ 'version' ], $date->format(DATE_RFC2822)))
+            ->setRawData($stats)
+            ->setSize($stats[ 'bytes' ]);
     }
 }
\ No newline at end of file

Spacing +6 added lines, -6 removed lines

@@ -52,7 +52,7 @@ 
             $this->instance = new MemcacheSoftware();
             $this->driverConnect();
 
-            if (array_key_exists('compress_data', $config) && $config[ 'compress_data' ] === true) {
+            if (array_key_exists('compress_data', $config) && $config['compress_data'] === true) {
                 $this->memcacheFlags = MEMCACHE_COMPRESSED;
             }
         }
@@ -128,7 +128,7 @@ 
      */
     protected 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],
@@ -137,7 +137,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) {
@@ -158,12 +158,12 @@ 
     public function getStats()
     {
         $stats = (array) $this->instance->getstats();
-        $date = (new \DateTime())->setTimestamp(time() - $stats[ 'uptime' ]);
+        $date = (new \DateTime())->setTimestamp(time() - $stats['uptime']);
 
         return (new driverStatistic())
           ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats[ 'version' ], $date->format(DATE_RFC2822)))
+          ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats['version'], $date->format(DATE_RFC2822)))
           ->setRawData($stats)
-          ->setSize($stats[ 'bytes' ]);
+          ->setSize($stats['bytes']);
     }
 }
\ No newline at end of file

src/phpFastCache/Drivers/Ssdb/Driver.php

Indentation +7 added lines, -7 removed lines

@@ -139,10 +139,10 @@ 
     {
         try {
             $server = isset($this->config[ 'ssdb' ]) ? $this->config[ 'ssdb' ] : [
-              'host' => "127.0.0.1",
-              'port' => 8888,
-              'password' => '',
-              'timeout' => 2000,
+                'host' => "127.0.0.1",
+                'port' => 8888,
+                'password' => '',
+                'timeout' => 2000,
             ];
 
             $host = $server[ 'host' ];
@@ -183,9 +183,9 @@ 
          * using hardcoded offset of pair key-value :-(
          */
         $stat->setInfo(sprintf("Ssdb-server v%s with a total of %s call(s).\n For more information see RawData.", $info[ 2 ], $info[ 6 ]))
-          ->setRawData($info)
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setSize($this->instance->dbsize());
+            ->setRawData($info)
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setSize($this->instance->dbsize());
 
         return $stat;
     }

Spacing +6 added lines, -6 removed lines

@@ -138,17 +138,17 @@ 
     protected function driverConnect()
     {
         try {
-            $server = isset($this->config[ 'ssdb' ]) ? $this->config[ 'ssdb' ] : [
+            $server = isset($this->config['ssdb']) ? $this->config['ssdb'] : [
               'host' => "127.0.0.1",
               'port' => 8888,
               'password' => '',
               'timeout' => 2000,
             ];
 
-            $host = $server[ 'host' ];
-            $port = isset($server[ 'port' ]) ? (int) $server[ 'port' ] : 8888;
-            $password = isset($server[ 'password' ]) ? $server[ 'password' ] : '';
-            $timeout = !empty($server[ 'timeout' ]) ? (int) $server[ 'timeout' ] : 2000;
+            $host = $server['host'];
+            $port = isset($server['port']) ? (int) $server['port'] : 8888;
+            $password = isset($server['password']) ? $server['password'] : '';
+            $timeout = !empty($server['timeout']) ? (int) $server['timeout'] : 2000;
             $this->instance = new SimpleSSDB($host, $port, $timeout);
             if (!empty($password)) {
                 $this->instance->auth($password);
@@ -182,7 +182,7 @@ 
          * Data returned by Ssdb are very poorly formatted
          * using hardcoded offset of pair key-value :-(
          */
-        $stat->setInfo(sprintf("Ssdb-server v%s with a total of %s call(s).\n For more information see RawData.", $info[ 2 ], $info[ 6 ]))
+        $stat->setInfo(sprintf("Ssdb-server v%s with a total of %s call(s).\n For more information see RawData.", $info[2], $info[6]))
           ->setRawData($info)
           ->setData(implode(', ', array_keys($this->itemInstances)))
           ->setSize($this->instance->dbsize());

src/phpFastCache/Proxy/phpFastCacheAbstractProxy.php

Spacing +2 added lines, -2 removed lines

@@ -74,9 +74,9 @@
      */
     public function __call($name, $args)
     {
-        if(method_exists($this->instance, $name)){
+        if (method_exists($this->instance, $name)) {
             return call_user_func_array([$this->instance, $name], $args);
-        }else{
+        } else {
             throw new \BadMethodCallException(sprintf('Method %s does not exists', $name));
         }
     }

Braces +1 added lines, -1 removed lines

@@ -273,7 +273,7 @@
         
         if (!is_dir($path)) {
             throw new phpFastCacheDriverException("Can't read PATH:" . $path, 94);
-        }else{
+        } else{
             $size = Directory::dirSize($path);
         }
 

src/phpFastCache/CacheManager.php

Braces +3 added lines, -3 removed lines

@@ -95,13 +95,13 @@ 
             $class = self::getNamespacePath() . $driver . '\Driver';
             try{
                 self::$instances[ $instance ] = new $class($config);
-            }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);
                     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);
                 }
             }
@@ -198,7 +198,7 @@ 
             self::$config = array_merge(self::$config, $name);
         } else if (is_string($name)){
             self::$config[ $name ] = $value;
-        }else{
+        } else{
             throw new \InvalidArgumentException('Invalid variable type: $name');
         }
     }

Indentation +25 added lines, -25 removed lines

@@ -50,15 +50,15 @@ 
      * @var array
      */
     protected static $config = [
-      'itemDetailedDate' => false,// Specify if the item must provide detailed creation/modification dates
-      'defaultTtl' => 900,// Default time-to-live in second
-      'securityKey' => 'auto',// The securityKey that will be used to create sub-directory
-      'htaccess' => true,// Auto-generate .htaccess if tit is missing
-      'default_chmod' => 0777, // 0777 recommended
-      'path' => '',// if not set will be the value of sys_get_temp_dir()
-      'fallback' => false, //Fall back when old driver is not support
-      'limited_memory_each_object' => 4096, // maximum size (bytes) of object store in memory
-      'compress_data' => false, // compress stored data, if the backend supports it
+        'itemDetailedDate' => false,// Specify if the item must provide detailed creation/modification dates
+        'defaultTtl' => 900,// Default time-to-live in second
+        'securityKey' => 'auto',// The securityKey that will be used to create sub-directory
+        'htaccess' => true,// Auto-generate .htaccess if tit is missing
+        'default_chmod' => 0777, // 0777 recommended
+        'path' => '',// if not set will be the value of sys_get_temp_dir()
+        'fallback' => false, //Fall back when old driver is not support
+        'limited_memory_each_object' => 4096, // maximum size (bytes) of object store in memory
+        'compress_data' => false, // compress stored data, if the backend supports it
     ];
 
     /**
@@ -106,7 +106,7 @@ 
                 }
             }
         } 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.
+            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');
         }
 
@@ -219,21 +219,21 @@ 
     public static function getStaticSystemDrivers()
     {
         return [
-          'Sqlite',
-          'Files',
-          'Apc',
-          'Apcu',
-          'Memcache',
-          'Memcached',
-          'Couchbase',
-          'Mongodb',
-          'Predis',
-          'Redis',
-          'Ssdb',
-          'Leveldb',
-          'Wincache',
-          'Xcache',
-          'Devnull',
+            'Sqlite',
+            'Files',
+            'Apc',
+            'Apcu',
+            'Memcache',
+            'Memcached',
+            'Couchbase',
+            'Mongodb',
+            'Predis',
+            'Redis',
+            'Ssdb',
+            'Leveldb',
+            'Wincache',
+            'Xcache',
+            'Devnull',
         ];
     }
 

Spacing +18 added lines, -18 removed lines

@@ -50,12 +50,12 @@ 
      * @var array
      */
     protected static $config = [
-      'itemDetailedDate' => false,// Specify if the item must provide detailed creation/modification dates
-      'defaultTtl' => 900,// Default time-to-live in second
-      'securityKey' => 'auto',// The securityKey that will be used to create sub-directory
-      'htaccess' => true,// Auto-generate .htaccess if tit is missing
+      'itemDetailedDate' => false, // Specify if the item must provide detailed creation/modification dates
+      'defaultTtl' => 900, // Default time-to-live in second
+      'securityKey' => 'auto', // The securityKey that will be used to create sub-directory
+      'htaccess' => true, // Auto-generate .htaccess if tit is missing
       'default_chmod' => 0777, // 0777 recommended
-      'path' => '',// if not set will be the value of sys_get_temp_dir()
+      'path' => '', // if not set will be the value of sys_get_temp_dir()
       'fallback' => false, //Fall back when old driver is not support
       'limited_memory_each_object' => 4096, // maximum size (bytes) of object store in memory
       'compress_data' => false, // compress stored data, if the backend supports it
@@ -90,27 +90,27 @@ 
         }
 
         $instance = crc32($driver . serialize($config));
-        if (!isset(self::$instances[ $instance ])) {
+        if (!isset(self::$instances[$instance])) {
             $badPracticeOmeter[$driver] = 1;
             $class = self::getNamespacePath() . $driver . '\Driver';
-            try{
-                self::$instances[ $instance ] = new $class($config);
-            }catch(phpFastCacheDriverCheckException $e){
+            try {
+                self::$instances[$instance] = new $class($config);
+            } 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] = new $class($config);
                     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];
     }
 
     /**
@@ -143,7 +143,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);
     }
@@ -198,9 +198,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 \InvalidArgumentException('Invalid variable type: $name');
         }
     }

src/autoload.php

Indentation +1 added lines, -1 removed lines

@@ -51,5 +51,5 @@
 
 if (class_exists('Composer\Autoload\ClassLoader')) {
     trigger_error('Your project already makes use of Composer. You SHOULD use the composer dependency "phpfastcache/phpfastcache" instead of hard-autoloading.',
-      E_USER_WARNING);
+        E_USER_WARNING);
 }
\ No newline at end of file

Spacing +3 added lines, -3 removed lines

@@ -18,9 +18,9 @@ 
 /**
  * Register Autoload
  */
-spl_autoload_register(function ($entity) {
+spl_autoload_register(function($entity) {
     $module = explode('\\', $entity, 2);
-    if (!in_array($module[ 0 ], ['phpFastCache', 'Psr'])) {
+    if (!in_array($module[0], ['phpFastCache', 'Psr'])) {
         /**
          * Not a part of phpFastCache file
          * then we return here.
@@ -32,7 +32,7 @@ 
 
         if (is_readable($path)) {
             require_once $path;
-        }else{
+        } else {
             trigger_error('Cannot locate the Psr/Cache files', E_USER_ERROR);
             return;
         }

Braces +1 added lines, -1 removed lines

@@ -273,7 +273,7 @@
         
         if (!is_dir($path)) {
             throw new phpFastCacheDriverException("Can't read PATH:" . $path, 94);
-        }else{
+        } else{
             $size = Directory::dirSize($path);
         }
 

tests/Autoload.test.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
 if (!class_exists('phpFastCache\CacheManager')) {
     echo "[FAIL] Autoload failed to find the CacheManager\n";
     $status = 255;
-}else{
+} else {
     echo "[PASS] Autoload successfully found the CacheManager\n";
 }
 
@@ -28,7 +28,7 @@ 
 if (!interface_exists('Psr\Cache\CacheItemInterface')) {
     echo "[FAIL] Autoload failed to find the Psr CacheItemInterface\n";
     $status = 255;
-}else{
+} else {
     echo "[PASS] Autoload successfully found the Psr CacheItemInterface\n";
 }
 

Braces +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
 if (!class_exists('phpFastCache\CacheManager')) {
     echo "[FAIL] Autoload failed to find the CacheManager\n";
     $status = 255;
-}else{
+} else{
     echo "[PASS] Autoload successfully found the CacheManager\n";
 }
 
@@ -28,7 +28,7 @@ 
 if (!interface_exists('Psr\Cache\CacheItemInterface')) {
     echo "[FAIL] Autoload failed to find the Psr CacheItemInterface\n";
     $status = 255;
-}else{
+} else{
     echo "[PASS] Autoload successfully found the Psr CacheItemInterface\n";
 }
 

src/phpFastCache/Core/Pool/CacheItemPoolTrait.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()

Braces +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
                     } else {
                         $item->setHit(true);
                     }
-                }else{
+                } else{
                     $item->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]));
                 }
 

Spacing +10 added lines, -10 removed lines

@@ -61,7 +61,7 @@ 
                     $item->set($this->driverUnwrapData($driverArray));
                     $item->expiresAt($this->driverUnwrapEdate($driverArray));
 
-                    if($this->config['itemDetailedDate']){
+                    if ($this->config['itemDetailedDate']) {
 
                         /**
                          * If the itemDetailedDate has been
@@ -85,8 +85,8 @@ 
                     } else {
                         $item->setHit(true);
                     }
-                }else{
-                    $item->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]));
+                } else {
+                    $item->expiresAfter(abs((int) $this->getConfig()['defaultTtl']));
                 }
 
             }
@@ -94,7 +94,7 @@ 
             throw new \InvalidArgumentException(sprintf('$key must be a string, got type "%s" instead.', gettype($key)));
         }
 
-        return $this->itemInstances[ $key ];
+        return $this->itemInstances[$key];
     }
 
     /**
@@ -105,7 +105,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 {
@@ -122,7 +122,7 @@ 
     {
         $collection = [];
         foreach ($keys as $key) {
-            $collection[ $key ] = $this->getItem($key);
+            $collection[$key] = $this->getItem($key);
         }
 
         return $collection;
@@ -203,7 +203,7 @@ 
          * @var ExtendedCacheItemInterface $item
          */
         if (!array_key_exists($item->getKey(), $this->itemInstances)) {
-            $this->itemInstances[ $item->getKey() ] = $item;
+            $this->itemInstances[$item->getKey()] = $item;
         }
         if ($this->driverWrite($item) && $this->driverWriteTags($item)) {
             $item->setHit(true);
@@ -222,10 +222,10 @@ 
     public function saveDeferred(CacheItemInterface $item)
     {
         if (!array_key_exists($item->getKey(), $this->itemInstances)) {
-            $this->itemInstances[ $item->getKey() ] = $item;
+            $this->itemInstances[$item->getKey()] = $item;
         }
 
-        return $this->deferredList[ $item->getKey() ] = $item;
+        return $this->deferredList[$item->getKey()] = $item;
     }
 
     /**
@@ -238,7 +238,7 @@ 
         foreach ($this->deferredList as $key => $item) {
             $result = $this->save($item);
             if ($return !== false) {
-                unset($this->deferredList[ $key ]);
+                unset($this->deferredList[$key]);
                 $return = $result;
             }
         }

src/phpFastCache/Core/Pool/DriverBaseTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -115,7 +115,7 @@
 
 
     /**
-     * @param $file
+     * @param string $file
      * @return string
      * @throws \Exception
      */

Indentation +3 added lines, -3 removed lines

@@ -159,9 +159,9 @@
     public function driverPreWrap(ExtendedCacheItemInterface $item)
     {
         $wrap = [
-          self::DRIVER_DATA_WRAPPER_INDEX => $item->get(),
-          self::DRIVER_TAGS_WRAPPER_INDEX => $item->getTags(),
-          self::DRIVER_EDATE_WRAPPER_INDEX => $item->getExpirationDate(),
+            self::DRIVER_DATA_WRAPPER_INDEX => $item->get(),
+            self::DRIVER_TAGS_WRAPPER_INDEX => $item->getTags(),
+            self::DRIVER_EDATE_WRAPPER_INDEX => $item->getExpirationDate(),
         ];
 
         if($this->config['itemDetailedDate']){

Braces +1 added lines, -1 removed lines

@@ -273,7 +273,7 @@
         
         if (!is_dir($path)) {
             throw new phpFastCacheDriverException("Can't read PATH:" . $path, 94);
-        }else{
+        } else{
             $size = Directory::dirSize($path);
         }
 

Spacing +14 added lines, -14 removed lines

@@ -52,7 +52,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;
         }
     }
 
@@ -164,16 +164,16 @@ 
           self::DRIVER_EDATE_WRAPPER_INDEX => $item->getExpirationDate(),
         ];
 
-        if($this->config['itemDetailedDate']){
-            $wrap[ self::DRIVER_MDATE_WRAPPER_INDEX ] = new \DateTime();
+        if ($this->config['itemDetailedDate']) {
+            $wrap[self::DRIVER_MDATE_WRAPPER_INDEX] = new \DateTime();
             /**
              * If the creation date exists
              * reuse it else set a new Date
              */
-            $wrap[ self::DRIVER_CDATE_WRAPPER_INDEX ] = $item->getCreationDate() ?: new \DateTime();
-        }else{
-            $wrap[ self::DRIVER_MDATE_WRAPPER_INDEX ] = null;
-            $wrap[ self::DRIVER_CDATE_WRAPPER_INDEX ] = null;
+            $wrap[self::DRIVER_CDATE_WRAPPER_INDEX] = $item->getCreationDate() ?: new \DateTime();
+        } else {
+            $wrap[self::DRIVER_MDATE_WRAPPER_INDEX] = null;
+            $wrap[self::DRIVER_CDATE_WRAPPER_INDEX] = null;
         }
 
         return $wrap;
@@ -185,7 +185,7 @@ 
      */
     public function driverUnwrapData(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_DATA_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_DATA_WRAPPER_INDEX];
     }
 
     /**
@@ -194,7 +194,7 @@ 
      */
     public function driverUnwrapTags(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_TAGS_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_TAGS_WRAPPER_INDEX];
     }
 
 
@@ -204,7 +204,7 @@ 
      */
     public function driverUnwrapEdate(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_EDATE_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_EDATE_WRAPPER_INDEX];
     }
 
     /**
@@ -213,7 +213,7 @@ 
      */
     public function driverUnwrapCdate(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_CDATE_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_CDATE_WRAPPER_INDEX];
     }
 
 
@@ -223,7 +223,7 @@ 
      */
     public function driverUnwrapMdate(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_MDATE_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_MDATE_WRAPPER_INDEX];
     }
 
     /**
@@ -248,7 +248,7 @@ 
          * on tags item, it can leads
          * to an infinite recursive calls
          */
-        if(strpos($item->getKey(), self::DRIVER_TAGS_KEY_PREFIX ) === 0){
+        if (strpos($item->getKey(), self::DRIVER_TAGS_KEY_PREFIX) === 0) {
             throw new \LogicException('Trying to set tag(s) to an Tag item index: ' . $item->getKey());
         }
 
@@ -291,7 +291,7 @@ 
         foreach ($tagsItems as $tagsItem) {
             $data = (array) $tagsItem->get();
 
-            unset($data[ $item->getKey() ]);
+            unset($data[$item->getKey()]);
             $tagsItem->set($data);
 
             /**

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

Doc Comments +3 added lines, -4 removed lines

@@ -25,7 +25,6 @@ 
     public $tmp = [];
 
     /**
-     * @param bool $skip_create_path
      * @param $config
      * @return string
      * @throws \Exception
@@ -118,7 +117,7 @@ 
     }
 
     /**
-     * @param $keyword
+     * @param string|false $keyword
      * @param bool $skip
      * @return string
      * @throws phpFastCacheDriverException
@@ -165,7 +164,7 @@ 
 
     /**
      * @param $filename
-     * @return mixed
+     * @return string
      */
     protected static function cleanFileName($filename)
     {
@@ -180,7 +179,7 @@ 
     }
 
     /**
-     * @param $path
+     * @param string $path
      * @param bool $create
      * @throws \Exception
      */

Indentation +3 added lines, -3 removed lines

@@ -185,9 +185,9 @@
     protected static function cleanFileName($filename)
     {
         $regex = [
-          '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/',
-          '/\.$/',
-          '/^\./',
+            '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/',
+            '/\.$/',
+            '/^\./',
         ];
         $replace = ['-', '', ''];
 

Spacing +15 added lines, -15 removed lines

@@ -34,30 +34,30 @@ 
     {
         $tmp_dir = rtrim(ini_get('upload_tmp_dir') ? ini_get('upload_tmp_dir') : sys_get_temp_dir(), '\\/') . DIRECTORY_SEPARATOR . 'phpfastcache';
 
-        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__, '/') . 'PathSeekerTrait.php/';
+                $document_root_path = rtrim($_SERVER['DOCUMENT_ROOT'], '/') . '/../';
+                $path = isset($_SERVER['DOCUMENT_ROOT']) && is_writable($document_root_path) ? $document_root_path : rtrim(__DIR__, '/') . 'PathSeekerTrait.php/';
             }
 
-            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'];
         }
 
         if ($getBasePath === true) {
             return $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') {
-            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,7 +73,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)) {
@@ -97,8 +97,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);
@@ -171,10 +171,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'];
         }
     }
 
@@ -204,7 +204,7 @@ 
         if ($create === true) {
             if (!is_writable($path)) {
                 try {
-                    if(!chmod($path, 0777)){
+                    if (!chmod($path, 0777)) {
                         throw new phpFastCacheDriverException('Chmod failed on : ' . $path);
                     }
                 } catch (phpFastCacheDriverException $e) {