-
Notifications
You must be signed in to change notification settings - Fork 391
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
phpstan internal error in console Method newEloquentBuilder() was not found #808
Comments
Thanks @parallels999 , |
Well, phpstan was never a part of the workflow, so i have no idea. Im gonna need to add phpstan to our pipeline. Until then i have no idea what was fixed. Its not an issue i would ever see 🤷 |
@parallels999 not yet. |
@erikn69 |
in my code, I use
I have phpstan v1.10.14, PHP 8.1, Laravel 10.9 |
@fgibaux try dev branch laravel-auditing/composer.json Lines 67 to 69 in eadb80b
|
@parallels999 ah thank you! it works without errors when using v14-dev |
@fgibaux the new version will be released very soon with corrections and performance improvements |
First of all: Thanks for fixing this! 💪🔥 |
Actual Behaviour
When i use Audit::query or SomeModel->audits()->where() then phpstan throwin exception
` Error
`
this happen after last release with change model to contract: release diff
Expected Behaviour
Phpstan has no warning when use builder on Audit model
Steps to Reproduce
List all the steps needed to reproduce the issue you're having.
Make sure to include code (affected models, configuration, ...).
Pastebins, screenshots and/or other resources that may help us understand what's going on, will also be appreciated.
In a nutshell, help us, help you.
Possible Solutions
AuditContract should be reveiwed.
The text was updated successfully, but these errors were encountered: