PHPSocialNetwork / phpfastcache

src/phpFastCache/Drivers/Apcu/Driver.php

Indentation +4 added lines, -4 removed lines

@@ -152,9 +152,9 @@
         $stats = (array) apc_cache_info('user');
         $date = (new \DateTime())->setTimestamp($stats['start_time']);
         return (new driverStatistic())
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setInfo(sprintf("The APCU cache is up since %s, and have %d item(s) in cache.\n For more information see RawData.", $date->format(DATE_RFC2822), $stats['num_entries']))
-          ->setRawData($stats)
-          ->setSize($stats['mem_size']);
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setInfo(sprintf("The APCU cache is up since %s, and have %d item(s) in cache.\n For more information see RawData.", $date->format(DATE_RFC2822), $stats['num_entries']))
+            ->setRawData($stats)
+            ->setSize($stats['mem_size']);
     }
 }
\ No newline at end of file

src/phpFastCache/Drivers/Devfalse/Driver.php

Indentation +3 added lines, -3 removed lines

@@ -140,9 +140,9 @@
     {
         $stat = new driverStatistic();
         $stat->setInfo('[Devfalse] A void info string')
-          ->setSize(0)
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData(false);
+            ->setSize(0)
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setRawData(false);
 
         return $stat;
     }

src/phpFastCache/Drivers/Memcached/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],
             ];
         }
 
@@ -178,9 +178,9 @@ 
         $stats = (array) $this->instance->getstats();
         $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 +2 added lines, -2 removed lines

