LoggerEssentials / ErrorHandling

rector.php

Indentation +7 added lines, -7 removed lines

@@ -5,11 +5,11 @@
 use Rector\Config\RectorConfig;
 
 return RectorConfig::configure()
-    ->withPaths([
-        __DIR__ . '/src',
-        __DIR__ . '/tests',
-    ])
+	->withPaths([
+		__DIR__ . '/src',
+		__DIR__ . '/tests',
+	])
 	->withPhpSets()
-    ->withTypeCoverageLevel(0)
-    ->withDeadCodeLevel(0)
-    ->withCodeQualityLevel(0);
+	->withTypeCoverageLevel(0)
+	->withDeadCodeLevel(0)
+	->withCodeQualityLevel(0);

Spacing +2 added lines, -2 removed lines

@@ -6,8 +6,8 @@
 
 return RectorConfig::configure()
     ->withPaths([
-        __DIR__ . '/src',
-        __DIR__ . '/tests',
+        __DIR__.'/src',
+        __DIR__.'/tests',
     ])
 	->withPhpSets()
     ->withTypeCoverageLevel(0)

src/StackTracePrinter.php

Indentation +49 added lines, -49 removed lines

@@ -9,8 +9,8 @@ 
 	 * @param Throwable $e
 	 * @param string $messageIntro
 	 */
