spiral / framework

src/Http/tests/PipelineTest.php

Spacing +2 added lines, -2 removed lines

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

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

Spacing +6 added lines, -6 removed lines

@@ -33,7 +33,7 @@ 
     public function emptyItemProvider(): iterable
     {
         $values = ['', ' '];
-        foreach ($values as $value) {
+        foreach ($values as $value){
             yield from [
                 [AcceptHeaderItem::fromString($value)],
                 [new AcceptHeaderItem($value)],
@@ -82,11 +82,11 @@ 
      */
     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 +100,7 @@ 
     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 +140,9 @@ 
             ]
         ];
 
-        foreach ($set as $params) {
+        foreach ($set as $params){
             $formattedParams = [];
-            foreach ($params['passed'] as $k => $v) {
+            foreach ($params['passed'] as $k => $v){
                 $formattedParams[] = "$k=$v";
             }
 

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

Spacing +1 added lines, -1 removed lines

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

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

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
         $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);
         }
 

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/tests/ServerTest.php

Spacing +2 added lines, -2 removed lines

@@ -164,7 +164,7 @@ 
         $this->container->bind(ServerRequestInterface::class, $request);
 
         $this->assertSame(
-            ['PATH' => 'sample',],
+            ['PATH' => 'sample', ],
             $this->input->server->__debugInfo()
         );
     }
@@ -178,7 +178,7 @@ 
         $this->container->bind(ServerRequestInterface::class, $request);
 
         $this->assertSame(
-            ['PATH' => 'sample',],
+            ['PATH' => 'sample', ],
             iterator_to_array($this->input->server)
         );
     }

src/Http/src/CallableHandler.php

Spacing +10 added lines, -10 removed lines

@@ -51,21 +51,21 @@ 
         $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) {
-                $output = ob_get_clean() . $output;
+        }finally{
+            while (ob_get_level() > $outputLevel + 1){
+                $output = ob_get_clean().$output;
             }
         }
 
         return $this->wrapResponse(
             $response,
             $result,
-            ob_get_clean() . $output
+            ob_get_clean().$output
         );
     }
 
@@ -80,17 +80,17 @@ 
      */
     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);
         }
 

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

Spacing +6 added lines, -6 removed lines

@@ -47,13 +47,13 @@ 
         Pipeline $pipeline,
         ResponseFactoryInterface $responseFactory,
         ContainerInterface $container
-    ) {
+    ){
         $this->config = $config;
         $this->pipeline = $pipeline;
         $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 +72,11 @@ 
      */
     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 +93,7 @@ 
      */
     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.');
         }
 

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/Pipeline.php

Spacing +2 added lines, -2 removed lines

@@ -73,12 +73,12 @@
      */
     public function handle(Request $request): Response
     {
-        if (empty($this->handler)) {
+        if (empty($this->handler)){
             throw new PipelineException('Unable to run pipeline, no handler given.');
         }
 
         $position = $this->position++;
-        if (isset($this->middleware[$position])) {
+        if (isset($this->middleware[$position])){
             return $this->middleware[$position]->process($request, $this);
         }
 

Braces +6 added lines, -3 removed lines

@@ -73,16 +73,19 @@
      */
     public function handle(Request $request): Response
     {
-        if (empty($this->handler)) {
+        if (empty($this->handler))
+        {
             throw new PipelineException('Unable to run pipeline, no handler given.');
         }
 
         $position = $this->position++;
-        if (isset($this->middleware[$position])) {
+        if (isset($this->middleware[$position]))
+        {
             return $this->middleware[$position]->process($request, $this);
         }
 
-        return $this->scope->runScope([Request::class => $request], function () use ($request) {
+        return $this->scope->runScope([Request::class => $request], function () use ($request)
+        {
             return $this->handler->handle($request);
         });
     }

src/Http/src/Exception/ClientException.php

Spacing +2 added lines, -2 removed lines

@@ -34,11 +34,11 @@
      */
     public function __construct(?int $code = null, string $message = '', ?\Throwable $previous = null)
     {
-        if (empty($code) && empty($this->code)) {
+        if (empty($code) && empty($this->code)){
             $code = self::BAD_DATA;
         }
 
-        if (empty($message)) {
+        if (empty($message)){
             $message = "Http Error - {$code}";
         }
 

Braces +4 added lines, -2 removed lines

@@ -34,11 +34,13 @@
      */
     public function __construct(?int $code = null, string $message = '', ?\Throwable $previous = null)
     {
-        if (empty($code) && empty($this->code)) {
+        if (empty($code) && empty($this->code))
+        {
             $code = self::BAD_DATA;
         }
 
-        if (empty($message)) {
+        if (empty($message))
+        {
             $message = "Http Error - {$code}";
         }