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
Class Rap2hpoutre\LaravelLogViewer\LogViewerController does not exist #230
Comments
Make sure you are dumping the autoloader when you update: composer update -oa |
Hi @mikemand Yeah been trying that and it didn't work for me. Is the package working for you right now? For the mean time, this is how I'm making it work. HomeController.php: |
same problem |
Yes, it's working for me. Are your routes cached? Check the |
Oh, no sorry. My fault, I was checking a site with an old version still. The new one isn't working for me. I'm investigating. |
Ok, I got it working. I used the class-based way of routing, but it should work for the original string-based way too. Route::get('logs', [\Rap2hpoutre\Controllers\LogViewerController::class, 'index']) Edit: I think I see the problem. The namespace changed for the controller. Instead of |
I think that's the issue! We would just need to update the README file in regards to the update. Thank you @mikemand ! |
In my opinion, Composer needs some way to supply upgrade steps (if applicable) when it is upgrading packages. I spend more time checking each package for an upgrade guide (and if they don't have one, comparing versions) than I do coding sometimes. 😢 Glad you got it working! |
Oops. I should not have merged #227 that fast. I re-open this issue since it's a problem that other people may have in the next days. I'm not sure I have to fix this or leave as is and write something in README though... |
Hi @raymondTheDev @mikemand @Scaenicus @tiagoa @DjeeBay @timothymarois & @avesgit ! Thank you for your patience and sorry for the mess. I merged a PR that introduced a breaking change and released it in a minor version (v1.5.0): my bad, I'm sorry about that. Unfortunately, it means that you will have to revert your change too, I'm sincerely sorry. Since only 20+ hours as passed since the release I guess we could consider not too much projects are affected, and the next users that will upgrade would not see the problem. I prefer preserving the semantical version (again maybe I'm wrong and sorry for the mess) to avoid unexpected behavior for other users. Feel free to send me feedback! |
In other words, you have to update then keep using:
|
Resolved now! |
Just did a composer update on a Laravel project and now it's showing.
Class Rap2hpoutre\LaravelLogViewer\LogViewerController does not exist
This was working fine before the composer update
Thank you.
The text was updated successfully, but these errors were encountered: