spiral / framework

src/Framework/Domain/GuardInterceptor.php

Spacing +9 added lines, -9 removed lines

@@ -52,7 +52,7 @@ 
     {
         $permission = $this->getPermissions($controller, $action);
 
-        if ($permission !== null && !$this->guard->allows($permission[0], $parameters)) {
+        if ($permission !== null && !$this->guard->allows($permission[0], $parameters)){
             throw new ControllerException($permission[2], $permission[1]);
         }
 
@@ -71,14 +71,14 @@ 
     private function getPermissions(string $controller, string $action): ?array
     {
         $key = sprintf('%s:%s', $controller, $action);
-        if (array_key_exists($key, $this->cache)) {
+        if (array_key_exists($key, $this->cache)){
             return $this->cache[$key];
         }
 
         $this->cache[$key] = null;
-        try {
+        try{
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }catch (\ReflectionException $e){
             return null;
         }
 
@@ -96,7 +96,7 @@ 
             Guarded::class
         );
 
-        if ($action === null) {
+        if ($action === null){
             return null;
         }
 
@@ -119,7 +119,7 @@ 
             $guarded->errorMessage ?? sprintf('Unauthorized access `%s`', $guarded->permission ?? $method->getName())
         ];
 
-        if ($guarded->permission === null && $ns === null) {
+        if ($guarded->permission === null && $ns === null){
             throw new InterceptorException(
                 sprintf(
                     'Unable to apply @Guarded without name or @GuardNamespace on `%s`->`%s`',
@@ -129,16 +129,16 @@ 
             );
         }
 
-        if ($ns !== null) {
+        if ($ns !== null){
             $permission[0] = sprintf('%s.%s', $ns->namespace, $permission[0]);
         }
 
-        if ($this->namespace !== null) {
+        if ($this->namespace !== null){
             // global namespace
             $permission[0] = sprintf('%s.%s', $this->namespace, $permission[0]);
         }
 
-        switch ($guarded->else) {
+        switch ($guarded->else){
             case 'unauthorized':
                 $permission[1] = ControllerException::UNAUTHORIZED;
                 break;

Braces +19 added lines, -9 removed lines

@@ -52,7 +52,8 @@ 
     {
         $permission = $this->getPermissions($controller, $action);
 
-        if ($permission !== null && !$this->guard->allows($permission[0], $parameters)) {
+        if ($permission !== null && !$this->guard->allows($permission[0], $parameters))
+        {
             throw new ControllerException($permission[2], $permission[1]);
         }
 
@@ -71,14 +72,18 @@ 
     private function getPermissions(string $controller, string $action): ?array
     {
         $key = sprintf('%s:%s', $controller, $action);
-        if (array_key_exists($key, $this->cache)) {
+        if (array_key_exists($key, $this->cache))
+        {
             return $this->cache[$key];
         }
 
         $this->cache[$key] = null;
-        try {
+        try
+        {
             $method = new \ReflectionMethod($controller, $action);
-        } catch (\ReflectionException $e) {
+        }
+        catch (\ReflectionException $e)
+        {
             return null;
         }
 
@@ -96,7 +101,8 @@ 
             Guarded::class
         );
 
-        if ($action === null) {
+        if ($action === null)
+        {
             return null;
         }
 
@@ -119,7 +125,8 @@ 
             $guarded->errorMessage ?? sprintf('Unauthorized access `%s`', $guarded->permission ?? $method->getName())
         ];
 
-        if ($guarded->permission === null && $ns === null) {
+        if ($guarded->permission === null && $ns === null)
+        {
             throw new InterceptorException(
                 sprintf(
                     'Unable to apply @Guarded without name or @GuardNamespace on `%s`->`%s`',
@@ -129,16 +136,19 @@ 
             );
         }
 
-        if ($ns !== null) {
+        if ($ns !== null)
+        {
             $permission[0] = sprintf('%s.%s', $ns->namespace, $permission[0]);
         }
 
-        if ($this->namespace !== null) {
+        if ($this->namespace !== null)
+        {
             // global namespace
             $permission[0] = sprintf('%s.%s', $this->namespace, $permission[0]);
         }
 
-        switch ($guarded->else) {
+        switch ($guarded->else)
+        {
             case 'unauthorized':
                 $permission[1] = ControllerException::UNAUTHORIZED;
                 break;

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/DirectoryLoader.php

Spacing +7 added lines, -7 removed lines

@@ -46,9 +46,9 @@ 
      */
     public function has(string $section): bool
     {
-        foreach (static::LOADERS as $extension => $class) {
+        foreach (static::LOADERS as $extension => $class){
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (file_exists($filename)) {
+            if (file_exists($filename)){
                 return true;
             }
         }
@@ -61,15 +61,15 @@ 
      */
     public function load(string $section): array
     {
-        foreach (static::LOADERS as $extension => $class) {
+        foreach (static::LOADERS as $extension => $class){
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (!file_exists($filename)) {
+            if (!file_exists($filename)){
                 continue;
             }
 
-            try {
+            try{
                 return $this->getLoader($extension)->loadFile($section, $filename);
-            } catch (LoaderException $e) {
+            }catch (LoaderException $e){
                 throw new LoaderException("Unable to load config `{$section}`.", $e->getCode(), $e);
             }
         }
@@ -83,7 +83,7 @@ 
      */
     private function getLoader(string $extension): FileLoaderInterface
     {
-        if (isset($this->loaders[$extension])) {
+        if (isset($this->loaders[$extension])){
             return $this->loaders[$extension];
         }
 

Braces +15 added lines, -7 removed lines

@@ -46,9 +46,11 @@ 
      */
     public function has(string $section): bool
     {
-        foreach (static::LOADERS as $extension => $class) {
+        foreach (static::LOADERS as $extension => $class)
+        {
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (file_exists($filename)) {
+            if (file_exists($filename))
+            {
                 return true;
             }
         }
@@ -61,15 +63,20 @@ 
      */
     public function load(string $section): array
     {
-        foreach (static::LOADERS as $extension => $class) {
+        foreach (static::LOADERS as $extension => $class)
+        {
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (!file_exists($filename)) {
+            if (!file_exists($filename))
+            {
                 continue;
             }
 
-            try {
+            try
+            {
                 return $this->getLoader($extension)->loadFile($section, $filename);
-            } catch (LoaderException $e) {
+            }
+            catch (LoaderException $e)
+            {
                 throw new LoaderException("Unable to load config `{$section}`.", $e->getCode(), $e);
             }
         }
@@ -83,7 +90,8 @@ 
      */
     private function getLoader(string $extension): FileLoaderInterface
     {
-        if (isset($this->loaders[$extension])) {
+        if (isset($this->loaders[$extension]))
+        {
             return $this->loaders[$extension];
         }
 

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