spiral / framework

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);
         }
 

src/Config/src/Patch/Delete.php

Spacing +6 added lines, -6 removed lines

@@ -45,20 +45,20 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try{
             $target = &$this->dotGet($config, $this->position);
 
-            if ($this->key !== null) {
+            if ($this->key !== null){
                 unset($target[$this->key]);
-            } else {
-                foreach ($target as $key => $value) {
-                    if ($value === $this->value) {
+            }else{
+                foreach ($target as $key => $value){
+                    if ($value === $this->value){
                         unset($target[$key]);
                         break;
                     }
                 }
             }
-        } catch (DotNotFoundException $e) {
+        }catch (DotNotFoundException $e){
             // doing nothing when section not found
         }
 

Braces +14 added lines, -6 removed lines

@@ -45,20 +45,28 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try
+        {
             $target = &$this->dotGet($config, $this->position);
 
-            if ($this->key !== null) {
+            if ($this->key !== null)
+            {
                 unset($target[$this->key]);
-            } else {
-                foreach ($target as $key => $value) {
-                    if ($value === $this->value) {
+            }
+            else
+            {
+                foreach ($target as $key => $value)
+                {
+                    if ($value === $this->value)
+                    {
                         unset($target[$key]);
                         break;
                     }
                 }
             }
-        } catch (DotNotFoundException $e) {
+        }
+        catch (DotNotFoundException $e)
+        {
             // doing nothing when section not found
         }
 

src/Config/src/Patch/Group.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
      */
     public function patch(array $config): array
     {
-        foreach ($this->patches as $patch) {
+        foreach ($this->patches as $patch){
             $config = $patch->patch($config);
         }
 

Braces +2 added lines, -1 removed lines

@@ -32,7 +32,8 @@
      */
     public function patch(array $config): array
     {
-        foreach ($this->patches as $patch) {
+        foreach ($this->patches as $patch)
+        {
             $config = $patch->patch($config);
         }
 

src/Config/src/Patch/Append.php

Spacing +4 added lines, -4 removed lines

@@ -46,15 +46,15 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try{
             $target = &$this->dotGet($config, $this->position);
 
-            if ($this->key !== null) {
+            if ($this->key !== null){
                 $target[$this->key] = $this->value;
-            } else {
+            }else{
                 $target[] = $this->value;
             }
-        } catch (DotNotFoundException $e) {
+        }catch (DotNotFoundException $e){
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }
 

Braces +10 added lines, -4 removed lines

@@ -46,15 +46,21 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try
+        {
             $target = &$this->dotGet($config, $this->position);
 
-            if ($this->key !== null) {
+            if ($this->key !== null)
+            {
                 $target[$this->key] = $this->value;
-            } else {
+            }
+            else
+            {
                 $target[] = $this->value;
             }
-        } catch (DotNotFoundException $e) {
+        }
+        catch (DotNotFoundException $e)
+        {
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }
 

src/Config/src/Patch/Traits/DotTrait.php

Spacing +4 added lines, -4 removed lines

@@ -23,14 +23,14 @@
     private function &dotGet(array &$data, string $name)
     {
         //Generating path relative to a given name and prefix
-        $path = (!empty($this->prefix) ? $this->prefix . '.' : '') . $name;
-        if (empty($path)) {
+        $path = (!empty($this->prefix) ? $this->prefix.'.' : '').$name;
+        if (empty($path)){
             return $data;
         }
 
         $path = explode('.', rtrim($path, '.'));
-        foreach ($path as $step) {
-            if (!is_array($data) || !array_key_exists($step, $data)) {
+        foreach ($path as $step){
+            if (!is_array($data) || !array_key_exists($step, $data)){
                 throw new DotNotFoundException("Unable to find config element '{$name}'.");
             }
             $data = &$data[$step];

Braces +6 added lines, -3 removed lines

@@ -24,13 +24,16 @@
     {
         //Generating path relative to a given name and prefix
         $path = (!empty($this->prefix) ? $this->prefix . '.' : '') . $name;
-        if (empty($path)) {
+        if (empty($path))
+        {
             return $data;
         }
 
         $path = explode('.', rtrim($path, '.'));
-        foreach ($path as $step) {
-            if (!is_array($data) || !array_key_exists($step, $data)) {
+        foreach ($path as $step)
+        {
+            if (!is_array($data) || !array_key_exists($step, $data))
+            {
                 throw new DotNotFoundException("Unable to find config element '{$name}'.");
             }
             $data = &$data[$step];