PHPSocialNetwork / phpfastcache

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

Doc Comments +4 added lines, -4 removed lines

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

Spacing +20 added lines, -20 removed lines

@@ -34,26 +34,26 @@ 
     {
         $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'];
         }
 
-        $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');
             }
@@ -72,14 +72,14 @@ 
             return $full_path;
         }
 
-        if (!isset($this->tmp[ $full_pathx ]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
+        if (!isset($this->tmp[$full_pathx]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
             if (!@file_exists($full_path)) {
                 @mkdir($full_path, $this->setChmodAuto(), true);
-            }else if (!@is_writable($full_path)) {
+            } else if (!@is_writable($full_path)) {
                 @chmod($full_path, $this->setChmodAuto());
             }
 
-            if ($this->config[ 'autoTmpFallback' ] && !@is_writable($full_path)) {
+            if ($this->config['autoTmpFallback'] && !@is_writable($full_path)) {
                 /**
                  * Switch back to tmp dir
                  * again if the path is not writable
@@ -93,8 +93,8 @@ 
                 throw new phpFastCacheIOException('PLEASE CREATE OR CHMOD ' . $full_path . ' - 0777 OR ANY WRITABLE PERMISSION!');
             }
 
-            $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);
@@ -167,10 +167,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'];
         }
     }
 
@@ -200,7 +200,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) {
@@ -268,7 +268,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))
@@ -280,10 +280,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);

Braces +2 added lines, -2 removed lines

@@ -75,7 +75,7 @@ 
         if (!isset($this->tmp[ $full_pathx ]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
             if (!@file_exists($full_path)) {
                 @mkdir($full_path, $this->setChmodAuto(), true);
-            }else if (!@is_writable($full_path)) {
+            } else if (!@is_writable($full_path)) {
                 @chmod($full_path, $this->setChmodAuto());
             }
 
@@ -283,7 +283,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);

src/phpFastCache/CacheManager.php

Indentation +28 added lines, -28 removed lines

@@ -56,18 +56,18 @@ 
      * @var ExtendedCacheItemPoolInterface[]
      */
     protected static $config = [
-      'itemDetailedDate' => false,// Specify if the item must provide detailed creation/modification dates
-      'autoTmpFallback' => false,// Automatically attempt to fallback to temporary directory if the cache fails to write on the specified directory
-      'secureFileManipulation' => false,// Provide a secure file manipulation mechanism, on intensive usage the performance can be affected.
-      'ignoreSymfonyNotice' => false,// Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
-      '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
+        'autoTmpFallback' => false,// Automatically attempt to fallback to temporary directory if the cache fails to write on the specified directory
+        'secureFileManipulation' => false,// Provide a secure file manipulation mechanism, on intensive usage the performance can be affected.
+        'ignoreSymfonyNotice' => false,// Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
+        '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
     ];
 
     /**
@@ -121,7 +121,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');
         }
 
@@ -260,21 +260,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 +24 added lines, -24 removed lines

@@ -56,15 +56,15 @@ 
      * @var ExtendedCacheItemPoolInterface[]
      */
     protected static $config = [
-      'itemDetailedDate' => false,// Specify if the item must provide detailed creation/modification dates
-      'autoTmpFallback' => false,// Automatically attempt to fallback to temporary directory if the cache fails to write on the specified directory
-      'secureFileManipulation' => false,// Provide a secure file manipulation mechanism, on intensive usage the performance can be affected.
-      'ignoreSymfonyNotice' => false,// Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
-      '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
+      'autoTmpFallback' => false, // Automatically attempt to fallback to temporary directory if the cache fails to write on the specified directory
+      'secureFileManipulation' => false, // Provide a secure file manipulation mechanism, on intensive usage the performance can be affected.
+      'ignoreSymfonyNotice' => false, // Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
+      '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
@@ -100,32 +100,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];
     }
 
     /**
@@ -186,7 +186,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);
     }
@@ -239,9 +239,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');
         }
     }