spiral / framework

src/Snapshots/FileSnapshotter.php

Spacing +7 added lines, -7 removed lines

@@ -53,7 +53,7 @@ 
         HandlerInterface $handler,
         FilesInterface $files,
         LoggerInterface $logger = null
-    ) {
+    ){
         $this->directory = $directory;
         $this->maxFiles = $maxFiles;
         $this->verbosity = $verbosity;
@@ -69,7 +69,7 @@ 
     {
         $snapshot = new Snapshot($this->getID($e), $e);
 
-        if ($this->logger !== null) {
+        if ($this->logger !== null){
             $this->logger->error($snapshot->getMessage());
         }
 
@@ -101,17 +101,17 @@ 
     protected function rotateSnapshots(): void
     {
         $finder = new Finder();
-        $finder->in($this->directory)->sort(function (SplFileInfo $a, SplFileInfo $b) {
+        $finder->in($this->directory)->sort(function (SplFileInfo $a, SplFileInfo $b){
             return $b->getMTime() - $a->getMTime();
         });
 
         $count = 0;
-        foreach ($finder as $file) {
+        foreach ($finder as $file){
             $count++;
-            if ($count > $this->maxFiles) {
-                try {
+            if ($count > $this->maxFiles){
+                try{
                     $this->files->delete($file->getRealPath());
-                } catch (FilesException $e) {
+                }catch (FilesException $e){
                     // ignore
                 }
             }

Braces +13 added lines, -6 removed lines

@@ -69,7 +69,8 @@ 
     {
         $snapshot = new Snapshot($this->getID($e), $e);
 
-        if ($this->logger !== null) {
+        if ($this->logger !== null)
+        {
             $this->logger->error($snapshot->getMessage());
         }
 
@@ -101,17 +102,23 @@ 
     protected function rotateSnapshots(): void
     {
         $finder = new Finder();
-        $finder->in($this->directory)->sort(function (SplFileInfo $a, SplFileInfo $b) {
+        $finder->in($this->directory)->sort(function (SplFileInfo $a, SplFileInfo $b)
+        {
             return $b->getMTime() - $a->getMTime();
         });
 
         $count = 0;
-        foreach ($finder as $file) {
+        foreach ($finder as $file)
+        {
             $count++;
-            if ($count > $this->maxFiles) {
-                try {
+            if ($count > $this->maxFiles)
+            {
+                try
+                {
                     $this->files->delete($file->getRealPath());
-                } catch (FilesException $e) {
+                }
+                catch (FilesException $e)
+                {
                     // ignore
                 }
             }

tests/Framework/Http/RrTest.php

Spacing +7 added lines, -7 removed lines

@@ -43,7 +43,7 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r){
                 $this->assertSame('Hello, dave.', (string)$r->getBody());
                 return true;
             })
@@ -65,7 +65,7 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r){
                 $this->assertContains('500', (string)$r->getBody());
                 return true;
             })
@@ -74,7 +74,7 @@ 
         $psr->shouldReceive('acceptRequest')->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);
@@ -82,7 +82,7 @@ 
         $this->app->get(RrDispacher::class)->serve($psr);
 
         $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);
@@ -103,7 +103,7 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r){
                 $this->assertContains('undefined', (string)$r->getBody());
                 return true;
             })
@@ -112,7 +112,7 @@ 
         $psr->shouldReceive('acceptRequest')->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);
@@ -120,7 +120,7 @@ 
         $this->app->get(RrDispacher::class)->serve($psr);
 
         $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 +6 added lines, -3 removed lines

@@ -43,7 +43,8 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r)
+            {
                 $this->assertSame('Hello, dave.', (string)$r->getBody());
                 return true;
             })
@@ -65,7 +66,8 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r)
+            {
                 $this->assertContains('500', (string)$r->getBody());
                 return true;
             })
@@ -103,7 +105,8 @@ 
         );
 
         $psr->shouldReceive('respond')->once()->with(
-            \Mockery::on(function ($r) {
+            \Mockery::on(function ($r)
+            {
                 $this->assertContains('undefined', (string)$r->getBody());
                 return true;
             })

src/Bootloader/SnapshotsBootloader.php

Spacing +2 added lines, -2 removed lines

@@ -46,9 +46,9 @@
         DirectoriesInterface $dirs,
         FilesInterface $files,
         LoggerInterface $logger = null
-    ) {
+    ){
         return new FileSnapshotter(
-            $dirs->get('runtime') . '/snapshots/',
+            $dirs->get('runtime').'/snapshots/',
             $env->get('SNAPSHOT_MAX_FILES', self::MAX_SNAPSHOTS),
             $env->get('SNAPSHOT_VERBOSITY', HandlerInterface::VERBOSITY_VERBOSE),
             new PlainHandler(),

src/Http/RrDispacher.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/Http/ErrorHandler/PlainRenderer.php

Spacing +2 added lines, -2 removed lines

@@ -40,9 +40,9 @@
     public function renderException(Request $request, int $code, string $message): Response
     {
         $response = $this->responseFactory->createResponse($code);
-        if ($request->getHeaderLine('Accept') == 'application/json') {
+        if ($request->getHeaderLine('Accept') == 'application/json'){
             $response->getBody()->write(json_encode(['status' => 500, 'error' => $message]));
-        } else {
+        }else{
             $response->getBody()->write("Error code: {$code}");
         }
 

Braces +5 added lines, -2 removed lines

@@ -40,9 +40,12 @@
     public function renderException(Request $request, int $code, string $message): Response
     {
         $response = $this->responseFactory->createResponse($code);
-        if ($request->getHeaderLine('Accept') == 'application/json') {
+        if ($request->getHeaderLine('Accept') == 'application/json')
+        {
             $response->getBody()->write(json_encode(['status' => 500, 'error' => $message]));
-        } else {
+        }
+        else
+        {
             $response->getBody()->write("Error code: {$code}");
         }
 

src/Http/Middleware/ErrorHandlerMiddleware.php

Spacing +13 added lines, -13 removed lines

@@ -57,7 +57,7 @@ 
         RendererInterface $renderer,
         ResponseFactoryInterface $responseFactory,
         ContainerInterface $container
-    ) {
+    ){
         $this->suppressErrors = $suppressErrors;
         $this->renderer = $renderer;
         $this->responseFactory = $responseFactory;
@@ -71,24 +71,24 @@ 
      */
     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) {
+        }catch (\Throwable $e){
             $snapshotter = $this->getOptional(SnapshotterInterface::class);
-            if ($snapshotter !== null) {
+            if ($snapshotter !== null){
                 /** @var SnapshotterInterface $snapshotter */
                 $snapshotter->register($e);
             }
 
             $code = 500;
 
-            if (!$this->suppressErrors) {
+            if (!$this->suppressErrors){
                 return $this->renderException($request, $e);
             }
         }
@@ -109,7 +109,7 @@ 
     {
         $response = $this->responseFactory->createResponse(500);
 
-        if ($request->getHeaderLine('Accept') == 'application/json') {
+        if ($request->getHeaderLine('Accept') == 'application/json'){
             $response = $response->withHeader('Content-Type', 'application/json');
             $handler = new JsonHandler();
             $response->getBody()->write(json_encode(
@@ -119,10 +119,10 @@ 
                     true
                 )
             ));
-        } else {
+        }else{
             $handler = new HtmlHandler();
             $state = $this->getOptional(StateInterface::class);
-            if ($state !== null) {
+            if ($state !== null){
                 $handler = $handler->withState($state);
             }
 
@@ -156,9 +156,9 @@ 
      */
     private function getOptional(string $class)
     {
-        try {
+        try{
             return $this->container->get($class);
-        } catch (\Throwable | ContainerExceptionInterface $se) {
+        }catch (\Throwable | ContainerExceptionInterface $se){
             return null;
         }
     }

Braces +29 added lines, -12 removed lines

@@ -71,24 +71,34 @@ 
      */
     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) {
+        }
+        catch (\Throwable $e)
+        {
             $snapshotter = $this->getOptional(SnapshotterInterface::class);
-            if ($snapshotter !== null) {
+            if ($snapshotter !== null)
+            {
                 /** @var SnapshotterInterface $snapshotter */
                 $snapshotter->register($e);
             }
 
             $code = 500;
 
-            if (!$this->suppressErrors) {
+            if (!$this->suppressErrors)
+            {
                 return $this->renderException($request, $e);
             }
         }
@@ -109,7 +119,8 @@ 
     {
         $response = $this->responseFactory->createResponse(500);
 
-        if ($request->getHeaderLine('Accept') == 'application/json') {
+        if ($request->getHeaderLine('Accept') == 'application/json')
+        {
             $response = $response->withHeader('Content-Type', 'application/json');
             $handler = new JsonHandler();
             $response->getBody()->write(json_encode(
@@ -119,10 +130,13 @@ 
                     true
                 )
             ));
-        } else {
+        }
+        else
+        {
             $handler = new HtmlHandler();
             $state = $this->getOptional(StateInterface::class);
-            if ($state !== null) {
+            if ($state !== null)
+            {
                 $handler = $handler->withState($state);
             }
 
@@ -156,9 +170,12 @@ 
      */
     private function getOptional(string $class)
     {
-        try {
+        try
+        {
             return $this->container->get($class);
-        } catch (\Throwable | ContainerExceptionInterface $se) {
+        }
+        catch (\Throwable | ContainerExceptionInterface $se)
+        {
             return null;
         }
     }

src/Http/SapiDispatcher.php

Spacing +6 added lines, -6 removed lines

@@ -62,12 +62,12 @@ 
         $http = $this->container->get(Http::class);
         $emitter = $emitter ?? $this->container->get(EmitterInterface::class);
 
-        try {
+        try{
             $response = $http->handle($this->initRequest());
             $emitter->emit($response);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $this->handleException($emitter, $e);
-        } finally {
+        }finally{
             $this->finalizer->finalize(false);
         }
     }
@@ -88,16 +88,16 @@ 
     {
         $handler = new HtmlHandler();
 
-        try {
+        try{
             /** @var SnapshotInterface $snapshot */
             $this->container->get(SnapshotterInterface::class)->register($e);
 
             // 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){
             // nothing to report
         }
 

Braces +15 added lines, -6 removed lines

@@ -62,12 +62,17 @@ 
         $http = $this->container->get(Http::class);
         $emitter = $emitter ?? $this->container->get(EmitterInterface::class);
 
-        try {
+        try
+        {
             $response = $http->handle($this->initRequest());
             $emitter->emit($response);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $this->handleException($emitter, $e);
-        } finally {
+        }
+        finally
+        {
             $this->finalizer->finalize(false);
         }
     }
@@ -88,16 +93,20 @@ 
     {
         $handler = new HtmlHandler();
 
-        try {
+        try
+        {
             /** @var SnapshotInterface $snapshot */
             $this->container->get(SnapshotterInterface::class)->register($e);
 
             // 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)
+        {
             // nothing to report
         }
 

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 RrDispacher::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 RrDispacher::class:
                     $state->setTag('dispatcher', 'roadrunner');
                     break;

src/Debug/StateCollector/HttpCollector.php

Spacing +3 added lines, -3 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function populate(StateInterface $state): void
     {
-        if (!$this->request === null) {
+        if (!$this->request === null){
             return;
         }
 
@@ -48,11 +48,11 @@ 
 
         $state->setVariable('headers', $this->request->getHeaders());
 
-        if ($this->request->getQueryParams() !== []) {
+        if ($this->request->getQueryParams() !== []){
             $state->setVariable('query', $this->request->getQueryParams());
         }
 
-        if ($this->request->getParsedBody() !== null) {
+        if ($this->request->getParsedBody() !== null){
             $state->setVariable('data', $this->request->getParsedBody());
         }
     }

Braces +6 added lines, -3 removed lines

@@ -39,7 +39,8 @@ 
      */
     public function populate(StateInterface $state): void
     {
-        if (!$this->request === null) {
+        if (!$this->request === null)
+        {
             return;
         }
 
@@ -48,11 +49,13 @@ 
 
         $state->setVariable('headers', $this->request->getHeaders());
 
-        if ($this->request->getQueryParams() !== []) {
+        if ($this->request->getQueryParams() !== [])
+        {
             $state->setVariable('query', $this->request->getQueryParams());
         }
 
-        if ($this->request->getParsedBody() !== null) {
+        if ($this->request->getParsedBody() !== null)
+        {
             $state->setVariable('data', $this->request->getParsedBody());
         }
     }