Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when upserting article through API #57

Closed
j-norwood-young opened this issue Mar 25, 2020 · 3 comments
Closed

Error when upserting article through API #57

j-norwood-young opened this issue Mar 25, 2020 · 3 comments

Comments

@j-norwood-young
Copy link

I've just upgraded (after a while) and am getting the following error in my client when posting to the /api/articles/upsert endpoint:

"message": "Declaration of Recca0120\\LaravelTracy\\Exceptions\\Handler::report(Throwable $e) must be compatible with Illuminate\\Contracts\\Debug\\ExceptionHandler::report(Exception $e)", 
"exception": "Symfony\\Component\\Debug\\Exception\\FatalErrorException", 
"file": "/var/www/workspace/remp/Beam/vendor/recca0120/laravel-tracy/src/Exceptions/Handler.php", 
"line": 14, 
@rootpd
Copy link
Member

rootpd commented Mar 25, 2020

Hi Jason,

  • Did you run composer install after you updated? Did it run correctly until the end?
  • Could you maybe try to remove your vendor folde rand try again?
  • If it's still not working, can you send me the compose.lock file you're at so we can have a look?

Thanks!

@j-norwood-young
Copy link
Author

I've temporarily fixed it by editing /Beam/vendor/recca0120/laravel-tracy/src/Exceptions/Handler.php and changing Throwable to Exception. I'd ideally like a permanent fix though.

public function renderForConsole($output, Exception $e)
public function report(Exception $e)
public function render($request, Exception $e)

@rootpd
Copy link
Member

rootpd commented Jan 26, 2021

This was fixed some time ago by updating the recca0129/laravel-tracy dependency and is OK now - the declarations do match. Closing the issue up.

@rootpd rootpd closed this as completed Jan 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants