PHPSocialNetwork / phpfastcache

src/phpFastCache/Drivers/Cookie/Driver.php

Spacing +5 added lines, -5 removed lines

@@ -79,7 +79,7 @@ 
             $keyword = self::PREFIX . $item->getKey();
             $v = json_encode($this->driverPreWrap($item));
 
-            if (isset($this->config[ 'limited_memory_each_object' ]) && strlen($v) > $this->config[ 'limited_memory_each_object' ]) {
+            if (isset($this->config['limited_memory_each_object']) && strlen($v) > $this->config['limited_memory_each_object']) {
                 return false;
             }
 
@@ -100,7 +100,7 @@ 
         // return null if no caching
         // return value if in caching
         $keyword = self::PREFIX . $key;
-        $x = isset($_COOKIE[ $keyword ]) ? $this->decode(json_decode($_COOKIE[ $keyword ], true)) : false;
+        $x = isset($_COOKIE[$keyword]) ? $this->decode(json_decode($_COOKIE[$keyword], true)) : false;
 
         if ($x == false) {
             return null;
@@ -121,7 +121,7 @@ 
     {
         $this->driverConnect();
         $keyword = self::PREFIX . $key;
-        $x = isset($_COOKIE[ $keyword ]) ? $this->decode(json_decode($_COOKIE[ $keyword ])->t) : false;
+        $x = isset($_COOKIE[$keyword]) ? $this->decode(json_decode($_COOKIE[$keyword])->t) : false;
 
         return $x ? $x - time() : $x;
     }
@@ -139,7 +139,7 @@ 
         if ($item instanceof Item) {
             $this->driverConnect();
             $keyword = self::PREFIX . $item->getKey();
-            $_COOKIE[ $keyword ] = null;
+            $_COOKIE[$keyword] = null;
 
             return @setcookie($keyword, null, -10);
         } else {
@@ -156,7 +156,7 @@ 
         $this->driverConnect();
         foreach ($_COOKIE as $keyword => $value) {
             if (strpos($keyword, self::PREFIX) !== false) {
-                $_COOKIE[ $keyword ] = null;
+                $_COOKIE[$keyword] = null;
                 $result = @setcookie($keyword, null, -10);
                 if ($return !== false) {
                     $return = $result;

src/phpFastCache/Drivers/Leveldb/Driver.php

Doc Comments +1 added lines, -1 removed lines

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

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()));
     }
 
     /**

src/phpFastCache/Cache/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

@@ -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 +5 added lines, -5 removed lines

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

src/phpFastCache/Drivers/Files/Driver.php

Doc Comments +4 added lines, -4 removed lines

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

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

Spacing +1 added lines, -1 removed lines

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

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 +2 added lines, -3 removed lines

@@ -81,8 +81,7 @@ 
             return unlink($source);
         }
 
-        $files = new RecursiveIteratorIterator
-        (
+        $files = new RecursiveIteratorIterator(
           new RecursiveDirectoryIterator($source, RecursiveDirectoryIterator::SKIP_DOTS),
           RecursiveIteratorIterator::CHILD_FIRST
         );
@@ -95,7 +94,7 @@ 
                 if (self::rrmdir($fileinfo->getRealPath()) === false) {
                     return false;
                 }
-            } else if(unlink($fileinfo->getRealPath()) === false) {
+            } else if (unlink($fileinfo->getRealPath()) === false) {
                 return false;
             }
         }

src/phpFastCache/Entities/driverStatistic.php

Doc Comments +4 added lines, -4 removed lines

@@ -44,7 +44,7 @@ 
     protected $rawData;
 
     /**
-     * @return string|bool Return infos or false if no information available
+     * @return string Return infos or false if no information available
      */
     public function getInfo()
     {
@@ -52,7 +52,7 @@ 
     }
 
     /**
-     * @return int|bool Return size in octet or false if no information available
+     * @return string Return size in octet or false if no information available
      */
     public function getSize()
     {
@@ -60,7 +60,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getData()
     {
@@ -68,7 +68,7 @@ 
     }
 
     /**
-     * @param $info
+     * @param string $info
      * @return $this
      */
     public function setInfo($info)

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/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)

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,

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