[GH-ISSUE #815] ErrorHandler only logs error levels set by error_reporting #313

New issue

Closed

opened 2026-03-04 02:14:03 +03:00 by kerem · 4 comments

kerem commented

2026-03-04 02:14:03 +03:00

Owner

Originally created by @masterada on GitHub (Jul 6, 2016).
Original GitHub issue: https://github.com/Seldaek/monolog/issues/815

In ErrorHandler.php, we should be able to turn this check off:

    if (!(error_reporting() & $code)) {
        return;
    }

As this check makes it harder to replace legacy error handling with Monolog's error handler.

Originally created by @masterada on GitHub (Jul 6, 2016). Original GitHub issue: https://github.com/Seldaek/monolog/issues/815 In ErrorHandler.php, we should be able to turn this check off: ``` if (!(error_reporting() & $code)) { return; } ``` As this check makes it harder to replace legacy error handling with Monolog's error handler.

kerem closed this issue

2026-03-04 02:14:04 +03:00

kerem commented

2026-03-04 02:14:05 +03:00

Author

Owner

@masterada commented on GitHub (Jul 6, 2016):

#816

@masterada commented on GitHub (Jul 6, 2016): #816

kerem commented

2026-03-04 02:14:05 +03:00

Author

Owner

@keywan-ghadami-oxid commented on GitHub (Jul 28, 2016):

Hi i have the similar problem because the Monolog ErrorHandler stops on this condition it does not even call parent ErrorHandlers which makes it incompatible with other ErrorHandlers.

I did not notice that you already created a pull request please have a look at
https://github.com/keywan-ghadami-oxid/monolog/pull/1/files
Maybe we can combine them what do you think?

@keywan-ghadami-oxid commented on GitHub (Jul 28, 2016): Hi i have the similar problem because the Monolog ErrorHandler stops on this condition it does not even call parent ErrorHandlers which makes it incompatible with other ErrorHandlers. I did not notice that you already created a pull request please have a look at https://github.com/keywan-ghadami-oxid/monolog/pull/1/files Maybe we can combine them what do you think?

kerem commented

2026-03-04 02:14:05 +03:00

Author

Owner

@xabbuh commented on GitHub (Aug 4, 2016):

I guess this can be closed as it was solved by #816 and is part of Monolog 1.21.

@xabbuh commented on GitHub (Aug 4, 2016): I guess this can be closed as it was solved by #816 and is part of Monolog 1.21.

kerem commented

2026-03-04 02:14:05 +03:00

Author

Owner

@Seldaek commented on GitHub (Aug 4, 2016):

Right sorry I missed that it was still open :)

@Seldaek commented on GitHub (Aug 4, 2016): Right sorry I missed that it was still open :)

kerem referenced this issue

2026-03-04 03:04:43 +03:00

[PR #313] [MERGED] [RFC] Add limit to batched records #1074