spiral / framework

tests/Framework/Jobs/DispatcherTest.php

Spacing +4 added lines, -4 removed lines

@@ -41,7 +41,7 @@ 
         $this->assertNull($this->app->getEnvironment()->get('FIRED'));
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "id": "1",
                   "job": "spiral.app.job.TestJob"
@@ -69,7 +69,7 @@ 
         $this->app->getContainer()->bind(Worker::class, $w);
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "id": "1",
                   "job": "spiral.app.job.ErrorJob"
@@ -85,7 +85,7 @@ 
         $w->shouldReceive('receive')->once()->andReturn(null);
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(0, $files);
@@ -93,7 +93,7 @@ 
         $this->app->get(JobDispatcher::class)->serve();
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(1, $files);

Braces +4 added lines, -2 removed lines

@@ -41,7 +41,8 @@ 
         $this->assertNull($this->app->getEnvironment()->get('FIRED'));
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "id": "1",
                   "job": "spiral.app.job.TestJob"
@@ -69,7 +70,8 @@ 
         $this->app->getContainer()->bind(Worker::class, $w);
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "id": "1",
                   "job": "spiral.app.job.ErrorJob"

tests/Framework/Http/SessionTest.php

Spacing +6 added lines, -6 removed lines

@@ -18,7 +18,7 @@ 
 {
     public function testSetSid(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -32,7 +32,7 @@ 
 
     public function testSessionResume(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -56,7 +56,7 @@ 
 
     public function testSessionRegenerateId(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -73,7 +73,7 @@ 
         $this->assertSame(200, $result->getStatusCode());
         $this->assertSame('2', $result->getBody()->__toString());
 
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             $this->session()->regenerateID(false);
 
             return ++$this->session()->getSection('cli')->value;
@@ -92,7 +92,7 @@ 
 
     public function testDestroySession(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -105,7 +105,7 @@ 
         ]);
         $this->assertSame(200, $result->getStatusCode());
         $this->assertSame('2', $result->getBody()->__toString());
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             $this->session()->destroy();
             $this->assertFalse($this->session()->isStarted());
 

Braces +12 added lines, -6 removed lines

@@ -18,7 +18,8 @@ 
 {
     public function testSetSid(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -32,7 +33,8 @@ 
 
     public function testSessionResume(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -56,7 +58,8 @@ 
 
     public function testSessionRegenerateId(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -73,7 +76,8 @@ 
         $this->assertSame(200, $result->getStatusCode());
         $this->assertSame('2', $result->getBody()->__toString());
 
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             $this->session()->regenerateID(false);
 
             return ++$this->session()->getSection('cli')->value;
@@ -92,7 +96,8 @@ 
 
     public function testDestroySession(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -105,7 +110,8 @@ 
         ]);
         $this->assertSame(200, $result->getStatusCode());
         $this->assertSame('2', $result->getBody()->__toString());
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             $this->session()->destroy();
             $this->assertFalse($this->session()->isStarted());
 

tests/Framework/Http/SapiTest.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
 
     public function setUp(): void
     {
-        $this->bufferEmitter = new class() implements EmitterInterface {
+        $this->bufferEmitter = new class() implements EmitterInterface{
             public $response;
 
             public function emit(ResponseInterface $response): bool
@@ -59,7 +59,7 @@ 
         $e = $this->bufferEmitter;
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(0, $files);
@@ -68,7 +68,7 @@ 
         $this->app->get(SapiDispatcher::class)->serve($e);
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(1, $files);
@@ -85,7 +85,7 @@ 
         ]);
 
         $files = $app->get(FilesInterface::class)->getFiles(
-            $app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(0, $files);
@@ -94,7 +94,7 @@ 
         $app->get(SapiDispatcher::class)->serve($e);
 
         $files = $app->get(FilesInterface::class)->getFiles(
-            $app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(1, $files);

Braces +2 added lines, -1 removed lines

@@ -25,7 +25,8 @@
 
     public function setUp(): void
     {
-        $this->bufferEmitter = new class() implements EmitterInterface {
+        $this->bufferEmitter = new class() implements EmitterInterface
+        {
             public $response;
 
             public function emit(ResponseInterface $response): bool

src/Cookies/CookieManager.php

Spacing +5 added lines, -5 removed lines

@@ -118,7 +118,7 @@ 
         string $domain = null,
         bool $secure = null,
         bool $httpOnly = true
-    ) {
+    ){
         $this->getCookieQueue()->set($name, $value, $lifetime, $path, $domain, $secure, $httpOnly);
 
         return $this;
@@ -169,9 +169,9 @@ 
      */
     private function getRequest(): ServerRequestInterface
     {
-        try {
+        try{
             return $this->container->get(ServerRequestInterface::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }catch (NotFoundExceptionInterface $e){
             throw new ScopeException('Unable to receive active request', $e->getCode(), $e);
         }
     }
@@ -183,9 +183,9 @@ 
      */
     private function getCookieQueue(): CookieQueue
     {
-        try {
+        try{
             return $this->container->get(CookieQueue::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }catch (NotFoundExceptionInterface $e){
             throw new ScopeException('Unable to receive cookie queue, invalid request scope', $e->getCode(), $e);
         }
     }

Braces +10 added lines, -4 removed lines

@@ -169,9 +169,12 @@ 
      */
     private function getRequest(): ServerRequestInterface
     {
-        try {
+        try
+        {
             return $this->container->get(ServerRequestInterface::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }
+        catch (NotFoundExceptionInterface $e)
+        {
             throw new ScopeException('Unable to receive active request', $e->getCode(), $e);
         }
     }
@@ -183,9 +186,12 @@ 
      */
     private function getCookieQueue(): CookieQueue
     {
-        try {
+        try
+        {
             return $this->container->get(CookieQueue::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }
+        catch (NotFoundExceptionInterface $e)
+        {
             throw new ScopeException('Unable to receive cookie queue, invalid request scope', $e->getCode(), $e);
         }
     }

src/Auth/AuthScope.php

Spacing +2 added lines, -2 removed lines

@@ -94,9 +94,9 @@
      */
     private function getAuthContext(): AuthContextInterface
     {
-        try {
+        try{
             return $this->container->get(AuthContextInterface::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }catch (NotFoundExceptionInterface $e){
             throw new ScopeException('Unable to resolve auth context, invalid scope', $e->getCode(), $e);
         }
     }

Braces +5 added lines, -2 removed lines

@@ -94,9 +94,12 @@
      */
     private function getAuthContext(): AuthContextInterface
     {
-        try {
+        try
+        {
             return $this->container->get(AuthContextInterface::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }
+        catch (NotFoundExceptionInterface $e)
+        {
             throw new ScopeException('Unable to resolve auth context, invalid scope', $e->getCode(), $e);
         }
     }

src/Security/GuardScope.php

Spacing +6 added lines, -6 removed lines

@@ -41,7 +41,7 @@ 
         PermissionsInterface $permissions,
         ContainerInterface $actorScope,
         array $roles = []
-    ) {
+    ){
         $this->roles = $roles;
         $this->container = $actorScope;
         $this->permissions = $permissions;
@@ -55,8 +55,8 @@ 
     public function allows(string $permission, array $context = []): bool
     {
         $allows = false;
-        foreach ($this->getRoles() as $role) {
-            if (!$this->permissions->hasRole($role)) {
+        foreach ($this->getRoles() as $role){
+            if (!$this->permissions->hasRole($role)){
                 continue;
             }
 
@@ -103,13 +103,13 @@ 
      */
     public function getActor(): ActorInterface
     {
-        if (!is_null($this->actor)) {
+        if (!is_null($this->actor)){
             return $this->actor;
         }
 
-        try {
+        try{
             return $this->container->get(ActorInterface::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }catch (NotFoundExceptionInterface $e){
             throw new ScopeException('Unable to resolve Actor from the scope', $e->getCode(), $e);
         }
     }

Braces +11 added lines, -5 removed lines

@@ -55,8 +55,10 @@ 
     public function allows(string $permission, array $context = []): bool
     {
         $allows = false;
-        foreach ($this->getRoles() as $role) {
-            if (!$this->permissions->hasRole($role)) {
+        foreach ($this->getRoles() as $role)
+        {
+            if (!$this->permissions->hasRole($role))
+            {
                 continue;
             }
 
@@ -103,13 +105,17 @@ 
      */
     public function getActor(): ActorInterface
     {
-        if (!is_null($this->actor)) {
+        if (!is_null($this->actor))
+        {
             return $this->actor;
         }
 
-        try {
+        try
+        {
             return $this->container->get(ActorInterface::class);
-        } catch (NotFoundExceptionInterface $e) {
+        }
+        catch (NotFoundExceptionInterface $e)
+        {
             throw new ScopeException('Unable to resolve Actor from the scope', $e->getCode(), $e);
         }
     }

src/Bootloader/Auth/HttpAuthBootloader.php

Spacing +2 added lines, -2 removed lines

@@ -87,8 +87,8 @@
         $registry = new TransportRegistry();
         $registry->setDefaultTransport($config->getDefaultTransport());
 
-        foreach ($config->getTransports() as $name => $transport) {
-            if ($transport instanceof Autowire) {
+        foreach ($config->getTransports() as $name => $transport){
+            if ($transport instanceof Autowire){
                 $transport = $transport->resolve($factory);
             }
 

Braces +4 added lines, -2 removed lines

@@ -87,8 +87,10 @@
         $registry = new TransportRegistry();
         $registry->setDefaultTransport($config->getDefaultTransport());
 
-        foreach ($config->getTransports() as $name => $transport) {
-            if ($transport instanceof Autowire) {
+        foreach ($config->getTransports() as $name => $transport)
+        {
+            if ($transport instanceof Autowire)
+            {
                 $transport = $transport->resolve($factory);
             }
 

src/Bootloader/Auth/SecurityActorBootloader.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     private function actor(AuthContextInterface $context): ActorInterface
     {
         $actor = $context->getActor();
-        if ($actor instanceof ActorInterface) {
+        if ($actor instanceof ActorInterface){
             return $actor;
         }
 

Braces +2 added lines, -1 removed lines

@@ -38,7 +38,8 @@
     private function actor(AuthContextInterface $context): ActorInterface
     {
         $actor = $context->getActor();
-        if ($actor instanceof ActorInterface) {
+        if ($actor instanceof ActorInterface)
+        {
             return $actor;
         }
 

src/Session/Middleware/SessionMiddleware.php

Spacing +8 added lines, -8 removed lines

@@ -60,7 +60,7 @@ 
         CookiesConfig $cookiesConfig,
         SessionFactory $factory,
         ScopeInterface $scope
-    ) {
+    ){
         $this->config = $config;
         $this->httpConfig = $httpConfig;
         $this->cookiesConfig = $cookiesConfig;
@@ -79,14 +79,14 @@ 
             $this->fetchID($request)
         );
 
-        try {
+        try{
             $response = $this->scope->runScope(
                 [SessionInterface::class => $session],
                 function () use ($handler, $request, $session) {
                     return $handler->handle($request->withAttribute(static::ATTRIBUTE, $session));
                 }
             );
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $session->abort();
             throw $e;
         }
@@ -105,14 +105,14 @@ 
         Request $request,
         Response $response
     ): Response {
-        if (!$session->isStarted()) {
+        if (!$session->isStarted()){
             return $response;
         }
 
         $session->commit();
 
         //SID changed
-        if ($this->fetchID($request) != $session->getID()) {
+        if ($this->fetchID($request) != $session->getID()){
             return $this->withCookie($request, $response, $session->getID());
         }
 
@@ -129,7 +129,7 @@ 
     protected function fetchID(Request $request): ?string
     {
         $cookies = $request->getCookieParams();
-        if (empty($cookies[$this->config->getCookie()])) {
+        if (empty($cookies[$this->config->getCookie()])){
             return null;
         }
 
@@ -162,8 +162,8 @@ 
     protected function clientSignature(Request $request): string
     {
         $signature = '';
-        foreach (static::SIGNATURE_HEADERS as $header) {
-            $signature .= $request->getHeaderLine($header) . ';';
+        foreach (static::SIGNATURE_HEADERS as $header){
+            $signature .= $request->getHeaderLine($header).';';
         }
 
         return hash('sha256', $signature);

Braces +15 added lines, -7 removed lines

@@ -79,14 +79,18 @@ 
             $this->fetchID($request)
         );
 
-        try {
+        try
+        {
             $response = $this->scope->runScope(
                 [SessionInterface::class => $session],
-                function () use ($handler, $request, $session) {
+                function () use ($handler, $request, $session)
+                {
                     return $handler->handle($request->withAttribute(static::ATTRIBUTE, $session));
                 }
             );
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $session->abort();
             throw $e;
         }
@@ -105,14 +109,16 @@ 
         Request $request,
         Response $response
     ): Response {
-        if (!$session->isStarted()) {
+        if (!$session->isStarted())
+        {
             return $response;
         }
 
         $session->commit();
 
         //SID changed
-        if ($this->fetchID($request) != $session->getID()) {
+        if ($this->fetchID($request) != $session->getID())
+        {
             return $this->withCookie($request, $response, $session->getID());
         }
 
@@ -129,7 +135,8 @@ 
     protected function fetchID(Request $request): ?string
     {
         $cookies = $request->getCookieParams();
-        if (empty($cookies[$this->config->getCookie()])) {
+        if (empty($cookies[$this->config->getCookie()]))
+        {
             return null;
         }
 
@@ -162,7 +169,8 @@ 
     protected function clientSignature(Request $request): string
     {
         $signature = '';
-        foreach (static::SIGNATURE_HEADERS as $header) {
+        foreach (static::SIGNATURE_HEADERS as $header)
+        {
             $signature .= $request->getHeaderLine($header) . ';';
         }