-
Notifications
You must be signed in to change notification settings - Fork 28
Conversation
Added root namespace for \Throwable. Without that i got: PHP message: PHP Fatal error: Uncaught TypeError: Argument 1 passed to EasyCorp\EasyLog\EasyLogFormatter::formatThrowable() must be an instance of EasyCorp\EasyLog\Throwable, instance of Symfony\Component\Security\Core\Exception\AccessDeniedException given
I would prefer to see a That said I think this would lead to an error in <7.0 so I would require |
Hmm I'm having this same with
|
And now looked that change - makes sense :D |
@javiereguiluz , looking forward to this being merged! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm facing this bug as well.
Note that a very minimal test suite could prevent such issues. |
Thanks for this and sorry for having taken so long to merge it. |
This PR was merged into the master branch. Discussion ---------- Update EasyLogFormatter.php Added root namespace for \Throwable. Without that i got: PHP message: PHP Fatal error: Uncaught TypeError: Argument 1 passed to EasyCorp\EasyLog\EasyLogFormatter::formatThrowable() must be an instance of EasyCorp\EasyLog\Throwable, instance of Symfony\Component\Security\Core\Exception\AccessDeniedException given Commits ------- 41b6783 Update EasyLogFormatter.php
This fix was included in the just released 1.0.6 version (https://github.com/EasyCorp/easy-log-handler/releases/tag/v1.0.6) |
Thanks a lot @javiereguiluz ! |
Added root namespace for \Throwable. Without that i got:
PHP message: PHP Fatal error: Uncaught TypeError: Argument 1 passed to EasyCorp\EasyLog\EasyLogFormatter::formatThrowable() must be an instance of EasyCorp\EasyLog\Throwable, instance of Symfony\Component\Security\Core\Exception\AccessDeniedException given