PHPSocialNetwork / phpfastcache

src/phpFastCache/Proxy/phpFastCacheAbstractProxy.php

Spacing +2 added lines, -2 removed lines

@@ -74,9 +74,9 @@
      */
     public function __call($name, $args)
     {
-        if(method_exists($this->instance, $name)){
+        if (method_exists($this->instance, $name)) {
             return call_user_func_array([$this->instance, $name], $args);
-        }else{
+        } else {
             throw new \BadMethodCallException(sprintf('Method %s does not exists', $name));
         }
     }

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

Braces +3 added lines, -3 removed lines

@@ -95,13 +95,13 @@ 
             $class = self::getNamespacePath() . $driver . '\Driver';
             try{
                 self::$instances[ $instance ] = new $class($config);
-            }catch(phpFastCacheDriverCheckException $e){
+            } catch(phpFastCacheDriverCheckException $e){
                 $fallback = self::standardizeDriverName($config['fallback']);
                 if($fallback && $fallback !== $driver){
                     $class = self::getNamespacePath() . $fallback . '\Driver';
                     self::$instances[ $instance ] = new $class($config);
                     trigger_error(sprintf('The "%s" driver is unavailable at the moment, the fallback driver "%s" has been used instead.', $driver, $fallback), E_USER_WARNING);
-                }else{
+                } else{
                     throw new phpFastCacheDriverCheckException($e->getMessage(), $e->getCode(), $e);
                 }
             }
@@ -198,7 +198,7 @@ 
             self::$config = array_merge(self::$config, $name);
         } else if (is_string($name)){
             self::$config[ $name ] = $value;
-        }else{
+        } else{
             throw new \InvalidArgumentException('Invalid variable type: $name');
         }
     }

Indentation +27 added lines, -27 removed lines

@@ -57,15 +57,15 @@ 
      * @var array
      */
     protected static $config = [
-      'securityKey' => 'auto',// The securityKey that will be used to create sub-directory
-      'ignoreSymfonyNotice' => false,// Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
-      'defaultTtl' => 900,// Default time-to-live in second
-      'htaccess' => true,// Auto-generate .htaccess if tit is missing
-      'default_chmod' => 0777, // 0777 recommended
-      'path' => '',// if not set will be the value of sys_get_temp_dir()
-      'fallback' => false, //Fall back when old driver is not support
-      'limited_memory_each_object' => 4096, // maximum size (bytes) of object store in memory
-      'compress_data' => false, // compress stored data, if the backend supports it
+        'securityKey' => 'auto',// The securityKey that will be used to create sub-directory
+        'ignoreSymfonyNotice' => false,// Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
+        'defaultTtl' => 900,// Default time-to-live in second
+        'htaccess' => true,// Auto-generate .htaccess if tit is missing
+        'default_chmod' => 0777, // 0777 recommended
+        'path' => '',// if not set will be the value of sys_get_temp_dir()
+        'fallback' => false, //Fall back when old driver is not support
+        'limited_memory_each_object' => 4096, // maximum size (bytes) of object store in memory
+        'compress_data' => false, // compress stored data, if the backend supports it
     ];
 
     /**
@@ -116,7 +116,7 @@ 
                 }
             }
         } 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');
         }
 
@@ -227,23 +227,23 @@ 
     public static function getStaticSystemDrivers()
     {
         return [
-          'Sqlite',
-          'Files',
-          'Apc',
-          'Apcu',
-          'Memcache',
-          'Memcached',
-          'Couchbase',
-          'Mongodb',
-          'Predis',
-          'Redis',
-          'Ssdb',
-          'Leveldb',
-          'Wincache',
-          'Xcache',
-          'Zenddisk',
-          'Zendshm',
-          'Devnull',
+            'Sqlite',
+            'Files',
+            'Apc',
+            'Apcu',
+            'Memcache',
+            'Memcached',
+            'Couchbase',
+            'Mongodb',
+            'Predis',
+            'Redis',
+            'Ssdb',
+            'Leveldb',
+            'Wincache',
+            'Xcache',
+            'Zenddisk',
+            'Zendshm',
+            'Devnull',
         ];
     }
 

Spacing +19 added lines, -19 removed lines

@@ -57,12 +57,12 @@ 
      * @var array
      */
     protected static $config = [
-      'securityKey' => 'auto',// The securityKey that will be used to create sub-directory
-      'ignoreSymfonyNotice' => false,// Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
-      'defaultTtl' => 900,// Default time-to-live in second
-      'htaccess' => true,// Auto-generate .htaccess if tit is missing
+      'securityKey' => 'auto', // The securityKey that will be used to create sub-directory
+      'ignoreSymfonyNotice' => false, // Ignore Symfony notice for Symfony project which do not makes use of PhpFastCache's Symfony Bundle
+      'defaultTtl' => 900, // Default time-to-live in second
+      'htaccess' => true, // Auto-generate .htaccess if tit is missing
       'default_chmod' => 0777, // 0777 recommended
-      '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()
       'fallback' => false, //Fall back when old driver is not support
       'limited_memory_each_object' => 4096, // maximum size (bytes) of object store in memory
       'compress_data' => false, // compress stored data, if the backend supports it
@@ -97,30 +97,30 @@ 
         }
 
         $instance = crc32($driver . serialize($config));
