terrylinooo / shieldon

src/Shieldon/Security/Xss.php

Indentation +39 added lines, -39 removed lines

@@ -36,9 +36,9 @@ 
 use function time;
 use function version_compare;
 
- /**
-  * Cross-Site Scripting protection.
-  */
+    /**
+     * Cross-Site Scripting protection.
+     */
 class Xss
 {
     /**
@@ -285,7 +285,7 @@ 
         unset($original);
 
         // Remove evil attributes such as style, onclick and xmlns
-		$str = $this->removeEvilAttributes($str, $isImage);
+        $str = $this->removeEvilAttributes($str, $isImage);
 
         /*
          * Sanitize naughty HTML elements
@@ -449,62 +449,62 @@ 
 	 * @param boolean $is_image TRUE if this is an image
 	 * @return string The string with the evil attributes removed
 	 */
-	protected function removeEvilAttributes($str, $is_image)
-	{
-		// All javascript event handlers (e.g. onload, onclick, onmouseover), style, and xmlns
-		$evilAttributes = array('on\w*', 'style', 'xmlns', 'formaction', 'form', 'xlink:href');
+    protected function removeEvilAttributes($str, $is_image)
+    {
+        // All javascript event handlers (e.g. onload, onclick, onmouseover), style, and xmlns
+        $evilAttributes = array('on\w*', 'style', 'xmlns', 'formaction', 'form', 'xlink:href');
 
-		if ($is_image) {
-			/*
+        if ($is_image) {
+            /*
 			 * Adobe Photoshop puts XML metadata into JFIF images,
 			 * including namespacing, so we have to allow this for images.
 			 */
-			unset($evilAttributes[array_search('xmlns', $evilAttributes)]);
-		}
+            unset($evilAttributes[array_search('xmlns', $evilAttributes)]);
+        }
 
-		do {
-			$count = 0;
-			$attribs = array();
+        do {
+            $count = 0;
+            $attribs = array();
 
-			// find occurrences of illegal attribute strings with quotes (042 and 047 are octal quotes)
-			preg_match_all(
+            // find occurrences of illegal attribute strings with quotes (042 and 047 are octal quotes)
+            preg_match_all(
                 '/(?<!\w)(' . implode('|', $evilAttributes) . ')\s*=\s*(\042|\047)([^\\2]*?)(\\2)/is',
                 $str,
                 $matches,
                 PREG_SET_ORDER
             );
 
-			foreach ($matches as $attr) {
-				$attribs[] = preg_quote($attr[0], '/');
-			}
+            foreach ($matches as $attr) {
+                $attribs[] = preg_quote($attr[0], '/');
+            }
 
-			// find occurrences of illegal attribute strings without quotes
-			preg_match_all(
+            // find occurrences of illegal attribute strings without quotes
+            preg_match_all(
                 '/(?<!\w)(' . implode('|', $evilAttributes) . ')\s*=\s*([^\s>]*)/is',
                 $str,
                 $matches,
                 PREG_SET_ORDER
             );
 
-			foreach ($matches as $attr) {
-				$attribs[] = preg_quote($attr[0], '/');
-			}
+            foreach ($matches as $attr) {
+                $attribs[] = preg_quote($attr[0], '/');
+            }
 
-			// replace illegal attribute strings that are inside an html tag
-			if (count($attribs) > 0) {
-				$str = preg_replace(
+            // replace illegal attribute strings that are inside an html tag
+            if (count($attribs) > 0) {
+                $str = preg_replace(
                     '/(<?)(\/?[^><]+?)([^A-Za-z<>\-])(.*?)(' . implode('|', $attribs) . ')(.*?)([\s><]?)([><]*)/i',
                     '$1$2 $4$6$7$8',
                     $str,
                     -1,
                     $count
                 );
-			}
+            }
 
-		} while ($count);
+        } while ($count);
 
-		return $str;
-	}
+        return $str;
+    }
 
     /**
      * Random Hash for protecting URLs
@@ -895,16 +895,16 @@ 
      * @return bool
      */
     protected function isPHP($version): bool
-	{
+    {
         static $_isPHP;
 
-		$version = (string) $version;
+        $version = (string) $version;
 
-		if (! isset($_isPHP[$version])) {
-			$_isPHP[$version] = version_compare(PHP_VERSION, $version, '>=');
-		}
+        if (! isset($_isPHP[$version])) {
+            $_isPHP[$version] = version_compare(PHP_VERSION, $version, '>=');
+        }
 
-		return $_isPHP[$version];
-	}
+        return $_isPHP[$version];
+    }
 }
 

Spacing +35 added lines, -35 removed lines

@@ -90,13 +90,13 @@ 
 
         $this->deniedRegexList = [
             'javascript\s*:',
-            '\bon\w+=\S+(?=.*>)',       // Inline JavaScript.
+            '\bon\w+=\S+(?=.*>)', // Inline JavaScript.
             '(document|(document\.)?window)\.(location|on\w*)',
             'expression\s*(\(|&\#40;)', // CSS and IE
-            'vbscript\s*:',             // IE, surprise!
-            'wscript\s*:',              // IE
-            'jscript\s*:',              // IE
-            'vbs\s*:',                  // IE
+            'vbscript\s*:', // IE, surprise!
+            'wscript\s*:', // IE
+            'jscript\s*:', // IE
+            'vbs\s*:', // IE
             'Redirect\s+30\d:',
             "([\"'])?data\s*:[^\\1]*?base64[^\\1]*?,[^\\1]*?\\1?",
         ];
@@ -223,9 +223,9 @@ 
          */
         $words = [
             'javascript', 'expression', 'vbscript', 'jscript', 'wscript',
-            'vbs',        'script',     'base64',   'applet',  'alert', 
-            'document',   'write',      'cookie',   'window',  'confirm',
-            'prompt',     'eval',
+            'vbs', 'script', 'base64', 'applet', 'alert', 
+            'document', 'write', 'cookie', 'window', 'confirm',
+            'prompt', 'eval',
         ];
 
         foreach ($words as $word) {
@@ -280,7 +280,7 @@ 
                 );
             }
 
-        } while($original !== $str);
+        } while ($original !== $str);
 
         unset($original);
 
