PHPSocialNetwork / phpfastcache

src/phpFastCache/Core/MemcacheDriverCollisionDetectorTrait.php

Indentation +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
                 return true;
             } else if (constant($CONSTANT_NAME) !== $driverName) {
                 trigger_error('Memcache collision detected, you used both Memcache and Memcached driver in your script, this may leads to unexpected behaviours',
-                  E_USER_WARNING);
+                    E_USER_WARNING);
 
                 return false;
             }

src/phpFastCache/Cache/ItemBaseTrait.php

Spacing +2 added lines, -2 removed lines

@@ -349,7 +349,7 @@ 
     public function removeTag($tagName)
     {
         if (($key = array_search($tagName, $this->tags)) !== false) {
-            unset($this->tags[ $key ]);
+            unset($this->tags[$key]);
             $this->removedTags[] = $tagName;
         }
 
@@ -392,7 +392,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;
     }

src/phpFastCache/Drivers/Cookie/Item.php

Indentation +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
             $this->driver->setItem($this);
         } else {
             throw new \InvalidArgumentException(sprintf('$key must be a string, got type "%s" instead.',
-              gettype($key)));
+                gettype($key)));
         }
     }
 

src/phpFastCache/Cache/DriverBaseTrait.php

Indentation +3 added lines, -3 removed lines

@@ -214,9 +214,9 @@
     public function driverPreWrap(ExtendedCacheItemInterface $item)
     {
         return [
-          self::DRIVER_DATA_WRAPPER_INDEX => $item->get(),
-          self::DRIVER_TIME_WRAPPER_INDEX => $item->getExpirationDate(),
-          self::DRIVER_TAGS_WRAPPER_INDEX => $item->getTags(),
+            self::DRIVER_DATA_WRAPPER_INDEX => $item->get(),
+            self::DRIVER_TIME_WRAPPER_INDEX => $item->getExpirationDate(),
+            self::DRIVER_TAGS_WRAPPER_INDEX => $item->getTags(),
         ];
     }
 

Spacing +6 added lines, -6 removed lines

@@ -60,7 +60,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;
         }
     }
 
@@ -179,7 +179,7 @@ 
      */
     public function driverUnwrapData(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_DATA_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_DATA_WRAPPER_INDEX];
     }
 
     /**
@@ -188,7 +188,7 @@ 
      */
     public function driverUnwrapTags(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_TAGS_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_TAGS_WRAPPER_INDEX];
     }
 
 
@@ -198,7 +198,7 @@ 
      */
     public function driverUnwrapTime(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_TIME_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_TIME_WRAPPER_INDEX];
     }
 
     /**
@@ -223,7 +223,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());
         }
 
@@ -266,7 +266,7 @@ 
         foreach ($tagsItems as $tagsItem) {
             $data = (array) $tagsItem->get();
 
-            unset($data[ $item->getKey() ]);
+            unset($data[$item->getKey()]);
             $tagsItem->set($data);
 
             /**

Doc Comments +1 added lines, -1 removed lines

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

src/phpFastCache/Drivers/Leveldb/Driver.php

Indentation +3 added lines, -3 removed lines

@@ -176,9 +176,9 @@
     public function getStats()
     {
         return (new driverStatistic())
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setInfo('Number of files used to build the cache: ' . Directory::getFileCount($this->getLeveldbFile()))
-          ->setSize(Directory::dirSize($this->getLeveldbFile()));
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setInfo('Number of files used to build the cache: ' . Directory::getFileCount($this->getLeveldbFile()))
+            ->setSize(Directory::dirSize($this->getLeveldbFile()));
     }
 
     /**

Unused Use Statements -1 removed lines

@@ -17,7 +17,6 @@
 use LevelDB as LeveldbClient;
 use phpFastCache\Core\DriverAbstract;
 use phpFastCache\Core\PathSeekerTrait;
-use phpFastCache\Core\StandardPsr6StructureTrait;
 use phpFastCache\Entities\driverStatistic;
 use phpFastCache\Exceptions\phpFastCacheDriverCheckException;
 use phpFastCache\Exceptions\phpFastCacheDriverException;

Doc Comments +1 added lines, -1 removed lines

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

src/phpFastCache/Drivers/Files/Driver.php

Indentation +3 added lines, -3 removed lines

@@ -254,9 +254,9 @@
         }
 
         $stat->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData([])
-          ->setSize(Directory::dirSize($path))
-          ->setInfo('Number of files used to build the cache: ' . Directory::getFileCount($path));
+            ->setRawData([])
+            ->setSize(Directory::dirSize($path))
+            ->setInfo('Number of files used to build the cache: ' . Directory::getFileCount($path));
 
         return $stat;
     }

Unused Use Statements -1 removed lines

@@ -16,7 +16,6 @@
 
 use phpFastCache\Core\DriverAbstract;
 use phpFastCache\Core\PathSeekerTrait;
-use phpFastCache\Core\StandardPsr6StructureTrait;
 use phpFastCache\Entities\driverStatistic;
 use phpFastCache\Exceptions\phpFastCacheDriverCheckException;
 use phpFastCache\Exceptions\phpFastCacheDriverException;

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
             /**
              * Skip if Existing Caching in Options
              */