-        if (!isset(self::$instances[ $instance ])) {
+        if (!isset(self::$instances[$instance])) {
             $badPracticeOmeter[$driver] = 1;
-            if(!$config['ignoreSymfonyNotice'] && interface_exists('Symfony\Component\HttpKernel\KernelInterface') && !class_exists('phpFastCache\Bundle\phpFastCacheBundle')){
+            if (!$config['ignoreSymfonyNotice'] && interface_exists('Symfony\Component\HttpKernel\KernelInterface') && !class_exists('phpFastCache\Bundle\phpFastCacheBundle')) {
                 trigger_error('A Symfony Bundle to make the PhpFastCache integration more easier is now available here: https://github.com/PHPSocialNetwork/phpfastcache-bundle', E_USER_NOTICE);
             }
             $class = self::getNamespacePath() . $driver . '\Driver';
-            try{
-                self::$instances[ $instance ] = new $class($config);
-            }catch(phpFastCacheDriverCheckException $e){
+            try {
+                self::$instances[$instance] = new $class($config);
+            } catch (phpFastCacheDriverCheckException $e) {
                 $fallback = self::standardizeDriverName($config['fallback']);
-                if($fallback && $fallback !== $driver){
+                if ($fallback && $fallback !== $driver) {
                     $class = self::getNamespacePath() . $fallback . '\Driver';
-                    self::$instances[ $instance ] = new $class($config);
+                    self::$instances[$instance] = new $class($config);
                     trigger_error(sprintf('The "%s" driver is unavailable at the moment, the fallback driver "%s" has been used instead.', $driver, $fallback), E_USER_WARNING);
-                }else{
+                } else {
                     throw new phpFastCacheDriverCheckException($e->getMessage(), $e->getCode(), $e);
                 }
             }
-        } else if(++$badPracticeOmeter[$driver] >= 5){
+        } 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];
     }
 
     /**
@@ -153,7 +153,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);
     }
@@ -206,9 +206,9 @@ 
     {
         if (is_array($name)) {
             self::$config = array_merge(self::$config, $name);
-        } else if (is_string($name)){
-            self::$config[ $name ] = $value;
-        }else{
+        } else if (is_string($name)) {
+            self::$config[$name] = $value;
+        } else {
             throw new \InvalidArgumentException('Invalid variable type: $name');
         }
     }

src/phpFastCache/Core/PathSeekerTrait.php

Indentation +3 added lines, -3 removed lines

@@ -185,9 +185,9 @@
     protected static function cleanFileName($filename)
     {
         $regex = [
-          '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/',
-          '/\.$/',
-          '/^\./',
+            '/[\?\[\]\/\\\=\<\>\:\;\,\'\"\&\$\#\*\(\)\|\~\`\!\{\}]/',
+            '/\.$/',
+            '/^\./',
         ];
         $replace = ['-', '', ''];
 

Doc Comments +3 added lines, -3 removed lines

@@ -141,7 +141,7 @@ 
     }
 
     /**
-     * @param $keyword
+     * @param string|false $keyword
      * @param bool $skip
      * @return string
      * @throws phpFastCacheDriverException
@@ -187,7 +187,7 @@ 
 
     /**
      * @param $filename
-     * @return mixed
+     * @return string
      */
     protected static function cleanFileName($filename)
     {
@@ -202,7 +202,7 @@ 
     }
 
     /**
-     * @param $path
+     * @param string $path
      * @param bool $create
      * @throws \Exception
      */

Spacing +13 added lines, -13 removed lines

@@ -40,10 +40,10 @@ 
          * Calculate the security key
          */
         {
-            $securityKey = array_key_exists('securityKey', $this->config) ? $this->config[ 'securityKey' ] : '';
+            $securityKey = array_key_exists('securityKey', $this->config) ? $this->config['securityKey'] : '';
             if (!$securityKey || $securityKey === 'auto') {
-                if (isset($_SERVER[ 'HTTP_HOST' ])) {
-                    $securityKey = preg_replace('/^www./', '', strtolower(str_replace(':', '_', $_SERVER[ 'HTTP_HOST' ])));
+                if (isset($_SERVER['HTTP_HOST'])) {
+                    $securityKey = preg_replace('/^www./', '', strtolower(str_replace(':', '_', $_SERVER['HTTP_HOST'])));
                 } else {
                     $securityKey = ($this->isPHPModule() ? 'web' : 'cli');
                 }
@@ -58,10 +58,10 @@ 
          * with the security key and the driver name
          */
         $tmp_dir = rtrim($tmp_dir, '/') . DIRECTORY_SEPARATOR;
-        if (empty($this->config[ 'path' ]) || !is_string($this->config[ 'path' ])) {
+        if (empty($this->config['path']) || !is_string($this->config['path'])) {
             $path = $tmp_dir;
         } else {
-            $path = rtrim($this->config[ 'path' ], '/') . DIRECTORY_SEPARATOR;
+            $path = rtrim($this->config['path'], '/') . DIRECTORY_SEPARATOR;
         }
         $path_suffix = $securityKey . DIRECTORY_SEPARATOR . $this->getDriverName();
         $full_path = Directory::getAbsolutePath($path . $path_suffix);
@@ -74,12 +74,12 @@ 
          * return the temp dir
          */
         if ($readonly === true) {
-            if(!@file_exists($full_path) || !@is_writable($full_path)){
+            if (!@file_exists($full_path) || !@is_writable($full_path)) {
                 return $full_path_tmp;
             }
             return $full_path;
-        }else{
-            if (!isset($this->tmp[ $full_path_hash ]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
+        } else {
+            if (!isset($this->tmp[$full_path_hash]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
                 if (!@file_exists($full_path)) {
                     @mkdir($full_path, $this->setChmodAuto(), true);
                 } elseif (!@is_writable($full_path)) {
@@ -99,8 +99,8 @@ 
                     }
                 }
 
-                $this->tmp[ $full_path_hash ] = $full_path;
-                $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config[ 'htaccess' ] : false);
+                $this->tmp[$full_path_hash] = $full_path;
+                $this->htaccessGen($full_path, array_key_exists('htaccess', $this->config) ? $this->config['htaccess'] : false);
             }
         }
         return realpath($full_path);
@@ -174,10 +174,10 @@ 
      */
     public function setChmodAuto()
     {
-        if (!isset($this->config[ 'default_chmod' ]) || $this->config[ 'default_chmod' ] == '' || is_null($this->config[ 'default_chmod' ])) {
+        if (!isset($this->config['default_chmod']) || $this->config['default_chmod'] == '' || is_null($this->config['default_chmod'])) {
             return 0777;
         } else {
-            return $this->config[ 'default_chmod' ];
+            return $this->config['default_chmod'];
         }
     }
 
@@ -207,7 +207,7 @@ 
         if ($create === true) {
             if (!is_writable($path)) {
                 try {
-                    if(!chmod($path, 0777)){
+                    if (!chmod($path, 0777)) {
                         throw new phpFastCacheDriverException('Chmod failed on : ' . $path);
                     }
                 } catch (phpFastCacheDriverException $e) {

Braces +2 added lines, -2 removed lines

@@ -78,11 +78,11 @@
                 return $full_path_tmp;
             }
             return $full_path;
-        }else{
+        } else{
             if (!isset($this->tmp[ $full_path_hash ]) || (!@file_exists($full_path) || !@is_writable($full_path))) {
                 if (!@file_exists($full_path)) {
                     @mkdir($full_path, $this->setChmodAuto(), true);
-                }elseif (!@is_writable($full_path)) {
+                } elseif (!@is_writable($full_path)) {
                     if (!@chmod($full_path, $this->setChmodAuto()))
                     {
                         /**

src/autoload.php

Spacing +3 added lines, -3 removed lines

@@ -18,9 +18,9 @@ 
 /**
  * Register Autoload
  */
-spl_autoload_register(function ($entity) {
+spl_autoload_register(function($entity) {
     $module = explode('\\', $entity, 2);
-    if (!in_array($module[ 0 ], ['phpFastCache', 'Psr'])) {
+    if (!in_array($module[0], ['phpFastCache', 'Psr'])) {
         /**
          * Not a part of phpFastCache file
          * then we return here.
@@ -32,7 +32,7 @@ 
 
         if (is_readable($path)) {
             require_once $path;
-        }else{
+        } else {
             trigger_error('Cannot locate the Psr/Cache files', E_USER_ERROR);
             return;
         }

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

Indentation +1 added lines, -1 removed lines

@@ -46,6 +46,6 @@
 });
 
 if ((!defined('PFC_IGNORE_COMPOSER_WARNING') || !PFC_IGNORE_COMPOSER_WARNING) && class_exists('Composer\Autoload\ClassLoader')) {
-  trigger_error('Your project already makes use of Composer. You SHOULD use the composer dependency "phpfastcache/phpfastcache" instead of hard-autoloading.',
+    trigger_error('Your project already makes use of Composer. You SHOULD use the composer dependency "phpfastcache/phpfastcache" instead of hard-autoloading.',
     E_USER_WARNING);
 }
\ No newline at end of file

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

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)

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/Core/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()){