spiral / framework

tests/Framework/Jobs/DispatcherTest.php

Spacing +4 added lines, -4 removed lines

@@ -38,7 +38,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"
@@ -66,7 +66,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"
@@ -82,7 +82,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);
@@ -90,7 +90,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

@@ -38,7 +38,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"
@@ -66,7 +67,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/app/src/Bootloader/AppBootloader.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
         PermissionsInterface $rbac,
         ViewsBootloader $views,
         ValidationBootloader $validation
-    ) {
+    ){
         $rbac->addRole('user');
         $rbac->associate('user', '*');
 
@@ -40,7 +40,7 @@ 
 
         $router->setDefault($route->withDefaults(['name' => 'Dave']));
 
-        $views->addDirectory('custom', __DIR__ . '/../../views/custom/');
+        $views->addDirectory('custom', __DIR__.'/../../views/custom/');
         $views->addEngine(TestEngine::class);
 
         $validation->addAlias('aliased', 'notEmpty');

tests/app/src/Job/ErrorJob.php

Braces +3 added lines, -1 removed lines

@@ -12,8 +12,10 @@
 
 class ErrorJob extends AbstractJob
 {
-    public function do()
+    public function do{
+        ()
     {
         throw new \ErrorException("bad job");
     }
+    }
 }
\ No newline at end of file

tests/app/src/Job/TestJob.php

Braces +3 added lines, -1 removed lines

@@ -13,8 +13,10 @@
 
 class TestJob extends AbstractJob
 {
-    public function do(EnvironmentInterface $env)
+    public function do{
+        (EnvironmentInterface $env)
     {
         $env->set('FIRED', true);
     }
+    }
 }
\ No newline at end of file

src/Http/Middleware/SessionMiddleware.php

Braces +15 added lines, -7 removed lines

@@ -69,14 +69,18 @@ 
             $this->fetchID($request)
         );
 
-        try {
+        try
+        {
             $response = $this->scope->runScope(
                 [SessionInterface::class => $session],
-                function () use ($session, $request, $handler) {
+                function () use ($session, $request, $handler)
+                {
                     return $handler->handle($request->withAttribute(static::ATTRIBUTE, $session));
                 }
             );
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $session->abort();
             throw $e;
         }
@@ -95,14 +99,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());
         }
 