-            if (isset($this->config[ 'skipExisting' ]) && $this->config[ 'skipExisting' ] == true && file_exists($file_path)) {
+            if (isset($this->config['skipExisting']) && $this->config['skipExisting'] == true && file_exists($file_path)) {
                 $content = $this->readfile($file_path);
                 $old = $this->decode($content);
                 $toWrite = false;

Doc Comments +4 added lines, -4 removed lines

@@ -60,7 +60,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean|null
      * @throws \InvalidArgumentException
      */
     protected function driverWrite(CacheItemInterface $item)
@@ -166,7 +166,7 @@ 
     /**
      * @param string $optionName
      * @param mixed $optionValue
-     * @return bool
+     * @return boolean|null
      * @throws \InvalidArgumentException
      */
     public static function isValidOption($optionName, $optionValue)
@@ -194,7 +194,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getValidOptions()
     {
@@ -202,7 +202,7 @@ 
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     public static function getRequiredOptions()
     {

src/phpFastCache/Util/Directory.php

Indentation +2 added lines, -2 removed lines

@@ -83,8 +83,8 @@
 
         $files = new RecursiveIteratorIterator
         (
-          new RecursiveDirectoryIterator($source, RecursiveDirectoryIterator::SKIP_DOTS),
-          RecursiveIteratorIterator::CHILD_FIRST
+            new RecursiveDirectoryIterator($source, RecursiveDirectoryIterator::SKIP_DOTS),
+            RecursiveIteratorIterator::CHILD_FIRST
         );
         
         foreach ($files as $fileinfo) {

Spacing +3 added lines, -4 removed lines

@@ -85,8 +85,7 @@ 
             return unlink($source);
         }
 
-        $files = new RecursiveIteratorIterator
-        (
+        $files = new RecursiveIteratorIterator(
           new RecursiveDirectoryIterator($source, RecursiveDirectoryIterator::SKIP_DOTS),
           RecursiveIteratorIterator::CHILD_FIRST
         );
@@ -99,7 +98,7 @@ 
                 if (self::rrmdir($fileinfo->getRealPath()) === false) {
                     return false;
                 }
-            } else if(unlink($fileinfo->getRealPath()) === false) {
+            } else if (unlink($fileinfo->getRealPath()) === false) {
                 return false;
             }
         }
@@ -136,7 +135,7 @@ 
         /**
          * Allows to dereference char
          */
-        $__FILE__ = preg_replace('~^(([a-z0-9\-]+)://)~', '', __FILE__);// remove file protocols such as "phar://" etc.
+        $__FILE__ = preg_replace('~^(([a-z0-9\-]+)://)~', '', __FILE__); // remove file protocols such as "phar://" etc.
         $prefix = $__FILE__[0] === DIRECTORY_SEPARATOR ? DIRECTORY_SEPARATOR : '';
         return $prefix . implode(DIRECTORY_SEPARATOR, $absolutes);
     }

src/phpFastCache/Drivers/Redis/Driver.php

Indentation +4 added lines, -4 removed lines

@@ -159,9 +159,9 @@
         $date = (new \DateTime())->setTimestamp(time() - $info[ 'uptime_in_seconds' ]);
 
         return (new driverStatistic())
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData($info)
-          ->setSize($info[ 'used_memory' ])
-          ->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.", $info[ 'redis_version' ], $date->format(DATE_RFC2822)));
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setRawData($info)
+            ->setSize($info[ 'used_memory' ])
+            ->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.", $info[ 'redis_version' ], $date->format(DATE_RFC2822)));
     }
 }
