spiral / framework

src/Framework/GRPC/ServiceLocator.php

Spacing +4 added lines, -4 removed lines

@@ -38,15 +38,15 @@
     public function getServices(): array
     {
         $result = [];
-        foreach ($this->classes->getClasses(ServiceInterface::class) as $service) {
-            if (!$service->isInstantiable()) {
+        foreach ($this->classes->getClasses(ServiceInterface::class) as $service){
+            if (!$service->isInstantiable()){
                 continue;
             }
 
             $instance = $this->container->get($service->getName());
 
-            foreach ($service->getInterfaces() as $interface) {
-                if ($interface->isSubclassOf(ServiceInterface::class)) {
+            foreach ($service->getInterfaces() as $interface){
+                if ($interface->isSubclassOf(ServiceInterface::class)){
                     $result[$interface->getName()] = $instance;
                 }
             }

Braces +8 added lines, -4 removed lines

@@ -38,15 +38,19 @@
     public function getServices(): array
     {
         $result = [];
-        foreach ($this->classes->getClasses(ServiceInterface::class) as $service) {
-            if (!$service->isInstantiable()) {
+        foreach ($this->classes->getClasses(ServiceInterface::class) as $service)
+        {
+            if (!$service->isInstantiable())
+            {
                 continue;
             }
 
             $instance = $this->container->get($service->getName());
 
-            foreach ($service->getInterfaces() as $interface) {
-                if ($interface->isSubclassOf(ServiceInterface::class)) {
+            foreach ($service->getInterfaces() as $interface)
+            {
+                if ($interface->isSubclassOf(ServiceInterface::class))
+                {
                     $result[$interface->getName()] = $instance;
                 }
             }

src/Framework/GRPC/GRPCDispatcher.php

Spacing +5 added lines, -5 removed lines

@@ -39,7 +39,7 @@ 
         EnvironmentInterface $env,
         FinalizerInterface $finalizer,
         ContainerInterface $container
-    ) {
+    ){
         $this->env = $env;
         $this->finalizer = $finalizer;
         $this->container = $container;
@@ -69,14 +69,14 @@ 
         $worker = $this->container->get(Worker::class);
         $locator = $this->container->get(LocatorInterface::class);
 
-        foreach ($locator->getServices() as $interface => $service) {
+        foreach ($locator->getServices() as $interface => $service){
             $server->registerService($interface, $service);
         }
 
         $server->serve(
             $worker,
             function (\Throwable $e = null): void {
-                if ($e !== null) {
+                if ($e !== null){
                     $this->handleException($e);
                 }
 
@@ -90,9 +90,9 @@ 
      */
     protected function handleException(\Throwable $e): void
     {
-        try {
+        try{
             $this->container->get(SnapshotterInterface::class)->register($e);
-        } catch (\Throwable | ContainerExceptionInterface $se) {
+        }catch (\Throwable | ContainerExceptionInterface $se){
             // no need to notify when unable to register an exception
         }
     }

Braces +9 added lines, -4 removed lines

@@ -69,14 +69,16 @@ 
         $worker = $this->container->get(Worker::class);
         $locator = $this->container->get(LocatorInterface::class);
 
-        foreach ($locator->getServices() as $interface => $service) {
+        foreach ($locator->getServices() as $interface => $service)
+        {
             $server->registerService($interface, $service);
         }
 
         $server->serve(
             $worker,
             function (\Throwable $e = null): void {
-                if ($e !== null) {
+                if ($e !== null)
+                {
                     $this->handleException($e);
                 }
 
@@ -90,9 +92,12 @@ 
      */
     protected function handleException(\Throwable $e): void
     {
-        try {
+        try
+        {
             $this->container->get(SnapshotterInterface::class)->register($e);
-        } catch (\Throwable | ContainerExceptionInterface $se) {
+        }
+        catch (\Throwable | ContainerExceptionInterface $se)
+        {
             // no need to notify when unable to register an exception
         }
     }

tests/app/src/ViewEngine/TestEngine.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
 
     public function compile(string $path, ContextInterface $context): void
     {
-        if ($path === 'custom:error') {
+        if ($path === 'custom:error'){
             throw new EngineException('Unable to compile custom:error');
         }
     }

Braces +2 added lines, -1 removed lines

@@ -32,7 +32,8 @@
 
     public function compile(string $path, ContextInterface $context): void
     {
-        if ($path === 'custom:error') {
+        if ($path === 'custom:error')
+        {
             throw new EngineException('Unable to compile custom:error');
         }
     }

tests/Framework/GRPC/GenerateTest.php

Spacing +9 added lines, -9 removed lines

@@ -36,14 +36,14 @@ 
     {
         exec('protoc 2>&1', $out);
 
-        if (strpos(implode("\n", $out), '--php_out') === false) {
+        if (strpos(implode("\n", $out), '--php_out') === false){
             $this->markTestSkipped('Protoc binary is missing');
         }
 
         parent::setUp();
 
         $fs = new Files();
-        $this->proto = \realpath($fs->normalizePath($this->app->dir('app') . 'proto/service.proto'));
+        $this->proto = \realpath($fs->normalizePath($this->app->dir('app').'proto/service.proto'));
     }
 
     public function tearDown(): void
@@ -52,12 +52,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');
         }
     }
 
@@ -85,8 +85,8 @@ 
             'proto' => $this->proto
         ]);
 
-        $this->assertFileExists($this->app->dir('app') . 'src/Service/EchoInterface.php');
-        $this->assertFileExists($this->app->dir('app') . 'src/Service/Sub/Message.php');
-        $this->assertFileExists($this->app->dir('app') . 'src/GPBMetadata/Service.php');
+        $this->assertFileExists($this->app->dir('app').'src/Service/EchoInterface.php');
+        $this->assertFileExists($this->app->dir('app').'src/Service/Sub/Message.php');
+        $this->assertFileExists($this->app->dir('app').'src/GPBMetadata/Service.php');
     }
 }

Braces +6 added lines, -3 removed lines

@@ -36,7 +36,8 @@ 
     {
         exec('protoc 2>&1', $out);
 
-        if (strpos(implode("\n", $out), '--php_out') === false) {
+        if (strpos(implode("\n", $out), '--php_out') === false)
+        {
             $this->markTestSkipped('Protoc binary is missing');
         }
 
@@ -52,11 +53,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');
         }
     }

tests/Framework/GRPC/DispatcherTest.php

Spacing +13 added lines, -13 removed lines

@@ -23,7 +23,7 @@ 
     public function setUp(): void
     {
         exec('protoc 2>&1', $out);
-        if (strpos(implode("\n", $out), '--php_out') === false) {
+        if (strpos(implode("\n", $out), '--php_out') === false){
             $this->markTestSkipped('Protoc binary is missing');
         }
 
@@ -32,10 +32,10 @@ 
         $fs = new Files();
 
         $this->runCommandDebug('grpc:generate', [
-            'proto' => realpath($fs->normalizePath($this->app->dir('app') . 'proto/service.proto'))
+            'proto' => realpath($fs->normalizePath($this->app->dir('app').'proto/service.proto'))
         ]);
 
-        $output = $this->app->dir('app') . 'src/Service/EchoService.php';
+        $output = $this->app->dir('app').'src/Service/EchoService.php';
         file_put_contents($output, GenerateTest::SERVICE);
     }
 
@@ -45,12 +45,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');
         }
     }
 
@@ -76,7 +76,7 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "service": "service.Echo",
                   "method": "Ping",
@@ -88,7 +88,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());
@@ -100,7 +100,7 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = null;
                 return true;
             })
@@ -120,7 +120,7 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "service": "service.Echo",
                   "method": "Invalid",
@@ -132,7 +132,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;
             })
@@ -140,7 +140,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(implode("\n", $out), '--php_out') === false) {
+        if (strpos(implode("\n", $out), '--php_out') === false)
+        {
             $this->markTestSkipped('Protoc binary is missing');
         }
 
@@ -45,11 +46,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');
         }
     }
@@ -76,7 +79,8 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "service": "service.Echo",
                   "method": "Ping",
@@ -88,7 +92,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());
@@ -100,7 +105,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })
@@ -120,7 +126,8 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "service": "service.Echo",
                   "method": "Invalid",
@@ -132,7 +139,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;
             })
@@ -140,7 +148,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })

src/Config/src/Loader/PhpLoader.php

Spacing +2 added lines, -2 removed lines

@@ -36,11 +36,11 @@
      */
     public function loadFile(string $section, string $filename): array
     {
-        try {
+        try{
             return ContainerScope::runScope($this->container, function () use ($filename) {
                 return (require $filename);
             });
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             throw new LoaderException($e->getMessage(), $e->getCode(), $e);
         }
     }

Braces +7 added lines, -3 removed lines

@@ -36,11 +36,15 @@
      */
     public function loadFile(string $section, string $filename): array
     {
-        try {
-            return ContainerScope::runScope($this->container, function () use ($filename) {
+        try
+        {
+            return ContainerScope::runScope($this->container, function () use ($filename)
+            {
                 return (require $filename);
             });
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw new LoaderException($e->getMessage(), $e->getCode(), $e);
         }
     }

src/Config/src/Loader/JsonLoader.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
         $content = file_get_contents($filename);
         $data = json_decode($content, true);
 
-        if (is_null($data)) {
+        if (is_null($data)){
             throw new LoaderException(json_last_error_msg(), json_last_error());
         }
 

Braces +2 added lines, -1 removed lines

@@ -23,7 +23,8 @@
         $content = file_get_contents($filename);
         $data = json_decode($content, true);
 
-        if (is_null($data)) {
+        if (is_null($data))
+        {
             throw new LoaderException(json_last_error_msg(), json_last_error());
         }
 

src/Config/src/ConfigManager.php

Spacing +11 added lines, -11 removed lines

@@ -69,11 +69,11 @@ 
      */
     public function setDefaults(string $section, array $data): void
     {
-        if (isset($this->defaults[$section])) {
+        if (isset($this->defaults[$section])){
             throw new ConfiguratorException("Unable to set default config `{$section}` more than once.");
         }
 
-        if (isset($this->data[$section])) {
+        if (isset($this->data[$section])){
             throw new ConfigDeliveredException("Unable to set default config `{$section}`, config has been loaded.");
         }
 
@@ -85,8 +85,8 @@ 
      */
     public function modify(string $section, PatchInterface $patch): array
     {
-        if (isset($this->instances[$section])) {
-            if ($this->strict) {
+        if (isset($this->instances[$section])){
+            if ($this->strict){
                 throw new ConfigDeliveredException(
                     "Unable to patch config `{$section}`, config object has already been delivered."
                 );
@@ -97,9 +97,9 @@ 
 
         $data = $this->getConfig($section);
 
-        try {
+        try{
             return $this->data[$section] = $patch->patch($data);
-        } catch (PatchException $e) {
+        }catch (PatchException $e){
             throw new PatchException("Unable to modify config `{$section}`.", $e->getCode(), $e);
         }
     }
@@ -109,18 +109,18 @@ 
      */
     public function getConfig(string $section = null): array
     {
-        if (isset($this->data[$section])) {
+        if (isset($this->data[$section])){
             return $this->data[$section];
         }
 
-        if (isset($this->defaults[$section])) {
+        if (isset($this->defaults[$section])){
             $data = [];
-            if ($this->loader->has($section)) {
+            if ($this->loader->has($section)){
                 $data = $this->loader->load($section);
             }
 
             $data = array_merge($this->defaults[$section], $data);
-        } else {
+        }else{
             $data = $this->loader->load($section);
         }
 
@@ -133,7 +133,7 @@ 
     public function createInjection(\ReflectionClass $class, string $context = null)
     {
         $config = $class->getConstant('CONFIG');
-        if (isset($this->instances[$config])) {
+        if (isset($this->instances[$config])){
             return $this->instances[$config];
         }
 

Braces +24 added lines, -11 removed lines

@@ -69,11 +69,13 @@ 
      */
     public function setDefaults(string $section, array $data): void
     {
-        if (isset($this->defaults[$section])) {
+        if (isset($this->defaults[$section]))
+        {
             throw new ConfiguratorException("Unable to set default config `{$section}` more than once.");
         }
 
-        if (isset($this->data[$section])) {
+        if (isset($this->data[$section]))
+        {
             throw new ConfigDeliveredException("Unable to set default config `{$section}`, config has been loaded.");
         }
 
@@ -85,8 +87,10 @@ 
      */
     public function modify(string $section, PatchInterface $patch): array
     {
-        if (isset($this->instances[$section])) {
-            if ($this->strict) {
+        if (isset($this->instances[$section]))
+        {
+            if ($this->strict)
+            {
                 throw new ConfigDeliveredException(
                     "Unable to patch config `{$section}`, config object has already been delivered."
                 );
@@ -97,9 +101,12 @@ 
 
         $data = $this->getConfig($section);
 
-        try {
+        try
+        {
             return $this->data[$section] = $patch->patch($data);
-        } catch (PatchException $e) {
+        }
+        catch (PatchException $e)
+        {
             throw new PatchException("Unable to modify config `{$section}`.", $e->getCode(), $e);
         }
     }
@@ -109,18 +116,23 @@ 
      */
     public function getConfig(string $section = null): array
     {
-        if (isset($this->data[$section])) {
+        if (isset($this->data[$section]))
+        {
             return $this->data[$section];
         }
 
-        if (isset($this->defaults[$section])) {
+        if (isset($this->defaults[$section]))
+        {
             $data = [];
-            if ($this->loader->has($section)) {
+            if ($this->loader->has($section))
+            {
                 $data = $this->loader->load($section);
             }
 
             $data = array_merge($this->defaults[$section], $data);
-        } else {
+        }
+        else
+        {
             $data = $this->loader->load($section);
         }
 
@@ -133,7 +145,8 @@ 
     public function createInjection(\ReflectionClass $class, string $context = null)
     {
         $config = $class->getConstant('CONFIG');
-        if (isset($this->instances[$config])) {
+        if (isset($this->instances[$config]))
+        {
             return $this->instances[$config];
         }
 

src/Config/src/Patch/Set.php

Spacing +2 added lines, -2 removed lines

@@ -45,10 +45,10 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try{
             $target = &$this->dotGet($config, $this->key);
             $target = $this->value;
-        } catch (DotNotFoundException $e) {
+        }catch (DotNotFoundException $e){
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }
 

Braces +5 added lines, -2 removed lines

@@ -45,10 +45,13 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try
+        {
             $target = &$this->dotGet($config, $this->key);
             $target = $this->value;
-        } catch (DotNotFoundException $e) {
+        }
+        catch (DotNotFoundException $e)
+        {
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }