[PR #928] [CLOSED] Php5.3 autoload errors #1418

New issue

Open

opened 2026-03-04 03:12:44 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 03:12:44 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Seldaek/monolog/pull/928
Author: @fabriceparallel
Created: 2/21/2017
Status: ❌ Closed

Base: master ← Head: php5.3_autoload_errors

📝 Commits (2)

c82afaa Add missed return phpdoc tag. (#908)
8140f60 make ErrorHandler extends LogLevel to avoid autoloading issue when error is triggered on compile

📊 Changes

2 files changed (+20 additions, -19 deletions)

View changed files

📝 src/Monolog/ErrorHandler.php (+19 -19)
📝 src/Monolog/Handler/RavenHandler.php (+1 -0)

📄 Description

See this PHP5.3 bug

When an error is triggered on compile-time the autoloader is not called which create a "LogLevel class not found" error.

I make the ErrorHandler extend LogLevel so ErrorHandler will already have all the log level it's looking for, and the autoloader won't be called.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Seldaek/monolog/pull/928 **Author:** [@fabriceparallel](https://github.com/fabriceparallel) **Created:** 2/21/2017 **Status:** ❌ Closed **Base:** `master` ← **Head:** `php5.3_autoload_errors` --- ### 📝 Commits (2) - [`c82afaa`](https://github.com/Seldaek/monolog/commit/c82afaa30356d20e08cc74fec7ab9b6ecd38288a) Add missed return phpdoc tag. (#908) - [`8140f60`](https://github.com/Seldaek/monolog/commit/8140f6026f59a2be9f80e562cf6ad6d441241593) make ErrorHandler extends LogLevel to avoid autoloading issue when error is triggered on compile ### 📊 Changes **2 files changed** (+20 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `src/Monolog/ErrorHandler.php` (+19 -19) 📝 `src/Monolog/Handler/RavenHandler.php` (+1 -0) </details> ### 📄 Description See this [PHP5.3 bug](https://bugs.php.net/bug.php?id=47987) When an error is triggered on compile-time the autoloader is not called which create a "`LogLevel` class not found" error. I make the ErrorHandler extend LogLevel so ErrorHandler will already have all the log level it's looking for, and the autoloader won't be called. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>