spiral / framework

src/Http/RrDispatcher.php

Spacing +8 added lines, -8 removed lines

@@ -49,7 +49,7 @@ 
         FinalizerInterface $finalizer,
         ContainerInterface $container,
         FactoryInterface $factory
-    ) {
+    ){
         $this->env = $env;
         $this->finalizer = $finalizer;
         $this->container = $container;
@@ -75,12 +75,12 @@ 
 
         /** @var Http $http */
         $http = $this->container->get(Http::class);
-        while ($request = $client->acceptRequest()) {
-            try {
+        while ($request = $client->acceptRequest()){
+            try{
                 $client->respond($http->handle($request));
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 $this->handleException($client, $e);
-            } finally {
+            }finally{
                 $this->finalizer->finalize(false);
             }
         }
@@ -94,17 +94,17 @@ 
     {
         $handler = new HtmlHandler();
 
-        try {
+        try{
             /** @var SnapshotInterface $snapshot */
             $snapshot = $this->container->get(SnapshotterInterface::class)->register($e);
             error_log($snapshot->getMessage());
 
             // 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){
             error_log((string)$e);
         }
 

Braces +17 added lines, -7 removed lines

@@ -75,12 +75,18 @@ 
 
         /** @var Http $http */
         $http = $this->container->get(Http::class);
-        while ($request = $client->acceptRequest()) {
-            try {
+        while ($request = $client->acceptRequest())
+        {
+            try
+            {
                 $client->respond($http->handle($request));
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 $this->handleException($client, $e);
-            } finally {
+            }
+            finally
+            {
                 $this->finalizer->finalize(false);
             }
         }
@@ -94,17 +100,21 @@ 
     {
         $handler = new HtmlHandler();
 
-        try {
+        try
+        {
             /** @var SnapshotInterface $snapshot */
             $snapshot = $this->container->get(SnapshotterInterface::class)->register($e);
             error_log($snapshot->getMessage());
 
             // 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)
+        {
             error_log((string)$e);
         }
 

src/Debug/StateCollector/EnvironmentCollector.php

Spacing +2 added lines, -2 removed lines

@@ -43,8 +43,8 @@
     {
         $state->setTag('php', phpversion());
 
-        if ($this->container->get(DispatcherInterface::class)) {
-            switch (get_class($this->container->get(DispatcherInterface::class))) {
+        if ($this->container->get(DispatcherInterface::class)){
+            switch (get_class($this->container->get(DispatcherInterface::class))){
                 case RrDispatcher::class:
                     $state->setTag('dispatcher', 'roadrunner');
                     break;

Braces +4 added lines, -2 removed lines

@@ -43,8 +43,10 @@
     {
         $state->setTag('php', phpversion());
 
-        if ($this->container->get(DispatcherInterface::class)) {
-            switch (get_class($this->container->get(DispatcherInterface::class))) {
+        if ($this->container->get(DispatcherInterface::class))
+        {
+            switch (get_class($this->container->get(DispatcherInterface::class)))
+            {
                 case RrDispatcher::class:
                     $state->setTag('dispatcher', 'roadrunner');
                     break;

src/Bootloader/Http/HttpBootloader.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
 
         $kernel->addDispatcher($factory->make(SapiDispatcher::class));
 
-        if (class_exists(PSR7Client::class)) {
+        if (class_exists(PSR7Client::class)){
             $kernel->addDispatcher($factory->make(RrDispatcher::class));
         }
     }

Braces +2 added lines, -1 removed lines

@@ -74,7 +74,8 @@
 
         $kernel->addDispatcher($factory->make(SapiDispatcher::class));
 
-        if (class_exists(PSR7Client::class)) {
+        if (class_exists(PSR7Client::class))
+        {
             $kernel->addDispatcher($factory->make(RrDispatcher::class));
         }
     }

src/Bootloader/Auth/AuthBootloader.php

Spacing +7 added lines, -7 removed lines

@@ -52,8 +52,8 @@ 
      */
     public function getActor(TokenInterface $token): ?object
     {
-        foreach ($this->getProviders() as $provider) {
-            if (!$provider instanceof ActorProviderInterface) {
+        foreach ($this->getProviders() as $provider){
+            if (!$provider instanceof ActorProviderInterface){
                 throw new AuthException(
                     sprintf(
                         'Expected `ActorProviderInterface`, got `%s`',
@@ -63,12 +63,12 @@ 
             }
 
             $actor = $provider->getActor($token);
-            if ($actor !== null) {
+            if ($actor !== null){
                 return $actor;
             }
         }
 
-        if ($this->actorProvider === []) {
+        if ($this->actorProvider === []){
             throw new AuthException('No actor provider');
         }
 
@@ -90,13 +90,13 @@ 
      */
     private function getProviders(): \Generator
     {
-        foreach ($this->actorProvider as $provider) {
-            if ($provider instanceof Autowire) {
+        foreach ($this->actorProvider as $provider){
+            if ($provider instanceof Autowire){
                 yield $provider->resolve($this->factory);
                 continue;
             }
 
-            if (is_object($provider)) {
+            if (is_object($provider)){
                 yield $provider;
                 continue;
             }

Braces +14 added lines, -7 removed lines

@@ -52,8 +52,10 @@ 
      */
     public function getActor(TokenInterface $token): ?object
     {
-        foreach ($this->getProviders() as $provider) {
-            if (!$provider instanceof ActorProviderInterface) {
+        foreach ($this->getProviders() as $provider)
+        {
+            if (!$provider instanceof ActorProviderInterface)
+            {
                 throw new AuthException(
                     sprintf(
                         'Expected `ActorProviderInterface`, got `%s`',
@@ -63,12 +65,14 @@ 
             }
 
             $actor = $provider->getActor($token);
-            if ($actor !== null) {
+            if ($actor !== null)
+            {
                 return $actor;
             }
         }
 
-        if ($this->actorProvider === []) {
+        if ($this->actorProvider === [])
+        {
             throw new AuthException('No actor provider');
         }
 
@@ -90,13 +94,16 @@ 
      */
     private function getProviders(): \Generator
     {
-        foreach ($this->actorProvider as $provider) {
-            if ($provider instanceof Autowire) {
+        foreach ($this->actorProvider as $provider)
+        {
+            if ($provider instanceof Autowire)
+            {
                 yield $provider->resolve($this->factory);
                 continue;
             }
 
-            if (is_object($provider)) {
+            if (is_object($provider))
+            {
                 yield $provider;
                 continue;
             }

tests/Framework/Validation/EntityCheckerTest.php

Spacing +1 added lines, -1 removed lines

@@ -142,7 +142,7 @@
             ['value' => $value] + $data,
             ['value' => [['entity::unique', User::class, $field, $fields]]]
         );
-        if ($context !== null) {
+        if ($context !== null){
             $validator = $validator->withContext($context);
         }
 

Braces +2 added lines, -1 removed lines

@@ -142,7 +142,8 @@
             ['value' => $value] + $data,
             ['value' => [['entity::unique', User::class, $field, $fields]]]
         );
-        if ($context !== null) {
+        if ($context !== null)
+        {
             $validator = $validator->withContext($context);
         }
 

src/Validation/Checker/EntityChecker.php

Spacing +7 added lines, -7 removed lines

@@ -38,7 +38,7 @@ 
     public function exists($value, string $role, ?string $field = null): bool
     {
         $repository = $this->orm->getRepository($role);
-        if ($field === null) {
+        if ($field === null){
             return $repository->findByPK($value) !== null;
         }
 
@@ -57,7 +57,7 @@ 
         $values = $this->withValues($withFields);
         $values[$field] = $value;
 
-        if ($this->isProvidedByContext($role, $values)) {
+        if ($this->isProvidedByContext($role, $values)){
             return true;
         }
 
@@ -71,8 +71,8 @@ 
     private function withValues(array $fields): array
     {
         $values = [];
-        foreach ($fields as $field) {
-            if ($this->getValidator()->hasValue($field)) {
+        foreach ($fields as $field){
+            if ($this->getValidator()->hasValue($field)){
                 $values[$field] = $this->getValidator()->getValue($field);
             }
         }
@@ -88,13 +88,13 @@ 
     private function isProvidedByContext(string $role, array $values): bool
     {
         $entity = $this->getValidator()->getContext();
-        if (!is_object($entity) || !$this->orm->getHeap()->has($entity)) {
+        if (!is_object($entity) || !$this->orm->getHeap()->has($entity)){
             return false;
         }
 
         $extract = $this->orm->getMapper($role)->extract($entity);
-        foreach ($values as $field => $value) {
-            if (!isset($extract[$field]) || $extract[$field] !== $value) {
+        foreach ($values as $field => $value){
+            if (!isset($extract[$field]) || $extract[$field] !== $value){
                 return false;
             }
         }

Braces +14 added lines, -7 removed lines

@@ -38,7 +38,8 @@ 
     public function exists($value, string $role, ?string $field = null): bool
     {
         $repository = $this->orm->getRepository($role);
-        if ($field === null) {
+        if ($field === null)
+        {
             return $repository->findByPK($value) !== null;
         }
 
@@ -57,7 +58,8 @@ 
         $values = $this->withValues($withFields);
         $values[$field] = $value;
 
-        if ($this->isProvidedByContext($role, $values)) {
+        if ($this->isProvidedByContext($role, $values))
+        {
             return true;
         }
 
@@ -71,8 +73,10 @@ 
     private function withValues(array $fields): array
     {
         $values = [];
-        foreach ($fields as $field) {
-            if ($this->getValidator()->hasValue($field)) {
+        foreach ($fields as $field)
+        {
+            if ($this->getValidator()->hasValue($field))
+            {
                 $values[$field] = $this->getValidator()->getValue($field);
             }
         }
@@ -88,13 +92,16 @@ 
     private function isProvidedByContext(string $role, array $values): bool
     {
         $entity = $this->getValidator()->getContext();
-        if (!is_object($entity) || !$this->orm->getHeap()->has($entity)) {
+        if (!is_object($entity) || !$this->orm->getHeap()->has($entity))
+        {
             return false;
         }
 
         $extract = $this->orm->getMapper($role)->extract($entity);
-        foreach ($values as $field => $value) {
-            if (!isset($extract[$field]) || $extract[$field] !== $value) {
+        foreach ($values as $field => $value)
+        {
+            if (!isset($extract[$field]) || $extract[$field] !== $value)
+            {
                 return false;
             }
         }

tests/Framework/Encrypter/KeyCommandTest.php

Spacing +9 added lines, -9 removed lines

@@ -25,7 +25,7 @@ 
     public function testMountFileNotFound(): void
     {
         $out = $this->runCommand('encrypt:key', [
-            '-m' => __DIR__ . '/.env'
+            '-m' => __DIR__.'/.env'
         ]);
 
         $this->assertStringContainsString('Unable to find', $out);
@@ -33,18 +33,18 @@ 
 
     public function testReplace(): void
     {
-        file_put_contents(__DIR__ . '/.env', '{encrypt-key}');
+        file_put_contents(__DIR__.'/.env', '{encrypt-key}');
 
         $out = $this->runCommand('encrypt:key', [
-            '-m' => __DIR__ . '/.env'
+            '-m' => __DIR__.'/.env'
         ]);
 
         $this->assertStringContainsString('key has been updated', $out);
 
-        $body = file_get_contents(__DIR__ . '/.env');
+        $body = file_get_contents(__DIR__.'/.env');
         $this->assertStringContainsString($body, $out);
 
-        unlink(__DIR__ . '/.env');
+        unlink(__DIR__.'/.env');
     }
 
     public function testReplaceCurrent(): void
@@ -55,18 +55,18 @@ 
             'ENCRYPTER_KEY' => $key
         ]);
 
-        file_put_contents(__DIR__ . '/.env', $key);
+        file_put_contents(__DIR__.'/.env', $key);
 
         $out = $app->console()->run('encrypt:key', [
-            '-m' => __DIR__ . '/.env'
+            '-m' => __DIR__.'/.env'
         ]);
         $out = $out->getOutput()->fetch();
 
         $this->assertStringContainsString('key has been updated', $out);
 
-        $body = file_get_contents(__DIR__ . '/.env');
+        $body = file_get_contents(__DIR__.'/.env');
         $this->assertStringContainsString($body, $out);
 
-        unlink(__DIR__ . '/.env');
+        unlink(__DIR__.'/.env');
     }
 }

tests/Framework/GRPC/DispatcherTest.php

Spacing +13 added lines, -13 removed lines

@@ -23,14 +23,14 @@ 
     public function setUp(): void
     {
         exec('protoc 2>&1', $out);
-        if (strpos(join("\n", $out), '--php_out') === false) {
+        if (strpos(join("\n", $out), '--php_out') === false){
             $this->markTestSkipped('Protoc binary is missing');
         }
 
         parent::setUp();
 
         $fs = new Files();
-        $proto = $fs->normalizePath($this->app->dir('app') . 'proto/service.proto');
+        $proto = $fs->normalizePath($this->app->dir('app').'proto/service.proto');
 
         // protoc can't figure relative paths
         $proto = str_replace('Framework/../', '', $proto);
@@ -39,7 +39,7 @@ 
             'proto' => $proto
         ]);
 
-        file_put_contents($this->app->dir('app') . 'src/Service/EchoService.php', GenerateTest::SERVICE);
+        file_put_contents($this->app->dir('app').'src/Service/EchoService.php', GenerateTest::SERVICE);
     }
 
     public function tearDown(): void
@@ -48,12 +48,12 @@ 
 
         $fs = new Files();
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/Service')) {
-            $fs->deleteDirectory($this->app->dir('app') . 'src/Service');
+        if ($fs->isDirectory($this->app->dir('app').'src/Service')){
+            $fs->deleteDirectory($this->app->dir('app').'src/Service');
         }
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/GPBMetadata')) {
-            $fs->deleteDirectory($this->app->dir('app') . 'src/GPBMetadata');
+        if ($fs->isDirectory($this->app->dir('app').'src/GPBMetadata')){
+            $fs->deleteDirectory($this->app->dir('app').'src/GPBMetadata');
         }
     }
 
@@ -79,7 +79,7 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "service": "service.Echo",
                   "method": "Ping",
@@ -91,7 +91,7 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('send')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out){
                 $msg = new Message();
                 $msg->mergeFromString($out);
                 $this->assertSame('hello', $msg->getMsg());
@@ -103,7 +103,7 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = null;
                 return true;
             })
@@ -123,7 +123,7 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "service": "service.Echo",
                   "method": "Invalid",
@@ -135,7 +135,7 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('error')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out){
                 $this->assertStringContainsString('Method `Invalid` not found', $out);
                 return true;
             })
@@ -143,7 +143,7 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = null;
                 return true;
             })

Braces +18 added lines, -9 removed lines

@@ -23,7 +23,8 @@ 
     public function setUp(): void
     {
         exec('protoc 2>&1', $out);
-        if (strpos(join("\n", $out), '--php_out') === false) {
+        if (strpos(join("\n", $out), '--php_out') === false)
+        {
             $this->markTestSkipped('Protoc binary is missing');
         }
 
@@ -48,11 +49,13 @@ 
 
         $fs = new Files();
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/Service')) {
+        if ($fs->isDirectory($this->app->dir('app') . 'src/Service'))
+        {
             $fs->deleteDirectory($this->app->dir('app') . 'src/Service');
         }
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/GPBMetadata')) {
+        if ($fs->isDirectory($this->app->dir('app') . 'src/GPBMetadata'))
+        {
             $fs->deleteDirectory($this->app->dir('app') . 'src/GPBMetadata');
         }
     }
@@ -79,7 +82,8 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "service": "service.Echo",
                   "method": "Ping",
@@ -91,7 +95,8 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('send')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out)
+            {
                 $msg = new Message();
                 $msg->mergeFromString($out);
                 $this->assertSame('hello', $msg->getMsg());
@@ -103,7 +108,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })
@@ -123,7 +129,8 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "service": "service.Echo",
                   "method": "Invalid",
@@ -135,7 +142,8 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('error')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out)
+            {
                 $this->assertStringContainsString('Method `Invalid` not found', $out);
                 return true;
             })
@@ -143,7 +151,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })

tests/Framework/Cycle/MigrateTest.php

Spacing +5 added lines, -5 removed lines

@@ -53,13 +53,13 @@
         $output = $this->runCommandDebug('cycle:migrate', ['-r' => true]);
         $this->assertStringContainsString('default.users', $output);
 
-        $user = file_get_contents(__DIR__ . '/../../app/src/User/User.php');
-        unlink(__DIR__ . '/../../app/src/User/User.php');
-        try {
+        $user = file_get_contents(__DIR__.'/../../app/src/User/User.php');
+        unlink(__DIR__.'/../../app/src/User/User.php');
+        try{
             $output = $this->runCommandDebug('cycle:migrate', ['-r' => true]);
             $this->assertStringContainsString('drop foreign key', $output);
-        } finally {
-            file_put_contents(__DIR__ . '/../../app/src/User/User.php', $user);
+        }finally{
+            file_put_contents(__DIR__.'/../../app/src/User/User.php', $user);
         }
     }
 }

Braces +5 added lines, -2 removed lines

@@ -55,10 +55,13 @@
 
         $user = file_get_contents(__DIR__ . '/../../app/src/User/User.php');
         unlink(__DIR__ . '/../../app/src/User/User.php');
-        try {
+        try
+        {
             $output = $this->runCommandDebug('cycle:migrate', ['-r' => true]);
             $this->assertStringContainsString('drop foreign key', $output);
-        } finally {
+        }
+        finally
+        {
             file_put_contents(__DIR__ . '/../../app/src/User/User.php', $user);
         }
     }