spiral / framework

src/Framework/Http/SapiDispatcher.php

Spacing +7 added lines, -7 removed lines

@@ -56,7 +56,7 @@ 
         // On demand to save some memory.
 
         // Disable buffering
-        while (\ob_get_level() > 0) {
+        while (\ob_get_level() > 0){
             \ob_end_flush();
         }
         /**
@@ -66,12 +66,12 @@ 
         $http = $this->container->get(Http::class);
         $emitter = $emitter ?? $this->container->get(EmitterInterface::class);
 
-        try {
+        try{
             $response = $http->handle($this->initRequest());
             $emitter->emit($response);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $this->handleException($emitter, $e);
-        } finally {
+        }finally{
             $this->finalizer->finalize(false);
         }
     }
@@ -92,16 +92,16 @@ 
     {
         $handler = new HtmlHandler();
 
-        try {
+        try{
             /** @var SnapshotInterface $snapshot */
             $this->container->get(SnapshotterInterface::class)->register($e);
 
             // on demand
             $state = $this->container->get(StateInterface::class);
-            if ($state !== null) {
+            if ($state !== null){
                 $handler = $handler->withState($state);
             }
-        } catch (\Throwable | ContainerExceptionInterface $se) {
+        }catch (\Throwable | ContainerExceptionInterface $se){
             // nothing to report
         }
 

Braces +17 added lines, -7 removed lines

@@ -56,7 +56,8 @@ 
         // On demand to save some memory.
 
         // Disable buffering
-        while (\ob_get_level() > 0) {
+        while (\ob_get_level() > 0)
+        {
             \ob_end_flush();
         }
         /**
@@ -66,12 +67,17 @@ 
         $http = $this->container->get(Http::class);
         $emitter = $emitter ?? $this->container->get(EmitterInterface::class);
 
-        try {
+        try
+        {
             $response = $http->handle($this->initRequest());
             $emitter->emit($response);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $this->handleException($emitter, $e);
-        } finally {
+        }
+        finally
+        {
             $this->finalizer->finalize(false);
         }
     }
@@ -92,16 +98,20 @@ 
     {
         $handler = new HtmlHandler();
 
-        try {
+        try
+        {
             /** @var SnapshotInterface $snapshot */
             $this->container->get(SnapshotterInterface::class)->register($e);
 
             // on demand
             $state = $this->container->get(StateInterface::class);
-            if ($state !== null) {
+            if ($state !== null)
+            {
                 $handler = $handler->withState($state);
             }
-        } catch (\Throwable | ContainerExceptionInterface $se) {
+        }
+        catch (\Throwable | ContainerExceptionInterface $se)
+        {
             // nothing to report
         }
 

src/Http/src/Emitter/SapiEmitter.php

Spacing +8 added lines, -8 removed lines

@@ -54,13 +54,13 @@ 
     private function emitBody(ResponseInterface $response): void
     {
         $body = $response->getBody();
-        if ($body->isSeekable()) {
+        if ($body->isSeekable()){
             $body->rewind();
         }
-        if (!$body->isReadable()) {
+        if (!$body->isReadable()){
             return;
         }
-        while (!$body->eof()) {
+        while (!$body->eof()){
             echo $body->read($this->bufferSize);
             flush();
         }
@@ -77,11 +77,11 @@ 
      */
     private function assertNoPreviousOutput(): void
     {
-        if (headers_sent()) {
+        if (headers_sent()){
             throw new EmitterException('Unable to emit response, headers already send.');
         }
 
-        if (ob_get_level() > 0 && ob_get_length() > 0) {
+        if (ob_get_level() > 0 && ob_get_length() > 0){
             throw new EmitterException('Unable to emit response, found non closed buffered output.');
         }
     }
@@ -107,7 +107,7 @@ 
             'HTTP/%s %d%s',
             $response->getProtocolVersion(),
             $statusCode,
-            ($reasonPhrase ? ' ' . $reasonPhrase : '')
+            ($reasonPhrase ? ' '.$reasonPhrase : '')
         ), true, $statusCode);
     }
 
@@ -125,10 +125,10 @@ 
     {
         $statusCode = $response->getStatusCode();
 
-        foreach ($response->getHeaders() as $header => $values) {
+        foreach ($response->getHeaders() as $header => $values){
             $name = $this->filterHeader($header);
             $first = $name === 'Set-Cookie' ? false : true;
-            foreach ($values as $value) {
+            foreach ($values as $value){
                 header(sprintf(
                     '%s: %s',
                     $name,

Braces +14 added lines, -7 removed lines

@@ -54,13 +54,16 @@ 
     private function emitBody(ResponseInterface $response): void
     {
         $body = $response->getBody();
-        if ($body->isSeekable()) {
+        if ($body->isSeekable())
+        {
             $body->rewind();
         }
-        if (!$body->isReadable()) {
+        if (!$body->isReadable())
+        {
             return;
         }
-        while (!$body->eof()) {
+        while (!$body->eof())
+        {
             echo $body->read($this->bufferSize);
             flush();
         }
@@ -77,11 +80,13 @@ 
      */
     private function assertNoPreviousOutput(): void
     {
-        if (headers_sent()) {
+        if (headers_sent())
+        {
             throw new EmitterException('Unable to emit response, headers already send.');
         }
 
-        if (ob_get_level() > 0 && ob_get_length() > 0) {
+        if (ob_get_level() > 0 && ob_get_length() > 0)
+        {
             throw new EmitterException('Unable to emit response, found non closed buffered output.');
         }
     }
@@ -125,10 +130,12 @@ 
     {
         $statusCode = $response->getStatusCode();
 
-        foreach ($response->getHeaders() as $header => $values) {
+        foreach ($response->getHeaders() as $header => $values)
+        {
             $name = $this->filterHeader($header);
             $first = $name === 'Set-Cookie' ? false : true;
-            foreach ($values as $value) {
+            foreach ($values as $value)
+            {
                 header(sprintf(
                     '%s: %s',
                     $name,

src/Http/tests/SapiEmitter/Support/httpFunctionMocks.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 
 use Spiral\Tests\Http\SapiEmitter\Support\HTTPFunctions;
 
-if (!function_exists(__NAMESPACE__ . '\\headers_sent')) {
+if (!function_exists(__NAMESPACE__.'\\headers_sent')){
     /**
      * Mock for the headers_sent() function for Emitter class.
      */

Braces +2 added lines, -1 removed lines

@@ -6,7 +6,8 @@
 
 use Spiral\Tests\Http\SapiEmitter\Support\HTTPFunctions;
 
-if (!function_exists(__NAMESPACE__ . '\\headers_sent')) {
+if (!function_exists(__NAMESPACE__ . '\\headers_sent'))
+{
     /**
      * Mock for the headers_sent() function for Emitter class.
      */

src/Http/tests/SapiEmitter/Support/HTTPFunctions.php

Spacing +8 added lines, -8 removed lines

@@ -63,14 +63,14 @@ 
      */
     public static function header(string $string, bool $replace = true, ?int $http_response_code = null): void
     {
-        if (strpos($string, 'HTTP/') !== 0) {
+        if (strpos($string, 'HTTP/') !== 0){
             $header = strtolower(explode(':', $string, 2)[0]);
-            if ($replace || !array_key_exists($header, self::$headers)) {
+            if ($replace || !array_key_exists($header, self::$headers)){
                 self::$headers[$header] = [];
             }
             self::$headers[$header][] = $string;
         }
-        if ($http_response_code !== null) {
+        if ($http_response_code !== null){
             self::$responseCode = $http_response_code;
         }
     }
@@ -80,9 +80,9 @@ 
      */
     public static function header_remove(?string $header = null): void
     {
-        if ($header === null) {
+        if ($header === null){
             self::$headers = [];
-        } else {
+        }else{
             unset(self::$headers[strtolower($header)]);
         }
     }
@@ -95,8 +95,8 @@ 
     public static function headers_list(): array
     {
         $result = [];
-        foreach (self::$headers as $values) {
-            foreach ($values as $header) {
+        foreach (self::$headers as $values){
+            foreach ($values as $header){
                 $result[] = $header;
             }
         }
@@ -108,7 +108,7 @@ 
      */
     public static function http_response_code(?int $response_code = null): int
     {
-        if ($response_code !== null) {
+        if ($response_code !== null){
             self::$responseCode = $response_code;
         }
         return self::$responseCode;

Braces +17 added lines, -8 removed lines

@@ -63,14 +63,17 @@ 
      */
     public static function header(string $string, bool $replace = true, ?int $http_response_code = null): void
     {
-        if (strpos($string, 'HTTP/') !== 0) {
+        if (strpos($string, 'HTTP/') !== 0)
+        {
             $header = strtolower(explode(':', $string, 2)[0]);
-            if ($replace || !array_key_exists($header, self::$headers)) {
+            if ($replace || !array_key_exists($header, self::$headers))
+            {
                 self::$headers[$header] = [];
             }
             self::$headers[$header][] = $string;
         }
-        if ($http_response_code !== null) {
+        if ($http_response_code !== null)
+        {
             self::$responseCode = $http_response_code;
         }
     }
@@ -80,9 +83,12 @@ 
      */
     public static function header_remove(?string $header = null): void
     {
-        if ($header === null) {
+        if ($header === null)
+        {
             self::$headers = [];
-        } else {
+        }
+        else
+        {
             unset(self::$headers[strtolower($header)]);
         }
     }
@@ -95,8 +101,10 @@ 
     public static function headers_list(): array
     {
         $result = [];
-        foreach (self::$headers as $values) {
-            foreach ($values as $header) {
+        foreach (self::$headers as $values)
+        {
+            foreach ($values as $header)
+            {
                 $result[] = $header;
             }
         }
@@ -108,7 +116,8 @@ 
      */
     public static function http_response_code(?int $response_code = null): int
     {
-        if ($response_code !== null) {
+        if ($response_code !== null)
+        {
             self::$responseCode = $response_code;
         }
         return self::$responseCode;

src/Http/tests/SapiEmitter/SapiEmitterTest.php

Indentation +6 added lines, -6 removed lines

@@ -132,12 +132,12 @@
     {
         $body = 'Example body';
         $response = $this->createResponse(200, [], $body)
-                         ->withHeader('X-Test', '1')
-                         ->withAddedHeader('X-Test', '2')
-                         ->withAddedHeader('X-Test', '3; 3.5')
-                         ->withHeader('Cookie-Set', '1')
-                         ->withAddedHeader('cookie-Set', '2')
-                         ->withAddedHeader('Cookie-set', '3');
+                            ->withHeader('X-Test', '1')
+                            ->withAddedHeader('X-Test', '2')
+                            ->withAddedHeader('X-Test', '3; 3.5')
+                            ->withHeader('Cookie-Set', '1')
+                            ->withAddedHeader('cookie-Set', '2')
+                            ->withAddedHeader('Cookie-set', '3');
 
         (new SapiEmitter())->emit($response);
         $this->assertEquals(200, $this->getResponseCode());

Spacing +8 added lines, -8 removed lines

@@ -65,7 +65,7 @@ 
         $this->assertEquals(200, $this->getResponseCode());
         $this->assertCount(2, $this->getHeaders());
         $this->assertContains('X-Test: 1', $this->getHeaders());
-        $this->assertContains('Content-Length: ' . $length, $this->getHeaders());
+        $this->assertContains('Content-Length: '.$length, $this->getHeaders());
         $this->expectOutputString('Example body');
     }
 
@@ -118,12 +118,12 @@ 
         $this->expectException(EmitterException::class);
         $this->expectExceptionMessage('Unable to emit response, found non closed buffered output.');
 
-        try {
+        try{
             echo 'some data';
             $this->createEmitter()->emit($response);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             throw $e;
-        } finally {
+        }finally{
             \ob_end_clean();
         }
     }
@@ -154,7 +154,7 @@ 
     private function createEmitter(?int $bufferSize = null): SapiEmitter
     {
         $emitter = new SapiEmitter();
-        if ($bufferSize !== null) {
+        if ($bufferSize !== null){
             $emitter->bufferSize = $bufferSize;
         }
         return $emitter;
@@ -169,12 +169,12 @@ 
         $response = (new Response())
             ->withStatus($status)
             ->withProtocolVersion($version);
-        foreach ($headers as $header => $value) {
+        foreach ($headers as $header => $value){
             $response = $response->withHeader($header, $value);
         }
-        if ($body instanceof StreamInterface) {
+        if ($body instanceof StreamInterface){
             $response = $response->withBody($body);
-        } elseif (is_string($body)) {
+        } elseif (is_string($body)){
             $response->getBody()->write($body);
         }
         return $response;

Braces +17 added lines, -7 removed lines

@@ -118,12 +118,17 @@ 
         $this->expectException(EmitterException::class);
         $this->expectExceptionMessage('Unable to emit response, found non closed buffered output.');
 
-        try {
+        try
+        {
             echo 'some data';
             $this->createEmitter()->emit($response);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw $e;
-        } finally {
+        }
+        finally
+        {
             \ob_end_clean();
         }
     }
@@ -154,7 +159,8 @@ 
     private function createEmitter(?int $bufferSize = null): SapiEmitter
     {
         $emitter = new SapiEmitter();
-        if ($bufferSize !== null) {
+        if ($bufferSize !== null)
+        {
             $emitter->bufferSize = $bufferSize;
         }
         return $emitter;
@@ -169,12 +175,16 @@ 
         $response = (new Response())
             ->withStatus($status)
             ->withProtocolVersion($version);
-        foreach ($headers as $header => $value) {
+        foreach ($headers as $header => $value)
+        {
             $response = $response->withHeader($header, $value);
         }
-        if ($body instanceof StreamInterface) {
+        if ($body instanceof StreamInterface)
+        {
             $response = $response->withBody($body);
-        } elseif (is_string($body)) {
+        }
+        elseif (is_string($body))
+        {
             $response->getBody()->write($body);
         }
         return $response;