PHPSocialNetwork / phpfastcache

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/Entities/driverStatistic.php

Doc Comments +3 added lines, -3 removed lines

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

Doc Comments +1 added lines, -1 removed lines

@@ -334,7 +334,7 @@
 
     /**
      * @internal This method de-register an item from $this->itemInstances
-     * @param CacheItemInterface|string $item
+     * @param CacheItemInterface $item
      * @throws \InvalidArgumentException
      */
     protected function deregisterItem($item)

Spacing +14 added lines, -14 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,7 +302,7 @@ 
      */
     public function detachItem(CacheItemInterface $item)
     {
-        if(isset($this->itemInstances[$item->getKey()])){
+        if (isset($this->itemInstances[$item->getKey()])) {
             $this->deregisterItem($item);
         }
     }
@@ -324,9 +324,9 @@ 
      */
     public function attachItem(CacheItemInterface $item)
     {
-        if(isset($this->itemInstances[$item->getKey()]) && spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])){
+        if (isset($this->itemInstances[$item->getKey()]) && spl_object_hash($item) !== spl_object_hash($this->itemInstances[$item->getKey()])) {
             throw new \LogicException('The item already exists and cannot be overwritten because the Spl object hash mismatches ! You probably tried to re-attach a detached item which has been already retrieved from cache.');
-        }else{
+        } else {
             $this->itemInstances[$item->getKey()] = $item;
         }
     }
@@ -339,15 +339,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();
         }
     }
@@ -363,8 +363,8 @@ 
      */
     public function isAttached(CacheItemInterface $item)
     {
-        if(isset($this->itemInstances[$item->getKey()])){
-            return spl_object_hash($item) === spl_object_hash($this->itemInstances[ $item->getKey() ]);
+        if (isset($this->itemInstances[$item->getKey()])) {
+            return spl_object_hash($item) === spl_object_hash($this->itemInstances[$item->getKey()]);
         }
         return null;
     }

Braces +3 added lines, -3 removed lines

@@ -326,7 +326,7 @@ 
     {
         if(isset($this->itemInstances[$item->getKey()]) && spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])){
             throw new \LogicException('The item already exists and cannot be overwritten because the Spl object hash mismatches ! You probably tried to re-attach a detached item which has been already retrieved from cache.');
-        }else{
+        } else{
             $this->itemInstances[$item->getKey()] = $item;
         }
     }
@@ -342,9 +342,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/AttachingDetachingMethods.test.php

Braces +4 added lines, -5 removed lines

@@ -24,10 +24,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{
     $key = 'test_attaching_detaching';
 
     $itemDetached = $driverInstance->getItem($key);
@@ -37,8 +37,7 @@ 
     if(!$driverInstance->isAttached($itemDetached))
     {
         echo '[PASS] ExtendedCacheItemPoolInterface::isAttached() identified $itemDetached as being detached.' . "\n";
-    }
-    else
+    } else
     {
         echo '[FAIL] ExtendedCacheItemPoolInterface::isAttached() failed to identify $itemDetached as to be detached.' . "\n";
         $status = 1;
@@ -48,7 +47,7 @@ 
         $driverInstance->attachItem($itemDetached);
         echo '[FAIL] ExtendedCacheItemPoolInterface::attachItem() attached $itemDetached without trowing an error.' . "\n";
         $status = 1;
-    }catch(\LogicException $e){
+    } catch(\LogicException $e){
         echo '[PASS] ExtendedCacheItemPoolInterface::attachItem() failed to attach $itemDetached by trowing a LogicException exception.' . "\n";
     }
 

Spacing +5 added lines, -5 removed lines

@@ -24,17 +24,17 @@ 
 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 {
     $key = 'test_attaching_detaching';
 
     $itemDetached = $driverInstance->getItem($key);
     $driverInstance->detachItem($itemDetached);
     $itemAttached = $driverInstance->getItem($key);
 
-    if($driverInstance->isAttached($itemDetached) !== true)
+    if ($driverInstance->isAttached($itemDetached) !== true)
     {
         echo '[PASS] ExtendedCacheItemPoolInterface::isAttached() identified $itemDetached as being detached.' . "\n";
     }
@@ -44,11 +44,11 @@ 
         $status = 1;
     }
 
