Skip to content

Commit

Permalink
Merge pull request #6987 from kenjis/remove-isFakerDeprecationError
Browse files Browse the repository at this point in the history
refactor: remove Workaround for Faker deprecation errors in PHP 8.2
  • Loading branch information
samsonasik authored Dec 17, 2022
2 parents 90fb1fe + 5482f56 commit a68891a
Showing 1 changed file with 0 additions and 33 deletions.
33 changes: 0 additions & 33 deletions system/Debug/Exceptions.php
Original file line number Diff line number Diff line change
Expand Up @@ -159,11 +159,6 @@ public function exceptionHandler(Throwable $exception)
public function errorHandler(int $severity, string $message, ?string $file = null, ?int $line = null)
{
if ($this->isDeprecationError($severity)) {
// @TODO Remove if Faker is fixed.
if ($this->isFakerDeprecationError($message, $file, $line)) {
return true;
}

if (! $this->config->logDeprecations || (bool) env('CODEIGNITER_SCREAM_DEPRECATIONS')) {
throw new ErrorException($message, 0, $severity, $file, $line);
}
Expand All @@ -178,34 +173,6 @@ public function errorHandler(int $severity, string $message, ?string $file = nul
return false; // return false to propagate the error to PHP standard error handler
}

/**
* Workaround for Faker deprecation errors in PHP 8.2.
*
* @see https://github.com/FakerPHP/Faker/issues/479
*/
private function isFakerDeprecationError(string $message, ?string $file = null, ?int $line = null)
{
if (
defined('VENDORPATH')
&& strpos($file, VENDORPATH . 'fakerphp/faker/') !== false
&& $message === 'Use of "static" in callables is deprecated'
) {
log_message(
LogLevel::WARNING,
'[DEPRECATED] {message} in {errFile} on line {errLine}.',
[
'message' => $message,
'errFile' => clean_path($file ?? ''),
'errLine' => $line ?? 0,
]
);

return true;
}

return false;
}

/**
* Checks to see if any errors have happened during shutdown that
* need to be caught and handle them.
Expand Down

0 comments on commit a68891a

Please sign in to comment.