@@ -393,7 +393,7 @@ 
         $originExif = @exif_read_data($imageAbsPath);
         $filteredExif = [];
 
-        if (! empty($originExif)) {
+        if (!empty($originExif)) {
             $filteredExif = $this->clean($originExif, true);
         }
 
@@ -620,21 +620,21 @@ 
             "'", '"', '&', '$', '#',
             '{', '}', '[', ']', '=',
             ';', '?', '%20', '%22',
-            '%3c',		// <
-            '%253c',	// <
-            '%3e',		// >
-            '%0e',		// >
-            '%28',		// (
-            '%29',		// )
-            '%2528',	// (
-            '%26',		// &
-            '%24',		// $
-            '%3f',		// ?
-            '%3b',		// ;
+            '%3c', // <
+            '%253c', // <
+            '%3e', // >
+            '%0e', // >
+            '%28', // (
+            '%29', // )
+            '%2528', // (
+            '%26', // &
+            '%24', // $
+            '%3f', // ?
+            '%3b', // ;
             '%3d'		// =
         ];
 
-        if (! $relativePath) {
+        if (!$relativePath) {
             $bad[] = './';
             $bad[] = '/';
         }
@@ -676,19 +676,19 @@ 
     protected function sanitizeNaughtyHtml(array $matches): string
     {
         static $naughtyTags = [
-            'alert',    'prompt', 'confirm',    'applet',  'audio',
-            'basefont', 'base',   'behavior',   'bgsound', 'blink', 
-            'body',     'embed',  'expression', 'form',    'frameset', 
-            'frame',    'head',   'html',       'ilayer',  'iframe', 
-            'input',    'button', 'select',     'isindex', 'layer', 
-            'link',     'meta',   'keygen',     'object',  'plaintext',
-            'style',    'script', 'textarea',   'title',   'math',
-            'video',    'svg',    'xml',        'xss',
+            'alert', 'prompt', 'confirm', 'applet', 'audio',
+            'basefont', 'base', 'behavior', 'bgsound', 'blink', 
+            'body', 'embed', 'expression', 'form', 'frameset', 
+            'frame', 'head', 'html', 'ilayer', 'iframe', 
+            'input', 'button', 'select', 'isindex', 'layer', 
+            'link', 'meta', 'keygen', 'object', 'plaintext',
+            'style', 'script', 'textarea', 'title', 'math',
+            'video', 'svg', 'xml', 'xss',
         ];
 
         static $evilAttributes = [
-            'on\w+', 'style',      'xmlns',     'seekSegmentTime', 
-            'form',  'xlink:href', 'FSCommand', 'formaction',
+            'on\w+', 'style', 'xmlns', 'seekSegmentTime', 
+            'form', 'xlink:href', 'FSCommand', 'formaction',
         ];
 
         // First, escape unclosed tags
@@ -721,7 +721,7 @@ 
                 // of numerous XSS issues we've had.
                 $matches['attributes'] = preg_replace('#^[^a-z]+#i', '', $matches['attributes']);
 
-                if (! preg_match($attributesPattern, $matches['attributes'], $attribute, PREG_OFFSET_CAPTURE)) {
+                if (!preg_match($attributesPattern, $matches['attributes'], $attribute, PREG_OFFSET_CAPTURE)) {
                     // No (valid) attribute found? Discard everything else inside the tag
                     break;
                 }
@@ -745,7 +745,7 @@ 
 
             $attributes = empty($attributes)
                 ? ''
-                : ' '.implode(' ', $attributes);
+                : ' ' . implode(' ', $attributes);
             return '<' . $matches['slash'] . $matches['tagName'] . $attributes . '>';
         }
 
@@ -900,7 +900,7 @@ 
 
 		$version = (string) $version;
 
-		if (! isset($_isPHP[$version])) {
+		if (!isset($_isPHP[$version])) {
 			$_isPHP[$version] = version_compare(PHP_VERSION, $version, '>=');
 		}
 

src/Shieldon/Security/Csrf.php

Indentation +3 added lines, -3 removed lines

@@ -19,9 +19,9 @@
 use function time;
 use function uniqid;
 
- /**
-  * Cross Site Request Forgery protection.
-  */
+    /**
+     * Cross Site Request Forgery protection.
+     */
 class Csrf
 {
     /**

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
      *
      * @var string
      */
-    protected $hash	= '';
+    protected $hash = '';
 
     /**
      * Token name.
@@ -79,7 +79,7 @@ 
         // Let's start checking process.
 
         // Do the tokens exist in both the _POST and _SESSION?
-        if (! isset($_POST[$this->name], $_SESSION[$this->name])) {
+        if (!isset($_POST[$this->name], $_SESSION[$this->name])) {
             return false;
         }
 

src/Shieldon/Driver/RedisDriver.php

Spacing +9 added lines, -9 removed lines

@@ -52,7 +52,7 @@ 
     {
         $this->channel = $channel;
 
-        if (! empty($this->channel)) {
+        if (!empty($this->channel)) {
             $this->tableFilterLogs = $this->channel . ':shieldon_filter_logs';
             $this->tableRuleList = $this->channel . ':shieldon_rule_list';
             $this->tableSessions = $this->channel . ':shieldon_sessions';
@@ -69,8 +69,8 @@ 
      */
     protected function doInitialize(bool $dbCheck = true): void
     {
-        if (! $this->isInitialized) {
-            if (! empty($this->channel)) {
+        if (!$this->isInitialized) {
+            if (!empty($this->channel)) {
                 $this->setChannel($this->channel);
             }
         }
@@ -93,7 +93,7 @@ 
 
                 $keys = $this->redis->keys($this->getNamespace($type) . ':*');
 
-                foreach($keys as $key) {
+                foreach ($keys as $key) {
                     $content = $this->redis->get($key);
                     $content = json_decode($content, true);
 
@@ -120,7 +120,7 @@ 
     {
         $results = [];
 
-        if (! $this->checkExist($ip, $type)) {
+        if (!$this->checkExist($ip, $type)) {
             return $results;
         }
 
@@ -139,7 +139,7 @@ 
                 $content = $this->redis->get($this->getKeyName($ip, $type));
                 $resultData = json_decode($content, true);
 
-                if (! empty($resultData['log_data'])) {
+                if (!empty($resultData['log_data'])) {
                     $results = $resultData['log_data']; 
                 }
         }
@@ -220,8 +220,8 @@ 
         foreach (['rule', 'filter_log', 'session'] as $type) {
             $keys = $this->redis->keys($this->getNamespace($type) . ':*');
 
-            if (! empty($keys)) {
-                foreach($keys as $key) {
+            if (!empty($keys)) {
+                foreach ($keys as $key) {
                     $this->redis->del($key);
                 }
             }
@@ -240,7 +240,7 @@ 
     private function getKeyName(string $ip, string $type = 'filter_log'): string
     {
         switch ($type) {
-            case 'filter_log'    : return $this->tableFilterLogs     . ':' . $ip;
+            case 'filter_log'    : return $this->tableFilterLogs . ':' . $ip;
             case 'session': return $this->tableSessions . ':' . $ip;
             case 'rule'   : return $this->tableRuleList . ':' . $ip;
         }

src/Shieldon/Driver/FileDriver.php

Spacing +16 added lines, -16 removed lines

@@ -76,8 +76,8 @@ 
      */
     protected function doInitialize(bool $dbCheck = true): void
     {
-        if (! $this->isInitialized) {
-            if (! empty($this->channel)) {
+        if (!$this->isInitialized) {
+            if (!empty($this->channel)) {
                 $this->setChannel($this->channel);
             }
 
@@ -107,7 +107,7 @@ 
                     $it = new RecursiveDirectoryIterator($dir, RecursiveDirectoryIterator::SKIP_DOTS);
                     $files = new RecursiveIteratorIterator($it, RecursiveIteratorIterator::CHILD_FIRST);
     
-                    foreach($files as $file) {
+                    foreach ($files as $file) {
                         if ($file->isFile()) {
     
                             $content = json_decode(file_get_contents($file->getPath() . '/' . $file->getFilename()), true);
@@ -139,7 +139,7 @@ 
     {
         $results = [];
 
-        if (! file_exists($this->getFilename($ip, $type))) {
+        if (!file_exists($this->getFilename($ip, $type))) {
             return $results;
         }
 
@@ -159,7 +159,7 @@ 
                 $fileContent = file_get_contents($this->getFilename($ip, $type));
                 $resultData = json_decode($fileContent, true);
 
-                if (! empty($resultData['log_data'])) {
+                if (!empty($resultData['log_data'])) {
                     $results = $resultData['log_data']; 
                 }
                 break;
@@ -243,7 +243,7 @@ 
                 $it = new RecursiveDirectoryIterator($dir, RecursiveDirectoryIterator::SKIP_DOTS);
                 $files = new RecursiveIteratorIterator($it, RecursiveIteratorIterator::CHILD_FIRST);
     
-                foreach($files as $file) {
+                foreach ($files as $file) {
                     if ($file->isDir()) {
                         // @codeCoverageIgnoreStart
                         rmdir($file->getRealPath());
@@ -268,9 +268,9 @@ 
 
         // Check if are Shieldon directories removed or not.
         $result = (
-            ! is_dir($this->getDirectory('filter_log')) && 
-            ! is_dir($this->getDirectory('rule'))       && 
-            ! is_dir($this->getDirectory('session'))
+            !is_dir($this->getDirectory('filter_log')) && 
+            !is_dir($this->getDirectory('rule')) && 
+            !is_dir($this->getDirectory('session'))
         );
 
         $this->createDirectory();
@@ -289,18 +289,18 @@ 
 
         $checkingFile = $this->directory . '/' . $this->channel . '_' . $this->checkPoint;
 
-        if (! file_exists($checkingFile)) {
+        if (!file_exists($checkingFile)) {
             $originalUmask = umask(0);
 
-            if (! is_dir($this->getDirectory('filter_log'))) {
+            if (!is_dir($this->getDirectory('filter_log'))) {
                 $resultA = @mkdir($this->getDirectory('filter_log'), 0777, true);
             }
     
-            if (! is_dir($this->getDirectory('rule'))) {
+            if (!is_dir($this->getDirectory('rule'))) {
                 $resultB = @mkdir($this->getDirectory('rule'), 0777, true);
             }
     
-            if (! is_dir($this->getDirectory('session'))) {
+            if (!is_dir($this->getDirectory('session'))) {
                 $resultC = @mkdir($this->getDirectory('session'), 0777, true);
             }
 
@@ -324,7 +324,7 @@ 
      */
     protected function checkDirectory(): bool
     {
-        if (! is_dir($this->directory) || ! is_writable($this->directory)) {
+        if (!is_dir($this->directory) || !is_writable($this->directory)) {
             throw new RuntimeException('The directory defined by File Driver must be writable. (' . $this->directory . ')');
         }
 
@@ -358,8 +358,8 @@ 
 
         switch ($type) {
             case 'filter_log' : return $this->directory . '/' . $this->tableFilterLogs . '/' . $ip . '.' . $this->extension;
-            case 'session'    : return $this->directory . '/' . $this->tableSessions   . '/' . $ip . '.' . $this->extension;
-            case 'rule'       : return $this->directory . '/' . $this->tableRuleList   . '/' . $ip . '.' . $this->extension;
+            case 'session'    : return $this->directory . '/' . $this->tableSessions . '/' . $ip . '.' . $this->extension;
+            case 'rule'       : return $this->directory . '/' . $this->tableRuleList . '/' . $ip . '.' . $this->extension;
         }
 
         return '';

src/Shieldon/Driver/DriverProvider.php

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
     {
         $this->channel = $channel;
 
-        if (! empty($this->channel)) {
+        if (!empty($this->channel)) {
             $this->tableFilterLogs = $this->channel . '_shieldon_filter_logs';
             $this->tableRuleList = $this->channel . '_shieldon_rule_list';
             $this->tableSessions = $this->channel . '_shieldon_sessions';

src/Shieldon/Driver/SqlDriverProvider.php

Indentation +1 added lines, -1 removed lines

@@ -146,7 +146,7 @@
         return $results;
     }
 
-   /**
+    /**
      * {@inheritDoc}
      */
     protected function doFetchAll(string $type = 'filter_log'): array

Spacing +20 added lines, -20 removed lines

@@ -60,12 +60,12 @@ 
      */
     protected function doInitialize(bool $dbCheck = true): void
     {
-        if (! $this->isInitialized) {
-            if (! empty($this->channel)) {
+        if (!$this->isInitialized) {
+            if (!empty($this->channel)) {
                 $this->setChannel($this->channel);
             }
 
-            if ($dbCheck && ! $this->checkTableExists()) {
+            if ($dbCheck && !$this->checkTableExists()) {
                 $this->installSql();
             }
         }
@@ -93,7 +93,7 @@ 
                 $resultData = $query->fetch($this->db::FETCH_ASSOC);
 
                 // No data found.
-                if (is_bool($resultData) && ! $resultData) {
+                if (is_bool($resultData) && !$resultData) {
                     $resultData = [];
                 }
 
@@ -113,11 +113,11 @@ 
                 $resultData = $query->fetch($this->db::FETCH_ASSOC);
 
                 // No data found.
-                if (is_bool($resultData) && ! $resultData) {
+                if (is_bool($resultData) && !$resultData) {
                     $resultData = [];
                 }
 
-                if (! empty($resultData['log_data'])) {
+                if (!empty($resultData['log_data'])) {
                     $results = json_decode($resultData['log_data'], true); 
                 }
                 break;
@@ -133,7 +133,7 @@ 
                 $resultData = $query->fetch($this->db::FETCH_ASSOC);
 
                 // No data found.
-                if (is_bool($resultData) && ! $resultData) {
+                if (is_bool($resultData) && !$resultData) {
                     $resultData = [];
                 }
 
@@ -228,7 +228,7 @@ 
         $query->execute();
         $result = $query->fetch();
 
-        if (! empty($result[$field])) {
+        if (!empty($result[$field])) {
             return true; 
         }
 
@@ -277,9 +277,9 @@ 
     protected function doDelete(string $ip, string $type = 'filter_log'): bool
     {
         switch ($type) {
-            case 'rule'      : return $this->remove($this->tableRuleList,   ['log_ip' => $ip]);
+            case 'rule'      : return $this->remove($this->tableRuleList, ['log_ip' => $ip]);
             case 'filter_log': return $this->remove($this->tableFilterLogs, ['log_ip' => $ip]);
-            case 'session'   : return $this->remove($this->tableSessions,   ['id'     => $ip]);
+            case 'session'   : return $this->remove($this->tableSessions, ['id'     => $ip]);
         }
 
         return false;
@@ -305,14 +305,14 @@ 
     private function update(string $table, array $data, array $where)
     {
         $placeholder = [];
-        foreach($data as $k => $v) {
+        foreach ($data as $k => $v) {
             $placeholder[] = "$k = :$k";
         }
 
         $dataPlaceholder = implode(', ', $placeholder);
 
         $placeholder = [];
-        foreach($where as $k => $v) {
+        foreach ($where as $k => $v) {
             $placeholder[] = "$k = :$k";
         }
 
@@ -324,7 +324,7 @@ 
 
             $bind = array_merge($data, $where);
     
-            foreach($bind as $k => $v) {
+            foreach ($bind as $k => $v) {
 
                 // @codeCoverageIgnoreStart
 
@@ -352,7 +352,7 @@ 
 
         // @codeCoverageIgnoreStart
         
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             throw $e->getMessage();
             return false;
         }
@@ -372,7 +372,7 @@ 
     {
         $placeholderField = [];
         $placeholderValue = [];
-        foreach($data as $k => $v) {
+        foreach ($data as $k => $v) {
             $placeholderField[] = "`$k`";
             $placeholderValue[] = ":$k";
         }
@@ -384,7 +384,7 @@ 
             $sql = 'INSERT INTO ' . $table . ' (' . $dataPlaceholderField . ') VALUES (' . $dataPlaceholderValue . ')';
             $query = $this->db->prepare($sql);
 
-            foreach($data as $k => $v) {
+            foreach ($data as $k => $v) {
 
                 // @codeCoverageIgnoreStart
 
@@ -412,7 +412,7 @@ 
 
         // @codeCoverageIgnoreStart
 
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             return false;
         }
 
@@ -431,7 +431,7 @@ 
     {
 
         $placeholder = [];
-        foreach($where as $k => $v) {
+        foreach ($where as $k => $v) {
             $placeholder[] = "`$k` = :$k";
         }
 
@@ -442,7 +442,7 @@ 
             $sql = 'DELETE FROM ' . $table . ' WHERE ' . $dataPlaceholder;
             $query = $this->db->prepare($sql);
 
-            foreach($where as $k => $v) {
+            foreach ($where as $k => $v) {
 
                 // @codeCoverageIgnoreStart
 
@@ -465,7 +465,7 @@ 
 
         // @codeCoverageIgnoreStart
 
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             return false;
         }
 

src/Shieldon/Driver/SqliteDriver.php

Indentation +5 added lines, -5 removed lines

@@ -27,11 +27,11 @@
         parent::__construct($pdo, $debug);
     }
 
-      /**
-     * Create SQL tables that Shieldon needs.
-     *
-     * @return bool
-     */
+        /**
+         * Create SQL tables that Shieldon needs.
+         *
+         * @return bool
+         */
     protected function installSql(): bool
     {
         try {

src/Shieldon/IpTrait.php

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
      */
     public function setIp(string $ip = '', $queryRdns = false): void
     {
-        if (! empty($ip)) {
+        if (!empty($ip)) {
             $_SERVER['REMOTE_ADDR'] = $ip;
         }
 
@@ -55,8 +55,8 @@ 
 
             // Check if your IP is from localhost, perhaps your are in development environment?
             if (
-                (substr($this->ip, 0 , 8) === '192.168.') ||
-                (substr($this->ip, 0 , 6) === '127.0.')
+                (substr($this->ip, 0, 8) === '192.168.') ||
+                (substr($this->ip, 0, 6) === '127.0.')
             ) {
                 $this->setRdns('localhost');
             } else {

src/Shieldon/Captcha/ImageCaptcha.php

Spacing +7 added lines, -7 removed lines

@@ -89,12 +89,12 @@ 
             }
         }
 
-        if (! is_array($this->properties['colors'])) {
+        if (!is_array($this->properties['colors'])) {
             $this->properties['colors'] = $defaults['colors'];
         }
 
         foreach ($defaults['colors'] as $k => $v) {
-            if (! is_array($this->properties['colors'][$k])) {
+            if (!is_array($this->properties['colors'][$k])) {
                 $this->properties['colors'][$k] = $defaults['colors'][$k];
             }
         }
@@ -131,7 +131,7 @@ 
     public function form(): string
     {
         // @codeCoverageIgnoreStart
-        if (! extension_loaded('gd')) {
+        if (!extension_loaded('gd')) {
             return '';
         }
         // @codeCoverageIgnoreEnd
@@ -139,7 +139,7 @@ 
         $html = '';
         $base64image = $this->createCaptcha();
 
-        if (! empty($base64image)) {
+        if (!empty($base64image)) {
             $html = '<div style="padding: 0px; overflow: hidden; margin: 10px 0;">';
             $html .= '<div style="
                 border: 1px #dddddd solid;
@@ -208,7 +208,7 @@ 
         imagefilledrectangle($im, 0, 0, $this->properties['img_width'], $this->properties['img_height'], $colors['background']);
 
         // Create the spiral pattern.
-        $theta		= 1;
+        $theta = 1;
         $thetac		= 7;
         $radius		= 16;
         $circles	= 20;
@@ -237,7 +237,7 @@ 
         $y = 0;
 
         for ($i = 0; $i < $length; $i++) {
-            $y = mt_rand(0 , $this->properties['img_height'] / 2);
+            $y = mt_rand(0, $this->properties['img_height'] / 2);
             imagestring($im, 5, $x, $y, $this->word[$i], $colors['text']);
             $x += ($this->properties['font_spacing'] * 2);
         }
@@ -246,7 +246,7 @@ 
         imagerectangle($im, 0, 0, $this->properties['img_width'] - 1, $this->properties['img_height'] - 1, $colors['border']);
 
         // Generate image in base64 string.
-        ob_start ();
+        ob_start();
 
         if (function_exists('imagejpeg')) {
             $this->imageType = 'jpeg';