@@ -127,7 +127,7 @@ 
      */
     public function driverConnect()
     {
-        $servers = (!empty($this->config[ 'memcache' ]) && is_array($this->config[ 'memcache' ]) ? $this->config[ 'memcache' ] : []);
+        $servers = (!empty($this->config['memcache']) && is_array($this->config['memcache']) ? $this->config['memcache'] : []);
         if (count($servers) < 1) {
             $servers = [
               ['127.0.0.1', 11211],
@@ -136,7 +136,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) {

src/phpFastCache/Drivers/Devnull/Driver.php

Indentation +3 added lines, -3 removed lines

@@ -140,9 +140,9 @@
     {
         $stat = new driverStatistic();
         $stat->setInfo('[Devnull] A void info string')
-          ->setSize(0)
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData(null);
+            ->setSize(0)
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setRawData(null);
 
         return $stat;
     }

Doc Comments +1 added lines, -1 removed lines

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

src/phpFastCache/Drivers/Mongodb/Driver.php

Doc Comments +2 added lines, -2 removed lines

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

Indentation +25 added lines, -25 removed lines

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

Spacing +12 added lines, -12 removed lines

@@ -92,7 +92,7 @@ 
                 return false;
             }
 
-            return isset($result[ 'ok' ]) ? $result[ 'ok' ] == 1 : true;
+            return isset($result['ok']) ? $result['ok'] == 1 : true;
         } else {
             throw new \InvalidArgumentException('Cross-Driver type confusion detected');
         }
@@ -108,9 +108,9 @@ 
 
         if ($document) {
             return [
-              self::DRIVER_DATA_WRAPPER_INDEX => $this->decode($document[ self::DRIVER_DATA_WRAPPER_INDEX ]->bin),
-              self::DRIVER_TIME_WRAPPER_INDEX => (new \DateTime())->setTimestamp($document[ self::DRIVER_TIME_WRAPPER_INDEX ]->sec),
-              self::DRIVER_TAGS_WRAPPER_INDEX => $this->decode($document[ self::DRIVER_TAGS_WRAPPER_INDEX ]->bin),
+              self::DRIVER_DATA_WRAPPER_INDEX => $this->decode($document[self::DRIVER_DATA_WRAPPER_INDEX]->bin),
+              self::DRIVER_TIME_WRAPPER_INDEX => (new \DateTime())->setTimestamp($document[self::DRIVER_TIME_WRAPPER_INDEX]->sec),
+              self::DRIVER_TAGS_WRAPPER_INDEX => $this->decode($document[self::DRIVER_TAGS_WRAPPER_INDEX]->bin),
             ];
         } else {
             return null;
@@ -130,7 +130,7 @@ 
         if ($item instanceof Item) {
             $deletionResult = (array) $this->getCollection()->remove(['_id' => $item->getKey()], ["w" => 1]);
 
-            return (int) $deletionResult[ 'ok' ] === 1 && !$deletionResult[ 'err' ];
+            return (int) $deletionResult['ok'] === 1 && !$deletionResult['err'];
         } else {
             throw new \InvalidArgumentException('Cross-Driver type confusion detected');
         }
@@ -154,11 +154,11 @@ 
         if ($this->instance instanceof MongodbClient) {
             throw new LogicException('Already connected to Mongodb server');
         } else {
-            $host = isset($this->config[ 'host' ]) ? $this->config[ 'host' ] : '127.0.0.1';
-            $port = isset($server[ 'port' ]) ? $server[ 'port' ] : '27017';
-            $timeout = isset($server[ 'timeout' ]) ? $server[ 'timeout' ] : 3;
-            $password = isset($this->config[ 'password' ]) ? $this->config[ 'password' ] : '';
-            $username = isset($this->config[ 'username' ]) ? $this->config[ 'username' ] : '';
+            $host = isset($this->config['host']) ? $this->config['host'] : '127.0.0.1';
+            $port = isset($server['port']) ? $server['port'] : '27017';
+            $timeout = isset($server['timeout']) ? $server['timeout'] : 3;
+            $password = isset($this->config['password']) ? $this->config['password'] : '';
+            $username = isset($this->config['username']) ? $this->config['username'] : '';
 
 
             /**
@@ -206,8 +206,8 @@ 
         ]);
 
         $stats = (new driverStatistic())
-          ->setInfo('MongoDB version ' . $serverStatus[ 'version' ] . ', Uptime (in days): ' . round($serverStatus[ 'uptime' ] / 86400, 1) . "\n For more information see RawData.")
-          ->setSize((int) @$collStats[ 'size' ])
+          ->setInfo('MongoDB version ' . $serverStatus['version'] . ', Uptime (in days): ' . round($serverStatus['uptime'] / 86400, 1) . "\n For more information see RawData.")
+          ->setSize((int) @$collStats['size'])
           ->setData(implode(', ', array_keys($this->itemInstances)))
           ->setRawData([
             'serverStatus' => $serverStatus,

src/phpFastCache/Cache/ItemBaseTrait.php

Spacing +4 added lines, -4 removed lines

@@ -114,10 +114,10 @@ 
      */
     public function setHit($isHit)
     {
-        if(is_bool($isHit)){
+        if (is_bool($isHit)) {
             $this->isHit = $isHit;
             return $this;
-        }else{
+        } else {
             throw new \InvalidArgumentException('$isHit must be a boolean');
         }
     }
@@ -369,7 +369,7 @@ 
     public function removeTag($tagName)
     {
         if (($key = array_search($tagName, $this->tags)) !== false) {
-            unset($this->tags[ $key ]);
+            unset($this->tags[$key]);
             $this->removedTags[] = $tagName;
         }
 
@@ -415,7 +415,7 @@ 
     final public function __debugInfo()
     {
         $info = get_object_vars($this);
-        $info[ 'driver' ] = 'object(' . get_class($info[ 'driver' ]) . ')';
+        $info['driver'] = 'object(' . get_class($info['driver']) . ')';
 
         return (array) $info;
     }

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/Drivers/Wincache/Driver.php

Indentation +4 added lines, -4 removed lines

@@ -132,9 +132,9 @@
         $date = (new \DateTime())->setTimestamp(time() - $info['total_cache_uptime']);
 
         return (new driverStatistic())
-          ->setInfo(sprintf("The Wincache daemon is up since %s.\n For more information see RawData.", $date->format(DATE_RFC2822)))
-          ->setSize($memInfo['memory_free'] - $memInfo['memory_total'])
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData($memInfo);
+            ->setInfo(sprintf("The Wincache daemon is up since %s.\n For more information see RawData.", $date->format(DATE_RFC2822)))
+            ->setSize($memInfo['memory_free'] - $memInfo['memory_total'])
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setRawData($memInfo);
     }
 }
\ No newline at end of file

src/phpFastCache/CacheManager.php

Indentation +23 added lines, -23 removed lines

@@ -52,13 +52,13 @@ 
      * @var array
      */
     public static $config = [
-      'default_chmod' => 0777, // 0777 recommended
-      'fallback' => 'files', //Fall back when old driver is not support
-      'securityKey' => 'auto',
-      'htaccess' => true,
-      'path' => '',// if not set will be the value of sys_get_temp_dir()
-      "limited_memory_each_object" => 4096, // maximum size (bytes) of object store in memory
-      "compress_data" => false, // compress stored data, if the backend supports it
+        'default_chmod' => 0777, // 0777 recommended
+        'fallback' => 'files', //Fall back when old driver is not support
+        'securityKey' => 'auto',
+        'htaccess' => true,
+        'path' => '',// if not set will be the value of sys_get_temp_dir()
+        "limited_memory_each_object" => 4096, // maximum size (bytes) of object store in memory
+        "compress_data" => false, // compress stored data, if the backend supports it
     ];
 
     /**
@@ -92,7 +92,7 @@ 
             $class = self::getNamespacePath() . $driver . '\Driver';
             self::$instances[ $instance ] = new $class($config);
         } 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');
         }
 
@@ -181,21 +181,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 +7 added lines, -7 removed lines

@@ -56,7 +56,7 @@ 
       'fallback' => 'files', //Fall back when old driver is not support
       'securityKey' => 'auto',
       'htaccess' => true,
-      '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()
       "limited_memory_each_object" => 4096, // maximum size (bytes) of object store in memory
       "compress_data" => false, // compress stored data, if the backend supports it
     ];
@@ -87,16 +87,16 @@ 
         }
 
         $instance = crc32($driver . serialize($config));
-        if (!isset(self::$instances[ $instance ])) {
+        if (!isset(self::$instances[$instance])) {
             $badPracticeOmeter[$driver] = 1;
             $class = self::getNamespacePath() . $driver . '\Driver';
-            self::$instances[ $instance ] = new $class($config);
-        } else if(++$badPracticeOmeter[$driver] >= 5){
+            self::$instances[$instance] = new $class($config);
+        } 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];
     }
 
     /**
@@ -129,7 +129,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);
     }
@@ -171,7 +171,7 @@ 
         if (is_array($name)) {
             self::$config = array_merge(self::$config, $name);
         } else {
-            self::$config[ $name ] = $value;
+            self::$config[$name] = $value;
         }
     }
 

tests/Psr6InterfaceImplements.test.php

Spacing +2 added lines, -2 removed lines

@@ -23,10 +23,10 @@
 if (!is_object($driverInstance)) {
     echo '[FAIL] CacheManager::getInstance() returned an invalid variable type:' . gettype($driverInstance) . "\n";
     $status = 1;
-}else if(!($driverInstance instanceof CacheItemPoolInterface)){
+} else if (!($driverInstance instanceof CacheItemPoolInterface)) {
     echo '[FAIL] CacheManager::getInstance() returned an invalid class:' . get_class($driverInstance) . "\n";
     $status = 1;
-}else{
+} else {
     echo '[PASS] CacheManager::getInstance() returned a valid CacheItemPoolInterface object: ' . get_class($driverInstance) . "\n";
 }
 

Braces +2 added lines, -2 removed lines

@@ -23,10 +23,10 @@
 if (!is_object($driverInstance)) {
     echo '[FAIL] CacheManager::getInstance() returned an invalid variable type:' . gettype($driverInstance) . "\n";
     $status = 1;
-}else if(!($driverInstance instanceof CacheItemPoolInterface)){
+} else if(!($driverInstance instanceof CacheItemPoolInterface)){
     echo '[FAIL] CacheManager::getInstance() returned an invalid class:' . get_class($driverInstance) . "\n";
     $status = 1;
-}else{
+} else{
     echo '[PASS] CacheManager::getInstance() returned a valid CacheItemPoolInterface object: ' . get_class($driverInstance) . "\n";
 }