-    try{
+    try {
         $driverInstance->attachItem($itemDetached);
         echo '[FAIL] ExtendedCacheItemPoolInterface::attachItem() attached $itemDetached without trowing an error.' . "\n";
         $status = 1;
-    }catch(\LogicException $e){
+    } catch (\LogicException $e) {
         echo '[PASS] ExtendedCacheItemPoolInterface::attachItem() failed to attach $itemDetached by trowing a LogicException exception.' . "\n";
     }
 

src/phpFastCache/Drivers/Predis/Driver.php

Indentation +9 added lines, -9 removed lines

@@ -127,14 +127,14 @@ 
     protected function driverConnect()
     {
         $server = isset($this->config[ 'redis' ]) ? $this->config[ 'redis' ] : [
-          'host' => '127.0.0.1',
-          'port' => '6379',
-          'password' => '',
-          'database' => '',
+            'host' => '127.0.0.1',
+            'port' => '6379',
+            'password' => '',
+            'database' => '',
         ];
 
         $config = [
-          'host' => $server[ 'host' ],
+            'host' => $server[ 'host' ],
         ];
 
         $port = isset($server[ 'port' ]) ? $server[ 'port' ] : '';
@@ -184,9 +184,9 @@ 
         $date = (isset($info['Server'][ 'uptime_in_seconds' ]) ? (new \DateTime())->setTimestamp(time() - $info['Server'][ 'uptime_in_seconds' ]) : 'unknown date');
 
         return (new driverStatistic())
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData($this->instance->info())
-          ->setSize($size)
-          ->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.", $version, $date->format(DATE_RFC2822)));
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setRawData($this->instance->info())
+            ->setSize($size)
+            ->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.", $version, $date->format(DATE_RFC2822)));
     }
 }
\ No newline at end of file

Spacing +13 added lines, -13 removed lines

@@ -126,7 +126,7 @@ 
      */
     protected function driverConnect()
     {
-        $server = isset($this->config[ 'redis' ]) ? $this->config[ 'redis' ] : [
+        $server = isset($this->config['redis']) ? $this->config['redis'] : [
           'host' => '127.0.0.1',
           'port' => '6379',
           'password' => '',
@@ -134,32 +134,32 @@ 
         ];
 
         $config = [
-          'host' => $server[ 'host' ],
+          'host' => $server['host'],
         ];
 
-        $port = isset($server[ 'port' ]) ? $server[ 'port' ] : '';
+        $port = isset($server['port']) ? $server['port'] : '';
         if ($port != '') {
-            $config[ 'port' ] = $port;
+            $config['port'] = $port;
         }
 
-        $password = isset($server[ 'password' ]) ? $server[ 'password' ] : '';
+        $password = isset($server['password']) ? $server['password'] : '';
         if ($password != '') {
-            $config[ 'password' ] = $password;
+            $config['password'] = $password;
         }
 
-        $database = isset($server[ 'database' ]) ? $server[ 'database' ] : '';
+        $database = isset($server['database']) ? $server['database'] : '';
         if ($database != '') {
-            $config[ 'database' ] = $database;
+            $config['database'] = $database;
         }
 
-        $timeout = isset($server[ 'timeout' ]) ? $server[ 'timeout' ] : '';
+        $timeout = isset($server['timeout']) ? $server['timeout'] : '';
         if ($timeout != '') {
-            $config[ 'timeout' ] = $timeout;
+            $config['timeout'] = $timeout;
         }
 
-        $read_write_timeout = isset($server[ 'read_write_timeout' ]) ? $server[ 'read_write_timeout' ] : '';
+        $read_write_timeout = isset($server['read_write_timeout']) ? $server['read_write_timeout'] : '';
         if ($read_write_timeout != '') {
-            $config[ 'read_write_timeout' ] = $read_write_timeout;
+            $config['read_write_timeout'] = $read_write_timeout;
         }
 
         $this->instance = new PredisClient($config);
@@ -181,7 +181,7 @@ 
         $info = $this->instance->info();
         $size = (isset($info['Memory']['used_memory']) ? $info['Memory']['used_memory'] : 0);
         $version = (isset($info['Server']['redis_version']) ? $info['Server']['redis_version'] : 0);
-        $date = (isset($info['Server'][ 'uptime_in_seconds' ]) ? (new \DateTime())->setTimestamp(time() - $info['Server'][ 'uptime_in_seconds' ]) : 'unknown date');
+        $date = (isset($info['Server']['uptime_in_seconds']) ? (new \DateTime())->setTimestamp(time() - $info['Server']['uptime_in_seconds']) : 'unknown date');
 
         return (new driverStatistic())
           ->setData(implode(', ', array_keys($this->itemInstances)))

src/phpFastCache/Drivers/Devtrue/Driver.php

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)

Indentation +6 added lines, -6 removed lines

@@ -79,9 +79,9 @@ 
     protected function driverRead(CacheItemInterface $item)
     {
         return [
-          self::DRIVER_DATA_WRAPPER_INDEX => true,
-          self::DRIVER_TAGS_WRAPPER_INDEX => [],
-          self::DRIVER_TIME_WRAPPER_INDEX => new \DateTime(),
+            self::DRIVER_DATA_WRAPPER_INDEX => true,
+            self::DRIVER_TAGS_WRAPPER_INDEX => [],
+            self::DRIVER_TIME_WRAPPER_INDEX => new \DateTime(),
         ];
     }
 
@@ -131,9 +131,9 @@ 
     {
         $stat = new driverStatistic();
         $stat->setInfo('[Devtrue] A void info string')
-          ->setSize(0)
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData(true);
+            ->setSize(0)
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setRawData(true);
 
         return $stat;
     }

src/phpFastCache/Drivers/Zendshm/Driver.php

Spacing +2 added lines, -2 removed lines

@@ -133,8 +133,8 @@
         $stats = (array) zend_shm_cache_info();
         return (new driverStatistic())
             ->setData(implode(', ', array_keys($this->namespaces)))
-            ->setInfo(sprintf("The Zend memory have %d item(s) in cache.\n For more information see RawData.",$stats[ 'items_total' ]))
+            ->setInfo(sprintf("The Zend memory have %d item(s) in cache.\n For more information see RawData.", $stats['items_total']))
             ->setRawData($stats)
-            ->setSize($stats[ 'memory_total' ]);
+            ->setSize($stats['memory_total']);
     }
 }
