spiral / framework

src/Cookies/src/Middleware/CookiesMiddleware.php

Braces +30 added lines, -14 removed lines

@@ -50,8 +50,10 @@ 
     {
         $cookies = $request->getCookieParams();
 
-        foreach ($cookies as $name => $cookie) {
-            if (!$this->isProtected($name)) {
+        foreach ($cookies as $name => $cookie)
+        {
+            if (!$this->isProtected($name))
+            {
                 continue;
             }
 
@@ -66,7 +68,8 @@ 
      */
     protected function isProtected(string $cookie): bool
     {
-        if (\in_array($cookie, $this->config->getExcludedCookies(), true)) {
+        if (\in_array($cookie, $this->config->getExcludedCookies(), true))
+        {
             //Excluded
             return false;
         }
@@ -81,14 +84,17 @@ 
      */
     protected function packCookies(Response $response, CookieQueue $queue): Response
     {
-        if (empty($queue->getScheduled())) {
+        if (empty($queue->getScheduled()))
+        {
             return $response;
         }
 
         $cookies = $response->getHeader('Set-Cookie');
 
-        foreach ($queue->getScheduled() as $cookie) {
-            if (empty($cookie->getValue()) || !$this->isProtected($cookie->getName())) {
+        foreach ($queue->getScheduled() as $cookie)
+        {
+            if (empty($cookie->getValue()) || !$this->isProtected($cookie->getName()))
+            {
                 $cookies[] = $cookie->createHeader();
                 continue;
             }
@@ -101,26 +107,35 @@ 
 
     private function decodeCookie(array|string $cookie): mixed
     {
-        try {
-            if (\is_array($cookie)) {
+        try
+        {
+            if (\is_array($cookie))
+            {
                 return \array_map(fn (array|string $cookie): mixed => $this->decodeCookie($cookie), $cookie);
             }
-        } catch (DecryptException) {
+        }
+        catch (DecryptException)
+        {
             return null;
         }
 
-        switch ($this->config->getProtectionMethod()) {
+        switch ($this->config->getProtectionMethod())
+        {
             case CookiesConfig::COOKIE_ENCRYPT:
-                try {
+                try
+                {
                     return $this->encryption->getEncrypter()->decrypt($cookie);
-                } catch (DecryptException) {
+                }
+                catch (DecryptException)
+                {
                 }
                 return null;
             case CookiesConfig::COOKIE_HMAC:
                 $hmac = \substr($cookie, -1 * CookiesConfig::MAC_LENGTH);
                 $value = \substr($cookie, 0, \strlen($cookie) - \strlen($hmac));
 
-                if (\hash_equals($this->hmacSign($value), $hmac)) {
+                if (\hash_equals($this->hmacSign($value), $hmac))
+                {
                     return $value;
                 }
         }
@@ -143,7 +158,8 @@ 
     private function encodeCookie(Cookie $cookie): Cookie
     {
         $value = $cookie->getValue() ?? '';
-        if ($this->config->getProtectionMethod() === CookiesConfig::COOKIE_ENCRYPT) {
+        if ($this->config->getProtectionMethod() === CookiesConfig::COOKIE_ENCRYPT)
+        {
             $encryptor = $this->encryption->getEncrypter();
 
             return $cookie->withValue($encryptor->encrypt($value));

src/Core/tests/Scope/ProxyTest.php

Braces +14 added lines, -7 removed lines

@@ -49,7 +49,8 @@ 
                     // from the current `foo` scope
                     self::assertInstanceOf(KVLogger::class, $logger);
 
-                    for ($i = 0; $i < 10; $i++) {
+                    for ($i = 0; $i < 10; $i++)
+                    {
                         // because of proxy
                         self::assertNotInstanceOf(KVLogger::class, $carrier->getLogger());
                         self::assertSame('kv', $carrier->logger->getName());
@@ -69,7 +70,8 @@ 
                     // from the current `foo` scope
                     self::assertInstanceOf(FileLogger::class, $logger);
 
-                    for ($i = 0; $i < 10; $i++) {
+                    for ($i = 0; $i < 10; $i++)
+                    {
                         // because of proxy
                         self::assertNotInstanceOf(FileLogger::class, $carrier->getLogger());
                         self::assertSame('file', $carrier->logger->getName());
@@ -137,7 +139,8 @@ 
             ->bind(
                 ContextInterface::class,
                 new \Spiral\Core\Config\Injectable(
-                    new class implements InjectorInterface {
+                    new class implements InjectorInterface
+                    {
                         public function createInjection(\ReflectionClass $class, mixed $context = null): Context
                         {
                             return new Context($context);
@@ -178,7 +181,8 @@ 
             ->bind(
                 ContextInterface::class,
                 new \Spiral\Core\Config\Injectable(
-                    new class implements InjectorInterface {
+                    new class implements InjectorInterface
+                    {
                         public function createInjection(\ReflectionClass $class, mixed $context = null): Context
                         {
                             return new Context($context);
@@ -189,14 +193,16 @@ 
 
         FiberHelper::runFiberSequence(
             static fn(): mixed => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $ctx): void {
-                for ($i = 0; $i < 10; $i++) {
+                for ($i = 0; $i < 10; $i++)
+                {
                     self::assertInstanceOf(ReflectionParameter::class, $ctx->getValue(), 'Context injected');
                     self::assertSame('ctx', $ctx->getValue()->getName());
                     \Fiber::suspend();
                 }
             }),
             static fn(): mixed => $root->runScope(new Scope(name: 'foo'), static function (ContextInterface $context): void {
-                for ($i = 0; $i < 10; $i++) {
+                for ($i = 0; $i < 10; $i++)
+                {
                     self::assertInstanceOf(ReflectionParameter::class, $context->getValue(), 'Context injected');
                     self::assertSame('context', $context->getValue()->getName());
                     \Fiber::suspend();
@@ -257,7 +263,8 @@ 
     {
         $root = new Container();
         $context = (object)['destroyed' => false];
-        $class = new class($context) implements DestroyableInterface {
+        $class = new class($context) implements DestroyableInterface
+        {
             public function __construct(
                 private readonly \stdClass $context,
             ) {

src/Core/tests/ScopesTest.php

Braces +17 added lines, -7 removed lines

@@ -32,11 +32,14 @@ 
 
         $this->assertNull(ContainerScope::getContainer());
 
-        try {
+        try
+        {
             $this->assertTrue(ContainerScope::runScope($container, static function (): never {
                 throw new RuntimeException('exception');
             }));
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
         }
 
         $this->assertInstanceOf(RuntimeException::class, $e);
@@ -54,7 +57,8 @@ 
         $this->assertTrue($c->runScope([
             'bucket' => new Bucket('b'),
             'other'  => new SampleClass()
-        ], function ($c) {
+        ], function ($c)
+        {
             $this->assertSame('b', $c->get('bucket')->getName());
             $this->assertTrue($c->has('other'));
 
@@ -76,21 +80,25 @@ 
         $this->assertTrue($c->runScope([
             'bucket' => new Bucket('b'),
             'other'  => new SampleClass()
-        ], function ($c) {
+        ], function ($c)
+        {
             $this->assertSame('b', $c->get('bucket')->getName());
             $this->assertTrue($c->has('other'));
 
             return $c->get('bucket')->getName() == 'b' && $c->has('other');
         }));
 
-        try {
+        try
+        {
             $this->assertTrue($c->runScope([
                 'bucket' => new Bucket('b'),
                 'other'  => new SampleClass()
             ], function () use ($c): void {
                 throw new RuntimeException('exception');
             }));
-        } catch (\Throwable) {
+        }
+        catch (\Throwable)
+        {
         }
 
         $this->assertSame('a', $c->get('bucket')->getName());
@@ -138,7 +146,9 @@ 
     public function testHasInstanceAfterMakeWithoutAliasInScope(): void
     {
         $container = new Container();
-        $container->bindSingleton('test', new #[Singleton] class {});
+        $container->bindSingleton('test', new #[Singleton] class
+        {
+});
         $container->make('test');
 
         $container->runScoped(function (Container $container): void {

src/Http/tests/PipelineTest.php

Braces +4 added lines, -2 removed lines

@@ -60,7 +60,8 @@ 
 
     public function testMiddlewareProcessingEventShouldBeDispatched(): void
     {
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface
+        {
             public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
             {
                 return (new ResponseFactory(new HttpConfig(['headers' => []])))->createResponse(200);
@@ -89,7 +90,8 @@ 
         $this->container->getBinder('http')
             ->bind(ServerRequestInterface::class, static fn(CurrentRequest $cr): ?ServerRequestInterface => $cr->get());
 
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface
+        {
             public function process(
                 ServerRequestInterface $request,
                 RequestHandlerInterface $handler,