PHPSocialNetwork / phpfastcache

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/ExtendedCacheItemPoolTrait.php

Spacing +9 added lines, -9 removed lines

@@ -47,7 +47,7 @@ 
      */
     public function getItemsAsJsonString(array $keys = [], $option = 0, $depth = 512)
     {
-        $callback = function(CacheItemInterface $item){
+        $callback = function(CacheItemInterface $item) {
             return $item->get();
         };
         return json_encode(array_map($callback, array_values($this->getItems($keys))), $option, $depth);
@@ -75,7 +75,7 @@ 
                  *
                  * #headache
                  */
-                return array_filter($this->getItems(array_unique(array_keys($items))), function(ExtendedCacheItemInterface $item){
+                return array_filter($this->getItems(array_unique(array_keys($items))), function(ExtendedCacheItemInterface $item) {
                     return $item->isHit();
                 });
             } else {
@@ -117,7 +117,7 @@ 
      */
     public function getItemsByTagsAsJsonString(array $tagNames, $option = 0, $depth = 512)
     {
-        $callback = function(CacheItemInterface $item){
+        $callback = function(CacheItemInterface $item) {
             return $item->get();
         };
 
@@ -302,15 +302,15 @@ 
      */
     protected function deregisterItem($item)
     {
-        if($item instanceof CacheItemInterface){
-            unset($this->itemInstances[ $item->getKey() ]);
+        if ($item instanceof CacheItemInterface) {
+            unset($this->itemInstances[$item->getKey()]);
 
-        }else if(is_string($item)){
-            unset($this->itemInstances[ $item ]);
-        }else{
+        } else if (is_string($item)) {
+            unset($this->itemInstances[$item]);
+        } else {
             throw new \InvalidArgumentException('Invalid type for $item variable');
         }
-        if(gc_enabled()){
+        if (gc_enabled()) {
             gc_collect_cycles();
         }
     }

Braces +2 added lines, -2 removed lines

@@ -305,9 +305,9 @@
         if($item instanceof CacheItemInterface){
             unset($this->itemInstances[ $item->getKey() ]);
 
-        }else if(is_string($item)){
+        } else if(is_string($item)){
             unset($this->itemInstances[ $item ]);
-        }else{
+        } else{
             throw new \InvalidArgumentException('Invalid type for $item variable');
         }
         if(gc_enabled()){

tests/AbstractProxy.test.php

Spacing +2 added lines, -2 removed lines

@@ -42,10 +42,10 @@
 if (!is_object($driverInstance->getItem('test'))) {
     echo '[FAIL] $driverInstance->getItem() returned an invalid var type:' . gettype($driverInstance) . "\n";
     $status = 1;
-}else if(!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)){
+} else if (!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)) {
     echo '[FAIL] $driverInstance->getItem() returned an invalid class that does not implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
     $status = 1;
-}else{
+} else {
     echo '[PASS] $driverInstance->getItem() returned a valid class that implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
 }
 

Braces +2 added lines, -2 removed lines

@@ -42,10 +42,10 @@
 if (!is_object($driverInstance->getItem('test'))) {
     echo '[FAIL] $driverInstance->getItem() returned an invalid var type:' . gettype($driverInstance) . "\n";
     $status = 1;
-}else if(!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)){
+} else if(!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)){
     echo '[FAIL] $driverInstance->getItem() returned an invalid class that does not implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
     $status = 1;
-}else{
+} else{
     echo '[PASS] $driverInstance->getItem() returned a valid class that implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
 }
 

src/phpFastCache/Drivers/Mongo/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 +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;
     }

Spacing +13 added lines, -13 removed lines

@@ -59,7 +59,7 @@ 
      */
     public function driverCheck()
     {
-        if(class_exists('MongoDB\Driver\Manager')){
+        if (class_exists('MongoDB\Driver\Manager')) {
             trigger_error('PhpFastCache currently only support the pecl Mongo extension.<br />
             The Support for the MongoDB extension will be added coming soon.', E_USER_ERROR);
         }
@@ -94,7 +94,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');
         }
@@ -112,9 +112,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;
@@ -134,7 +134,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');
         }
@@ -158,11 +158,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'] : '';
 
 
             /**
@@ -210,8 +210,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,