\ No newline at end of file

src/phpFastCache/Helper/ActOnAll.php

Braces +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
                     } else {
                         $item->setHit(true);
                     }
-                }else{
+                } else{
                     $item->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]));
                 }
 

Spacing +7 added lines, -7 removed lines

@@ -21,7 +21,7 @@ 
      */
     public function __construct()
     {
-        $this->instances =& CacheManager::getInternalInstances();
+        $this->instances = & CacheManager::getInternalInstances();
     }
 
     /**
@@ -29,22 +29,22 @@ 
      */
     protected function getGenericCallback()
     {
-        return function($method, $args){
+        return function($method, $args) {
             $getterMethod = (strpos($method, 'get') === 0);
             $return = false;
 
-            if($getterMethod){
+            if ($getterMethod) {
                 $return = [];
             }
 
             foreach ($this->instances as $instance)
             {
                 $reflectionMethod = new \ReflectionMethod(get_class($instance), $method);
-                if($getterMethod){
-                    $return[ $instance->getDriverName() ] = $reflectionMethod->invokeArgs($instance, $args);
-                }else{
+                if ($getterMethod) {
+                    $return[$instance->getDriverName()] = $reflectionMethod->invokeArgs($instance, $args);
+                } else {
                     $result = $reflectionMethod->invokeArgs($instance, $args);
-                    if($result !== false){
+                    if ($result !== false) {
                         $return = $result;
                     }
                 }

src/phpFastCache/EventManager.php

Braces +3 added lines, -3 removed lines

@@ -84,13 +84,13 @@
             if(is_callable($arguments[0])){
                 if(isset($arguments[1]) && is_string($arguments[0])){
                     $this->events[$name][$arguments[1]] = $arguments[0];
-                }else {
+                } else {
                     $this->events[$name][] = $arguments[0];
                 }
-            }else{
+            } else{
                 throw new \InvalidArgumentException(sprintf('Expected Callable, got "%s"', gettype($arguments[0])));
             }
-        }else{
+        } else{
             throw new \BadMethodCallException('An event must start with "on" such as "onCacheGetItem"');
         }
     }

Spacing +9 added lines, -9 removed lines

@@ -66,8 +66,8 @@ 
     public function dispatch($eventName, ...$args)
     {
         $eventName = 'on' . ucfirst($eventName);
-        if(array_key_exists($eventName, $this->events)){
-            foreach ($this->events[ $eventName ] as $event) {
+        if (array_key_exists($eventName, $this->events)) {
+            foreach ($this->events[$eventName] as $event) {
                 call_user_func_array($event, $args);
             }
         }
@@ -81,17 +81,17 @@ 
      */
     public function __call($name, $arguments)
     {
-        if(strpos($name, 'on') === 0){
-            if(is_callable($arguments[0])){
-                if(isset($arguments[1]) && is_string($arguments[0])){
+        if (strpos($name, 'on') === 0) {
+            if (is_callable($arguments[0])) {
+                if (isset($arguments[1]) && is_string($arguments[0])) {
                     $this->events[$name][$arguments[1]] = $arguments[0];
-                }else {
+                } else {
                     $this->events[$name][] = $arguments[0];
                 }
-            }else{
+            } else {
                 throw new \InvalidArgumentException(sprintf('Expected Callable, got "%s"', gettype($arguments[0])));
             }
-        }else{
+        } else {
             throw new \BadMethodCallException('An event must start with "on" such as "onCacheGetItem"');
         }
     }
@@ -103,7 +103,7 @@ 
      */
     public function unbindEventCallback($eventName, $callbackName)
     {
-        if(isset($this->events[$eventName][$callbackName])){
+        if (isset($this->events[$eventName][$callbackName])) {
             unset($this->events[$eventName][$callbackName]);
             return true;
         }