spiral / framework

src/Http/src/Request/HeadersBag.php

Spacing +8 added lines, -8 removed lines

@@ -10,19 +10,19 @@ 
  */
 final class HeadersBag extends InputBag
 {
-    public function has(int|string $name): bool
+    public function has(int | string $name): bool
     {
-        return parent::has($this->normalize((string) $name));
+        return parent::has($this->normalize((string)$name));
     }
 
     /**
      * @param false|string $implode Implode header lines, false to return header as array.
      */
-    public function get(int|string $name, mixed $default = null, bool|string $implode = ','): array|string|null
+    public function get(int | string $name, mixed $default = null, bool | string $implode = ','): array | string | null
     {
-        $value = parent::get($this->normalize((string) $name), $default);
+        $value = parent::get($this->normalize((string)$name), $default);
 
-        if (\is_string($implode) && !empty($implode) && \is_array($value)) {
+        if (\is_string($implode) && !empty($implode) && \is_array($value)){
             return \implode($implode, $value);
         }
 
@@ -34,12 +34,12 @@ 
      */
     public function fetch(array $keys, bool $fill = false, mixed $filler = null, ?string $implode = ','): array
     {
-        $keys = \array_map(fn (string $header): string => $this->normalize($header), $keys);
+        $keys = \array_map(fn (string $header) : string => $this->normalize($header), $keys);
 
         $values = parent::fetch($keys, $fill, $filler);
 
-        if (!empty($implode)) {
-            foreach ($values as &$value) {
+        if (!empty($implode)){
+            foreach ($values as &$value){
                 $value = \implode($implode, $value);
                 unset($value);
             }

Braces +6 added lines, -3 removed lines

@@ -22,7 +22,8 @@ 
     {
         $value = parent::get($this->normalize((string) $name), $default);
 
-        if (\is_string($implode) && !empty($implode) && \is_array($value)) {
+        if (\is_string($implode) && !empty($implode) && \is_array($value))
+        {
             return \implode($implode, $value);
         }
 
@@ -38,8 +39,10 @@ 
 
         $values = parent::fetch($keys, $fill, $filler);
 
-        if (!empty($implode)) {
-            foreach ($values as &$value) {
+        if (!empty($implode))
+        {
+            foreach ($values as &$value)
+            {
                 $value = \implode($implode, $value);
                 unset($value);
             }

src/Http/src/Middleware/ErrorHandlerMiddleware/EnvSuppressErrors.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 {
     public function __construct(
         private DebugMode $debugMode
-    ) {
+    ){
     }
 
     public function suppressed(): bool

src/Http/src/Middleware/JsonPayloadMiddleware.php

Spacing +6 added lines, -6 removed lines

@@ -21,16 +21,16 @@ 
      */
     public function __construct(
         private readonly JsonPayloadConfig $config
-    ) {
+    ){
     }
 
     public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
     {
-        if ($this->isJsonPayload($request)) {
+        if ($this->isJsonPayload($request)){
             $body = (string)$request->getBody();
-            if ($body !== '') {
+            if ($body !== ''){
                 $request = $request->withParsedBody(\json_decode($body, true));
-                if (\json_last_error() !== 0) {
+                if (\json_last_error() !== 0){
                     throw new ClientException(400, 'invalid json payload');
                 }
             }
@@ -43,8 +43,8 @@ 
     {
         $contentType = $request->getHeaderLine('Content-Type');
 
-        foreach ($this->config->getContentTypes() as $allowedType) {
-            if (\stripos($contentType, $allowedType) === 0) {
+        foreach ($this->config->getContentTypes() as $allowedType){
+            if (\stripos($contentType, $allowedType) === 0){
                 return true;
             }
         }

Braces +10 added lines, -5 removed lines

@@ -26,11 +26,14 @@ 
 
     public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
     {
-        if ($this->isJsonPayload($request)) {
+        if ($this->isJsonPayload($request))
+        {
             $body = (string)$request->getBody();
-            if ($body !== '') {
+            if ($body !== '')
+            {
                 $request = $request->withParsedBody(\json_decode($body, true));
-                if (\json_last_error() !== 0) {
+                if (\json_last_error() !== 0)
+                {
                     throw new ClientException(400, 'invalid json payload');
                 }
             }
@@ -43,8 +46,10 @@ 
     {
         $contentType = $request->getHeaderLine('Content-Type');
 
-        foreach ($this->config->getContentTypes() as $allowedType) {
-            if (\stripos($contentType, $allowedType) === 0) {
+        foreach ($this->config->getContentTypes() as $allowedType)
+        {
+            if (\stripos($contentType, $allowedType) === 0)
+            {
                 return true;
             }
         }

src/Http/src/CallableHandler.php

Spacing +11 added lines, -11 removed lines

@@ -24,7 +24,7 @@ 
     public function __construct(
         callable $callable,
         private readonly ResponseFactoryInterface $responseFactory
-    ) {
+    ){
         $this->callable = $callable;
     }
 
@@ -39,21 +39,21 @@ 
         $output = $result = null;
 
         $response = $this->responseFactory->createResponse(200);
-        try {
+        try{
             $result = \call_user_func($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
         );
     }
 
@@ -66,17 +66,17 @@ 
      */
     private function wrapResponse(Response $response, mixed $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

@@ -39,13 +39,19 @@ 
         $output = $result = null;
 
         $response = $this->responseFactory->createResponse(200);
-        try {
+        try
+        {
             $result = \call_user_func($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;
             }
         }
@@ -66,17 +72,22 @@ 
      */
     private function wrapResponse(Response $response, mixed $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/Header/AcceptHeader.php

Spacing +18 added lines, -18 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function __construct(array $items = [])
     {
-        foreach ($items as $item) {
+        foreach ($items as $item){
             $this->addItem($item);
         }
     }
@@ -41,9 +41,9 @@ 
         $header = new static();
 
         $parts = \explode(',', $raw);
-        foreach ($parts as $part) {
+        foreach ($parts as $part){
             $part = \trim($part);
-            if ($part !== '') {
+            if ($part !== ''){
                 $header->addItem($part);
             }
         }
@@ -51,7 +51,7 @@ 
         return $header;
     }
 
-    public function add(AcceptHeaderItem|string $item): self
+    public function add(AcceptHeaderItem | string $item): self
     {
         $header = clone $this;
         $header->addItem($item);
@@ -74,7 +74,7 @@ 
      */
     public function getAll(): array
     {
-        if (!$this->sorted) {
+        if (!$this->sorted){
             /**
              * Sort item in descending order.
              */
@@ -89,14 +89,14 @@ 
     /**
      * Add new item to list.
      */
-    private function addItem(AcceptHeaderItem|string $item): void
+    private function addItem(AcceptHeaderItem | string $item): void
     {
-        if (\is_string($item)) {
+        if (\is_string($item)){
             $item = AcceptHeaderItem::fromString($item);
         }
 
         $value = \strtolower($item->getValue());
-        if ($value !== '' && (!$this->has($value) || self::compare($item, $this->get($value)) === 1)) {
+        if ($value !== '' && (!$this->has($value) || self::compare($item, $this->get($value)) === 1)){
             $this->sorted = false;
             $this->items[$value] = $item;
         }
@@ -106,11 +106,11 @@ 
      * Compare to header items, witch one is preferable.
      * Return 1 if first value preferable or -1 if second, 0 in case of same weight.
      */
-    private static function compare(AcceptHeaderItem|string $a, AcceptHeaderItem|string $b): int
+    private static function compare(AcceptHeaderItem | string $a, AcceptHeaderItem | string $b): int
     {
-        if ($a->getQuality() === $b->getQuality()) {
+        if ($a->getQuality() === $b->getQuality()){
             // If quality are same value with more params has more weight.
-            if (\count($a->getParams()) === \count($b->getParams())) {
+            if (\count($a->getParams()) === \count($b->getParams())){
                 // If quality and params then check for specific type or subtype.
                 // Means */* or * has less weight.
                 return static::compareValue($a->getValue(), $b->getValue());
@@ -129,11 +129,11 @@ 
     private static function compareValue(string $a, string $b): int
     {
         // Check "Accept" headers values with it is type and subtype.
-        if (\str_contains($a, '/') && \str_contains($b, '/')) {
+        if (\str_contains($a, '/') && \str_contains($b, '/')){
             [$typeA, $subtypeA] = \explode('/', $a, 2);
             [$typeB, $subtypeB] = \explode('/', $b, 2);
 
-            if ($typeA === $typeB) {
+            if ($typeA === $typeB){
                 return static::compareAtomic($subtypeA, $subtypeB);
             }
 
@@ -145,23 +145,23 @@ 
 
     private static function compareAtomic(string $a, string $b): int
     {
-        if (\mb_strpos($a, '*/') === 0) {
+        if (\mb_strpos($a, '*/') === 0){
             $a = '*';
         }
 
-        if (\mb_strpos($b, '*/') === 0) {
+        if (\mb_strpos($b, '*/') === 0){
             $b = '*';
         }
 
-        if (\strtolower($a) === \strtolower($b)) {
+        if (\strtolower($a) === \strtolower($b)){
             return 0;
         }
 
-        if ($a === '*') {
+        if ($a === '*'){
             return -1;
         }
 
-        if ($b === '*') {
+        if ($b === '*'){
             return 1;
         }
 

Braces +30 added lines, -15 removed lines

@@ -26,7 +26,8 @@ 
      */
     public function __construct(array $items = [])
     {
-        foreach ($items as $item) {
+        foreach ($items as $item)
+        {
             $this->addItem($item);
         }
     }
@@ -41,9 +42,11 @@ 
         $header = new static();
 
         $parts = \explode(',', $raw);
-        foreach ($parts as $part) {
+        foreach ($parts as $part)
+        {
             $part = \trim($part);
-            if ($part !== '') {
+            if ($part !== '')
+            {
                 $header->addItem($part);
             }
         }
@@ -74,7 +77,8 @@ 
      */
     public function getAll(): array
     {
-        if (!$this->sorted) {
+        if (!$this->sorted)
+        {
             /**
              * Sort item in descending order.
              */
@@ -91,12 +95,14 @@ 
      */
     private function addItem(AcceptHeaderItem|string $item): void
     {
-        if (\is_string($item)) {
+        if (\is_string($item))
+        {
             $item = AcceptHeaderItem::fromString($item);
         }
 
         $value = \strtolower($item->getValue());
-        if ($value !== '' && (!$this->has($value) || self::compare($item, $this->get($value)) === 1)) {
+        if ($value !== '' && (!$this->has($value) || self::compare($item, $this->get($value)) === 1))
+        {
             $this->sorted = false;
             $this->items[$value] = $item;
         }
@@ -108,9 +114,11 @@ 
      */
     private static function compare(AcceptHeaderItem|string $a, AcceptHeaderItem|string $b): int
     {
-        if ($a->getQuality() === $b->getQuality()) {
+        if ($a->getQuality() === $b->getQuality())
+        {
             // If quality are same value with more params has more weight.
-            if (\count($a->getParams()) === \count($b->getParams())) {
+            if (\count($a->getParams()) === \count($b->getParams()))
+            {
                 // If quality and params then check for specific type or subtype.
                 // Means */* or * has less weight.
                 return static::compareValue($a->getValue(), $b->getValue());
@@ -129,11 +137,13 @@ 
     private static function compareValue(string $a, string $b): int
     {
         // Check "Accept" headers values with it is type and subtype.
-        if (\str_contains($a, '/') && \str_contains($b, '/')) {
+        if (\str_contains($a, '/') && \str_contains($b, '/'))
+        {
             [$typeA, $subtypeA] = \explode('/', $a, 2);
             [$typeB, $subtypeB] = \explode('/', $b, 2);
 
-            if ($typeA === $typeB) {
+            if ($typeA === $typeB)
+            {
                 return static::compareAtomic($subtypeA, $subtypeB);
             }
 
@@ -145,23 +155,28 @@ 
 
     private static function compareAtomic(string $a, string $b): int
     {
-        if (\mb_strpos($a, '*/') === 0) {
+        if (\mb_strpos($a, '*/') === 0)
+        {
             $a = '*';
         }
 
-        if (\mb_strpos($b, '*/') === 0) {
+        if (\mb_strpos($b, '*/') === 0)
+        {
             $b = '*';
         }
 
-        if (\strtolower($a) === \strtolower($b)) {
+        if (\strtolower($a) === \strtolower($b))
+        {
             return 0;
         }
 
-        if ($a === '*') {
+        if ($a === '*')
+        {
             return -1;
         }
 
-        if ($b === '*') {
+        if ($b === '*')
+        {
             return 1;
         }
 

src/Http/src/Header/AcceptHeaderItem.php

Spacing +10 added lines, -10 removed lines

@@ -27,17 +27,17 @@ 
 
     public function __toString(): string
     {
-        if ($this->value === '') {
+        if ($this->value === ''){
             return '';
         }
 
         $parts = [$this->value];
 
-        if ($this->quality < 1) {
+        if ($this->quality < 1){
             $parts[] = \sprintf('q=%s', $this->quality);
         }
 
-        foreach ($this->getParams() as $name => $value) {
+        foreach ($this->getParams() as $name => $value){
             $parts[] = \sprintf('%s=%s', $name, $value);
         }
 
@@ -55,20 +55,20 @@ 
         $quality = 1.0;
         $params = [];
 
-        foreach ($elements as $element) {
+        foreach ($elements as $element){
             $parsed = \explode('=', \trim($element), 2);
 
             // Wrong params must be ignored
-            if (\count($parsed) !== 2) {
+            if (\count($parsed) !== 2){
                 continue;
             }
 
             $name = \trim($parsed[0]);
             $value = \trim($parsed[1]);
 
-            if (\strcasecmp($name, 'q') === 0) {
+            if (\strcasecmp($name, 'q') === 0){
                 $quality = (float)$value;
-            } else {
+            }else{
                 $params[$name] = $value;
             }
         }
@@ -127,15 +127,15 @@ 
 
     private function setParams(array $params): void
     {
-        foreach ($params as $name => $value) {
-            if (\is_numeric($name) || !\is_scalar($value)) {
+        foreach ($params as $name => $value){
+            if (\is_numeric($name) || !\is_scalar($value)){
                 continue;
             }
 
             $name = \trim($name);
             $value = \trim((string)$value);
 
-            if ($name === '' || $value === '') {
+            if ($name === '' || $value === ''){
                 continue;
             }
 

Braces +21 added lines, -10 removed lines

@@ -27,17 +27,20 @@ 
 
     public function __toString(): string
     {
-        if ($this->value === '') {
+        if ($this->value === '')
+        {
             return '';
         }
 
         $parts = [$this->value];
 
-        if ($this->quality < 1) {
+        if ($this->quality < 1)
+        {
             $parts[] = \sprintf('q=%s', $this->quality);
         }
 
-        foreach ($this->getParams() as $name => $value) {
+        foreach ($this->getParams() as $name => $value)
+        {
             $parts[] = \sprintf('%s=%s', $name, $value);
         }
 
@@ -55,20 +58,25 @@ 
         $quality = 1.0;
         $params = [];
 
-        foreach ($elements as $element) {
+        foreach ($elements as $element)
+        {
             $parsed = \explode('=', \trim($element), 2);
 
             // Wrong params must be ignored
-            if (\count($parsed) !== 2) {
+            if (\count($parsed) !== 2)
+            {
                 continue;
             }
 
             $name = \trim($parsed[0]);
             $value = \trim($parsed[1]);
 
-            if (\strcasecmp($name, 'q') === 0) {
+            if (\strcasecmp($name, 'q') === 0)
+            {
                 $quality = (float)$value;
-            } else {
+            }
+            else
+            {
                 $params[$name] = $value;
             }
         }
@@ -127,15 +135,18 @@ 
 
     private function setParams(array $params): void
     {
-        foreach ($params as $name => $value) {
-            if (\is_numeric($name) || !\is_scalar($value)) {
+        foreach ($params as $name => $value)
+        {
+            if (\is_numeric($name) || !\is_scalar($value))
+            {
                 continue;
             }
 
             $name = \trim($name);
             $value = \trim((string)$value);
 
-            if ($name === '' || $value === '') {
+            if ($name === '' || $value === '')
+            {
                 continue;
             }
 

src/Http/src/ResponseWrapper.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
         private readonly ResponseFactoryInterface $responseFactory,
         private readonly StreamFactoryInterface $streamFactory,
         private readonly FilesInterface $files
-    ) {
+    ){
     }
 
     public function create(int $code): ResponseInterface
@@ -39,7 +39,7 @@ 
      * @throws ResponseException
      */
     public function redirect(
-        string|UriInterface $uri,
+        string | UriInterface $uri,
         int $code = 302
     ): ResponseInterface {
         return $this->responseFactory->createResponse($code)->withHeader('Location', (string)$uri);
@@ -67,8 +67,8 @@ 
         string $name = '',
         string $mime = 'application/octet-stream'
     ): ResponseInterface {
-        if (empty($name)) {
-            if (!\is_string($filename)) {
+        if (empty($name)){
+            if (!\is_string($filename)){
                 throw new ResponseException('Unable to resolve public filename');
             }
 
@@ -82,7 +82,7 @@ 
         $response = $response->withHeader('Content-Length', (string)$stream->getSize());
         $response = $response->withHeader(
             'Content-Disposition',
-            'attachment; filename="' . \addcslashes($name, '"') . '"'
+            'attachment; filename="'.\addcslashes($name, '"').'"'
         );
 
         return $response->withBody($stream);

Braces +4 added lines, -2 removed lines

@@ -67,8 +67,10 @@
         string $name = '',
         string $mime = 'application/octet-stream'
     ): ResponseInterface {
-        if (empty($name)) {
-            if (!\is_string($filename)) {
+        if (empty($name))
+        {
+            if (!\is_string($filename))
+            {
                 throw new ResponseException('Unable to resolve public filename');
             }
 

src/SendIt/tests/JobTest.php

Spacing +3 added lines, -3 removed lines

@@ -57,13 +57,13 @@ 
 
         $this->mailer->expects('send')->with($email)->andThrow(new TransportException('failed'));
 
-        try {
+        try{
             $this->getHandler()->handle(
                 MailQueue::JOB_NAME,
                 'id',
                 json_encode(MessageSerializer::pack($this->getMail()))
             );
-        } catch (TransportException $e) {
+        }catch (TransportException $e){
         }
     }
 
@@ -121,7 +121,7 @@ 
     private function expectRenderer(Email $email): void
     {
         $this->renderer->expects('render')->withArgs(
-            function (Message $message) {
+            function (Message $message){
                 $this->assertSame($message->getSubject(), 'test');
                 return true;
             }

Braces +7 added lines, -3 removed lines

@@ -57,13 +57,16 @@ 
 
         $this->mailer->expects('send')->with($email)->andThrow(new TransportException('failed'));
 
-        try {
+        try
+        {
             $this->getHandler()->handle(
                 MailQueue::JOB_NAME,
                 'id',
                 json_encode(MessageSerializer::pack($this->getMail()))
             );
-        } catch (TransportException $e) {
+        }
+        catch (TransportException $e)
+        {
         }
     }
 
@@ -121,7 +124,8 @@ 
     private function expectRenderer(Email $email): void
     {
         $this->renderer->expects('render')->withArgs(
-            function (Message $message) {
+            function (Message $message)
+            {
                 $this->assertSame($message->getSubject(), 'test');
                 return true;
             }

src/SendIt/tests/JobsQueueTest.php

Braces +2 added lines, -1 removed lines

@@ -33,7 +33,8 @@
         $mail->setBCC('admin2@google.com');
 
         $queue->expects('push')->withArgs(
-            function ($job, $data, Options $options) use ($mail) {
+            function ($job, $data, Options $options) use ($mail)
+            {
                 $this->assertSame(MailQueue::JOB_NAME, $job);
                 $this->assertSame($data, MessageSerializer::pack($mail));
                 $this->assertSame('mailer', $options->getQueue());