iqltechnologies / brighty

brighty/system/Test/ReflectionHelper.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
         $refMethod->setAccessible(true);
         $obj = (gettype($obj) === 'object') ? $obj : null;
 
-        return static function (...$args) use ($obj, $refMethod) {
+        return static function(...$args) use ($obj, $refMethod) {
             return $refMethod->invokeArgs($obj, $args);
         };
     }

brighty/system/Test/CIUnitTestCase.php

Spacing +8 added lines, -11 removed lines

@@ -231,7 +231,7 @@ 
     {
         parent::setUp();
 
-        if (! $this->app) {
+        if (!$this->app) {
             $this->app = $this->createApplication();
         }
 
@@ -399,14 +399,13 @@ 
     {
         $found = false;
 
-        if (! function_exists('xdebug_get_headers')) {
+        if (!function_exists('xdebug_get_headers')) {
             $this->markTestSkipped('XDebug not found.');
         }
 
         foreach (xdebug_get_headers() as $emitted) {
             $found = $ignoreCase ?
-                    (stripos($emitted, $header) === 0) :
-                    (strpos($emitted, $header) === 0);
+                    (stripos($emitted, $header) === 0) : (strpos($emitted, $header) === 0);
             if ($found) {
                 break;
             }
@@ -427,20 +426,19 @@ 
     {
         $found = false;
 
-        if (! function_exists('xdebug_get_headers')) {
+        if (!function_exists('xdebug_get_headers')) {
             $this->markTestSkipped('XDebug not found.');
         }
 
         foreach (xdebug_get_headers() as $emitted) {
             $found = $ignoreCase ?
-                    (stripos($emitted, $header) === 0) :
-                    (strpos($emitted, $header) === 0);
+                    (stripos($emitted, $header) === 0) : (strpos($emitted, $header) === 0);
             if ($found) {
                 break;
             }
         }
 
-        $success = ! $found;
+        $success = !$found;
         $this->assertTrue($success, "Found header for {$header}");
     }
 
@@ -523,14 +521,13 @@ 
      */
     protected function getHeaderEmitted(string $header, bool $ignoreCase = false): ?string
     {
-        if (! function_exists('xdebug_get_headers')) {
+        if (!function_exists('xdebug_get_headers')) {
             $this->markTestSkipped('XDebug not found.');
         }
 
         foreach (xdebug_get_headers() as $emitted) {
             $found = $ignoreCase ?
-                    (stripos($emitted, $header) === 0) :
-                    (strpos($emitted, $header) === 0);
+                    (stripos($emitted, $header) === 0) : (strpos($emitted, $header) === 0);
             if ($found) {
                 return $emitted;
             }

brighty/system/Test/Fabricator.php

Spacing +5 added lines, -5 removed lines

@@ -105,7 +105,7 @@ 
             $model = model($model, false);
         }
 
-        if (! is_object($model)) {
+        if (!is_object($model)) {
             throw new InvalidArgumentException(lang('Fabricator.invalidModel'));
         }
 
@@ -124,7 +124,7 @@ 
 
         // Determine eligible date fields
         foreach (['createdField', 'updatedField', 'deletedField'] as $field) {
-            if (! empty($this->model->{$field})) {
+            if (!empty($this->model->{$field})) {
                 $this->dateFields[] = $this->model->{$field};
             }
         }
@@ -278,7 +278,7 @@ 
     {
         $this->formatters = [];
 
-        if (! empty($this->model->allowedFields)) {
+        if (!empty($this->model->allowedFields)) {
             foreach ($this->model->allowedFields as $field) {
                 $this->formatters[$field] = $this->guessFormatter($field);
             }
@@ -509,12 +509,12 @@ 
         // Determine which fields we will need
         $fields = [];
 
-        if (! empty($this->model->useTimestamps)) {
+        if (!empty($this->model->useTimestamps)) {
             $fields[$this->model->createdField] = $datetime;
             $fields[$this->model->updatedField] = $datetime;
         }
 
-        if (! empty($this->model->useSoftDeletes)) {
+        if (!empty($this->model->useSoftDeletes)) {
             $fields[$this->model->deletedField] = null;
         }
 

brighty/system/Test/FeatureTestTrait.php

Spacing +6 added lines, -6 removed lines

@@ -150,7 +150,7 @@ 
         // Clean up any open output buffers
         // not relevant to unit testing
         // @codeCoverageIgnoreStart
-        if (\ob_get_level() > 0 && (! isset($this->clean) || $this->clean === true)) {
+        if (\ob_get_level() > 0 && (!isset($this->clean) || $this->clean === true)) {
             \ob_end_clean();
         }
         // @codeCoverageIgnoreEnd
@@ -165,7 +165,7 @@ 
         $request = $this->setRequestBody($request);
 
         // Initialize the RouteCollection
-        if (! $routes = $this->routes) {
+        if (!$routes = $this->routes) {
             require APPPATH . 'Config/Routes.php';
 
             /**
@@ -188,7 +188,7 @@ 
             ->run($routes, true);
 
         $output = \ob_get_contents();
-        if (empty($response->getBody()) && ! empty($output)) {
+        if (empty($response->getBody()) && !empty($output)) {
             $response->setBody($output);
         }
 
@@ -319,7 +319,7 @@ 
      */
     protected function setupHeaders(IncomingRequest $request)
     {
-        if (! empty($this->headers)) {
+        if (!empty($this->headers)) {
             foreach ($this->headers as $name => $value) {
                 $request->setHeader($name, $value);
             }
@@ -342,7 +342,7 @@ 
     {
         // $params should set the query vars if present,
         // otherwise set it from the URL.
-        $get = ! empty($params) && $method === 'get'
+        $get = !empty($params) && $method === 'get'
             ? $params
             : $this->getPrivateProperty($request->uri, 'query');
 
@@ -384,7 +384,7 @@ 
             } elseif ($this->bodyFormat === 'xml') {
                 $formatMime = 'application/xml';
             }
-            if (! empty($formatMime) && ! empty($params)) {
+            if (!empty($formatMime) && !empty($params)) {
                 $formatted = Services::format()->getFormatter($formatMime)->format($params);
                 $request->setBody($formatted);
                 $request->setHeader('Content-Type', $formatMime);

brighty/system/Test/DOMParser.php

Spacing +9 added lines, -9 removed lines

@@ -36,7 +36,7 @@ 
      */
     public function __construct()
     {
-        if (! extension_loaded('DOM')) {
+        if (!extension_loaded('DOM')) {
             // always there in travis-ci
             // @codeCoverageIgnoreStart
             throw new BadMethodCallException('DOM extension is required, but not currently loaded.');
@@ -67,7 +67,7 @@ 
         // turning off some errors
         libxml_use_internal_errors(true);
 
-        if (! $this->dom->loadHTML($content)) {
+        if (!$this->dom->loadHTML($content)) {
             // unclear how we would get here, given that we are trapping libxml errors
             // @codeCoverageIgnoreStart
             libxml_clear_errors();
@@ -90,7 +90,7 @@ 
      */
     public function withFile(string $path)
     {
-        if (! is_file($path)) {
+        if (!is_file($path)) {
             throw new InvalidArgumentException(basename($path) . ' is not a valid file.');
         }
 
@@ -126,7 +126,7 @@ 
      */
     public function dontSee(?string $search = null, ?string $element = null): bool
     {
-        return ! $this->see($search, $element);
+        return !$this->see($search, $element);
     }
 
     /**
@@ -192,23 +192,23 @@ 
         $path = '';
 
         // By ID
-        if (! empty($selector['id'])) {
+        if (!empty($selector['id'])) {
             $path = empty($selector['tag'])
                 ? "id(\"{$selector['id']}\")"
                 : "//{$selector['tag']}[@id=\"{$selector['id']}\"]";
         }
         // By Class
-        elseif (! empty($selector['class'])) {
+        elseif (!empty($selector['class'])) {
             $path = empty($selector['tag'])
                 ? "//*[@class=\"{$selector['class']}\"]"
                 : "//{$selector['tag']}[@class=\"{$selector['class']}\"]";
         }
         // By tag only
-        elseif (! empty($selector['tag'])) {
+        elseif (!empty($selector['tag'])) {
             $path = "//{$selector['tag']}";
         }
 
-        if (! empty($selector['attr'])) {
+        if (!empty($selector['attr'])) {
             foreach ($selector['attr'] as $key => $value) {
                 $path .= "[@{$key}=\"{$value}\"]";
             }
@@ -216,7 +216,7 @@ 
 
         // $paths might contain a number of different
         // ready to go xpath portions to tack on.
-        if (! empty($paths) && is_array($paths)) {
+        if (!empty($paths) && is_array($paths)) {
             foreach ($paths as $extra) {
                 $path .= $extra;
             }

brighty/system/Test/DatabaseTestTrait.php

Spacing +2 added lines, -2 removed lines

@@ -182,8 +182,8 @@
      */
     protected function runSeeds()
     {
-        if (! empty($this->seed)) {
-            if (! empty($this->basePath)) {
+        if (!empty($this->seed)) {
+            if (!empty($this->basePath)) {
                 $this->seeder->setPath(rtrim($this->basePath, '/') . '/Seeds');
             }
 

brighty/system/Test/TestLogger.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         $file  = null;
 
         foreach ($trace as $row) {
-            if (! in_array($row['function'], ['log', 'log_message'], true)) {
+            if (!in_array($row['function'], ['log', 'log_message'], true)) {
                 $file = basename($row['file'] ?? '');
                 break;
             }

brighty/system/Test/TestResponse.php

Spacing +2 added lines, -2 removed lines

@@ -132,7 +132,7 @@ 
             return false;
         }
         // Empty bodies are not considered valid, unless in redirects
-        return ! ($status < 300 && empty($this->response->getBody()));
+        return !($status < 300 && empty($this->response->getBody()));
     }
 
     /**
@@ -224,7 +224,7 @@ 
      */
     public function getRedirectUrl(): ?string
     {
-        if (! $this->isRedirect()) {
+        if (!$this->isRedirect()) {
             return null;
         }
 

brighty/system/Test/bootstrap.php

Spacing +11 added lines, -11 removed lines

@@ -41,16 +41,16 @@ 
 $paths = new Paths();
 
 // Define necessary framework path constants
-defined('APPPATH')       || define('APPPATH', realpath(rtrim($paths->appDirectory, '\\/ ')) . DIRECTORY_SEPARATOR);
-defined('WRITEPATH')     || define('WRITEPATH', realpath(rtrim($paths->writableDirectory, '\\/ ')) . DIRECTORY_SEPARATOR);
-defined('SYSTEMPATH')    || define('SYSTEMPATH', realpath(rtrim($paths->systemDirectory, '\\/')) . DIRECTORY_SEPARATOR);
-defined('ROOTPATH')      || define('ROOTPATH', realpath(APPPATH . '../') . DIRECTORY_SEPARATOR);
-defined('CIPATH')        || define('CIPATH', realpath(SYSTEMPATH . '../') . DIRECTORY_SEPARATOR);
-defined('FCPATH')        || define('FCPATH', realpath(PUBLICPATH) . DIRECTORY_SEPARATOR);
-defined('TESTPATH')      || define('TESTPATH', realpath(HOMEPATH . 'tests/') . DIRECTORY_SEPARATOR);
-defined('SUPPORTPATH')   || define('SUPPORTPATH', realpath(TESTPATH . '_support/') . DIRECTORY_SEPARATOR);
+defined('APPPATH') || define('APPPATH', realpath(rtrim($paths->appDirectory, '\\/ ')) . DIRECTORY_SEPARATOR);
+defined('WRITEPATH') || define('WRITEPATH', realpath(rtrim($paths->writableDirectory, '\\/ ')) . DIRECTORY_SEPARATOR);
+defined('SYSTEMPATH') || define('SYSTEMPATH', realpath(rtrim($paths->systemDirectory, '\\/')) . DIRECTORY_SEPARATOR);
+defined('ROOTPATH') || define('ROOTPATH', realpath(APPPATH . '../') . DIRECTORY_SEPARATOR);
+defined('CIPATH') || define('CIPATH', realpath(SYSTEMPATH . '../') . DIRECTORY_SEPARATOR);
+defined('FCPATH') || define('FCPATH', realpath(PUBLICPATH) . DIRECTORY_SEPARATOR);
+defined('TESTPATH') || define('TESTPATH', realpath(HOMEPATH . 'tests/') . DIRECTORY_SEPARATOR);
+defined('SUPPORTPATH') || define('SUPPORTPATH', realpath(TESTPATH . '_support/') . DIRECTORY_SEPARATOR);
 defined('COMPOSER_PATH') || define('COMPOSER_PATH', realpath(HOMEPATH . 'vendor/autoload.php'));
-defined('VENDORPATH')    || define('VENDORPATH', realpath(HOMEPATH . 'vendor') . DIRECTORY_SEPARATOR);
+defined('VENDORPATH') || define('VENDORPATH', realpath(HOMEPATH . 'vendor') . DIRECTORY_SEPARATOR);
 
 // Load Common.php from App then System
 if (file_exists(APPPATH . 'Common.php')) {
@@ -60,7 +60,7 @@ 
 require_once SYSTEMPATH . 'Common.php';
 
 // Set environment values that would otherwise stop the framework from functioning during tests.
-if (! isset($_SERVER['app.baseURL'])) {
+if (!isset($_SERVER['app.baseURL'])) {
     $_SERVER['app.baseURL'] = 'http://example.com/';
 }
 
@@ -77,7 +77,7 @@ 
 require_once APPPATH . 'Config/Services.php';
 
 // Use Config\Services as CodeIgniter\Services
-if (! class_exists('CodeIgniter\Services', false)) {
+if (!class_exists('CodeIgniter\Services', false)) {
     class_alias('Config\Services', 'CodeIgniter\Services');
 }