spiral / framework

src/Exceptions/src/ExceptionHandler.php

Spacing +20 added lines, -20 removed lines

@@ -47,9 +47,9 @@ 
 
     public function getRenderer(?string $format = null): ?ExceptionRendererInterface
     {
-        if ($format !== null) {
-            foreach ($this->renderers as $renderer) {
-                if ($renderer->canRender($format)) {
+        if ($format !== null){
+            foreach ($this->renderers as $renderer){
+                if ($renderer->canRender($format)){
                     return $renderer;
                 }
             }
@@ -72,18 +72,18 @@ 
 
     public function report(\Throwable $exception): void
     {
-        if ($this->shouldNotReport($exception)) {
+        if ($this->shouldNotReport($exception)){
             return;
         }
 
-        foreach ($this->reporters as $reporter) {
-            try {
-                if ($reporter instanceof ExceptionReporterInterface) {
+        foreach ($this->reporters as $reporter){
+            try{
+                if ($reporter instanceof ExceptionReporterInterface){
                     $reporter->report($exception);
-                } else {
+                }else{
                     $reporter($exception);
                 }
-            } catch (\Throwable) {
+            }catch (\Throwable){
                 // Do nothing
             }
         }
@@ -91,10 +91,10 @@ 
 
     public function handleGlobalException(\Throwable $e): void
     {
-        if (\in_array(PHP_SAPI, ['cli', 'cli-server'], true)) {
+        if (\in_array(PHP_SAPI, ['cli', 'cli-server'], true)){
             $this->output ??= \defined('STDERR') ? \STDERR : \fopen('php://stderr', 'wb+');
             $format = 'cli';
-        } else {
+        }else{
             $this->output ??= \defined('STDOUT') ? \STDOUT : \fopen('php://stdout', 'wb+');
             $format = 'html';
         }
@@ -103,9 +103,9 @@ 
         $this->report($e);
 
         // There is possible an exception on the application termination
-        try {
+        try{
             \fwrite($this->output, $this->render($e, verbosity: $this->verbosity, format: $format));
-        } catch (\Throwable) {
+        }catch (\Throwable){
             $this->output = null;
         }
     }
@@ -129,7 +129,7 @@ 
     /**
      * @param ExceptionReporterInterface|Closure(\Throwable):void $reporter
      */
-    public function addReporter(ExceptionReporterInterface|Closure $reporter): void
+    public function addReporter(ExceptionReporterInterface | Closure $reporter): void
     {
         $this->reporters[] = $reporter;
     }
@@ -147,13 +147,13 @@ 
      */
     protected function handleShutdown(): void
     {
-        if (empty($error = \error_get_last())) {
+        if (empty($error = \error_get_last())){
             return;
         }
 
-        try {
+        try{
             $this->handleError($error['type'], $error['message'], $error['file'], $error['line']);
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             $this->handleGlobalException($e);
         }
     }
@@ -169,7 +169,7 @@ 
         string $errfile = '',
         int $errline = 0,
     ): bool {
-        if (!(\error_reporting() & $errno)) {
+        if (!(\error_reporting() & $errno)){
             return false;
         }
 
@@ -183,8 +183,8 @@ 
 
     protected function shouldNotReport(\Throwable $exception): bool
     {
-        foreach ($this->nonReportableExceptions as $nonReportableException) {
-            if ($exception instanceof $nonReportableException) {
+        foreach ($this->nonReportableExceptions as $nonReportableException){
+            if ($exception instanceof $nonReportableException){
                 return true;
             }
         }

Braces +43 added lines, -19 removed lines

@@ -47,9 +47,12 @@ 
 
     public function getRenderer(?string $format = null): ?ExceptionRendererInterface
     {
-        if ($format !== null) {
-            foreach ($this->renderers as $renderer) {
-                if ($renderer->canRender($format)) {
+        if ($format !== null)
+        {
+            foreach ($this->renderers as $renderer)
+            {
+                if ($renderer->canRender($format))
+                {
                     return $renderer;
                 }
             }
@@ -72,18 +75,26 @@ 
 
     public function report(\Throwable $exception): void
     {
-        if ($this->shouldNotReport($exception)) {
+        if ($this->shouldNotReport($exception))
+        {
             return;
         }
 
-        foreach ($this->reporters as $reporter) {
-            try {
-                if ($reporter instanceof ExceptionReporterInterface) {
+        foreach ($this->reporters as $reporter)
+        {
+            try
+            {
+                if ($reporter instanceof ExceptionReporterInterface)
+                {
                     $reporter->report($exception);
-                } else {
+                }
+                else
+                {
                     $reporter($exception);
                 }
-            } catch (\Throwable) {
+            }
+            catch (\Throwable)
+            {
                 // Do nothing
             }
         }
@@ -91,10 +102,13 @@ 
 
     public function handleGlobalException(\Throwable $e): void
     {
-        if (\in_array(PHP_SAPI, ['cli', 'cli-server'], true)) {
+        if (\in_array(PHP_SAPI, ['cli', 'cli-server'], true))
+        {
             $this->output ??= \defined('STDERR') ? \STDERR : \fopen('php://stderr', 'wb+');
             $format = 'cli';
-        } else {
+        }
+        else
+        {
             $this->output ??= \defined('STDOUT') ? \STDOUT : \fopen('php://stdout', 'wb+');
             $format = 'html';
         }
@@ -103,9 +117,12 @@ 
         $this->report($e);
 
         // There is possible an exception on the application termination
-        try {
+        try
+        {
             \fwrite($this->output, $this->render($e, verbosity: $this->verbosity, format: $format));
-        } catch (\Throwable) {
+        }
+        catch (\Throwable)
+        {
             $this->output = null;
         }
     }
@@ -147,13 +164,17 @@ 
      */
     protected function handleShutdown(): void
     {
-        if (empty($error = \error_get_last())) {
+        if (empty($error = \error_get_last()))
+        {
             return;
         }
 
-        try {
+        try
+        {
             $this->handleError($error['type'], $error['message'], $error['file'], $error['line']);
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             $this->handleGlobalException($e);
         }
     }
@@ -169,7 +190,8 @@ 
         string $errfile = '',
         int $errline = 0,
     ): bool {
-        if (!(\error_reporting() & $errno)) {
+        if (!(\error_reporting() & $errno))
+        {
             return false;
         }
 
@@ -183,8 +205,10 @@ 
 
     protected function shouldNotReport(\Throwable $exception): bool
     {
-        foreach ($this->nonReportableExceptions as $nonReportableException) {
-            if ($exception instanceof $nonReportableException) {
+        foreach ($this->nonReportableExceptions as $nonReportableException)
+        {
+            if ($exception instanceof $nonReportableException)
+            {
                 return true;
             }
         }

src/Exceptions/tests/ExceptionHandlerTest.php

Spacing +7 added lines, -7 removed lines

@@ -133,7 +133,7 @@ 
 
     private function makeEmptyErrorHandler(): ExceptionHandler
     {
-        return new class extends ExceptionHandler {
+        return new class extends ExceptionHandler{
             protected function bootBasicHandlers(): void
             {
             }
@@ -147,17 +147,17 @@ 
 
     public static function nonReportableExceptionsDataProvider(): \Traversable
     {
-        yield [new class extends ClientException {}];
+        yield [new class extends ClientException{}];
         yield [new NotFoundException()];
-        yield [new class extends NotFoundException {}];
+        yield [new class extends NotFoundException{}];
         yield [new ForbiddenException()];
-        yield [new class extends ForbiddenException {}];
+        yield [new class extends ForbiddenException{}];
         yield [new UnauthorizedException()];
-        yield [new class extends UnauthorizedException {}];
+        yield [new class extends UnauthorizedException{}];
         yield [new AuthorizationException()];
-        yield [new class extends AuthorizationException {}];
+        yield [new class extends AuthorizationException{}];
         yield [new ValidationException([])];
-        yield [new class([]) extends ValidationException {}];
+        yield [new class([]) extends ValidationException{}];
         yield [new TestException()];
     }
 }

Braces +20 added lines, -7 removed lines

@@ -133,7 +133,8 @@ 
 
     private function makeEmptyErrorHandler(): ExceptionHandler
     {
-        return new class extends ExceptionHandler {
+        return new class extends ExceptionHandler
+        {
             protected function bootBasicHandlers(): void
             {
             }
@@ -147,17 +148,29 @@ 
 
     public static function nonReportableExceptionsDataProvider(): \Traversable
     {
-        yield [new class extends ClientException {}];
+        yield [new class extends ClientException
+        {
+}];
         yield [new NotFoundException()];
-        yield [new class extends NotFoundException {}];
+        yield [new class extends NotFoundException
+        {
+}];
         yield [new ForbiddenException()];
-        yield [new class extends ForbiddenException {}];
+        yield [new class extends ForbiddenException
+        {
+}];
         yield [new UnauthorizedException()];
-        yield [new class extends UnauthorizedException {}];
+        yield [new class extends UnauthorizedException
+        {
+}];
         yield [new AuthorizationException()];
-        yield [new class extends AuthorizationException {}];
+        yield [new class extends AuthorizationException
+        {
+}];
         yield [new ValidationException([])];
-        yield [new class([]) extends ValidationException {}];
+        yield [new class([]) extends ValidationException
+        {
+}];
         yield [new TestException()];
     }
 }