-    public static function printException(Throwable $e, $messageIntro = ''): void {
-        self::p("%s[%s] %s\n", $messageIntro, get_class($e), $e->getMessage());
+	public static function printException(Throwable $e, $messageIntro = ''): void {
+		self::p("%s[%s] %s\n", $messageIntro, get_class($e), $e->getMessage());
 
 		foreach($e->getTrace() as $idx => $station) {
 			self::formatStation($idx, $station);
@@ -26,55 +26,55 @@ 
 	 * @param int $idx
 	 * @param array $station
 	 */
-    /**
-     * @param int $idx
-     * @param array<string, mixed> $station
-     */
-    private static function formatStation($idx, $station): void {
-        $defaults = [
-            'file' => null,
-            'line' => null,
-            'class' => null,
-            'function' => null,
-            'type' => null,
-            'args' => [],
-        ];
-        $station = array_merge($defaults, $station);
-        self::p("#%- 3s%s:%d\n", $idx, $station['file'] ?: 'unknown', $station['line']);
-        if($station['class'] !== null || $station['function'] !== null) {
-            $params = [];
-            $args = $station['args'];
-            if (!is_array($args)) {
-                $args = [];
-            }
-            foreach($args as $argument) {
-                if(is_array($argument)) {
-                    $params[] = sprintf('array%d', count($argument));
-                } elseif(is_object($argument)) {
-                    $params[] = sprintf('%s', get_class($argument));
-                } else {
-                    $params[] = gettype($argument);
-                }
-            }
-            $func = $station['function'];
-            if(is_string($func) && strpos($func, '{closure}') !== false && $station['class'] !== null) {
-                $station['function'] = '{closure}';
-            }
-            self::p("    %s%s%s%s%s%s\n", $station['class'], $station['type'], $station['function'], "(", join(', ', $params), ")");
-        }
-    }
+	/**
+	 * @param int $idx
+	 * @param array<string, mixed> $station
+	 */
+	private static function formatStation($idx, $station): void {
+		$defaults = [
+			'file' => null,
+			'line' => null,
+			'class' => null,
+			'function' => null,
+			'type' => null,
+			'args' => [],
+		];
+		$station = array_merge($defaults, $station);
+		self::p("#%- 3s%s:%d\n", $idx, $station['file'] ?: 'unknown', $station['line']);
+		if($station['class'] !== null || $station['function'] !== null) {
+			$params = [];
+			$args = $station['args'];
+			if (!is_array($args)) {
+				$args = [];
+			}
+			foreach($args as $argument) {
+				if(is_array($argument)) {
+					$params[] = sprintf('array%d', count($argument));
+				} elseif(is_object($argument)) {
+					$params[] = sprintf('%s', get_class($argument));
+				} else {
+					$params[] = gettype($argument);
+				}
+			}
+			$func = $station['function'];
+			if(is_string($func) && strpos($func, '{closure}') !== false && $station['class'] !== null) {
+				$station['function'] = '{closure}';
+			}
+			self::p("    %s%s%s%s%s%s\n", $station['class'], $station['type'], $station['function'], "(", join(', ', $params), ")");
+		}
+	}
 
 	/**
 	 * @param string $format
 	 */
-    private static function p($format = "\n"): void {
-        $fp = fopen('php://stderr', 'w');
-        if ($fp !== false) {
-            $args = array_slice(func_get_args(), 1);
-            /** @var array<bool|float|int|string|null> $args */
-            $args = $args;
-            fwrite($fp, vsprintf($format, $args));
-            fclose($fp);
-        }
-    }
+	private static function p($format = "\n"): void {
+		$fp = fopen('php://stderr', 'w');
+		if ($fp !== false) {
+			$args = array_slice(func_get_args(), 1);
+			/** @var array<bool|float|int|string|null> $args */
+			$args = $args;
+			fwrite($fp, vsprintf($format, $args));
+			fclose($fp);
+		}
+	}
 }

Spacing +7 added lines, -7 removed lines

@@ -12,11 +12,11 @@ 
     public static function printException(Throwable $e, $messageIntro = ''): void {
         self::p("%s[%s] %s\n", $messageIntro, get_class($e), $e->getMessage());
 
-		foreach($e->getTrace() as $idx => $station) {
+		foreach ($e->getTrace() as $idx => $station) {
 			self::formatStation($idx, $station);
 		}
 
-		if($e->getPrevious() instanceof Throwable) {
+		if ($e->getPrevious() instanceof Throwable) {
 			self::p();
 			self::printException($e->getPrevious(), 'Previous: ');
 		}
@@ -41,23 +41,23 @@ 
         ];
         $station = array_merge($defaults, $station);
         self::p("#%- 3s%s:%d\n", $idx, $station['file'] ?: 'unknown', $station['line']);
-        if($station['class'] !== null || $station['function'] !== null) {
+        if ($station['class'] !== null || $station['function'] !== null) {
             $params = [];
             $args = $station['args'];
             if (!is_array($args)) {
                 $args = [];
             }
-            foreach($args as $argument) {
-                if(is_array($argument)) {
+            foreach ($args as $argument) {
+                if (is_array($argument)) {
                     $params[] = sprintf('array%d', count($argument));
-                } elseif(is_object($argument)) {
+                } elseif (is_object($argument)) {
                     $params[] = sprintf('%s', get_class($argument));
                 } else {
                     $params[] = gettype($argument);
                 }
             }
             $func = $station['function'];
-            if(is_string($func) && strpos($func, '{closure}') !== false && $station['class'] !== null) {
+            if (is_string($func) && strpos($func, '{closure}') !== false && $station['class'] !== null) {
                 $station['function'] = '{closure}';
             }
             self::p("    %s%s%s%s%s%s\n", $station['class'], $station['type'], $station['function'], "(", join(', ', $params), ")");

src/CoreErrorHandlers.php

Spacing +14 added lines, -14 removed lines

@@ -30,11 +30,11 @@ 
 	 * @param int $bitmask
 	 */
 	public static function enableExceptionsForErrors(int $bitmask = E_ALL): void {
-		set_error_handler(function ($level, $message, $file, $line) use ($bitmask) {
-			if(0 === error_reporting()) {
+		set_error_handler(function($level, $message, $file, $line) use ($bitmask) {
+			if (0 === error_reporting()) {
 				return false;
 			}
-			if(($bitmask & $level) !== 0) {
+			if (($bitmask & $level) !== 0) {
 				throw new ErrorException($message, 0, $level, $file, $line);
 			}
 
@@ -47,7 +47,7 @@ 
 	 * @param string $logLevel PSR-3 Log-Level
 	 */
 	public static function registerAssertionHandler(LoggerInterface $logger, string $logLevel): void {
-		if(self::$assertionLogger === null) {
+		if (self::$assertionLogger === null) {
 			self::$assertionLogger = new LoggerCollection();
 
 			// PHP 8.0+ throws AssertionError by default (assert.exception=1)
@@ -55,8 +55,8 @@ 
 			$previousHandler = set_exception_handler(null);
 			restore_exception_handler();
 
-			set_exception_handler(static function (Throwable $exception) use ($previousHandler, $logLevel): void {
-				if($exception instanceof \AssertionError) {
+			set_exception_handler(static function(Throwable $exception) use ($previousHandler, $logLevel): void {
+				if ($exception instanceof \AssertionError) {
 					// Log the assertion failure
 					self::$assertionLogger?->log($logLevel, $exception->getMessage(), [
 						'file' => $exception->getFile(),
@@ -67,7 +67,7 @@ 
 				}
 
 				// Pass other exceptions to previous handler or re-throw
-				if($previousHandler !== null) {
+				if ($previousHandler !== null) {
 					$previousHandler($exception);
 				} else {
 					throw $exception;
@@ -81,11 +81,11 @@ 
 	 * @param LoggerInterface $logger
 	 */
 	public static function registerFatalErrorHandler(LoggerInterface $logger): void {
-		if(self::$fatalLogger === null) {
+		if (self::$fatalLogger === null) {
 			self::$fatalLogger = new LoggerCollection();
-			register_shutdown_function(function (): void {
+			register_shutdown_function(function(): void {
 				$error = error_get_last();
-				if($error !== null && in_array($error['type'], self::FATAL_LEVELS, true)) {
+				if ($error !== null && in_array($error['type'], self::FATAL_LEVELS, true)) {
 					$fl = new LogLevelRangeFilter(self::$fatalLogger ?? new LoggerCollection(), LogLevel::ERROR);
 					$fl->log(LogLevel::ALERT, $error['message'], $error);
 				}
@@ -95,14 +95,14 @@ 
 	}
 
 	public static function registerExceptionHandler(LoggerInterface $logger): void {
-		if(self::$exceptionLogger === null) {
+		if (self::$exceptionLogger === null) {
 			self::$exceptionLogger = new LoggerCollection();
-			set_exception_handler(static function (Throwable $exception): void {
+			set_exception_handler(static function(Throwable $exception): void {
 				$log = new LogLevelRangeFilter(self::$exceptionLogger ?? new LoggerCollection(), LogLevel::ERROR);
 				try {
 					$exceptionAsArray = self::getExceptionAsArray($exception, true, true);
 					$log->log(LogLevel::CRITICAL, $exception->getMessage(), ['exception' => $exceptionAsArray]);
-				} catch(Throwable) {
+				} catch (Throwable) {
 					$exceptionAsArray1 = self::getExceptionAsArray($exception, false, false);
 					$log->log(LogLevel::CRITICAL, $exception->getMessage(), ['exception' => $exceptionAsArray1]);
 				}
@@ -121,7 +121,7 @@ 
 	 * @phpstan-return TExceptionArray|null
 	 */
 	private static function getExceptionAsArray(?Throwable $exception, bool $previous, bool $withTrace): ?array {
-		if($exception === null) {
+		if ($exception === null) {
 			return null;
 		}