PHPSocialNetwork / phpfastcache

src/phpFastCache/Drivers/Memcached/Driver.php

Indentation +5 added lines, -5 removed lines

@@ -131,7 +131,7 @@ 
         $servers = (!empty($this->config[ 'memcache' ]) && is_array($this->config[ 'memcache' ]) ? $this->config[ 'memcache' ] : []);
         if (count($servers) < 1) {
             $servers = [
-              ['127.0.0.1', 11211],
+                ['127.0.0.1', 11211],
             ];
         }
 
@@ -161,9 +161,9 @@ 
         $date = (new \DateTime())->setTimestamp(time() - $stats[ 'uptime' ]);
 
         return (new driverStatistic())
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats[ 'version' ], $date->format(DATE_RFC2822)))
-          ->setRawData($stats)
-          ->setSize($stats[ 'bytes' ]);
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats[ 'version' ], $date->format(DATE_RFC2822)))
+            ->setRawData($stats)
+            ->setSize($stats[ 'bytes' ]);
     }
 }
\ No newline at end of file

Spacing +10 added lines, -10 removed lines

@@ -127,7 +127,7 @@ 
      */
     protected function driverConnect()
     {
-        $servers = (!empty($this->config[ 'memcache' ]) && is_array($this->config[ 'memcache' ]) ? $this->config[ 'memcache' ] : []);
+        $servers = (!empty($this->config['memcache']) && is_array($this->config['memcache']) ? $this->config['memcache'] : []);
         if (count($servers) < 1) {
             $servers = [
               ['127.0.0.1', 11211],
@@ -136,11 +136,11 @@ 
 
         foreach ($servers as $server) {
             try {
-                if (!$this->instance->addServer($server[ 0 ], $server[ 1 ])) {
+                if (!$this->instance->addServer($server[0], $server[1])) {
                     $this->fallback = true;
                 }
-                if(!empty($server[ 'sasl_user' ]) && !empty($server[ 'sasl_password'])){
-                    $this->instance->setSaslAuthData($server[ 'sasl_user' ], $server[ 'sasl_password']);
+                if (!empty($server['sasl_user']) && !empty($server['sasl_password'])) {
+                    $this->instance->setSaslAuthData($server['sasl_user'], $server['sasl_password']);
                 }
             } catch (\Exception $e) {
                 $this->fallback = true;
@@ -160,16 +160,16 @@ 
     public function getStats()
     {
         $stats = (array) $this->instance->getStats();
-        $stats[ 'uptime' ] = (isset($stats[ 'uptime' ]) ? $stats[ 'uptime' ] : 0);
-        $stats[ 'version' ] = (isset($stats[ 'version' ]) ? $stats[ 'version' ] : 'UnknownVersion');
-        $stats[ 'bytes' ] = (isset($stats[ 'bytes' ]) ? $stats[ 'version' ] : 0);
+        $stats['uptime'] = (isset($stats['uptime']) ? $stats['uptime'] : 0);
+        $stats['version'] = (isset($stats['version']) ? $stats['version'] : 'UnknownVersion');
+        $stats['bytes'] = (isset($stats['bytes']) ? $stats['version'] : 0);
 
-        $date = (new \DateTime())->setTimestamp(time() - $stats[ 'uptime' ]);
+        $date = (new \DateTime())->setTimestamp(time() - $stats['uptime']);
 
         return (new driverStatistic())
           ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats[ 'version' ], $date->format(DATE_RFC2822)))
+          ->setInfo(sprintf("The memcache daemon v%s is up since %s.\n For more information see RawData.", $stats['version'], $date->format(DATE_RFC2822)))
           ->setRawData($stats)
-          ->setSize($stats[ 'bytes' ]);
+          ->setSize($stats['bytes']);
     }
 }
\ No newline at end of file

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 +14 added lines, -14 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,15 +74,15 @@ 
          * 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);
-                }else if (!@is_writable($full_path)) {
+                } else if (!@is_writable($full_path)) {
                     @chmod($full_path, $this->setChmodAuto());
                 }
                 if (!@is_writable($full_path)) {
@@ -103,8 +103,8 @@ 
                 if (!@file_exists($full_path) || !@is_writable($full_path)) {
                     throw new phpFastCacheDriverException('PLEASE CREATE OR CHMOD ' . $full_path . ' - 0777 OR ANY WRITABLE PERMISSION!');
                 }
-                $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);
@@ -178,10 +178,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'];
         }
     }
 
@@ -211,7 +211,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);
-                }else if (!@is_writable($full_path)) {
+                } else if (!@is_writable($full_path)) {
                     @chmod($full_path, $this->setChmodAuto());
                 }
                 if (!@is_writable($full_path)) {

src/autoload.php

Indentation +1 added lines, -1 removed lines

@@ -51,5 +51,5 @@
 
 if (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.',
-      E_USER_WARNING);
+        E_USER_WARNING);
 }
\ No newline at end of file

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

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