iqltechnologies / brighty

brighty/system/Session/Handlers/FileHandler.php

Spacing +12 added lines, -12 removed lines

@@ -66,13 +66,13 @@ 
     {
         parent::__construct($config, $ipAddress);
 
-        if (! empty($config->sessionSavePath)) {
+        if (!empty($config->sessionSavePath)) {
             $this->savePath = rtrim($config->sessionSavePath, '/\\');
             ini_set('session.save_path', $config->sessionSavePath);
         } else {
             $sessionPath = rtrim(ini_get('session.save_path'), '/\\');
 
-            if (! $sessionPath) {
+            if (!$sessionPath) {
                 $sessionPath = WRITEPATH . 'session';
             }
 
@@ -94,11 +94,11 @@ 
      */
     public function open($path, $name): bool
     {
-        if (! is_dir($path) && ! mkdir($path, 0700, true)) {
+        if (!is_dir($path) && !mkdir($path, 0700, true)) {
             throw SessionException::forInvalidSavePath($this->savePath);
         }
 
-        if (! is_writable($path)) {
+        if (!is_writable($path)) {
             throw SessionException::forWriteProtectedSavePath($this->savePath);
         }
 
@@ -124,7 +124,7 @@ 
         // This might seem weird, but PHP 5.6 introduced session_reset(),
         // which re-reads session data
         if ($this->fileHandle === null) {
-            $this->fileNew = ! is_file($this->filePath . $id);
+            $this->fileNew = !is_file($this->filePath . $id);
 
             if (($this->fileHandle = fopen($this->filePath . $id, 'c+b')) === false) {
                 $this->logger->error("Session: Unable to open file '" . $this->filePath . $id . "'.");
@@ -140,7 +140,7 @@ 
                 return false;
             }
 
-            if (! isset($this->sessionID)) {
+            if (!isset($this->sessionID)) {
                 $this->sessionID = $id;
             }
 
@@ -184,7 +184,7 @@ 
             $this->sessionID = $id;
         }
 
-        if (! is_resource($this->fileHandle)) {
+        if (!is_resource($this->fileHandle)) {
             return false;
         }
 
@@ -192,7 +192,7 @@ 
             return ($this->fileNew) ? true : touch($this->filePath . $id);
         }
 
-        if (! $this->fileNew) {
+        if (!$this->fileNew) {
             ftruncate($this->fileHandle, 0);
             rewind($this->fileHandle);
         }
@@ -206,7 +206,7 @@ 
                 }
             }
 
-            if (! is_int($result)) {
+            if (!is_int($result)) {
                 $this->fingerprint = md5(substr($data, 0, $written));
                 $this->logger->error('Session: Unable to write data.');
 
@@ -270,7 +270,7 @@ 
     #[ReturnTypeWillChange]
     public function gc($max_lifetime)
     {
-        if (! is_dir($this->savePath) || ($directory = opendir($this->savePath)) === false) {
+        if (!is_dir($this->savePath) || ($directory = opendir($this->savePath)) === false) {
             $this->logger->debug("Session: Garbage collector couldn't list files under directory '" . $this->savePath . "'.");
 
             return false;
@@ -289,8 +289,8 @@ 
 
         while (($file = readdir($directory)) !== false) {
             // If the filename doesn't match this pattern, it's either not a session file or is not ours
-            if (! preg_match($pattern, $file)
-                || ! is_file($this->savePath . DIRECTORY_SEPARATOR . $file)
+            if (!preg_match($pattern, $file)
+                || !is_file($this->savePath . DIRECTORY_SEPARATOR . $file)
                 || ($mtime = filemtime($this->savePath . DIRECTORY_SEPARATOR . $file)) === false
                 || $mtime > $ts
             ) {

brighty/system/Session/Handlers/RedisHandler.php

Spacing +10 added lines, -10 removed lines

@@ -69,7 +69,7 @@ 
         }
 
         if (preg_match('#(?:tcp://)?([^:?]+)(?:\:(\d+))?(\?.+)?#', $this->savePath, $matches)) {
-            if (! isset($matches[3])) {
+            if (!isset($matches[3])) {
                 $matches[3] = ''; // Just to avoid undefined index notices below
             }
 
@@ -109,11 +109,11 @@ 
 
         $redis = new Redis();
 
-        if (! $redis->connect($this->savePath['host'], $this->savePath['port'], $this->savePath['timeout'])) {
+        if (!$redis->connect($this->savePath['host'], $this->savePath['port'], $this->savePath['timeout'])) {
             $this->logger->error('Session: Unable to connect to Redis with the configured settings.');
-        } elseif (isset($this->savePath['password']) && ! $redis->auth($this->savePath['password'])) {
+        } elseif (isset($this->savePath['password']) && !$redis->auth($this->savePath['password'])) {
             $this->logger->error('Session: Unable to authenticate to Redis instance.');
-        } elseif (isset($this->savePath['database']) && ! $redis->select($this->savePath['database'])) {
+        } elseif (isset($this->savePath['database']) && !$redis->select($this->savePath['database'])) {
             $this->logger->error('Session: Unable to select Redis database with index ' . $this->savePath['database']);
         } else {
             $this->redis = $redis;
@@ -136,7 +136,7 @@ 
     public function read($id)
     {
         if (isset($this->redis) && $this->lockSession($id)) {
-            if (! isset($this->sessionID)) {
+            if (!isset($this->sessionID)) {
                 $this->sessionID = $id;
             }
 
@@ -164,12 +164,12 @@ 
      */
     public function write($id, $data): bool
     {
-        if (! isset($this->redis)) {
+        if (!isset($this->redis)) {
             return false;
         }
 
         if ($this->sessionID !== $id) {
-            if (! $this->releaseLock() || ! $this->lockSession($id)) {
+            if (!$this->releaseLock() || !$this->lockSession($id)) {
                 return false;
             }
 
@@ -211,7 +211,7 @@ 
                         $this->redis->del($this->lockKey);
                     }
 
-                    if (! $this->redis->close()) {
+                    if (!$this->redis->close()) {
                         return false;
                     }
                 }
@@ -283,7 +283,7 @@ 
                 continue;
             }
 
-            if (! $this->redis->setex($lockKey, 300, (string) time())) {
+            if (!$this->redis->setex($lockKey, 300, (string) time())) {
                 $this->logger->error('Session: Error while trying to obtain lock for ' . $this->keyPrefix . $sessionID);
 
                 return false;
@@ -314,7 +314,7 @@ 
     protected function releaseLock(): bool
     {
         if (isset($this->redis, $this->lockKey) && $this->lock) {
-            if (! $this->redis->del($this->lockKey)) {
+            if (!$this->redis->del($this->lockKey)) {
                 $this->logger->error('Session: Error while trying to free lock for ' . $this->lockKey);
 
                 return false;

brighty/system/Session/Handlers/MemcachedHandler.php

Spacing +9 added lines, -9 removed lines

@@ -64,7 +64,7 @@ 
             $this->keyPrefix .= $this->ipAddress . ':';
         }
 
-        if (! empty($this->keyPrefix)) {
+        if (!empty($this->keyPrefix)) {
             ini_set('memcached.sess_prefix', $this->keyPrefix);
         }
 
@@ -88,7 +88,7 @@ 
             $serverList[] = $server['host'] . ':' . $server['port'];
         }
 
-        if (! preg_match_all('#,?([^,:]+)\:(\d{1,5})(?:\:(\d+))?#', $this->savePath, $matches, PREG_SET_ORDER)) {
+        if (!preg_match_all('#,?([^,:]+)\:(\d{1,5})(?:\:(\d+))?#', $this->savePath, $matches, PREG_SET_ORDER)) {
             $this->memcached = null;
             $this->logger->error('Session: Invalid Memcached save path format: ' . $this->savePath);
 
@@ -103,7 +103,7 @@ 
                 continue;
             }
 
-            if (! $this->memcached->addServer($match[1], $match[2], $match[3] ?? 0)) {
+            if (!$this->memcached->addServer($match[1], $match[2], $match[3] ?? 0)) {
                 $this->logger->error('Could not add ' . $match[1] . ':' . $match[2] . ' to Memcached server pool.');
             } else {
                 $serverList[] = $match[1] . ':' . $match[2];
@@ -131,7 +131,7 @@ 
     public function read($id)
     {
         if (isset($this->memcached) && $this->lockSession($id)) {
-            if (! isset($this->sessionID)) {
+            if (!isset($this->sessionID)) {
                 $this->sessionID = $id;
             }
 
@@ -153,12 +153,12 @@ 
      */
     public function write($id, $data): bool
     {
-        if (! isset($this->memcached)) {
+        if (!isset($this->memcached)) {
             return false;
         }
 
         if ($this->sessionID !== $id) {
-            if (! $this->releaseLock() || ! $this->lockSession($id)) {
+            if (!$this->releaseLock() || !$this->lockSession($id)) {
                 return false;
             }
 
@@ -195,7 +195,7 @@ 
                 $this->memcached->delete($this->lockKey);
             }
 
-            if (! $this->memcached->quit()) {
+            if (!$this->memcached->quit()) {
                 return false;
             }
 
@@ -258,7 +258,7 @@ 
                 continue;
             }
 
-            if (! $this->memcached->set($lockKey, time(), 300)) {
+            if (!$this->memcached->set($lockKey, time(), 300)) {
                 $this->logger->error('Session: Error while trying to obtain lock for ' . $this->keyPrefix . $sessionID);
 
                 return false;
@@ -286,7 +286,7 @@ 
     {
         if (isset($this->memcached, $this->lockKey) && $this->lock) {
             if (
-                ! $this->memcached->delete($this->lockKey)
+                !$this->memcached->delete($this->lockKey)
                 && $this->memcached->getResultCode() !== Memcached::RES_NOTFOUND
             ) {
                 $this->logger->error('Session: Error while trying to free lock for ' . $this->lockKey);

brighty/system/Test/FilterTestTrait.php

Spacing +6 added lines, -6 removed lines

@@ -126,14 +126,14 @@ 
      */
     protected function getFilterCaller($filter, string $position): Closure
     {
-        if (! in_array($position, ['before', 'after'], true)) {
+        if (!in_array($position, ['before', 'after'], true)) {
             throw new InvalidArgumentException('Invalid filter position passed: ' . $position);
         }
 
         if (is_string($filter)) {
             // Check for an alias (no namespace)
             if (strpos($filter, '\\') === false) {
-                if (! isset($this->filtersConfig->aliases[$filter])) {
+                if (!isset($this->filtersConfig->aliases[$filter])) {
                     throw new RuntimeException("No filter found with alias '{$filter}'");
                 }
 
@@ -144,21 +144,21 @@ 
             $filter = new $filter();
         }
 
-        if (! $filter instanceof FilterInterface) {
+        if (!$filter instanceof FilterInterface) {
             throw FilterException::forIncorrectInterface(get_class($filter));
         }
 
         $request = clone $this->request;
 
         if ($position === 'before') {
-            return static function (?array $params = null) use ($filter, $request) {
+            return static function(?array $params = null) use ($filter, $request) {
                 return $filter->before($request, $params);
             };
         }
 
         $response = clone $this->response;
 
-        return static function (?array $params = null) use ($filter, $request, $response) {
+        return static function(?array $params = null) use ($filter, $request, $response) {
             return $filter->after($request, $response, $params);
         };
     }
@@ -174,7 +174,7 @@ 
      */
     protected function getFiltersForRoute(string $route, string $position): array
     {
-        if (! in_array($position, ['before', 'after'], true)) {
+        if (!in_array($position, ['before', 'after'], true)) {
             throw new InvalidArgumentException('Invalid filter position passed:' . $position);
         }
 

brighty/system/Test/ControllerTestTrait.php

Spacing +4 added lines, -4 removed lines

@@ -99,7 +99,7 @@ 
             $this->appConfig = config('App');
         }
 
-        if (! $this->uri instanceof URI) {
+        if (!$this->uri instanceof URI) {
             $this->uri = Services::uri($this->appConfig->baseURL ?? 'http://example.com/', false);
         }
 
@@ -130,7 +130,7 @@ 
      */
     public function controller(string $name)
     {
-        if (! class_exists($name)) {
+        if (!class_exists($name)) {
             throw new InvalidArgumentException('Invalid Controller: ' . $name);
         }
 
@@ -151,7 +151,7 @@ 
      */
     public function execute(string $method, ...$params)
     {
-        if (! method_exists($this->controller, $method) || ! is_callable([$this->controller, $method])) {
+        if (!method_exists($this->controller, $method) || !is_callable([$this->controller, $method])) {
             throw new InvalidArgumentException('Method does not exist or is not callable in controller: ' . $method);
         }
 
@@ -177,7 +177,7 @@ 
         }
 
         // If the controller did not return a response then start one
-        if (! $response instanceof Response) {
+        if (!$response instanceof Response) {
             $response = $this->response;
         }
 

brighty/system/Test/Mock/MockCache.php

Spacing +4 added lines, -4 removed lines

@@ -114,7 +114,7 @@ 
     {
         $key = static::validateKey($key, $this->prefix);
 
-        if (! isset($this->cache[$key])) {
+        if (!isset($this->cache[$key])) {
             return false;
         }
 
@@ -154,7 +154,7 @@ 
 
         if (empty($data)) {
             $data = 0;
-        } elseif (! is_int($data)) {
+        } elseif (!is_int($data)) {
             return false;
         }
 
@@ -174,7 +174,7 @@ 
 
         if (empty($data)) {
             $data = 0;
-        } elseif (! is_int($data)) {
+        } elseif (!is_int($data)) {
             return false;
         }
 
@@ -216,7 +216,7 @@ 
     public function getMetaData(string $key)
     {
         // Misses return null
-        if (! array_key_exists($key, $this->expirations)) {
+        if (!array_key_exists($key, $this->expirations)) {
             return null;
         }
 

brighty/system/Test/Mock/MockCommon.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
  * the LICENSE file that was distributed with this source code.
  */
 
-if (! function_exists('is_cli')) {
+if (!function_exists('is_cli')) {
     /**
      * Is CLI?
      *

brighty/system/Test/Mock/MockConnection.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 
         $query->setQuery($sql, $binds, $setEscapeFlags);
 
-        if (! empty($this->swapPre) && ! empty($this->DBPrefix)) {
+        if (!empty($this->swapPre) && !empty($this->DBPrefix)) {
             $query->swapPrefix($this->DBPrefix, $this->swapPre);
         }
 

brighty/system/Test/ControllerTester.php

Spacing +4 added lines, -4 removed lines

@@ -99,7 +99,7 @@ 
             $this->appConfig = config('App');
         }
 
-        if (! $this->uri instanceof URI) {
+        if (!$this->uri instanceof URI) {
             $this->uri = Services::uri($this->appConfig->baseURL ?? 'http://example.com/', false);
         }
 
@@ -130,7 +130,7 @@ 
      */
     public function controller(string $name)
     {
-        if (! class_exists($name)) {
+        if (!class_exists($name)) {
             throw new InvalidArgumentException('Invalid Controller: ' . $name);
         }
 
@@ -151,7 +151,7 @@ 
      */
     public function execute(string $method, ...$params)
     {
-        if (! method_exists($this->controller, $method) || ! is_callable([$this->controller, $method])) {
+        if (!method_exists($this->controller, $method) || !is_callable([$this->controller, $method])) {
             throw new InvalidArgumentException('Method does not exist or is not callable in controller: ' . $method);
         }
 
@@ -191,7 +191,7 @@ 
                 $output = $response;
                 $result->response()->setBody($output);
                 $result->setBody($output);
-            } elseif (! empty($response) && ! empty($response->getBody())) {
+            } elseif (!empty($response) && !empty($response->getBody())) {
                 $result->setBody($response->getBody());
             } else {
                 $result->setBody('');