spiral / framework

src/Http/tests/AcceptHeaderTest.php

Braces +4 added lines, -2 removed lines

@@ -118,7 +118,8 @@ 
         $headers = $acceptHeader->getAll();
         $this->assertCount(count($expected), $headers);
 
-        foreach ($expected as $i => $value) {
+        foreach ($expected as $i => $value)
+        {
             $this->assertSame($value, $headers[$i]->getValue());
         }
     }
@@ -164,7 +165,8 @@ 
         $headers = $acceptHeader->getAll();
         $this->assertCount(count($expected), $headers);
 
-        foreach ($expected as $i => $value) {
+        foreach ($expected as $i => $value)
+        {
             $this->assertSame($value, (string)$headers[$i]);
         }
     }

src/Http/tests/HttpTest.php

Braces +24 added lines, -12 removed lines

@@ -44,7 +44,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function () {
+        $core->setHandler(function ()
+        {
             return 'hello world';
         });
 
@@ -73,7 +74,8 @@ 
     public function testHandlerInterface(): void
     {
         $core = $this->getCore();
-        $core->setHandler(new CallableHandler(function () {
+        $core->setHandler(new CallableHandler(function ()
+        {
             return 'hello world';
         }, new ResponseFactory(new HttpConfig(['headers' => []]))));
 
@@ -85,7 +87,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function ($req, $resp) {
+        $core->setHandler(function ($req, $resp)
+        {
             return $resp->withAddedHeader('hello', 'value');
         });
 
@@ -98,7 +101,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function ($req, $resp) {
+        $core->setHandler(function ($req, $resp)
+        {
             echo 'hello!';
 
             return $resp->withAddedHeader('hello', 'value');
@@ -114,7 +118,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function ($req, $resp) {
+        $core->setHandler(function ($req, $resp)
+        {
             echo 'hello!';
             $resp->getBody()->write('world ');
 
@@ -131,7 +136,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function () {
+        $core->setHandler(function ()
+        {
             ob_start();
             ob_start();
             echo 'hello!';
@@ -152,7 +158,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function () {
+        $core->setHandler(function ()
+        {
             return [
                 'status'  => 404,
                 'message' => 'not found',
@@ -168,7 +175,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function () {
+        $core->setHandler(function ()
+        {
             return new Json([
                 'status'  => 404,
                 'message' => 'not found',
@@ -184,7 +192,8 @@ 
     {
         $core = $this->getCore([HeaderMiddleware::class]);
 
-        $core->setHandler(function () {
+        $core->setHandler(function ()
+        {
             return 'hello?';
         });
 
@@ -201,7 +210,8 @@ 
         $core->getPipeline()->pushMiddleware(new Header2Middleware());
         $core->getPipeline()->riseMiddleware(new HeaderMiddleware());
 
-        $core->setHandler(function () {
+        $core->setHandler(function ()
+        {
             return 'hello?';
         });
 
@@ -218,7 +228,8 @@ 
         $core->getPipeline()->pushMiddleware(new HeaderMiddleware());
         $core->getPipeline()->riseMiddleware(new Header2Middleware());
 
-        $core->setHandler(function () {
+        $core->setHandler(function ()
+        {
             return 'hello?';
         });
 
@@ -232,7 +243,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function () {
+        $core->setHandler(function ()
+        {
             $this->assertTrue($this->container->has(ServerRequestInterface::class));
 
             return 'OK';

src/Http/tests/PipelineTest.php

Braces +4 added lines, -2 removed lines

@@ -26,7 +26,8 @@ 
     {
         $pipeline = new Pipeline(new Container());
 
-        $handler = new CallableHandler(function () {
+        $handler = new CallableHandler(function ()
+        {
             return 'response';
         }, new ResponseFactory(new HttpConfig(['headers' => []])));
 
@@ -41,7 +42,8 @@ 
     {
         $pipeline = new Pipeline(new Container());
 
-        $handler = new CallableHandler(function () {
+        $handler = new CallableHandler(function ()
+        {
             return 'response';
         }, new ResponseFactory(new HttpConfig(['headers' => []])));
 

src/Http/tests/AcceptHeaderItemTest.php

Braces +12 added lines, -6 removed lines

@@ -33,7 +33,8 @@ 
     public function emptyItemProvider(): iterable
     {
         $values = ['', ' '];
-        foreach ($values as $value) {
+        foreach ($values as $value)
+        {
             yield from [
                 [AcceptHeaderItem::fromString($value)],
                 [new AcceptHeaderItem($value)],
@@ -82,11 +83,13 @@ 
      */
     public function testItemQualityBoundaries(float $quality, AcceptHeaderItem $item): void
     {
-        if ($quality > 1) {
+        if ($quality > 1)
+        {
             $this->assertSame(1.0, $item->getQuality());
         }
 
-        if ($quality < 0) {
+        if ($quality < 0)
+        {
             $this->assertSame(0.0, $item->getQuality());
         }
 
@@ -100,7 +103,8 @@ 
     public function qualityBoundariesProvider(): iterable
     {
         $qualities = [-1, 0, 0.5, 1, 2];
-        foreach ($qualities as $quality) {
+        foreach ($qualities as $quality)
+        {
             yield from [
                 [$quality, AcceptHeaderItem::fromString("*;q=$quality")],
                 [$quality, AcceptHeaderItem::fromString("*;Q=$quality")],
@@ -140,9 +144,11 @@ 
             ]
         ];
 
-        foreach ($set as $params) {
+        foreach ($set as $params)
+        {
             $formattedParams = [];
-            foreach ($params['passed'] as $k => $v) {
+            foreach ($params['passed'] as $k => $v)
+            {
                 $formattedParams[] = "$k=$v";
             }
 

src/Http/tests/Diactoros/UploadedFileFactory.php

Braces +2 added lines, -1 removed lines

@@ -26,7 +26,8 @@
         string $clientFilename = null,
         string $clientMediaType = null
     ): UploadedFileInterface {
-        if ($size === null) {
+        if ($size === null)
+        {
             $size = $stream->getSize();
         }
 

src/Http/tests/Diactoros/ResponseFactory.php

Braces +2 added lines, -1 removed lines

@@ -37,7 +37,8 @@
         $response = new Response('php://memory', $code, []);
         $response = $response->withStatus($code, $reasonPhrase);
 
-        foreach ($this->config->getBaseHeaders() as $header => $value) {
+        foreach ($this->config->getBaseHeaders() as $header => $value)
+        {
             $response = $response->withAddedHeader($header, $value);
         }
 

src/Http/src/CallableHandler.php

Braces +19 added lines, -8 removed lines

@@ -51,13 +51,19 @@ 
         $output = $result = null;
 
         $response = $this->responseFactory->createResponse(200);
-        try {
+        try
+        {
             $result = ($this->callable)($request, $response);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             ob_get_clean();
             throw $e;
-        } finally {
-            while (ob_get_level() > $outputLevel + 1) {
+        }
+        finally
+        {
+            while (ob_get_level() > $outputLevel + 1)
+            {
                 $output = ob_get_clean() . $output;
             }
         }
@@ -80,17 +86,22 @@ 
      */
     private function wrapResponse(Response $response, $result = null, string $output = ''): Response
     {
-        if ($result instanceof Response) {
-            if (!empty($output) && $result->getBody()->isWritable()) {
+        if ($result instanceof Response)
+        {
+            if (!empty($output) && $result->getBody()->isWritable())
+            {
                 $result->getBody()->write($output);
             }
 
             return $result;
         }
 
-        if (is_array($result) || $result instanceof \JsonSerializable) {
+        if (is_array($result) || $result instanceof \JsonSerializable)
+        {
             $response = $this->writeJson($response, $result);
-        } else {
+        }
+        else
+        {
             $response->getBody()->write((string)$result);
         }
 

src/Http/src/Http.php

Braces +12 added lines, -5 removed lines

@@ -53,7 +53,8 @@ 
         $this->responseFactory = $responseFactory;
         $this->container = $container;
 
-        foreach ($this->config->getMiddleware() as $middleware) {
+        foreach ($this->config->getMiddleware() as $middleware)
+        {
             $this->pipeline->pushMiddleware($this->container->get($middleware));
         }
     }
@@ -72,11 +73,16 @@ 
      */
     public function setHandler($handler): self
     {
-        if ($handler instanceof RequestHandlerInterface) {
+        if ($handler instanceof RequestHandlerInterface)
+        {
             $this->handler = $handler;
-        } elseif (is_callable($handler)) {
+        }
+        elseif (is_callable($handler))
+        {
             $this->handler = new CallableHandler($handler, $this->responseFactory);
-        } else {
+        }
+        else
+        {
             throw new HttpException(
                 'Invalid handler is given, expects callable or RequestHandlerInterface.'
             );
@@ -93,7 +99,8 @@ 
      */
     public function handle(ServerRequestInterface $request): ResponseInterface
     {
-        if (empty($this->handler)) {
+        if (empty($this->handler))
+        {
             throw new HttpException('Unable to run HttpCore, no handler is set.');
         }
 

src/Http/src/SapiRequestFactory.php

Braces +55 added lines, -24 removed lines

@@ -124,12 +124,14 @@ 
         $uri = $this->getUri($server, $headers);
 
         $request = $this->requestFactory->createServerRequest($method, $uri, $server);
-        foreach ($headers as $name => $value) {
+        foreach ($headers as $name => $value)
+        {
             $request = $request->withAddedHeader($name, $value);
         }
 
         $protocol = '1.1';
-        if (!empty($_SERVER['SERVER_PROTOCOL'])) {
+        if (!empty($_SERVER['SERVER_PROTOCOL']))
+        {
             $protocol = str_replace('HTTP/', '', $_SERVER['SERVER_PROTOCOL']);
         }
 
@@ -140,15 +142,21 @@ 
             ->withCookieParams($cookies)
             ->withUploadedFiles($this->getUploadedFilesArray($files));
 
-        if ($body === null) {
+        if ($body === null)
+        {
             return $request;
         }
 
-        if (\is_resource($body)) {
+        if (\is_resource($body))
+        {
             $body = $this->streamFactory->createStreamFromResource($body);
-        } elseif (\is_string($body)) {
+        }
+        elseif (\is_string($body))
+        {
             $body = $this->streamFactory->createStream($body);
-        } elseif (!$body instanceof StreamInterface) {
+        }
+        elseif (!$body instanceof StreamInterface)
+        {
             throw new \InvalidArgumentException(
                 'Body parameter for ServerRequestFactory::createFromParameters() '
                 . 'must be instance of StreamInterface, resource or null.'
@@ -166,29 +174,39 @@ 
     private function getUri(array $server, array $headers): UriInterface
     {
         $uri = $this->uriFactory->createUri();
-        if (isset($server['HTTPS'])) {
+        if (isset($server['HTTPS']))
+        {
             $uri = $uri->withScheme($server['HTTPS'] === 'on' ? 'https' : 'http');
         }
 
-        if (isset($server['HTTP_HOST'])) {
-            if (1 === \preg_match('/^(.+)\:(\d+)$/', $server['HTTP_HOST'], $matches)) {
+        if (isset($server['HTTP_HOST']))
+        {
+            if (1 === \preg_match('/^(.+)\:(\d+)$/', $server['HTTP_HOST'], $matches))
+            {
                 $uri = $uri->withHost($matches[1])->withPort($matches[2]);
-            } else {
+            }
+            else
+            {
                 $uri = $uri->withHost($server['HTTP_HOST']);
             }
-        } elseif (isset($server['SERVER_NAME'])) {
+        }
+        elseif (isset($server['SERVER_NAME']))
+        {
             $uri = $uri->withHost($server['SERVER_NAME']);
         }
 
-        if (isset($server['SERVER_PORT'])) {
+        if (isset($server['SERVER_PORT']))
+        {
             $uri = $uri->withPort($server['SERVER_PORT']);
         }
 
-        if (isset($server['REQUEST_URI'])) {
+        if (isset($server['REQUEST_URI']))
+        {
             $uri = $uri->withPath(\explode('?', $server['REQUEST_URI'])[0]);
         }
 
-        if (isset($server['QUERY_STRING'])) {
+        if (isset($server['QUERY_STRING']))
+        {
             $uri = $uri->withQuery($server['QUERY_STRING']);
         }
 
@@ -200,12 +218,17 @@ 
      */
     private static function getHeadersFromGlobals(): array
     {
-        if (\function_exists('getallheaders')) {
+        if (\function_exists('getallheaders'))
+        {
             $headers = getallheaders();
-        } else {
+        }
+        else
+        {
             $headers = [];
-            foreach ($_SERVER as $name => $value) {
-                if (strncmp($name, 'HTTP_', 5) === 0) {
+            foreach ($_SERVER as $name => $value)
+            {
+                if (strncmp($name, 'HTTP_', 5) === 0)
+                {
                     $name = str_replace(' ', '-', ucwords(strtolower(str_replace('_', ' ', substr($name, 5)))));
                     $headers[$name] = $value;
                 }
@@ -222,7 +245,8 @@ 
     private function getUploadedFilesArray(array $filesArray): array
     {
         $files = [];
-        foreach ($filesArray as $class => $info) {
+        foreach ($filesArray as $class => $info)
+        {
             $files[$class] = [];
             $this->populateUploadedFileRecursive(
                 $files[$class],
@@ -250,8 +274,10 @@ 
      */
     private function populateUploadedFileRecursive(&$files, $names, $tempNames, $types, $sizes, $errors): void
     {
-        if (\is_array($names)) {
-            foreach ($names as $i => $name) {
+        if (\is_array($names))
+        {
+            foreach ($names as $i => $name)
+            {
                 $files[$i] = [];
                 $this->populateUploadedFileRecursive(
                     $files[$i],
@@ -262,10 +288,15 @@ 
                     $errors[$i]
                 );
             }
-        } else {
-            try {
+        }
+        else
+        {
+            try
+            {
                 $stream = $this->streamFactory->createStreamFromFile($tempNames);
-            } catch (\RuntimeException $e) {
+            }
+            catch (\RuntimeException $e)
+            {
                 $stream = $this->streamFactory->createStream();
             }