@@ -119,7 +125,8 @@ 
     protected function fetchID(Request $request): ?string
     {
         $cookies = $request->getCookieParams();
-        if (empty($cookies[$this->config->getCookie()])) {
+        if (empty($cookies[$this->config->getCookie()]))
+        {
             return null;
         }
 
@@ -152,7 +159,8 @@ 
     protected function clientSignature(Request $request): string
     {
         $signature = '';
-        foreach (static::SIGNATURE_HEADERS as $header) {
+        foreach (static::SIGNATURE_HEADERS as $header)
+        {
             $signature .= $request->getHeaderLine($header) . ';';
         }
 

Spacing +8 added lines, -8 removed lines

@@ -57,7 +57,7 @@ 
         CookiesConfig $cookiesConfig,
         SessionFactory $factory,
         ScopeInterface $scope
-    ) {
+    ){
         $this->config = $config;
         $this->httpConfig = $httpConfig;
         $this->cookiesConfig = $cookiesConfig;
@@ -76,14 +76,14 @@ 
             $this->fetchID($request)
         );
 
-        try {
+        try{
             $response = $this->scope->runScope(
                 [SessionInterface::class => $session],
                 function () use ($session, $request, $handler) {
                     return $handler->handle($request->withAttribute(static::ATTRIBUTE, $session));
                 }
             );
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $session->abort();
             throw $e;
         }
@@ -102,14 +102,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());
         }
 
@@ -126,7 +126,7 @@ 
     protected function fetchID(Request $request): ?string
     {
         $cookies = $request->getCookieParams();
-        if (empty($cookies[$this->config->getCookie()])) {
+        if (empty($cookies[$this->config->getCookie()])){
             return null;
         }
 
@@ -159,8 +159,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);

src/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/GRPC/GRPCDispatcher.php

Spacing +6 added lines, -6 removed lines

@@ -37,7 +37,7 @@ 
         EnvironmentInterface $env,
         FinalizerInterface $finalizer,
         ContainerInterface $container
-    ) {
+    ){
         $this->env = $env;
         $this->finalizer = $finalizer;
         $this->container = $container;
@@ -65,12 +65,12 @@ 
         $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) {
-            if ($e !== null) {
+        $server->serve($worker, function (\Throwable $e = null){
+            if ($e !== null){
                 $this->handleException($e);
             }
 
@@ -83,9 +83,9 @@ 
      */
     protected function handleException(\Throwable $e)
     {
-        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 +11 added lines, -5 removed lines

@@ -65,12 +65,15 @@ 
         $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) {
-            if ($e !== null) {
+        $server->serve($worker, function (\Throwable $e = null)
+        {
+            if ($e !== null)
+            {
                 $this->handleException($e);
             }
 
@@ -83,9 +86,12 @@ 
      */
     protected function handleException(\Throwable $e)
     {
-        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
         }
     }

src/Console/Sequence/RuntimeDirectory.php

Spacing +6 added lines, -6 removed lines

@@ -43,19 +43,19 @@ 
 
         $runtimeDirectory = $this->dirs->get('runtime');
 
-        if (!$this->files->exists($runtimeDirectory)) {
+        if (!$this->files->exists($runtimeDirectory)){
             $this->files->ensureDirectory($runtimeDirectory);
             $output->writeln("<comment>created</comment>");
             return;
-        } else {
+        }else{
             $output->writeln("<info>exists</info>");
         }
 
-        foreach ($this->files->getFiles($runtimeDirectory) as $filename) {
-            try {
+        foreach ($this->files->getFiles($runtimeDirectory) as $filename){
+            try{
                 $this->files->setPermissions($filename, FilesInterface::RUNTIME);
                 $this->files->setPermissions(dirname($filename), FilesInterface::RUNTIME);
-            } catch (\Throwable $e) {
+            }catch (\Throwable $e){
                 // @codeCoverageIgnoreStart
                 $output->writeln(sprintf(
                     "<fg=red>[errored]</fg=red> `%s`: <fg=red>%s</fg=red>",
@@ -66,7 +66,7 @@ 
                 // @codeCoverageIgnoreEnd
             }
 
-            if ($output->isVerbose()) {
+            if ($output->isVerbose()){
                 $output->writeln(sprintf(
                     "<fg=green>[updated]</fg=green> `%s`",
                     $this->files->relativePath($filename, $runtimeDirectory)

Braces +14 added lines, -6 removed lines

@@ -43,19 +43,26 @@ 
 
         $runtimeDirectory = $this->dirs->get('runtime');
 
-        if (!$this->files->exists($runtimeDirectory)) {
+        if (!$this->files->exists($runtimeDirectory))
+        {
             $this->files->ensureDirectory($runtimeDirectory);
             $output->writeln("<comment>created</comment>");
             return;
-        } else {
+        }
+        else
+        {
             $output->writeln("<info>exists</info>");
         }
 
-        foreach ($this->files->getFiles($runtimeDirectory) as $filename) {
-            try {
+        foreach ($this->files->getFiles($runtimeDirectory) as $filename)
+        {
+            try
+            {
                 $this->files->setPermissions($filename, FilesInterface::RUNTIME);
                 $this->files->setPermissions(dirname($filename), FilesInterface::RUNTIME);
-            } catch (\Throwable $e) {
+            }
+            catch (\Throwable $e)
+            {
                 // @codeCoverageIgnoreStart
                 $output->writeln(sprintf(
                     "<fg=red>[errored]</fg=red> `%s`: <fg=red>%s</fg=red>",
@@ -66,7 +73,8 @@ 
                 // @codeCoverageIgnoreEnd
             }
 
-            if ($output->isVerbose()) {
+            if ($output->isVerbose())
+            {
                 $output->writeln(sprintf(
                     "<fg=green>[updated]</fg=green> `%s`",
                     $this->files->relativePath($filename, $runtimeDirectory)

src/Http/Middleware/ErrorHandlerMiddleware.php

Spacing +8 added lines, -8 removed lines

@@ -44,7 +44,7 @@ 
         bool $suppressErrors,
         RendererInterface $renderer,
         SnapshotterInterface $snapshots = null
-    ) {
+    ){
         $this->suppressErrors = $suppressErrors;
         $this->renderer = $renderer;
         $this->snapshotter = $snapshots;
@@ -57,20 +57,20 @@ 
      */
     public function process(Request $request, Handler $handler): Response
     {
-        try {
+        try{
             return $handler->handle($request);
-        } catch (ClientException|RouterException $e) {
-            if ($e instanceof ClientException) {
+        }catch (ClientException | RouterException $e){
+            if ($e instanceof ClientException){
                 $code = $e->getCode();
-            } else {
+            }else{
                 $code = 404;
             }
-        } catch (\Throwable $e) {
-            if (!$this->suppressErrors) {
+        }catch (\Throwable $e){
+            if (!$this->suppressErrors){
                 throw $e;
             }
 
-            if ($this->snapshotter !== null) {
+            if ($this->snapshotter !== null){
                 $this->snapshotter->register($e);
             }
 

Braces +17 added lines, -7 removed lines

@@ -57,20 +57,30 @@
      */
     public function process(Request $request, Handler $handler): Response
     {
-        try {
+        try
+        {
             return $handler->handle($request);
-        } catch (ClientException|RouterException $e) {
-            if ($e instanceof ClientException) {
+        }
+        catch (ClientException|RouterException $e)
+        {
+            if ($e instanceof ClientException)
+            {
                 $code = $e->getCode();
-            } else {
+            }
+            else
+            {
                 $code = 404;
             }
-        } catch (\Throwable $e) {
-            if (!$this->suppressErrors) {
+        }
+        catch (\Throwable $e)
+        {
+            if (!$this->suppressErrors)
+            {
                 throw $e;
             }
 
-            if ($this->snapshotter !== null) {
+            if ($this->snapshotter !== null)
+            {
                 $this->snapshotter->register($e);
             }