\ No newline at end of file

Spacing +8 added lines, -8 removed lines

@@ -122,11 +122,11 @@ 
         } else {
             $this->instance = $this->instance ?: new RedisClient();
 
-            $host = isset($this->config[ 'host' ]) ? $this->config[ 'host' ] : '127.0.0.1';
-            $port = isset($this->config[ 'port' ]) ? (int) $this->config[ 'port' ] : '6379';
-            $password = isset($this->config[ 'password' ]) ? $this->config[ 'password' ] : '';
-            $database = isset($this->config[ 'database' ]) ? $this->config[ 'database' ] : '';
-            $timeout = isset($this->config[ 'timeout' ]) ? $this->config[ 'timeout' ] : '';
+            $host = isset($this->config['host']) ? $this->config['host'] : '127.0.0.1';
+            $port = isset($this->config['port']) ? (int) $this->config['port'] : '6379';
+            $password = isset($this->config['password']) ? $this->config['password'] : '';
+            $database = isset($this->config['database']) ? $this->config['database'] : '';
+            $timeout = isset($this->config['timeout']) ? $this->config['timeout'] : '';
 
             if (!$this->instance->connect($host, (int) $port, (int) $timeout)) {
                 return false;
@@ -156,12 +156,12 @@ 
     {
         // used_memory
         $info = $this->instance->info();
-        $date = (new \DateTime())->setTimestamp(time() - $info[ 'uptime_in_seconds' ]);
+        $date = (new \DateTime())->setTimestamp(time() - $info['uptime_in_seconds']);
 
         return (new driverStatistic())
           ->setData(implode(', ', array_keys($this->itemInstances)))
           ->setRawData($info)
-          ->setSize($info[ 'used_memory' ])
-          ->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.", $info[ 'redis_version' ], $date->format(DATE_RFC2822)));
+          ->setSize($info['used_memory'])
+          ->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.", $info['redis_version'], $date->format(DATE_RFC2822)));
     }
 }
\ No newline at end of file

src/phpFastCache/Drivers/Mongodb/Driver.php

Indentation +26 added lines, -26 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;
@@ -105,14 +105,14 @@ 
     protected function driverRead(CacheItemInterface $item)
     {
         $document = $this->getCollection()
-          ->findOne(['_id' => $item->getKey()],
+            ->findOne(['_id' => $item->getKey()],
             [self::DRIVER_DATA_WRAPPER_INDEX, self::DRIVER_TIME_WRAPPER_INDEX, self::DRIVER_TAGS_WRAPPER_INDEX  /*'d', 'e'*/]);
 
         if ($document) {
             return [
-              self::DRIVER_DATA_WRAPPER_INDEX => $this->decode($document[ self::DRIVER_DATA_WRAPPER_INDEX ]->bin),
-              self::DRIVER_TIME_WRAPPER_INDEX => (new \DateTime())->setTimestamp($document[ self::DRIVER_TIME_WRAPPER_INDEX ]->sec),
-              self::DRIVER_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;
@@ -167,10 +167,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]);
         }
     }
@@ -196,25 +196,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;
     }

Doc Comments +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     protected function driverWrite(CacheItemInterface $item)
@@ -149,7 +149,7 @@ 
     }
 
     /**
-     * @return bool
+     * @return boolean|null
      * @throws MongoConnectionException
      * @throws LogicException
      */

Spacing +12 added lines, -12 removed lines

@@ -84,7 +84,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');
         }
@@ -102,9 +102,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;
@@ -124,7 +124,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');
         }
@@ -148,11 +148,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($this->config[ 'port' ]) ? $this->config[ 'port' ] : '27017';
-            $timeout = isset($this->config[ 'timeout' ]) ? $this->config[ '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($this->config['port']) ? $this->config['port'] : '27017';
+            $timeout = isset($this->config['timeout']) ? $this->config['timeout'] : 3;
+            $password = isset($this->config['password']) ? $this->config['password'] : '';
+            $username = isset($this->config['username']) ? $this->config['username'] : '';
 
 
             /**
@@ -200,8 +200,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,